Revert "! r can we use multiline strings"

This reverts commit 04fe91a.
approvals · Sep 23, 2024 · 16fecf8 · 16fecf8
1 parent 0bc6aa5
commit 16fecf8
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 29 deletions.
diff --git a/...org/approvaltests/ApprovalsDuplicateVerifyExceptionTest.testExceptionMessage.approved.txt b/...org/approvaltests/ApprovalsDuplicateVerifyExceptionTest.testExceptionMessage.approved.txt
@@ -1,11 +1,11 @@
 org.approvaltests.ApprovalsDuplicateVerifyException: Already approved: file.txt
 By default, ApprovalTests only allows one verify() call per test.
-To find out more, visit:
+To find out more, visit: 
 https://github.com/approvals/ApprovalTests.Java/blob/master/approvaltests/docs/reference/Naming.md
 
 # Fixes
 1. separate your test into two tests
 2. add NamedParameters with the NamerFactory
-3. Override Approvals.settings() with either
-    a. allowMultipleVerifyCallsForThisClass
-    b. allowMultipleVerifyCallsForThisMethod
+3. Override Approvals.settings() with either 
+	a. allowMultipleVerifyCallsForThisClass
+	b. allowMultipleVerifyCallsForThisMethod
diff --git a/approvaltests-tests/src/test/java/org/approvaltests/inline/InlineApprovalsTest.java b/approvaltests-tests/src/test/java/org/approvaltests/inline/InlineApprovalsTest.java
@@ -200,11 +200,11 @@ void testSemiAutomaticMessage()
   void testSemiAutomaticWithPreviousApproved()
   {
     var expected = """
-      42
-      ***** DELETE ME TO APPROVE *****
-      vvvvv PREVIOUS RESULT      vvvvv
-      41
-      """;
+        42
+        ***** DELETE ME TO APPROVE *****
+        vvvvv PREVIOUS RESULT      vvvvv
+        41
+        """;
     var options = new Options().inline(expected, InlineOptions.semiAutomaticWithPreviousApproved());
     try
     {

diff --git a/approvaltests/pom.xml b/approvaltests/pom.xml
@@ -164,14 +164,6 @@
                     </execution>
                 </executions>
             </plugin>
-            <plugin>
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-compiler-plugin</artifactId>
-                <configuration>
-                    <source>15</source>
-                    <target>15</target>
-                </configuration>
-            </plugin>
         </plugins>
     </build>
 

diff --git a/approvaltests/src/main/java/org/approvaltests/ApprovalsDuplicateVerifyException.java b/approvaltests/src/main/java/org/approvaltests/ApprovalsDuplicateVerifyException.java
@@ -4,17 +4,11 @@ public class ApprovalsDuplicateVerifyException extends RuntimeException
 {
   public ApprovalsDuplicateVerifyException(String file)
   {
-    super("""
-        Already approved: %s
-        By default, ApprovalTests only allows one verify() call per test.
-        To find out more, visit:
-        https://github.com/approvals/ApprovalTests.Java/blob/master/approvaltests/docs/reference/Naming.md
-
-        # Fixes
-        1. separate your test into two tests
-        2. add NamedParameters with the NamerFactory
-        3. Override Approvals.settings() with either
-            a. allowMultipleVerifyCallsForThisClass
-            b. allowMultipleVerifyCallsForThisMethod""".formatted(file));;
+    super("Already approved: " + file + "\n"
+        + "By default, ApprovalTests only allows one verify() call per test.\n" + "To find out more, visit: \n"
+        + "https://github.com/approvals/ApprovalTests.Java/blob/master/approvaltests/docs/reference/Naming.md\n\n"
+        + "# Fixes\n" + "1. separate your test into two tests\n" + "2. add NamedParameters with the NamerFactory\n"
+        + "3. Override Approvals.settings() with either \n" + "\ta. allowMultipleVerifyCallsForThisClass\n"
+        + "\tb. allowMultipleVerifyCallsForThisMethod");
   }
 }