Skip to content

Add step to assert a field is not present#1760

Merged
skarimo merged 3 commits intomasterfrom
antonio.ramadas/add-step-definition
Mar 11, 2024
Merged

Add step to assert a field is not present#1760
skarimo merged 3 commits intomasterfrom
antonio.ramadas/add-step-definition

Conversation

@antonio-ramadas-dd
Copy link
Copy Markdown
Contributor

What does this PR do?

Add new step to assert on Cucumber that a field is missing. This allows us to be more comprehensive when writing tests.

Review checklist

Please check relevant items below:

  • This PR includes all newly recorded cassettes for any modified tests.

  • This PR does not rely on API client schema changes.

    • The CI should be fully passing.
  • Or, this PR relies on API schema changes and this is a Draft PR to include tests for that new functionality.

    • Note: CI shouldn't be run on this Draft PR, as its expected to fail without the corresponding schema changes.

@antonio-ramadas-dd
Copy link
Copy Markdown
Contributor Author

/merge

@dd-devflow
Copy link
Copy Markdown

dd-devflow bot commented Mar 11, 2024

🚂 MergeQueue

Pull request added to the queue.

This build is going to start soon! (estimated merge in less than 0s)

Use /merge -c to cancel this operation!

@dd-devflow
Copy link
Copy Markdown

dd-devflow bot commented Mar 11, 2024

🚨 MergeQueue

Gitlab pipeline didn't start on its own and we were unable to create it... Please retry.

If you need support, contact us on Slack #ci-interfaces with those details!

@skarimo skarimo merged commit c85ac3c into master Mar 11, 2024
@skarimo skarimo deleted the antonio.ramadas/add-step-definition branch March 11, 2024 19:10
github-actions bot pushed a commit that referenced this pull request Mar 11, 2024
* Add step to assert a field is not present

* Fix method naming

---------

Co-authored-by: skarimo <40482491+skarimo@users.noreply.github.com> c85ac3c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants