GavinJoyce / ember-headlessui

https://gavinjoyce.github.io/ember-headlessui/
Other
92 stars 34 forks source link

Fix CI / don't use test.each #150

Closed NullVoxPopuli closed 2 years ago

NullVoxPopuli commented 2 years ago

the tests on the main branch are failing, so I removed test.each in favor of more native JS to try pin down which test specifically is causing issues.

that test will then be skipped / marked as todo.

whovever needs nested dialogs can work to unfix it :sweat_smile:

NullVoxPopuli commented 2 years ago

Gonna merge this because all PRs are blocked until test stability is achieved. And while I still wouldn't consider tests stable, this is a step towards at least eliminating false negatives.