search

ScreenPyHQ / screenpy

Screenplay pattern base for Python automated UI test suites.
MIT License
27 stars 3 forks source link

#33: add plural handling for SeeAnyOf/SeeAllOf. #64

Closed perrygoy closed 1 year ago

perrygoy commented 1 year ago

Addressing #33.

This PR adds plural (and empty!) handling for SeeAnyOf and SeeAllOf. I thought i'd add some cheek if someone tries to do SeeAnyOf() or SeeAllOf() with no tests. It also adds tests for the describe method, which directly uses the new log_message attribute that handles logging the correct string.

bandophahita commented 1 year ago

I noticed marcel.shall(SeeAllOf()) will pass but marcel.shall(SeeAnyOf()) will fail. I would expect these two to be consistent.

bandophahita commented 1 year ago

I'm going to be touching See/any/all in the Quietly branch, so I can address the inconsistency there.