search

ember-decorators / babel-transforms

MIT License
6 stars 7 forks source link

test: add tests for class properties and decorators transform #3

Closed buschtoens closed 6 years ago

buschtoens commented 6 years ago

Closes #1

buschtoens commented 6 years ago

This adds pretty basic tests, that only verify, that the syntax actually can be used in the dummy app. I don't know a pure ember-cli way to explicitly test various build configurations and addon dependency graphs. Maybe ember-try has us covered, but I don't think so.

I guess these bad tests are better than no tests at all. However, we should come up with real integration tests, that test different dependency graphs and disableTransforms settings.

example-addon
  -> @ember-decorators/babel-transforms

example-app
  -> @ember-decorators/babel-transforms

example-parent-addon
  -> example-addon
    -> @ember-decorators/babel-transforms

example-parent-app
  -> example-parent-addon
    -> example-addon
      -> @ember-decorators/babel-transforms
dfreeman commented 6 years ago

I have some acceptance tests set up in ember-cli-dependency-lint that test behavior with different hierarchies of addons that might be useful for inspiration here.