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

Carve out Ada selftests in alr to its own Ada testsuite #1893

Merged
merged 8 commits into from
Mar 17, 2025

Conversation

mosteo
Copy link
Member

@mosteo mosteo commented Mar 12, 2025

Take advantage of the new alr test to remove test code from Alire itself.

Incidentally, I think the test named Assertions_Enabled would make for a better default test instead of the empty example we are generating now. This one will likely be left in by most people as it guarantees the testsuite is build in validation mode, whereas the current example requires removal or renaming and does nothing useful.

PR creation checklist
  • A test is included, if required by the changes.

@mosteo mosteo force-pushed the feat/ada-testsuite branch from f7f7f66 to 61f5cfd Compare March 12, 2025 09:32
@mosteo mosteo force-pushed the feat/ada-testsuite branch 2 times, most recently from 2c636e7 to d714094 Compare March 12, 2025 22:25
@mosteo mosteo marked this pull request as ready for review March 12, 2025 22:54
@mosteo mosteo requested a review from Fabien-Chouteau March 12, 2025 22:54
@mosteo mosteo merged commit 8d5ce2f into alire-project:master Mar 17, 2025
34 checks passed
@mosteo mosteo deleted the feat/ada-testsuite branch March 17, 2025 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants