Skip to content

Commit 51ccbcc

Browse files
committed
test: reuse dictionary fuzz fixtures without method overwrites
1 parent 51d806d commit 51ccbcc

1 file changed

Lines changed: 10 additions & 4 deletions

File tree

‎test/ipc_dictionary_tests.jl‎

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,13 @@ using Test, Arrow, Tables
2020
const AC = Arrow.AC
2121
const Meta = Arrow.Meta
2222
const FB = Arrow.FB
23-
include("support/SeededFuzz.jl")
23+
# Reuse the property suite's module in the full run: loading it twice
24+
# overwrites its global ArrowTypes extension registrations.
25+
const Fuzz = if isdefined(parentmodule(@__MODULE__), :PropertyTests)
26+
getfield(getfield(parentmodule(@__MODULE__), :PropertyTests), :SeededFuzz)
27+
else
28+
include("support/SeededFuzz.jl")
29+
end
2430

2531
# The normal suite consumes independently produced PyArrow 25.0.1 bytes.
2632
# Regenerate with: python test/support/generate_dictionary_fixtures.py
@@ -255,9 +261,9 @@ end
255261

256262
@testset "Physical dictionary delta layouts" begin
257263
for case in (
258-
SeededFuzz._logical_layout_case(),
259-
SeededFuzz._physical_layout_case(),
260-
SeededFuzz._union_layout_case(),
264+
Fuzz._logical_layout_case(),
265+
Fuzz._physical_layout_case(),
266+
Fuzz._union_layout_case(),
261267
)
262268
for (vf, pool) in zip(case.schema.fields, case.batches[1].columns)
263269
vf.type isa AC.DictionaryType && continue

0 commit comments

Comments
 (0)