Skip to content

[2.10] box: cover box_wait_limbo_acked#9327

Merged
sergepetrenko merged 1 commit intotarantool:release/2.10from
sergepetrenko:wait-limbo-acked-test-2.10
Nov 3, 2023
Merged

[2.10] box: cover box_wait_limbo_acked#9327
sergepetrenko merged 1 commit intotarantool:release/2.10from
sergepetrenko:wait-limbo-acked-test-2.10

Conversation

@sergepetrenko
Copy link
Collaborator

@sergepetrenko sergepetrenko commented Nov 2, 2023

Prior to this patch, there were many possible code execution options
that were not covered by tests. After this commit, any assert(false)
inside box_wait_limbo_acked cause a crash.

Closes #7318

NO_DOC=test
NO_CHANGELOG=test

(cherry picked from commit 7fce5be)

@coveralls
Copy link

coveralls commented Nov 2, 2023

Coverage Status

coverage: 85.233% (-0.006%) from 85.239%
when pulling 1fc0fd0 on sergepetrenko:wait-limbo-acked-test-2.10
into ba249cc
on tarantool:release/2.10
.

Prior to this patch, there were many possible code execution options
that were not covered by tests. After this commit, any assert(false)
inside box_wait_limbo_acked cause a crash.

Closes tarantool#7318

NO_DOC=test
NO_CHANGELOG=test

(cherry picked from commit 7fce5be)
@sergepetrenko sergepetrenko force-pushed the wait-limbo-acked-test-2.10 branch from b78f3d3 to 1fc0fd0 Compare November 3, 2023 09:40
@sergepetrenko sergepetrenko marked this pull request as ready for review November 3, 2023 09:54
@sergepetrenko sergepetrenko requested a review from a team as a code owner November 3, 2023 09:54
@sergepetrenko sergepetrenko added the full-ci Enables all tests for a pull request label Nov 3, 2023
@sergepetrenko sergepetrenko merged commit 93f6399 into tarantool:release/2.10 Nov 3, 2023
@sergepetrenko sergepetrenko deleted the wait-limbo-acked-test-2.10 branch December 2, 2023 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

full-ci Enables all tests for a pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants