Remove test_fixup_256, which cannot detect the bug
CI / test (arm64, -DCMAKE_BUILD_TYPE=Debug -DMSAN_TOOLCHAIN_PATH=/opt/msan, 22, https://minio.weaselab.dev/public/aarch64/msan-toolchain-22.1.8.tar.zst, debug-arm64, ubuntu-latest-arm64) (pull_request) Successful in 3m43s
CI / release (arm64, , ubuntu-latest-arm64) (pull_request) Successful in 3m24s
CI / pre-commit (pull_request) Successful in 2m24s
CI / test (amd64, -DCMAKE_BUILD_TYPE=Debug -DMSAN_TOOLCHAIN_PATH=/opt/msan, 21, https://minio.weaselab.dev/public/x86_64/msan-toolchain-21.1.8.tar.zst, debug, ubuntu-latest-amd64) (pull_request) Successful in 4m12s
CI / test (amd64, -DCMAKE_CXX_FLAGS=-DUSE_64_BIT=1, 21, , 64-bit-versions, ubuntu-latest-amd64) (pull_request) Successful in 3m28s
CI / test (amd64, -DCMAKE_C_COMPILER=gcc -DCMAKE_CXX_COMPILER=g++, 21, , gcc, ubuntu-latest-amd64) (pull_request) Successful in 7m40s
CI / test (amd64, -DUSE_SIMD_FALLBACK=ON, 21, , simd-fallback, ubuntu-latest-amd64) (pull_request) Successful in 4m14s
CI / release (amd64, -DMSAN_TOOLCHAIN_PATH=/opt/msan, ubuntu-latest-amd64) (pull_request) Successful in 7m11s
CI / coverage (pull_request) Successful in 3m57s
CI / test (arm64, -DCMAKE_BUILD_TYPE=Debug -DMSAN_TOOLCHAIN_PATH=/opt/msan, 22, https://minio.weaselab.dev/public/aarch64/msan-toolchain-22.1.8.tar.zst, debug-arm64, ubuntu-latest-arm64) (pull_request) Successful in 3m43s
CI / release (arm64, , ubuntu-latest-arm64) (pull_request) Successful in 3m24s
CI / pre-commit (pull_request) Successful in 2m24s
CI / test (amd64, -DCMAKE_BUILD_TYPE=Debug -DMSAN_TOOLCHAIN_PATH=/opt/msan, 21, https://minio.weaselab.dev/public/x86_64/msan-toolchain-21.1.8.tar.zst, debug, ubuntu-latest-amd64) (pull_request) Successful in 4m12s
CI / test (amd64, -DCMAKE_CXX_FLAGS=-DUSE_64_BIT=1, 21, , 64-bit-versions, ubuntu-latest-amd64) (pull_request) Successful in 3m28s
CI / test (amd64, -DCMAKE_C_COMPILER=gcc -DCMAKE_CXX_COMPILER=g++, 21, , gcc, ubuntu-latest-amd64) (pull_request) Successful in 7m40s
CI / test (amd64, -DUSE_SIMD_FALLBACK=ON, 21, , simd-fallback, ubuntu-latest-amd64) (pull_request) Successful in 4m14s
CI / release (amd64, -DMSAN_TOOLCHAIN_PATH=/opt/msan, ubuntu-latest-amd64) (pull_request) Successful in 7m11s
CI / coverage (pull_request) Successful in 3m57s
Every child of the Node256 in test_fixup_256 is written at the same version, so max(childMaxVersion[0..15]) and max(maxOfMax) agree and the buggy code passes. test_fixup_256_heterogeneous_versions covers the same Node256-under-a-range-end shape with distinct versions, so keep only that.
This commit is contained in:
1 parent
3d3184ef5f
commit
c94e9aaee6
1 file changed
-12
@@ -222,18 +222,6 @@ def test_merge_child_node48():
|
|||||||
cs.addWrites(1, write(b"\x00" * 8, b"\x00" * 10))
|
cs.addWrites(1, write(b"\x00" * 8, b"\x00" * 10))
|
||||||
|
|
||||||
|
|
||||||
def test_fixup_256():
|
|
||||||
# All children are written at the same version, so this test passes even
|
|
||||||
# if fixupMaxVersion only considers some of them. See
|
|
||||||
# test_fixup_256_heterogeneous_versions for that case.
|
|
||||||
with DebugConflictSet() as cs:
|
|
||||||
cs.addWrites(0, write(bytes([1])))
|
|
||||||
for i in range(256):
|
|
||||||
cs.addWrites(1, write(bytes([1, i])))
|
|
||||||
cs.addWrites(2, write(bytes([0]), bytes([1])))
|
|
||||||
cs.check(read(0, bytes([1]), bytes([2])))
|
|
||||||
|
|
||||||
|
|
||||||
def test_fixup_256_heterogeneous_versions():
|
def test_fixup_256_heterogeneous_versions():
|
||||||
# Regression test for issue #89: recomputing the maxVersion of a Node256
|
# Regression test for issue #89: recomputing the maxVersion of a Node256
|
||||||
# must consider all of its children, not just those with an index < 16.
|
# must consider all of its children, not just those with an index < 16.
|
||||||
|
|||||||
Reference in new issue
Block a user