Skip to content
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

Organize test suites in swayfmt #2494

Open
8 of 17 tasks
eureka-cpu opened this issue Aug 9, 2022 · 2 comments
Open
8 of 17 tasks

Organize test suites in swayfmt #2494

eureka-cpu opened this issue Aug 9, 2022 · 2 comments
Labels
forc formatter good first issue Good for newcomers testing General testing
Milestone

Comments

@eureka-cpu
Copy link
Contributor

eureka-cpu commented Aug 9, 2022

Edit: Extending this to re-organizing all tests in swayfmt

@eureka-cpu eureka-cpu added good first issue Good for newcomers forc testing General testing formatter labels Aug 9, 2022
@eureka-cpu eureka-cpu added this to the swayfmt-v2 milestone Aug 9, 2022
@eureka-cpu eureka-cpu changed the title Break current tests up, and add remaining tests for expr in sfv2 Organize test suites in swayfmt Oct 1, 2022
@cr-fuel cr-fuel self-assigned this Sep 19, 2023
@eureka-cpu
Copy link
Contributor Author

Lower priority but tests that are in swayfmt/tests/mod.rs need to be broken up into their respective item directories if they are only testing individual formatting of items.

@JoshuaBatty JoshuaBatty assigned eureka-cpu and unassigned cr-fuel Sep 29, 2023
@eureka-cpu eureka-cpu removed their assignment Nov 1, 2023
@JoshuaBatty
Copy link
Member

I'm not sure how much of this is still relevant. A once over cleaning up the test infra for sway-fmt while checking that all types are covered in the test suite is probably a good idea to do still.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
forc formatter good first issue Good for newcomers testing General testing
Projects
None yet
Development

No branches or pull requests

3 participants