Improve tests in part1.2

Author: 3750

part1.2-from-sequential-to-parallel/src/main/groovy/migration/FileService.groovy

@@ -5,11 +5,13 @@ class FileService {
     static void setText(File parent, String name, String text) {
         MetricService.incrementInvocationsCount()
         new File(parent, name).text = text
+        sleep text.length() * 20
     }
 
     static void appendText(File parent, String name, String text) {
         MetricService.incrementInvocationsCount()
         new File(parent, name).append(text)
+        sleep text.length() * 10
     }
 
 }

part1.2-from-sequential-to-parallel/src/test/groovy/migration/AppendTextSpec.groovy

@@ -2,28 +2,26 @@ package migration
 
 class AppendTextSpec extends BaseSpec {
 
-    def "append text"() {
-        given:
-        def name = "test"
+    def fileName = "test"
+
+    def setup() {
+        new File(rootDir, fileName).text = "initial text"
+    }
 
+    def "append text"() {
         when:
-        FileService.appendText(tempDir, name, "some text")
-        sleep 100
+        FileService.appendText(rootDir, fileName, "additional text")
 
         then:
-        new File(tempDir, name).text == 'some text'
+        new File(rootDir, fileName).text == 'initial textadditional text'
     }
 
     def "append empty text"() {
-        given:
-        def name = "test"
-
         when:
-        FileService.appendText(tempDir, name, '')
-        sleep 100
+        FileService.appendText(rootDir, fileName, '')
 
         then:
-        new File(tempDir, name).text == ''
+        new File(rootDir, fileName).text == 'initial text'
     }
 
 }

part1.2-from-sequential-to-parallel/src/test/groovy/migration/BaseSpec.groovy

@@ -7,13 +7,6 @@ import java.nio.file.Files
 
 class BaseSpec extends Specification {
 
-    static File tempDir = Files.createTempDirectory("parallel-test-execution").toFile()
-
-    def cleanup() {
-        tempDir.listFiles().each {
-            println "Removing $it"
-            it.delete()
-        }
-    }
+    static File rootDir = Files.createTempDirectory("parallel-test-execution").toFile()
 
 }

part1.2-from-sequential-to-parallel/src/test/groovy/migration/MetricsSpec.groovy

@@ -7,8 +7,7 @@ class MetricsSpec extends BaseSpec {
         def count = MetricService.getInvocationsCount()
 
         when:
-        FileService.appendText(tempDir, "test", "some text")
-        sleep 100
+        FileService.appendText(rootDir, "test", "some text")
 
         then:
         MetricService.getInvocationsCount() == count + 1
@@ -19,8 +18,7 @@ class MetricsSpec extends BaseSpec {
         def count = MetricService.getInvocationsCount()
 
         when:
-        FileService.setText(tempDir, "test", "some text")
-        sleep 100
+        FileService.setText(rootDir, "test", "some text")
 
         then:
         MetricService.getInvocationsCount() == count + 1

part1.2-from-sequential-to-parallel/src/test/groovy/migration/SetTextSpec.groovy

@@ -2,28 +2,32 @@ package migration
 
 class SetTextSpec extends BaseSpec {
 
+    def cleanup() {
+        rootDir.listFiles().each {
+            it.delete()
+        }
+    }
+
     def "create file with text"() {
         given:
-        def name = "test"
+        def fileName = "test 1"
 
         when:
-        FileService.setText(tempDir, name, "some text")
-        sleep 200
+        FileService.setText(rootDir, fileName, "some text")
 
         then:
-        new File(tempDir, name).text == 'some text'
+        new File(rootDir, fileName).text == 'some text'
     }
 
     def "create empty file"() {
         given:
-        def name = "test"
+        def fileName = "test 2"
 
         when:
-        FileService.setText(tempDir, name, "")
-        sleep 200
+        FileService.setText(rootDir, fileName, "")
 
         then:
-        new File(tempDir, name).text == ''
+        new File(rootDir, fileName).text == ''
     }
 
 }