Skip to content

Commit 8958988

Browse files
committed
Fix realizability test: correct expected return value counts (2D returns 5, 3D returns 29)
1 parent 05932b5 commit 8958988

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

test/test_realizability.jl

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -80,13 +80,13 @@ const TOL = 1e-10
8080
0.0, 3.0, 0.0, 0.0, 0.0, 0.0, 1.0, 0.0, 0.0, 3.0)
8181
@test all(isfinite.([S210r2, S120r2, S310r2, S220r2, S130r2]))
8282

83-
# Test realizable_2D directly (10 parameters, no H200/H020)
83+
# Test realizable_2D directly (takes 10 parameters, returns 5: S21, S12, S31, S22, S13)
8484
result = HyQMOM.realizable_2D(
8585
S300, S400, S110, S210, S310, S120, S220,
8686
S030, S130, S040
8787
)
8888

89-
@test length(result) == 10
89+
@test length(result) == 5
9090
@test all(isfinite, result)
9191
end
9292

@@ -145,7 +145,8 @@ const TOL = 1e-10
145145
S211, S021, S121, S031, S012, S112, S013, S022
146146
)
147147

148-
@test length(result) == 22
148+
# Returns 29 values: all 28 standardized moments + flag220
149+
@test length(result) == 29
149150
@test all(isfinite, result)
150151

151152
# Test with some correlations
@@ -158,7 +159,7 @@ const TOL = 1e-10
158159
S211, S021, S121, S031, S012, S112, S013, S022
159160
)
160161

161-
@test length(result2) == 22
162+
@test length(result2) == 29
162163
@test all(isfinite, result2)
163164
end
164165

0 commit comments

Comments
 (0)