Skip to content

Refactor unit tests to TUnit framework#19

Merged
jozefizso merged 2 commits into
mainfrom
tests/tunit
May 23, 2026
Merged

Refactor unit tests to TUnit framework#19
jozefizso merged 2 commits into
mainfrom
tests/tunit

Conversation

@jozefizso

@jozefizso jozefizso commented May 23, 2026

Copy link
Copy Markdown
Member

TUnit improves the performance a lot:

dotnet test

Test run summary: Passed!
  total: 39
  failed: 0
  succeeded: 36
  skipped: 3
  duration: 1s 895ms

Former xunit tests:

dotnet test

[xUnit.net 00:00:02.31]   Finished:    Kavod.Vba.Compression.Tests
  Kavod.Vba.Compression.Tests test net10.0 succeeded (3.0s)
[xUnit.net 00:00:02.43]   Finished:    Kavod.Vba.Compression.Tests
  Kavod.Vba.Compression.Tests test net8.0 succeeded (3.2s)

Test summary: total: 78, failed: 0, succeeded: 72, skipped: 6, duration: 3.2s
Build succeeded in 4.4s

@jozefizso jozefizso self-assigned this May 23, 2026
@jozefizso jozefizso added the chore A routine task label May 23, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 33d71136e3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/Directory.Build.props
@jozefizso jozefizso merged commit af50ac6 into main May 23, 2026
8 checks passed
@jozefizso jozefizso deleted the tests/tunit branch May 23, 2026 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore A routine task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant