Skip to content

Refactor/v11 push test case ready message earlier#1864

Open
luke-hill wants to merge 3 commits intov11/remove_fake_queryfrom
refactor/v11-push-test-case-ready-message-earlier
Open

Refactor/v11 push test case ready message earlier#1864
luke-hill wants to merge 3 commits intov11/remove_fake_queryfrom
refactor/v11-push-test-case-ready-message-earlier

Conversation

@luke-hill
Copy link
Copy Markdown
Contributor

Description

This attempts to bring the test case event message earlier

This breaks a couple of feature tests regarding event ordering and NME's being generated for missing ids

Type of change

Please delete options that are not relevant.

  • Refactoring (improvements to code design or tooling that don't change behaviour)

Please add an entry to the relevant section of CHANGELOG.md as part of this pull request.

Checklist:

Your PR is ready for review once the following checklist is
complete. You can also add some checks if you want to.

  • Tests have been added for any changes to behaviour of the code
  • New and existing tests are passing locally and on CI
  • bundle exec rubocop reports no offenses
  • RDoc comments have been updated
  • CHANGELOG.md has been updated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant