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

.NET8 SIL.Core.Tests #1348

Merged
merged 4 commits into from
Oct 17, 2024
Merged

.NET8 SIL.Core.Tests #1348

merged 4 commits into from
Oct 17, 2024

Conversation

josephmyers
Copy link
Collaborator

@josephmyers josephmyers commented Oct 3, 2024

This change is Reviewable

Copy link

github-actions bot commented Oct 3, 2024

LibPalaso Tests

    38 files  +    1      38 suites  +1   11m 3s ⏱️ + 1m 30s
 4 828 tests ±    0   4 597 ✅ ±    0  231 💤 ± 0  0 ❌ ±0 
12 640 runs  +1 447  12 040 ✅ +1 378  600 💤 +69  0 ❌ ±0 

Results for commit e94a8b3. ± Comparison against base commit 17401d1.

♻️ This comment has been updated with latest results.

Abort() throws an exception immediately (and is generally considered not recommended), where Interrupt() will/should end the thread when it reaches the WaitSleepJoin state. There may be some subtle differences here, but all its tests continue to pass for all three frameworks.

Also fixed corresponding tests that were failing with a runtime exception on .NET8
@josephmyers josephmyers marked this pull request as ready for review October 11, 2024 09:16
@josephmyers josephmyers requested a review from hahn-kev October 11, 2024 09:16
Copy link
Contributor

@hahn-kev hahn-kev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me.

One thing that might be nice is to make SIL.Core.Tests reference the CommandLineRunner test app project, then it'll always be built before the tests run.

@josephmyers josephmyers merged commit 3f8fe62 into master Oct 17, 2024
1 of 2 checks passed
@josephmyers josephmyers deleted the net8-core branch October 17, 2024 06:04
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.

3 participants