From c8abf2fb703a7d5b149a0508b65266bdccdb670e Mon Sep 17 00:00:00 2001
From: Connor Ward <c.ward20@imperial.ac.uk>
Date: Thu, 5 Dec 2024 09:55:44 +0000
Subject: [PATCH] Apply suggestions from code review

Co-authored-by: Josh Hope-Collins <joshua.hope-collins13@imperial.ac.uk>
Co-authored-by: David A. Ham <david.ham@imperial.ac.uk>
---
 tests/firedrake/regression/test_dg_advection.py | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/tests/firedrake/regression/test_dg_advection.py b/tests/firedrake/regression/test_dg_advection.py
index 0949937d3e..0559f2f159 100644
--- a/tests/firedrake/regression/test_dg_advection.py
+++ b/tests/firedrake/regression/test_dg_advection.py
@@ -38,13 +38,14 @@ def run_test(mesh):
     t = 0.0
     T = 10*dt
 
-    problem = LinearVariationalProblem(a_mass, action(arhs, D1), dD1)
+    problem = LinearVariationalProblem(a_mass, action(arhs, D1), dD1,
+                                       constant_jacobian=True)
     solver = LinearVariationalSolver(
         problem,
         solver_parameters={
-            'ksp_type': 'cg',
-            # specify superlu_dist as MUMPS fails in parallel on MacOS
-            'pc_factor_mat_solver_type': 'superlu_dist',
+            'ksp_type': 'preonly',
+            'pc_type': 'bjacobi',
+            'sub_pc_type': 'ilu'
         }
     )