Refactor stringassertion

core/src/main/java/eu/tsystems/mms/tic/testframework/internal/asserts/AbstractPropertyAssertion.java

@@ -40,7 +40,7 @@ public abstract class AbstractPropertyAssertion<T> implements ActualProperty<T>
     protected final AbstractPropertyAssertion<T> parent;
     protected PropertyAssertionConfig config;
 
-    public AbstractPropertyAssertion(AbstractPropertyAssertion parentAssertion, AssertionProvider<T> provider) {
+    public AbstractPropertyAssertion(AbstractPropertyAssertion<T> parentAssertion, AssertionProvider<T> provider) {
         this.parent = parentAssertion;
         this.provider = provider;
     }

core/src/main/java/eu/tsystems/mms/tic/testframework/internal/asserts/FileAssertion.java

@@ -29,8 +29,8 @@
  */
 public interface FileAssertion extends ActualProperty<File> {
     QuantityAssertion<Long> bytes();
-    StringAssertion<String> name();
-    StringAssertion<String> extension();
-    StringAssertion<String> mimetype();
+    StringAssertion name();
+    StringAssertion extension();
+    StringAssertion mimetype();
     BinaryAssertion<Boolean> exists();
 }

core/src/main/java/eu/tsystems/mms/tic/testframework/internal/asserts/ObjectAssertion.java

@@ -0,0 +1,36 @@
+/*
+ * Testerra
+ *
+ * (C) 2024, Sebastian Kiehne, Deutsche Telekom MMS GmbH, Deutsche Telekom AG
+ *
+ * Deutsche Telekom AG and all other contributors /
+ * copyright owners license this file to you under the Apache
+ * License, Version 2.0 (the "License"); you may not use this
+ * file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+package eu.tsystems.mms.tic.testframework.internal.asserts;
+
+public interface ObjectAssertion<T> extends BinaryAssertion<T> {
+    default boolean is(Object expected) {
+        return is(expected, null);
+    }
+
+    boolean is(Object expected, String subject);
+
+    default boolean isNot(Object expected) {
+        return isNot(expected, null);
+    }
+
+    boolean isNot(Object expected, String subject);
+}

core/src/main/java/eu/tsystems/mms/tic/testframework/internal/asserts/QuantityAssertion.java

@@ -28,16 +28,7 @@
  * Allows numeric range based assertions
  * @author Mike Reiche
  */
-public interface QuantityAssertion<TYPE> extends BinaryAssertion<TYPE> {
-    default boolean is(Object expected) {
-        return is(expected, null);
-    }
-    boolean is(Object expected, String subject);
-
-    default boolean isNot(Object expected) {
-        return isNot(expected, null);
-    }
-    boolean isNot(Object expected, String subject);
+public interface QuantityAssertion<TYPE> extends ObjectAssertion<TYPE> {
 
     default boolean isGreaterThan(long expected) {
         return isGreaterThan(new BigDecimal(expected));
@@ -124,7 +115,7 @@ default boolean isBetween(BigDecimal lower, BigDecimal higher) {
     }
     boolean isBetween(BigDecimal lower, BigDecimal higher, String subject);
 
-    <MAPPED_TYPE> StringAssertion<MAPPED_TYPE> map(Function<? super TYPE, MAPPED_TYPE> mapFunction);
+    <MAPPED_TYPE> QuantityAssertion<MAPPED_TYPE> map(Function<? super TYPE, MAPPED_TYPE> mapFunction);
 
     QuantityAssertion<BigDecimal> absolute();
 }

core/src/main/java/eu/tsystems/mms/tic/testframework/internal/asserts/StringAssertion.java

@@ -23,14 +23,15 @@
 
 import java.util.Arrays;
 import java.util.List;
+import java.util.function.Function;
 import java.util.regex.Pattern;
 import java.util.stream.Collectors;
 
 /**
  * Allows string based assertions
  * @author Mike Reiche
  */
-public interface StringAssertion<T> extends QuantityAssertion<T> {
+public interface StringAssertion extends ObjectAssertion<String> {
     BinaryAssertion <Boolean> contains(String expected);
     BinaryAssertion <Boolean> startsWith(String expected);
     BinaryAssertion <Boolean> endsWith(String expected);
@@ -54,4 +55,6 @@ default boolean isContaining(String text) {
     default boolean isNotContaining(String text) {
         return contains(text).is(false);
     }
+
+    StringAssertion map(Function<String, String> mapFunction);
 }

driver-ui/src/main/java/eu/tsystems/mms/tic/testframework/internal/asserts/AbstractTestedPropertyAssertion.java

@@ -46,7 +46,7 @@ public abstract class AbstractTestedPropertyAssertion<T> extends AbstractPropert
     protected static final Assertion assertionImpl = Testerra.getInjector().getInstance(Assertion.class);
     private static final TestController.Overrides overrides = Testerra.getInjector().getInstance(TestController.Overrides.class);
 
-    public AbstractTestedPropertyAssertion(AbstractPropertyAssertion parentAssertion, AssertionProvider<T> provider) {
+    public AbstractTestedPropertyAssertion(AbstractPropertyAssertion<T> parentAssertion, AssertionProvider<T> provider) {
         super(parentAssertion, provider);
     }
 

driver-ui/src/main/java/eu/tsystems/mms/tic/testframework/internal/asserts/DefaultBinaryAssertion.java

@@ -26,7 +26,7 @@
  * @author Mike Reiche
  */
 public class DefaultBinaryAssertion<T> extends AbstractTestedPropertyAssertion<T> implements BinaryAssertion<T> {
-    public DefaultBinaryAssertion(AbstractPropertyAssertion parentAssertion, AssertionProvider<T> provider) {
+    public DefaultBinaryAssertion(AbstractPropertyAssertion<T> parentAssertion, AssertionProvider<T> provider) {
         super(parentAssertion, provider);
     }
 

driver-ui/src/main/java/eu/tsystems/mms/tic/testframework/internal/asserts/DefaultFileAssertion.java

@@ -32,7 +32,7 @@
  */
 public class DefaultFileAssertion extends AbstractPropertyAssertion<File> implements FileAssertion {
 
-    public DefaultFileAssertion(AbstractPropertyAssertion parentAssertion, AssertionProvider<File> provider) {
+    public DefaultFileAssertion(AbstractPropertyAssertion<File> parentAssertion, AssertionProvider<File> provider) {
         super(parentAssertion, provider);
     }
 
@@ -52,7 +52,7 @@ public String createSubject() {
     }
 
     @Override
-    public StringAssertion<String> name() {
+    public StringAssertion name() {
         return propertyAssertionFactory.createWithParent(DefaultStringAssertion.class, this, new AssertionProvider<String>() {
             @Override
             public String getActual() {
@@ -67,7 +67,7 @@ public String createSubject() {
     }
 
     @Override
-    public StringAssertion<String> extension() {
+    public StringAssertion extension() {
         return propertyAssertionFactory.createWithParent(DefaultStringAssertion.class, this, new AssertionProvider<String>() {
             @Override
             public String getActual() {
@@ -82,7 +82,7 @@ public String createSubject() {
     }
 
     @Override
-    public StringAssertion<String> mimetype() {
+    public StringAssertion mimetype() {
         return propertyAssertionFactory.createWithParent(DefaultStringAssertion.class, this, new AssertionProvider<String>() {
             @Override
             public String getActual() {

driver-ui/src/main/java/eu/tsystems/mms/tic/testframework/internal/asserts/DefaultObjectAssertion.java

@@ -0,0 +1,56 @@
+/*
+ * Testerra
+ *
+ * (C) 2024, Sebastian Kiehne, Deutsche Telekom MMS GmbH, Deutsche Telekom AG
+ *
+ * Deutsche Telekom AG and all other contributors /
+ * copyright owners license this file to you under the Apache
+ * License, Version 2.0 (the "License"); you may not use this
+ * file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+package eu.tsystems.mms.tic.testframework.internal.asserts;
+
+public class DefaultObjectAssertion<T> extends DefaultBinaryAssertion<T> implements ObjectAssertion<T> {
+
+    public DefaultObjectAssertion(AbstractPropertyAssertion<T> parentAssertion, AssertionProvider<T> provider) {
+        super(parentAssertion, provider);
+    }
+
+    @Override
+    public boolean is(Object expected, String failMessage) {
+        if (expected instanceof Boolean) {
+            boolean expectedBoolean = (Boolean) expected;
+            return this.is(expectedBoolean, failMessage);
+        }
+        return testSequence(
+                provider,
+                (actual) -> assertionImpl.equals(actual, expected),
+                (actual) -> assertionImpl.formatExpectEquals(null, expected, createFailMessage(failMessage))
+        );
+    }
+
+    @Override
+    public boolean isNot(Object expected, String failMessage) {
+        if (expected instanceof Boolean) {
+            boolean expectedBoolean = (Boolean) expected;
+            return this.is(!expectedBoolean, failMessage);
+        } else {
+            return testSequence(
+                    provider,
+                    (actual) -> assertionImpl.notEquals(actual, expected),
+                    (actual) -> assertionImpl.formatExpectNotEquals(null, expected, createFailMessage(failMessage))
+            );
+        }
+    }
+}

driver-ui/src/main/java/eu/tsystems/mms/tic/testframework/internal/asserts/DefaultPatternAssertion.java

@@ -29,7 +29,7 @@
  */
 public class DefaultPatternAssertion extends AbstractTestedPropertyAssertion<Matcher> implements PatternAssertion {
 
-    public DefaultPatternAssertion(AbstractPropertyAssertion parentAssertion, AssertionProvider<Matcher> provider) {
+    public DefaultPatternAssertion(AbstractPropertyAssertion<Matcher> parentAssertion, AssertionProvider<Matcher> provider) {
         super(parentAssertion, provider);
     }
 

driver-ui/src/main/java/eu/tsystems/mms/tic/testframework/internal/asserts/DefaultPropertyAssertionFactory.java

@@ -31,10 +31,10 @@
  */
 public class DefaultPropertyAssertionFactory implements PropertyAssertionFactory, Loggable {
 
-    public <ASSERTION extends AbstractPropertyAssertion, TYPE> ASSERTION createAssertion(
+    public <ASSERTION extends AbstractPropertyAssertion, T> ASSERTION createAssertion(
             Class<ASSERTION> assertionClass,
             AbstractPropertyAssertion parentAssertion,
-            AssertionProvider<TYPE> provider
+            AssertionProvider<T> provider
     ) {
         ASSERTION assertion;
         try {
@@ -48,10 +48,10 @@ public <ASSERTION extends AbstractPropertyAssertion, TYPE> ASSERTION createAsser
     }
 
     @Override
-    public <ASSERTION extends AbstractPropertyAssertion, TYPE> ASSERTION createWithParent(
+    public <ASSERTION extends AbstractPropertyAssertion, T> ASSERTION createWithParent(
         Class<ASSERTION> assertionClass,
         AbstractPropertyAssertion parentAssertion,
-        AssertionProvider<TYPE> provider
+        AssertionProvider<T> provider
     ) {
         ASSERTION assertion = createAssertion(assertionClass, parentAssertion, provider);
         assertion.config = parentAssertion.config;

driver-ui/src/main/java/eu/tsystems/mms/tic/testframework/internal/asserts/DefaultQuantityAssertion.java

@@ -29,39 +29,12 @@
  *
  * @author Mike Reiche
  */
-public class DefaultQuantityAssertion<TYPE> extends DefaultBinaryAssertion<TYPE> implements QuantityAssertion<TYPE> {
+public class DefaultQuantityAssertion<TYPE> extends DefaultObjectAssertion<TYPE> implements QuantityAssertion<TYPE> {
 
     public DefaultQuantityAssertion(AbstractPropertyAssertion parentAssertion, AssertionProvider<TYPE> provider) {
         super(parentAssertion, provider);
     }
 
-    @Override
-    public boolean is(Object expected, String failMessage) {
-        if (expected instanceof Boolean) {
-            boolean expectedBoolean = (Boolean) expected;
-            return this.is(expectedBoolean, failMessage);
-        }
-        return testSequence(
-                provider,
-                (actual) -> assertionImpl.equals(actual, expected),
-                (actual) -> assertionImpl.formatExpectEquals(null, expected, createFailMessage(failMessage))
-        );
-    }
-
-    @Override
-    public boolean isNot(Object expected, String failMessage) {
-        if (expected instanceof Boolean) {
-            boolean expectedBoolean = (Boolean) expected;
-            return this.is(!expectedBoolean, failMessage);
-        } else {
-            return testSequence(
-                    provider,
-                    (actual) -> assertionImpl.notEquals(actual, expected),
-                    (actual) -> assertionImpl.formatExpectNotEquals(null, expected, createFailMessage(failMessage))
-            );
-        }
-    }
-
     private BigDecimal toBigDecimal(Object given) {
         if (given == null) {
             return null;
@@ -116,8 +89,8 @@ public boolean isBetween(BigDecimal lower, BigDecimal higher, String failMessage
     }
 
     @Override
-    public <MAPPED_TYPE> StringAssertion<MAPPED_TYPE> map(Function<? super TYPE, MAPPED_TYPE> mapFunction) {
-        return propertyAssertionFactory.createWithParent(DefaultStringAssertion.class, this, new AssertionProvider<MAPPED_TYPE>() {
+    public <MAPPED_TYPE> QuantityAssertion<MAPPED_TYPE> map(Function<? super TYPE, MAPPED_TYPE> mapFunction) {
+        return propertyAssertionFactory.createWithParent(DefaultQuantityAssertion.class, this, new AssertionProvider<MAPPED_TYPE>() {
             @Override
             public MAPPED_TYPE getActual() {
                 TYPE actual = provider.getActual();