Use @beforeeach instead of @BeforeAll.

Author: josealonso

exercises/concept/tim-from-marketing-2/.classpath

@@ -19,6 +19,13 @@
 			<attribute name="test" value="true"/>
 		</attributes>
 	</classpathentry>
+	<classpathentry kind="src" output="bin/test" path="build/gen/test/java">
+		<attributes>
+			<attribute name="gradle_scope" value="test"/>
+			<attribute name="gradle_used_by_scope" value="test"/>
+			<attribute name="test" value="true"/>
+		</attributes>
+	</classpathentry>
 	<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-17/"/>
 	<classpathentry kind="con" path="org.eclipse.buildship.core.gradleclasspathcontainer"/>
 	<classpathentry kind="output" path="bin"/>

exercises/concept/tim-from-marketing-2/.docs/hints.md

@@ -1,11 +1,9 @@
 # Hints
 
-## 1.- Print the name of all the employees
 
+## 1.- Print the name of all the employees
 WIP
 
 
 ## 2.- Print the name and department of a given employee
-
 WIP
-

exercises/concept/tim-from-marketing-2/.docs/instructions.md

@@ -1,13 +1,11 @@
 # Instructions
 
 In this exercise you will be writing code to print all the names of the factory employees.
-
 Employees have an ID, a name and a department name, like in the [tim-from-marketing](/exercises/concept/tim-from-marketing) exercise. 
 Assume that the ID of the first employee is 0, the ID of the second employee is 1, and so on. The three fields of an employee may be empty, that's why they are declared as Optional<T> types.
 The class constructor receives a parameter of type List<Optional<Employee>>, which is populated in the tests.
 
 ## 1.- Print the names of all the employees
-
 Implement the `printAllEmployeesNames()` method to print the names of all the employees, together with their id. If the employee does not exist, print "[id] - No employee found".
 
 ```java

exercises/concept/tim-from-marketing-2/.docs/introduction.md

@@ -5,7 +5,6 @@
 ## Introduction
 
 The **Optional<T>** type was introduced in Java 8 as a way to indicate that a method will return an object of type T or an empty value. It is present in type signatures of many core Java methods. 
-
 Before Java 8, developers had to implement null checks:
 
 ```java
@@ -65,7 +64,6 @@ public Optional<Integer> getEmployeeAge(String name) {
 ```
 
 It is important to understand that the Optional API does not eliminate the null checking, but it defers it until the end of a series of methods, as long as all those methods return an optional object. 
-
 The fields of the Employee class have an Optional type. Notice that this is not recommended, as explained by one well-known Java language architect in [this SO answer](https://stackoverflow.com/questions/26327957/should-java-8-getters-return-optional-type)
 
 ## Flatmap operation
@@ -89,4 +87,4 @@ List<Optional<String>> listOfOptionals = Arrays.asList(
         System.out.println(result); // Output: [Java, Kotlin]
     }
 }
-```
+```

exercises/concept/tim-from-marketing-2/.meta/design.md

@@ -5,7 +5,6 @@
 The goal of this exercise is to teach the student how to use the Optional API.
 We will use the most common methods: `ifPresent`, `orElse`, `ifPresentOrElse`, `orElseThrow`. 
 The `isPresent` and `get` methods are not presented, since they do not provide any value over an ordinary null check.
-
 Some methods of the Stream API are needed. This is a bit problematic, since they have not been explained in the current Java track.
 
 ## Learning objectives
@@ -30,13 +29,13 @@ This Concepts Exercise's Concepts are:
 This Concept Exercise's prerequisites Concepts are:
 
 - `custom classes`.
+- `lists`.
 - `generic-types`.
 - `streams`.
 
 ## Analyzer
 
 This exercise could benefit from the following rules in the [analyzer]:
-
 - `actionable`: If the solution uses `null` in any method, encourage the student to use `Optional<T>` instead.
 - `actionable`: If the solution uses the `get` or `isPresent` methods of the Optional<T> API, encourage the student to use `orElse`, `orElseThrow` or `ifPresentOrElse` instead. 
 - `informative`: TODO.

exercises/concept/tim-from-marketing-2/src/test/java/EmployeeServiceTest.java

@@ -1,6 +1,6 @@
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Tag;
-import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.TestInstance;
 import org.junit.jupiter.api.TestInstance.Lifecycle;
@@ -10,8 +10,7 @@
 import java.util.ArrayList;
 import java.util.Optional;
 
-// Annotation in order to use @BeforeAll in a non-static method
-@TestInstance(Lifecycle.PER_CLASS)
+// @TestInstance(Lifecycle.PER_CLASS)
 public class EmployeeServiceTest {
 
     private EmployeeService employeeService;
@@ -33,7 +32,7 @@ void initList() {
         listOfEmployees.add(Optional.of(new Employee(2, "John", "Engineering")));
     }
 
-    @BeforeAll
+    @BeforeEach
     void setup() {
         initList();
         employeeService = new EmployeeService(listOfEmployees);