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.

Sign up for GitHub

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

Jump to bottom

Add unit testing for SystemTestsCompareN #4194

Open
billsacks opened this issue Mar 11, 2022 · 5 comments
Open

Add unit testing for SystemTestsCompareN #4194

billsacks opened this issue Mar 11, 2022 · 5 comments
Labels
Low Priority tp: unit tests ty: enhancement

Comments

@billsacks
Copy link
Member

As discussed in #4169 , we should get unit tests in place for SystemTestsCompareN. See also some comments about unit tests in #3222 . We'd like to encourage migration from SystemTestsCompare2 to the more general SystemTestsCompareN, but I see having some unit tests in place as a prerequisite for encouraging that migration.

@alperaltuntas
@billsacks
Copy link
Member Author

(@alperaltuntas I don't mean to imply any urgency by opening this issue, I just want to track that this is still needed.)

@github-actions
Copy link
Contributor

github-actions bot commented May 3, 2023

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the Stale label May 3, 2023
@ekluzek ekluzek removed the Stale label May 3, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jun 3, 2023

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the Stale label Jun 3, 2023
@ekluzek ekluzek added Low Priority and removed Stale labels Jun 5, 2023
@ekluzek
Copy link
Contributor

ekluzek commented Jun 5, 2023

This does seem like something that would be worthwhile still.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 5, 2023

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the Stale label Jul 5, 2023
@billsacks billsacks removed the Stale label Jul 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Low Priority tp: unit tests ty: enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@billsacks @ekluzek