changeset 1105:0ba4609605d4 refactor/sbpoperators/inflation

REVIEW: Suggest changes to tests
author Vidar Stiernström <vidar.stiernstrom@it.uu.se>
date Fri, 20 May 2022 11:31:20 +0200
parents c0ab81e4c39c
children f80e69b0566b
files test/SbpOperators/boundaryops/boundary_operator_test.jl test/SbpOperators/volumeops/volume_operator_test.jl
diffstat 2 files changed, 4 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/test/SbpOperators/boundaryops/boundary_operator_test.jl	Fri May 20 11:30:46 2022 +0200
+++ b/test/SbpOperators/boundaryops/boundary_operator_test.jl	Fri May 20 11:31:20 2022 +0200
@@ -7,6 +7,7 @@
 import Sbplib.SbpOperators.Stencil
 import Sbplib.SbpOperators.BoundaryOperator
 
+# REVIEW: Remove the commented tests. It is tested in the user code.
 # TODO: What should happen to all the commented tests? Deleted? Replicated for user code?
 
 @testset "BoundaryOperator" begin
--- a/test/SbpOperators/volumeops/volume_operator_test.jl	Fri May 20 11:30:46 2022 +0200
+++ b/test/SbpOperators/volumeops/volume_operator_test.jl	Fri May 20 11:31:20 2022 +0200
@@ -10,6 +10,9 @@
 import Sbplib.SbpOperators.odd
 import Sbplib.SbpOperators.even
 
+# REVIEW: Remove the commented tests for 2D (it is tested in the user code), but
+# change testset Regions and testset Inferred, to test the 1D operators.
+
 @testset "VolumeOperator" begin
     inner_stencil = CenteredStencil(1/4, 2/4, 1/4)
     closure_stencils = (Stencil(1/2, 1/2; center=1), Stencil(0.,1.; center=2))