Adjust assignation to fit existing tests

mojohaus · Aug 23, 2021 · 868fae0 · 868fae0
1 parent 3ad787d
commit 868fae0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 11 deletions.
diff --git a/...main/java/org/codehaus/mojo/license/header/transformer/AbstractFileHeaderTransformer.java b/...main/java/org/codehaus/mojo/license/header/transformer/AbstractFileHeaderTransformer.java
@@ -274,13 +274,13 @@ public String getLineSeparator()
      */
     public void setLineSeparator( String lineSeparator )
     {
-        if ( lineSeparator != null )
+        try
         {
             this.lineSeparator = Eol.valueOf(lineSeparator).getEolString();
         }
-        else
+        catch ( IllegalArgumentException e )
         {
-            this.lineSeparator = null;
+            this.lineSeparator = lineSeparator;
         }
     }
 

diff --git a/.../java/org/codehaus/mojo/license/header/transformer/AbstractFileHeaderTransformerTest.java b/.../java/org/codehaus/mojo/license/header/transformer/AbstractFileHeaderTransformerTest.java
@@ -324,14 +324,8 @@ public void testSetLineSeparator()
         transformer.setLineSeparator("CRLF");
         Assert.assertEquals("\r\n", transformer.getLineSeparator());
 
-        transformer.setLineSeparator(null);
-        Assert.assertEquals(LINE_SEPARATOR, transformer.getLineSeparator());
-    }
-
-    @Test(expected = IllegalArgumentException.class)
-    public void testSetLineSeparatorUnknownEol()
-    {
-        transformer.setLineSeparator("INEXISTANT");
+        transformer.setLineSeparator("RANDOM");
+        Assert.assertEquals("RANDOM", transformer.getLineSeparator());
     }
 
     public static void assertEquals( FileHeader model, FileHeader model2 )