Skip to content

Test the classic addon wiring - #20

Open
aklkv wants to merge 2 commits into
mainmatter:mainfrom
aklkv:test/classic-wiring
Open

aklkv wants to merge 2 commits into
mainmatter:mainfrom
aklkv:test/classic-wiring

Conversation

@aklkv

@aklkv aklkv commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Part of splitting up #16.

Adds tests for index.js, so the classic wiring is covered before the package split moves things around. One fix fell out: baseDir() returned the app root, so ember-cli-htmlbars hashed the app's whole dependency graph for its cache key instead of this addon.

aklkv added 2 commits October 20, 2026 03:02
Nothing tested index.js. These stand the addon up with a stub host and
check the htmlbars registry entry, the parallel-babel descriptor
ember-cli rebuilds the plugin from in workers, option merging, and that
the babel plugin is added once.
ember-cli-htmlbars passes baseDir() to hash-for-dep to build its cache
key, and expects the plugin's own package (its own plugins use
__dirname). path.resolve('./') is the app's root, so every build hashed
the app's whole dependency graph instead.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant