From e44e161c898bd2a6e9e89b6d4cd847c22f85bf9b Mon Sep 17 00:00:00 2001 From: Christian Koop Date: Fri, 30 Dec 2022 18:23:02 +0100 Subject: [PATCH] Adds some additional test cases for MathUtilsTest --- .../com/songoda/core/math/MathUtilsTest.java | 29 ++++++++++--------- 1 file changed, 16 insertions(+), 13 deletions(-) diff --git a/Core/src/test/java/com/songoda/core/math/MathUtilsTest.java b/Core/src/test/java/com/songoda/core/math/MathUtilsTest.java index 20147be8..d0aba855 100644 --- a/Core/src/test/java/com/songoda/core/math/MathUtilsTest.java +++ b/Core/src/test/java/com/songoda/core/math/MathUtilsTest.java @@ -2,16 +2,19 @@ package com.songoda.core.math; import org.junit.jupiter.api.Test; -import static org.junit.jupiter.api.Assertions.assertEquals; -import static org.junit.jupiter.api.Assertions.assertThrowsExactly; -import static org.junit.jupiter.api.Assertions.assertTrue; +import static org.junit.jupiter.api.Assertions.*; class MathUtilsTest { private final String warningMessage = "MathTest"; - private final String warningMessageExpectedStart = warningMessage + " "; + private final String warningMessageExpectedStart = this.warningMessage + " "; @Test void eval() { + assertEquals(0, MathUtils.eval("0")); + assertEquals(10, MathUtils.eval("10")); + assertEquals(0.0, MathUtils.eval("0.0")); + assertEquals(10.5, MathUtils.eval("10.5")); + assertEquals(10.5 + 4, MathUtils.eval("10.5 + 4")); assertEquals(10.5 - 5, MathUtils.eval("10.5 - 5")); assertEquals(10 * 4, MathUtils.eval("10 * 4")); @@ -33,28 +36,28 @@ class MathUtilsTest { @Test void evalWithCommaAsDecimalSeparator() { - Exception ex = assertThrowsExactly(RuntimeException.class, () -> MathUtils.eval("1,0", warningMessage)); + Exception ex = assertThrowsExactly(RuntimeException.class, () -> MathUtils.eval("1,0", this.warningMessage)); - assertTrue(ex.getMessage().startsWith(warningMessageExpectedStart), - () -> "'" + ex.getMessage() + "' does not start with '" + warningMessageExpectedStart + "'"); + assertTrue(ex.getMessage().startsWith(this.warningMessageExpectedStart), + () -> "'" + ex.getMessage() + "' does not start with '" + this.warningMessageExpectedStart + "'"); assertTrue(ex.getMessage().contains("Unexpected: ")); } @Test void evalWithUnsupportedSyntax() { - Exception ex = assertThrowsExactly(RuntimeException.class, () -> MathUtils.eval("!2", warningMessage)); + Exception ex = assertThrowsExactly(RuntimeException.class, () -> MathUtils.eval("!2", this.warningMessage)); - assertTrue(ex.getMessage().startsWith(warningMessageExpectedStart), - () -> "'" + ex.getMessage() + "' does not start with '" + warningMessageExpectedStart + "'"); + assertTrue(ex.getMessage().startsWith(this.warningMessageExpectedStart), + () -> "'" + ex.getMessage() + "' does not start with '" + this.warningMessageExpectedStart + "'"); assertTrue(ex.getMessage().contains("Unexpected: ")); } @Test void evalWithUnsupportedFunction() { - Exception ex = assertThrowsExactly(RuntimeException.class, () -> MathUtils.eval("acos(90)", warningMessage)); + Exception ex = assertThrowsExactly(RuntimeException.class, () -> MathUtils.eval("acos(90)", this.warningMessage)); - assertTrue(ex.getMessage().startsWith(warningMessageExpectedStart), - () -> "'" + ex.getMessage() + "' does not start with '" + warningMessageExpectedStart + "'"); + assertTrue(ex.getMessage().startsWith(this.warningMessageExpectedStart), + () -> "'" + ex.getMessage() + "' does not start with '" + this.warningMessageExpectedStart + "'"); assertTrue(ex.getMessage().contains("Unknown function: ")); } }