Humanizr / Humanizer

Humanizer meets all your .NET needs for manipulating and displaying strings, enums, dates, times, timespans, numbers and quantities
Other
8.52k stars 947 forks source link

Remove duplicate test case #1518

Open martincostello opened 4 weeks ago

martincostello commented 4 weeks ago

Fix xunit duplicate test case warnings (shown below) by removing duplicate test case.

[xUnit.net 00:00:07.28] Humanizer.Tests: Skipping test case with duplicate ID 'a707b471d6ae2cb7f4db4d76f96963a59c21dc32' ('Pluralize(singular: "database", plural: "databases")' and 'Pluralize(singular: "database", plural: "databases")')
[xUnit.net 00:00:07.28] Humanizer.Tests: Skipping test case with duplicate ID '5309b170f3f5a43a7bbcb0f5fe07b994c6620cfc' ('PluralizeWordsWithUnknownPlurality(singular: "database", plural: "databases")' and 'PluralizeWordsWithUnknownPlurality(singular: "database", plural: "databases")')
[xUnit.net 00:00:07.28] Humanizer.Tests: Skipping test case with duplicate ID '282086b076dda6e9d4498250b67110352f93dd43' ('Singularize(singular: "database", plural: "databases")' and 'Singularize(singular: "database", plural: "databases")')
[xUnit.net 00:00:07.28] Humanizer.Tests: Skipping test case with duplicate ID '82227a9842c246b76306b19f58c22f5163e256ff' ('SingularizeWordsWithUnknownSingularity(singular: "database", plural: "databases")' and 'SingularizeWordsWithUnknownSingularity(singular: "database", plural: "databases")')

I kept the other entry as the non-duplicate as it had an explicit comment for an issue on it:

https://github.com/Humanizr/Humanizer/blob/facf3fef383b430de49b67beb00f3b834d2086c9/src/Humanizer.Tests/InflectorTests.cs#L392-L393

Here is a checklist you should tick through before submitting a pull request: