alexlafroscia / ember-cli-stencil

Automatic discovery of Stencil.js components for your Ember application
26 stars 7 forks source link

Syntax fix #8

Closed xcambar closed 5 years ago

xcambar commented 5 years ago

.reduce(...) requires a default value.

xcambar commented 5 years ago

I find the tests to run inconsistently (even locally, on the current master). Did you notice that already?

It seems to be related to the Web Components to be rendered too late for the test suite under some circumstances...

alexlafroscia commented 5 years ago

Hmm, interesting. I'm sure that's related to the other PR failing the build as well.

I haven't noticed flakey tests in the past, but I'll check it out. We may need to add a test helper that slows the tests down until whatever is available if it seems like it could be a timing issue.