-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[dart] adapter and example test for it #6
Conversation
Hi! Thank you for the contribution. I will take a look soon. A bit busy at my work 🙂 |
@skela thank you for all your work! But I found some issues, could you fix them?
|
Will fix asap, thx for taking a look ❤️ |
…th single quotes as opposed to quotes. also made sure it still works if test names have variable subs (dollar sign) formatters in em.
… that are marked as test (in particular, run All, and run previous relies on this allowing it)
…e to match using simple queries.
@quolpr ok i think i sorted out the issues u had now. ive never been able to get neotest to work for my tests in dart, so i was a bit cautious going all in on that solution, but i went with the best of both worlds, hope you dont mind. |
@skela will review hope this week, busy right now. Thanks for contribution! |
@skela works great! Thanks for the contribution 👍 |
Makes it so u can use quick test for dart/flutter tests.
It prob needs some configuration options to pass in which binary to use to run the tests, either dart, flutter or fwm flutter, but this is a good starting point.
Let me know if i need to change something.