-
Notifications
You must be signed in to change notification settings - Fork 62
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
Add Support for xunit.v3 #339
base: main
Are you sure you want to change the base?
Conversation
Add support for xunit.v3 Signed-off-by: Jonas Bengtsson <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution. Could you add a test project similar to the ones for MSTestV2 und NUnit?
<PackageTags>test;arch;archunit;xunit</PackageTags> | ||
<IncludeSource>False</IncludeSource> | ||
<Company>TNG Technology Consulting GmbH</Company> | ||
<PackageId>TngTech.ArchUnitNET.xUnit.v3</PackageId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<PackageId>TngTech.ArchUnitNET.xUnit.v3</PackageId> | |
<PackageId>TngTech.ArchUnitNET.xUnitV3</PackageId> |
Change the package name to be similar to the existing MSTest package
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have added two test projects:
TngTech.ArchUnitNET.xUnitTests
TngTech.ArchUnitNET.xUnitV3Tests
Previously there were no test project for TngTech.ArchUnitNET.xUnit
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, xUnit didn't need additional tests cause ArchUnitNETTests uses xUnit, so it's tested through that project. But probably doesn't hurt to have them 👍
Co-authored-by: Alexander Linne <[email protected]> Signed-off-by: Jonas Bengtsson <[email protected]>
Signed-off-by: Jonas Bengtsson <[email protected]>
Signed-off-by: Jonas Bengtsson <[email protected]>
Signed-off-by: Jonas Bengtsson <[email protected]>
Add support for xunit.v3
Fixes: #338