aurelia / testing

Simplifies the testing of UI components by providing an elegant, fluent interface for arranging test setups along with a number of runtime debug/test helpers.
MIT License
40 stars 27 forks source link

feat(update): update bindings helper #79

Open zewa666 opened 6 years ago

zewa666 commented 6 years ago

adds a helper function to flush the microtaskqueue so that testing binding changes becomes easier

zewa666 commented 6 years ago

Hey @bigopon I could need some help with getting the tests to work. Funny enough using the updateBindings helper in an app works flawlessly but not inside the testing repos tests themselves.

bigopon commented 6 years ago

I can reproduce it. I'll check and update

bigopon commented 6 years ago

The task queue that component view model observers uses is not the same task queue getting from global Container. I'm not sure why. Without proper debugger I think it's hard to observe the behavior. Maybe @jdanyow / @EisenbergEffect can give some help

zewa666 commented 6 years ago

I had the same Idea but even picking the aurelia.container during bootstrapping, which should be the right one, didnt help. I wonder whether its because of the way how the initialization is setup, pointing to the dist folder. I'll try to switch that similar to how I did it with i18n and see whether this helps.

silbinarywolf commented 5 years ago

@zewa666 Did you have any luck with this or have any other information that will help in getting this PR over the line?

I'd like to remove my setTimeout() hack for testing two-way binding :)

Alexander-Taran commented 5 years ago

@bigopon @zewa666 should we merge this?

bigopon commented 5 years ago

Let me fix it first. I'll create a PR shortly

bigopon commented 5 years ago

I think we will have to delay this feature for awhile, unless we fix some issues related to resources registration with container first. setTimeout(, 0) or await Promise.resolve() could be appropriate for now.