Skip to content

Commit 9e746c7

Browse files
committed
chore(test_result_t): fix skipped-test indentation
1 parent 155b554 commit 9e746c7

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

src/julienne/julienne_test_diagnosis_s.F90

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,8 @@
158158

159159
module procedure within_real
160160

161-
if (abs(operands%actual - operands%expected) < tolerance) then
161+
if (abs(operands%actual - operands%expected) <= tolerance) then
162+
! We use <= to allow for tolerance=0, which could never be satisfied if we used < instead:
162163
test_diagnosis = test_diagnosis_t(test_passed=.true., diagnostics_string="")
163164
else
164165
test_diagnosis = test_diagnosis_t(test_passed=.false. &
@@ -172,7 +173,8 @@
172173

173174
module procedure within_double_precision
174175

175-
if (abs(operands%actual - operands%expected) < tolerance) then
176+
if (abs(operands%actual - operands%expected) <= tolerance) then
177+
! We use <= to allow for tolerance=0, which could never be satisfied if we used < instead:
176178
test_diagnosis = test_diagnosis_t(test_passed=.true., diagnostics_string="")
177179
else
178180
test_diagnosis = test_diagnosis_t(test_passed=.false. &

src/julienne/julienne_test_result_s.f90

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818

1919
module procedure characterize
2020
if (.not. allocated(self%diagnosis_)) then
21-
characterization = "SKIPS on " // trim(self%description_%string()) // "."
21+
characterization = "SKIPS on " // trim(self%description_%string()) // "."
2222
else
2323
associate(test_passed => self%diagnosis_%test_passed())
2424
characterization = merge("passes on ", "FAILS on ", test_passed) // trim(self%description_%string()) // "."

0 commit comments

Comments
 (0)