Include "the" in the name of Targets #3679
john-phillips-uspto
started this conversation in
Ideas
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
When using net.serenitybdd.screenplay.targets.Target, a new Target is made by calling Target.the("header").locatedBy("#header").
In the resulting report only the exact name of the Target is recorded (i.e "she sees header that is displayed").
Would there be any utility to/issues with adding a "the" in front of the name so that the step reads "she sees the header that is displayed"?
It would require a little bit of care to make sure that we don't end up with "the the the header" when calling various Target methods, but aside from that it should be doable.
I can see this being an issue if people are already writing their Target names to account for the lack of "the". Do we have a sense of what a "normal" naming convention is for Targets?
Beta Was this translation helpful? Give feedback.
All reactions