From 38968256c942866bc8950f792170944cadf4a26e Mon Sep 17 00:00:00 2001 From: Miroslav Pokorny Date: Fri, 9 Oct 2026 22:21:49 +1100 Subject: [PATCH] SpreadsheetExpressionFunctionTestCase: XXXTesting constants replace some expressions --- ...preadsheetExpressionFunctionNumberColumnOrRowTest.java | 8 ++++---- ...eadsheetExpressionFunctionNumberColumnsOrRowsTest.java | 6 +++--- .../SpreadsheetExpressionFunctionNumberIfTest.java | 4 ++-- .../SpreadsheetExpressionFunctionNumberTestCase.java | 2 +- .../SpreadsheetExpressionFunctionNumberTypeTest.java | 4 ++-- .../function/SpreadsheetExpressionFunctionTestCase.java | 5 +---- 6 files changed, 13 insertions(+), 16 deletions(-) diff --git a/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberColumnOrRowTest.java b/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberColumnOrRowTest.java index 40dcfe26..305959dd 100644 --- a/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberColumnOrRowTest.java +++ b/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberColumnOrRowTest.java @@ -42,7 +42,7 @@ public void testColumnReferenceParameterMissing() { this.applyAndCheck2( SpreadsheetExpressionFunctionNumberColumnOrRow.COLUMN, Lists.empty(), - KIND.create( + EXPRESSION_NUMBER_KIND.create( REFERENCE.column() .value() ) @@ -54,7 +54,7 @@ public void testRowReferenceParameterMissing() { this.applyAndCheck2( SpreadsheetExpressionFunctionNumberColumnOrRow.ROW, Lists.empty(), - KIND.create( + EXPRESSION_NUMBER_KIND.create( REFERENCE.row() .value() ) @@ -68,7 +68,7 @@ public void testColumnReferenceParameterPresent() { this.applyAndCheck2( SpreadsheetExpressionFunctionNumberColumnOrRow.COLUMN, Lists.of(B5), - KIND.create( + EXPRESSION_NUMBER_KIND.create( B5.column() .value() ) @@ -80,7 +80,7 @@ public void testRowReferenceParameterPresent() { this.applyAndCheck2( SpreadsheetExpressionFunctionNumberColumnOrRow.ROW, Lists.of(B5), - KIND.create( + EXPRESSION_NUMBER_KIND.create( B5.row() .value() ) diff --git a/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberColumnsOrRowsTest.java b/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberColumnsOrRowsTest.java index cf08edc9..3b919752 100644 --- a/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberColumnsOrRowsTest.java +++ b/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberColumnsOrRowsTest.java @@ -66,7 +66,7 @@ public void testColumnReferenceParameter() { this.applyAndCheck2( SpreadsheetExpressionFunctionNumberColumnsOrRows.COLUMNS, Lists.of(B5), - KIND.one() + EXPRESSION_NUMBER_KIND.one() ); } @@ -75,7 +75,7 @@ public void testRowReferenceParameter() { this.applyAndCheck2( SpreadsheetExpressionFunctionNumberColumnsOrRows.ROWS, Lists.of(B5), - KIND.one() + EXPRESSION_NUMBER_KIND.one() ); } @@ -104,7 +104,7 @@ private void applyAndCheck3(final SpreadsheetExpressionFunctionNumberColumnsOrRo this.applyAndCheck2( function, Lists.of(selection), - KIND.create(expected) + EXPRESSION_NUMBER_KIND.create(expected) ); } diff --git a/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberIfTest.java b/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberIfTest.java index e453736c..6c63c11d 100644 --- a/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberIfTest.java +++ b/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberIfTest.java @@ -119,7 +119,7 @@ private void applyIfAndCheck(final SpreadsheetExpressionFunctionNumberIf functio condition ), this.createContext(), - KIND.create(expected) + EXPRESSION_NUMBER_KIND.create(expected) ); } @@ -131,7 +131,7 @@ private List prepareList(final List list) { private Object wrapIfNumber(final Object value) { return value instanceof Number ? - KIND.create((Number) value) : + EXPRESSION_NUMBER_KIND.create((Number) value) : value; } diff --git a/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberTestCase.java b/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberTestCase.java index e73f5f83..11bd6264 100644 --- a/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberTestCase.java +++ b/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberTestCase.java @@ -35,7 +35,7 @@ public final void applyAndCheck2(final F function, this.applyAndCheck2( function, parameters.stream() - .map(i -> i instanceof Number ? KIND.create((Number) i) : i) + .map(i -> i instanceof Number ? EXPRESSION_NUMBER_KIND.create((Number) i) : i) .collect(Collectors.toList()), this.createContext(), result diff --git a/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberTypeTest.java b/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberTypeTest.java index d211f13e..804e647e 100644 --- a/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberTypeTest.java +++ b/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionNumberTypeTest.java @@ -40,7 +40,7 @@ public void testNull() { @Test public void testNumber() { this.typeAndCheck( - KIND.create(123), + EXPRESSION_NUMBER_KIND.create(123), 1 ); } @@ -147,7 +147,7 @@ private void typeAndCheck(final Object value, final int expected) { this.applyAndCheck2( Lists.of(value), - KIND.create(expected) + EXPRESSION_NUMBER_KIND.create(expected) ); } diff --git a/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionTestCase.java b/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionTestCase.java index 3bda1912..d94ad9d6 100644 --- a/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionTestCase.java +++ b/src/test/java/walkingkooka/spreadsheet/expression/function/SpreadsheetExpressionFunctionTestCase.java @@ -49,7 +49,6 @@ import walkingkooka.spreadsheet.value.SpreadsheetCell; import walkingkooka.text.CharSequences; import walkingkooka.tree.expression.ExpressionEvaluationContexts; -import walkingkooka.tree.expression.ExpressionNumberKind; import walkingkooka.tree.expression.function.ExpressionFunctionTesting2; import java.math.MathContext; @@ -87,8 +86,6 @@ public abstract class SpreadsheetExpressionFunctionTestCase