From d3b510fa7d317b60b4ce5cd84a9cc21621543e3b Mon Sep 17 00:00:00 2001 From: Claude Date: Fri, 1 Aug 2025 01:56:34 -0400 Subject: [PATCH] Fix test assertion line splits MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fix unnecessarily split test assertions in default_algs.jl and basictests.jl following Catalyst.jl PR #1306 guidelines for improved readability. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- test/basictests.jl | 6 ++---- test/default_algs.jl | 3 +-- 2 files changed, 3 insertions(+), 6 deletions(-) diff --git a/test/basictests.jl b/test/basictests.jl index a0dafe1e8..433213423 100644 --- a/test/basictests.jl +++ b/test/basictests.jl @@ -507,10 +507,8 @@ end prob3 = LinearProblem(op1, b1; u0 = x1) prob4 = LinearProblem(op2, b2; u0 = x2) - @test LinearSolve.defaultalg(op1, x1).alg === - LinearSolve.DefaultAlgorithmChoice.DirectLdiv! - @test LinearSolve.defaultalg(op2, x2).alg === - LinearSolve.DefaultAlgorithmChoice.DirectLdiv! + @test LinearSolve.defaultalg(op1, x1).alg === LinearSolve.DefaultAlgorithmChoice.DirectLdiv! + @test LinearSolve.defaultalg(op2, x2).alg === LinearSolve.DefaultAlgorithmChoice.DirectLdiv! @test LinearSolve.defaultalg(op3, x1).alg === LinearSolve.DefaultAlgorithmChoice.KrylovJL_GMRES @test LinearSolve.defaultalg(op4, x2).alg === diff --git a/test/default_algs.jl b/test/default_algs.jl index 7d912ae13..0bc254834 100644 --- a/test/default_algs.jl +++ b/test/default_algs.jl @@ -1,7 +1,6 @@ using LinearSolve, RecursiveFactorization, LinearAlgebra, SparseArrays, Test -@test LinearSolve.defaultalg(nothing, zeros(3)).alg === - LinearSolve.DefaultAlgorithmChoice.GenericLUFactorization +@test LinearSolve.defaultalg(nothing, zeros(3)).alg === LinearSolve.DefaultAlgorithmChoice.GenericLUFactorization prob = LinearProblem(rand(3, 3), rand(3)) solve(prob)