Skip to content

Commit e27e6c1

Browse files
mtfishmanclaude
andcommitted
Hoist long broken = ... expression onto its own line
Define the LQ-regression `broken` predicate locally and pass `broken = broken` to `@test`, so the assertion line stays readable without a multi-line `broken = (...)` argument. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent d0464e5 commit e27e6c1

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

test/test_abstract_blocktype.jl

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -99,10 +99,10 @@ arrayts = (Array, JLArray)
9999
# `BlockSparseMatrix{T, AbstractMatrix{T}}` produce a `c * u` that does not
100100
# reproduce `a` (regression in MatrixAlgebraKit 0.6.6; fix tracked at
101101
# https://github.com/QuantumKitHub/MatrixAlgebraKit.jl/pull/219).
102-
@test c * u a broken = (
102+
broken =
103103
arrayt Array && f (right_orth, lq_compact, lq_full) &&
104-
pkgversion(MatrixAlgebraKit) < v"0.6.7"
105-
)
104+
pkgversion(MatrixAlgebraKit) < v"0.6.7"
105+
@test c * u a broken = broken
106106
# TODO: Fix comparison with UniformScaling on GPU.
107107
@test isisometric(u; side = :right) broken = arrayt Array
108108
end

0 commit comments

Comments
 (0)