Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

docs: Update README examples to use TUnit syntax instead of NUnit - #849

#849
Merged
samtrion merged 2 commits into
feature/tunit-skiponfailuredailydevops/extensions.test:feature/tunit-skiponfailurefrom
copilot/sub-pr-846-another-onedailydevops/extensions.test:copilot/sub-pr-846-another-oneCopy head branch name to clipboard
Nov 24, 2025
Merged

docs: Update README examples to use TUnit syntax instead of NUnit#849
samtrion merged 2 commits into
feature/tunit-skiponfailuredailydevops/extensions.test:feature/tunit-skiponfailurefrom
copilot/sub-pr-846-another-onedailydevops/extensions.test:copilot/sub-pr-846-another-oneCopy head branch name to clipboard

Conversation

Copilot AI commented Nov 24, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

What does this PR do?

Updates code examples in src/NetEvolve.Extensions.TUnit/README.md (lines 212-265) to use TUnit syntax instead of NUnit patterns:

  • Removed [TestFixture] attributes from class declarations
  • Converted test methods from void to async Task
  • Replaced NUnit assertion pattern (Assert.That(x, Is.True)) with TUnit pattern (await Assert.That(x).IsTrue())
  • Assertions updated: .IsTrue(), .IsLessThan()

Before:

[TestFixture]
public class FlakyTests
{
    [Test]
    public void Should_Handle_External_Service_Timeout()
    {
        var service = new ExternalService();
        Assert.That(service.IsAvailable(), Is.True);
    }
}

After:

public class FlakyTests
{
    [Test]
    public async Task Should_Handle_External_Service_Timeout()
    {
        var service = new ExternalService();
        await Assert.That(service.IsAvailable()).IsTrue();
    }
}

Why is it important?

Documentation examples were showing NUnit syntax for a TUnit library, creating confusion. Examples now align with actual test implementation in SkipOnFailureAttributeTests.cs.

Related issues

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI mentioned this pull request Nov 24, 2025
Merged
@coderabbitai

coderabbitai Bot commented Nov 24, 2025 •
edited
Loading

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • state:ready for merge

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch copilot/sub-pr-846-another-one

Tip

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

  • Provide your own instructions using the high_level_summary_instructions setting.
  • Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
  • Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example instruction:

"Divide the high-level summary into five sections:

  1. 📝 Description — Summarize the main change in 50–60 words, explaining what was done.
  2. 📓 References — List relevant issues, discussions, documentation, or related PRs.
  3. 📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
  4. 📊 Contributor Summary — Include a Markdown table showing contributions:
    | Contributor | Lines Added | Lines Removed | Files Changed |
  5. ✔️ Additional Notes — Add any extra reviewer context.
    Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.

Comment @coderabbitai help to get the list of available commands and usage tips.

@samtrion
docs: Update README examples with TUnit syntax
795ea4f 
Co-authored-by: samtrion <3283596+samtrion@users.noreply.github.com>
Copilot AI changed the title [WIP] Update README.md with TUnit syntax examples docs: Update README examples to use TUnit syntax instead of NUnit Nov 24, 2025
Copilot AI requested a review from samtrion November 24, 2025 11:24
@samtrion
samtrion marked this pull request as ready for review November 24, 2025 11:25
@samtrion
samtrion requested a review from a team as a code owner November 24, 2025 11:25
@samtrion
samtrion merged commit 4a7b058 into feature/tunit-skiponfailure Nov 24, 2025
3 checks passed
@samtrion
samtrion deleted the copilot/sub-pr-846-another-one branch November 24, 2025 11:25
samtrion added a commit that referenced this pull request Nov 24, 2025
@samtrion
feat: TUnit SkipOnFailureAttribute added (#846)
a0e725f 
* feat: TUnit `SkipOnFailureAttribute` added

* test: Updated PublicAPI definition

* docs: Update README examples to use TUnit syntax instead of NUnit (#849)

* Initial plan

* docs: Update README examples with TUnit syntax

Co-authored-by: samtrion <3283596+samtrion@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: samtrion <3283596+samtrion@users.noreply.github.com>

* refactor: SkipOnFailureAttribute tests to ensure deterministic behavior (#847)

* Initial plan

* refactor: Replace non-deterministic RNG test with deterministic failing test

Co-authored-by: samtrion <3283596+samtrion@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: samtrion <3283596+samtrion@users.noreply.github.com>

* fix(test): Added missing PublicAPI Definitions

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@samtrion samtrion Awaiting requested review from samtrion

Assignees

@samtrion samtrion

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@samtrion
Morty Proxy This is a proxified and sanitized view of the page, visit original site.