Football Match Reports: Teach default keyword instead of exceptions

exercises/concept/football-match-reports/.docs/instructions.md

@@ -27,11 +27,11 @@ FootballMatchReports.onField(10);
 // => "striker"
 ```
 
-## 2. Raise an alert if an unknown shirt number is encountered
+## 2. Output "invalid" if the shirt number is not part of the official list
 
-Modify the `FootballMatchReports.onField()` method to throw an `IllegalArgumentException` when a shirt number outside the range 1-11 is processed.
+Modify the `FootballMatchReports.onField()` method to return 'invalid' when a shirt number outside the range 1-11 is processed.
 
 ```java
 FootballMatchReports.onField(13);
-// => Throw IllegalArgumentException
+// => "invalid"
 ```

exercises/concept/football-match-reports/.meta/config.json

@@ -2,6 +2,9 @@
   "authors": [
     "Azumix"
   ],
+  "contributors": [
+    "AlvesJorge"
+  ],
   "files": {
     "solution": [
       "src/main/java/FootballMatchReports.java"

exercises/concept/football-match-reports/.meta/src/reference/java/FootballMatchReports.java

@@ -30,7 +30,7 @@ public static String onField(int shirtNum) {
                 playerDescription = "striker";
                 break;
             default:
-                throw new IllegalArgumentException();
+                playerDescription = "invalid";
         }
         return playerDescription;
     }

exercises/concept/football-match-reports/src/test/java/FootballMatchReportsTest.java

@@ -3,7 +3,6 @@
 import org.junit.jupiter.api.Test;
 
 import static org.assertj.core.api.Assertions.assertThat;
-import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
 
 public class FootballMatchReportsTest {
 
@@ -68,17 +67,15 @@ public void test_right_wing() {
 
     @Test
     @Tag("task:2")
-    @DisplayName("The onField method throws IllegalArgumentException for unknown shirt number")
+    @DisplayName("The onField method returns 'invalid' for invalid shirt number")
     public void test_exception() {
-        assertThatExceptionOfType(IllegalArgumentException.class)
-                .isThrownBy(() -> FootballMatchReports.onField(13));
+        assertThat(FootballMatchReports.onField(13)).isEqualTo("invalid");
     }
 
     @Test
     @Tag("task:2")
-    @DisplayName("The onField method throws IllegalArgumentException for negative shirt number")
+    @DisplayName("The onField method returns 'invalid' for negative shirt number")
     public void test_exception_negative_number() {
-        assertThatExceptionOfType(IllegalArgumentException.class)
-                .isThrownBy(() -> FootballMatchReports.onField(-1));
+        assertThat(FootballMatchReports.onField(-1)).isEqualTo("invalid");
     }
 }