Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[Backport stable/8.0] test: fix unfinished stubbing of command response writer #10638

Merged
merged 2 commits into from
Oct 7, 2022

Conversation

backport-action
Copy link
Collaborator

Description

Backport of #10605 to stable/8.0.

relates to #10604

The engine is shared across multiple tests and not stopped in-between.
With this change we only set up mocking of the response writer once and
not for every test method. This ensures that mocking does not interfere
with engine processing from previous tests which would show up as
unfinished mocking exceptions.

(cherry picked from commit 15a55cc)
(cherry picked from commit 7acd44d)
Copy link
Member

@lenaschoenburg lenaschoenburg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors r+

zeebe-bors-camunda bot added a commit that referenced this pull request Oct 7, 2022
10493: [Backport stable/8.0] fix(raft): handle exceptions on partition server init r=oleschoenburg a=megglos

# Description
Backport of #10450 to `stable/8.0`.

10566: [Backport stable/8.0] fix(helm): rename podSecurityContext to containerSecurityContext r=oleschoenburg a=backport-action

# Description
Backport of #10556 to `stable/8.0`.

relates to camunda/camunda-platform-helm#374

10624: [Backport stable/8.0] fix: take snapshot if nothing was exported since last snapshot r=oleschoenburg a=backport-action

# Description
Backport of #10611 to `stable/8.0`.

relates to #9761

10638: [Backport stable/8.0] test: fix unfinished stubbing of command response writer r=oleschoenburg a=backport-action

# Description
Backport of #10605 to `stable/8.0`.

relates to #10604

Co-authored-by: Meggle (Sebastian Bathke) <sebastian.bathke@camunda.com>
Co-authored-by: Ole Schönburg <ole.schoenburg@gmail.com>
@zeebe-bors-camunda
Copy link
Contributor

This PR was included in a batch that was canceled, it will be automatically retried

@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2022

Test Results

   865 files  ±  0     865 suites  ±0   1h 44m 10s ⏱️ +18s
6 154 tests +65  6 150 ✔️ +65  4 💤 ±0  0 ±0 
6 484 runs  +65  6 478 ✔️ +65  6 💤 ±0  0 ±0 

Results for commit 72a119e. ± Comparison against base commit ea42dac.

@zeebe-bors-camunda
Copy link
Contributor

Build succeeded:

@zeebe-bors-camunda zeebe-bors-camunda bot merged commit eb1d67f into stable/8.0 Oct 7, 2022
@zeebe-bors-camunda zeebe-bors-camunda bot deleted the backport-10605-to-stable/8.0 branch October 7, 2022 16:07
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.

None yet

2 participants