BAEL-3214 | Code review related changes - 2

Nikunj Gandhi · Nikunj Gandhi · commit 52934fdb209d · 2019-09-19T15:31:48.000-04:00
diff --git a/core-java-modules/core-java-io-2/src/test/java/com/baeldung/filewriter/FileWriterExampleUnitTest.java b/core-java-modules/core-java-io-2/src/test/java/com/baeldung/filewriter/FileWriterExampleUnitTest.java
@@ -11,39 +11,37 @@
 
 public class FileWriterExampleUnitTest {
 
-    private static final String FILE_NAME = "src/test/resources/FileWriterTest.txt";
-
     @After
     public void tearDown() throws IOException {
-        Files.delete(Path.of(FILE_NAME));
+        Files.delete(Path.of("src/test/resources/FileWriterTest.txt"));
     }
 
     @Test
     public void testWriteString() throws IOException {
-        try (FileWriter fileWriter = new FileWriter(FILE_NAME)) {
+        try (FileWriter fileWriter = new FileWriter("src/test/resources/FileWriterTest.txt")) {
             fileWriter.write("Hello Folks!");
         }
-        Assert.assertEquals("Hello Folks!", new String(Files.readAllBytes(Path.of(FILE_NAME))));
+        Assert.assertEquals("Hello Folks!", new String(Files.readAllBytes(Path.of("src/test/resources/FileWriterTest.txt"))));
     }
 
     @Test
     public void testAppendString() throws IOException {
-        try (FileWriter fileWriter = new FileWriter(FILE_NAME)) {
+        try (FileWriter fileWriter = new FileWriter("src/test/resources/FileWriterTest.txt")) {
             fileWriter.write("Hello Folks!");
         }
         // using another try with resources to reopen the file in append mode
-        try (FileWriter fileWriter = new FileWriter(FILE_NAME, true)) {
+        try (FileWriter fileWriter = new FileWriter("src/test/resources/FileWriterTest.txt", true)) {
             fileWriter.write("Hello Folks Again!");
         }
 
-        Assert.assertEquals("Hello Folks!" + "Hello Folks Again!", new String(Files.readAllBytes(Path.of(FILE_NAME))));
+        Assert.assertEquals("Hello Folks!" + "Hello Folks Again!", new String(Files.readAllBytes(Path.of("src/test/resources/FileWriterTest.txt"))));
     }
 
     @Test
     public void testWriteCharArray() throws IOException {
-        try (FileWriter fileWriter = new FileWriter(FILE_NAME)) {
+        try (FileWriter fileWriter = new FileWriter("src/test/resources/FileWriterTest.txt")) {
             fileWriter.write("Hello Folks!".toCharArray());
         }
-        Assert.assertEquals("Hello Folks!", new String(Files.readAllBytes(Path.of(FILE_NAME))));
+        Assert.assertEquals("Hello Folks!", new String(Files.readAllBytes(Path.of("src/test/resources/FileWriterTest.txt"))));
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -11,39 +11,37 @@`
`11`	`11`
`12`	`12`	`public class FileWriterExampleUnitTest {`
`13`	`13`
`14`		`- private static final String FILE_NAME = "src/test/resources/FileWriterTest.txt";`
`15`		`-`
`16`	`14`	`@After`
`17`	`15`	`public void tearDown() throws IOException {`
`18`		`- Files.delete(Path.of(FILE_NAME));`
	`16`	`+ Files.delete(Path.of("src/test/resources/FileWriterTest.txt"));`
`19`	`17`	`}`
`20`	`18`
`21`	`19`	`@Test`
`22`	`20`	`public void testWriteString() throws IOException {`
`23`		`- try (FileWriter fileWriter = new FileWriter(FILE_NAME)) {`
	`21`	`+ try (FileWriter fileWriter = new FileWriter("src/test/resources/FileWriterTest.txt")) {`
`24`	`22`	`fileWriter.write("Hello Folks!");`
`25`	`23`	`}`
`26`		`- Assert.assertEquals("Hello Folks!", new String(Files.readAllBytes(Path.of(FILE_NAME))));`
	`24`	`+ Assert.assertEquals("Hello Folks!", new String(Files.readAllBytes(Path.of("src/test/resources/FileWriterTest.txt"))));`
`27`	`25`	`}`
`28`	`26`
`29`	`27`	`@Test`
`30`	`28`	`public void testAppendString() throws IOException {`
`31`		`- try (FileWriter fileWriter = new FileWriter(FILE_NAME)) {`
	`29`	`+ try (FileWriter fileWriter = new FileWriter("src/test/resources/FileWriterTest.txt")) {`
`32`	`30`	`fileWriter.write("Hello Folks!");`
`33`	`31`	`}`
`34`	`32`	`// using another try with resources to reopen the file in append mode`
`35`		`- try (FileWriter fileWriter = new FileWriter(FILE_NAME, true)) {`
	`33`	`+ try (FileWriter fileWriter = new FileWriter("src/test/resources/FileWriterTest.txt", true)) {`
`36`	`34`	`fileWriter.write("Hello Folks Again!");`
`37`	`35`	`}`
`38`	`36`
`39`		`- Assert.assertEquals("Hello Folks!" + "Hello Folks Again!", new String(Files.readAllBytes(Path.of(FILE_NAME))));`
	`37`	`+ Assert.assertEquals("Hello Folks!" + "Hello Folks Again!", new String(Files.readAllBytes(Path.of("src/test/resources/FileWriterTest.txt"))));`
`40`	`38`	`}`
`41`	`39`
`42`	`40`	`@Test`
`43`	`41`	`public void testWriteCharArray() throws IOException {`
`44`		`- try (FileWriter fileWriter = new FileWriter(FILE_NAME)) {`
	`42`	`+ try (FileWriter fileWriter = new FileWriter("src/test/resources/FileWriterTest.txt")) {`
`45`	`43`	`fileWriter.write("Hello Folks!".toCharArray());`
`46`	`44`	`}`
`47`		`- Assert.assertEquals("Hello Folks!", new String(Files.readAllBytes(Path.of(FILE_NAME))));`
	`45`	`+ Assert.assertEquals("Hello Folks!", new String(Files.readAllBytes(Path.of("src/test/resources/FileWriterTest.txt"))));`
`48`	`46`	`}`
`49`	`47`	`}`