[#857] Aligned assertion and helper names with the documented naming rules. by AlexSkrypnyk · Pull Request #895 · drevops/behat-steps

[#857] Moved assertion qualifiers after the predicate so 'Not' sits directly after the subject.

AlexSkrypnyk

committed

1df412d

[#857] Kept 'Has' for what a subject holds and named compared values 'Equals' or 'Contains'.

AlexSkrypnyk

committed

8cb832a

[#857] Spelled existence 'Exists' and containment 'Contains' in assertion names.

AlexSkrypnyk

committed

c256d03

[#857] Named every assertion 'Assert' and renamed 'RuntimeException' preconditions away from it.

AlexSkrypnyk

committed

d8c50dd

[#857] Named every published helper with a verb, such as 'messageGetSelector()' and 'authIsLoggedIn()'.

AlexSkrypnyk

committed

09fd4c3

[#857] Dropped 'Clear' from the helper verb list, since only a hook uses it.

AlexSkrypnyk

committed

34cb9ee

[#857] Documented the assertion and helper verb rules, and listed the renames in 'MIGRATION.md'.

AlexSkrypnyk

committed

20c86f0

[#857] Stated the naming test reasons as plain cause and effect.

AlexSkrypnyk

committed

ec21bb9

[#857] Stated in 'CONTRIBUTING.md' what 'TraitMethodNamingTest' checks and what review still holds.

AlexSkrypnyk

committed

74e61cb

Navigation Menu

Repository navigation

[#857] Aligned assertion and helper names with the documented naming rules. - #895

[#857] Aligned assertion and helper names with the documented naming rules.#895
AlexSkrypnyk merged 9 commits into
4.xfrom
feature/857-align-method-names

Commits on Oct 1, 2026

[#857] Moved assertion qualifiers after the predicate so 'Not' sits directly after the subject.

[#857] Kept 'Has' for what a subject holds and named compared values 'Equals' or 'Contains'.

[#857] Spelled existence 'Exists' and containment 'Contains' in assertion names.

[#857] Named every assertion 'Assert' and renamed 'RuntimeException' preconditions away from it.

[#857] Named every published helper with a verb, such as 'messageGetSelector()' and 'authIsLoggedIn()'.

[#857] Dropped 'Clear' from the helper verb list, since only a hook uses it.

[#857] Documented the assertion and helper verb rules, and listed the renames in 'MIGRATION.md'.

[#857] Stated the naming test reasons as plain cause and effect.

[#857] Stated in 'CONTRIBUTING.md' what 'TraitMethodNamingTest' checks and what review still holds.