Skip to content

test: name the data-driven enum value tests after TUnit arguments - #1874

Open
vbreuss wants to merge 1 commit into
mainfrom
test/rename-inline-data-tests
Open

vbreuss wants to merge 1 commit into
mainfrom
test/rename-inline-data-tests

Conversation

@vbreuss

@vbreuss vbreuss commented Oct 18, 2026

Copy link
Copy Markdown
Member

Two ThatEnum.HasValue tests still referred to xUnit's [InlineData], although they have been driven by TUnit's [Arguments] since the migration to TUnit.

Changes

  • WhenExpectedComesFromInlineData_ShouldSucceed and WhenUnsignedExpectedComesFromInlineData_ShouldSucceed are renamed to ...FromAttributeArguments...; the alphabetical order within the class is unchanged.

Verification

  • ThatEnum+HasValue+Tests runs 16 tests on net48 and net8.0, all passing, as before.

Two `ThatEnum.HasValue` tests still referred to xUnit's `[InlineData]`, although they have been driven by TUnit's `[Arguments]` since the migration to TUnit.

## Changes

- `WhenExpectedComesFromInlineData_ShouldSucceed` and `WhenUnsignedExpectedComesFromInlineData_ShouldSucceed` are renamed to `...FromAttributeArguments...`; the alphabetical order within the class is unchanged.

## Verification

- `ThatEnum+HasValue+Tests` runs 16 tests on net48 and net8.0, all passing, as before.
@vbreuss vbreuss self-assigned this Oct 18, 2026
@vbreuss
vbreuss enabled auto-merge (squash) October 18, 2026 15:44
@sonarqubecloud

Copy link
Copy Markdown

This branch has not been deployed

No deployments
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.

1 participant