Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add step to assert a field is not present #1554

Merged
merged 3 commits into from
Mar 11, 2024

Conversation

antonio-ramadas-dd
Copy link
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 antonio-ramadas-dd marked this pull request as ready for review March 11, 2024 14:43
@antonio-ramadas-dd antonio-ramadas-dd requested a review from a team as a code owner March 11, 2024 14:43
@antonio-ramadas-dd
Copy link
Contributor Author

/merge

@dd-devflow
Copy link

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

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 457efca into master Mar 11, 2024
14 checks passed
@skarimo skarimo deleted the antonio.ramadas/add-step-definition branch March 11, 2024 19:11
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> 457efca
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants