volodya-lombrozo / jtcop

Maven Plugin for checking tests in Java projects
MIT License
18 stars 2 forks source link

feat(#248): start with requirements in disabled test #431

Closed h1alexbel closed 5 days ago

h1alexbel commented 1 week ago

In this pull I've created unit test as requirements start for #248.

related to #248


PR-Codex overview

This PR adds a new test case passesOnPluralWithSubjectAtTheBeginning to RulePresentSimpleMlTest for plural subject at the beginning, using @CsvSource and @ParameterizedTest.

Detailed summary

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

h1alexbel commented 1 week ago

@volodya-lombrozo please take a look if I'm understand requirements for valid test method name with present tense and plural correctly

h1alexbel commented 5 days ago

@volodya-lombrozo updated pull request. Take a look again, please

volodya-lombrozo commented 5 days ago

@rultor merge

rultor commented 5 days ago

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

rultor commented 5 days ago

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 18min)