You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Whenever a list of messages is checked in MessageContext, the step definitions expect to be passed a table of messages with a header indicating the message type, such as error messages:
Then I should see the following warning messages:
| warning messages |
| You are viewing the published version. To view the latest draft version, click here. |
When this header is not present the messages will not be checked. Currently no warning is being emitted when this happens, so when a developer uses the following table the test will be silently skipped:
Then I should see the following warning messages:
| You are viewing the published version. To view the latest draft version, click here. |
It is likely that developers miss this sometimes in their tests since it is not documented that the header is required, and is only apparent when reading the code. We recently discovered this and did a check in our existing test suite and found that in numerous occasions the header was missed: ec-europa/joinup-dev#1388
It would be nice if we could improve the documentation to clearly indicate that this header is required, and also fail the test if the header is missing.
The text was updated successfully, but these errors were encountered:
Whenever a list of messages is checked in
MessageContext
, the step definitions expect to be passed a table of messages with a header indicating the message type, such aserror messages
:When this header is not present the messages will not be checked. Currently no warning is being emitted when this happens, so when a developer uses the following table the test will be silently skipped:
It is likely that developers miss this sometimes in their tests since it is not documented that the header is required, and is only apparent when reading the code. We recently discovered this and did a check in our existing test suite and found that in numerous occasions the header was missed: ec-europa/joinup-dev#1388
It would be nice if we could improve the documentation to clearly indicate that this header is required, and also fail the test if the header is missing.
The text was updated successfully, but these errors were encountered: