Merge remote-tracking branch 'origin/camicia-exercice' into camicia-exercice

Author: thibault2705

.github/workflows/check-deprecated-exercises.yml

@@ -2,13 +2,19 @@ name: Deprecated
 
 on:
   pull_request:
+    paths:
+      - "exercises/**"
+  push:
+    branches:
+      - main
+  workflow_dispatch:
 
 jobs:
   test-deprecated:
     name: Check for deprecated exercises
     runs-on: ubuntu-24.04
     steps:
-      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
+      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd
         with:
           fetch-depth: 0
       - name: Test deprecated exercises using test-deprecated-exercises

.github/workflows/java.yml

@@ -15,11 +15,11 @@ jobs:
     name: Check if tests compile cleanly with starter sources
     runs-on: ubuntu-24.04
     steps:
-      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
-      - name: Set up JDK 1.17
-        uses: actions/setup-java@dded0888837ed1f317902acf8a20df0ad188d165
+      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd
+      - name: Set up JDK 21
+        uses: actions/setup-java@be666c2fcd27ec809703dec50e508c2fdc7f6654
         with:
-          java-version: 17
+          java-version: 21
           distribution: "temurin"
       - name: Check if tests compile cleanly with starter sources
         run: ./gradlew compileStarterTestJava --continue
@@ -29,11 +29,11 @@ jobs:
     name: Lint Java files using Checkstyle
     runs-on: ubuntu-24.04
     steps:
-      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
-      - name: Set up JDK 1.17
-        uses: actions/setup-java@dded0888837ed1f317902acf8a20df0ad188d165
+      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd
+      - name: Set up JDK 21
+        uses: actions/setup-java@be666c2fcd27ec809703dec50e508c2fdc7f6654
         with:
-          java-version: 17
+          java-version: 21
           distribution: "temurin"
       - name: Run checkstyle
         run: ./gradlew check --exclude-task test --continue
@@ -44,11 +44,23 @@ jobs:
     if: github.event_name == 'push' || github.event_name == 'workflow_dispatch'
     runs-on: ubuntu-24.04
     steps:
-      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
+      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd
       - name: Test all exercises using java-test-runner
         run: bin/test-with-test-runner
+      - name: Print summary
+        run: |
+          if [ -f exercises/build/summary.txt ]; then
+            echo "===== TEST SUMMARY ====="
+            cat exercises/build/summary.txt
+            echo "========================"
+          else
+            echo "===== ALL TESTS PASSED ====="
+            echo "No summary file was generated."
+            echo "============================="
+          fi
+        if: always() 
       - name: Archive test results
-        uses: actions/upload-artifact@330a01c490aca151604b8cf639adc76d48f6c5d4
+        uses: actions/upload-artifact@bbbca2ddaa5d8feaa63e36b76fdaad77386f024f
         with:
           name: test-results
           path: exercises/**/build/results.json
@@ -59,13 +71,25 @@ jobs:
     if: github.event_name == 'pull_request'
     runs-on: ubuntu-24.04
     steps:
-      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
+      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd
         with:
           fetch-depth: 0
       - name: Test changed exercises using java-test-runner
         run: bin/test-changed-exercise
+      - name: Print summary
+        run: |
+          if [ -f exercises/build/summary.txt ]; then
+            echo "===== TEST SUMMARY ====="
+            cat exercises/build/summary.txt
+            echo "========================"
+          else
+            echo "===== ALL TESTS PASSED ====="
+            echo "No summary file was generated."
+            echo "============================="
+          fi
+        if: always() 
       - name: Archive test results
-        uses: actions/upload-artifact@330a01c490aca151604b8cf639adc76d48f6c5d4
+        uses: actions/upload-artifact@bbbca2ddaa5d8feaa63e36b76fdaad77386f024f
         with:
           name: test-results
           path: |

.github/workflows/markdown.yml

@@ -17,6 +17,6 @@ jobs:
     name: Lint Markdown files
     runs-on: ubuntu-24.04
     steps:
-      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
+      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd
       - name: Lint markdown
-        uses: DavidAnson/markdownlint-cli2-action@992badcdf24e3b8eb7e87ff9287fe931bcb00c6e
+        uses: DavidAnson/markdownlint-cli2-action@ce4853d43830c74c1753b39f3cf40f71c2031eb9

bin/test-changed-exercise

@@ -35,7 +35,12 @@ echo "Changed exercises detected:"
 echo "$changed_exercises"
 echo "----------------------------------------"
 
+summary_dir="exercises/build"
+summary_file="${summary_dir}/summary.txt"
+mkdir -p "$summary_dir"
+
 # Run tests
+exit_code=0
 for dir in $changed_exercises; do
   slug=$(basename "$dir")
 
@@ -47,8 +52,27 @@ for dir in $changed_exercises; do
   mkdir -p "$(dirname "$results_path")"
 
   if [[ $dir == exercises/practice/* ]]; then
-    ./exercises/gradlew -p exercises ":practice:$slug:test" 2>&1 | tee "$results_path"
+    ./exercises/gradlew -p exercises ":practice:$slug:test" 2>&1 | tee "$results_path" || true
   elif [[ $dir == exercises/concept/* ]]; then
-    ./exercises/gradlew -p exercises ":concept:$slug:test" 2>&1 | tee "$results_path"
+    ./exercises/gradlew -p exercises ":concept:$slug:test" 2>&1 | tee "$results_path" || true
+  fi
+
+  # Detect failure
+  if grep -q "FAILED" "$results_path"; then
+    exit_code=1
+
+    # Determine practice/slug or concept/slug
+    relative_path=$(echo "$dir" | sed 's|^exercises/||')
+
+    # Create summary.txt with header only on first failure
+    if [ ! -f "$summary_file" ]; then
+        echo "The following exercises have test failures or test errors:" > "$summary_file"
+    fi
+
+    # Append the correct path (practice/slug or concept/slug)
+    echo "$relative_path" >> "$summary_file"
   fi
+
 done
+
+exit $exit_code

bin/test-with-test-runner

@@ -12,6 +12,10 @@ docker pull exercism/java-test-runner
 
 exit_code=0
 
+summary_dir="exercises/build"
+summary_file="${summary_dir}/summary.txt"
+mkdir -p "$summary_dir"
+
 function run_test_runner() {
     local slug=$1
     local solution_dir=$2
@@ -56,6 +60,18 @@ function verify_exercise() {
 
     if [[ $(jq -r '.status' "${results_file}") != "pass" ]]; then
         echo "${slug}: ${implementation_file_key} solution did not pass the tests"
+
+        # Determine practice/slug or concept/slug
+        local relative_path=$(echo "$dir" | sed -E 's|.*/exercises/||')
+
+        # Create summary.txt with header only on first failure
+        if [ ! -f "$summary_file" ]; then
+            echo "The following exercises have test failures or test errors:" > "$summary_file"
+        fi
+
+        # Append the correct path (practice/slug or concept/slug)
+        echo "$relative_path" >> "$summary_file"
+
         exit_code=1
     fi
 

concepts/datetime/.meta/config.json

@@ -1,5 +1,9 @@
 {
   "blurb": "There are several classes in Java to work with dates and time.",
-  "authors": ["sanderploegsma"],
-  "contributors": []
+  "authors": [
+    "sanderploegsma"
+  ],
+  "contributors": [
+    "kahgoh"
+  ]
 }

concepts/datetime/about.md

@@ -118,7 +118,18 @@ printer.format(date);
 // => "December 3, 2007"
 ```
 
+A [locale][locale] can also be specified when creating the custom format to format and parse for different regions:
+
+```java
+DateTimeFormatter.ofPattern("MMMM d, yyyy", Locale.FRENCH).format(date);
+// => décembre 3, 2007
+
+DateTimeFormatter.ofPattern("MMMM d, yyyy", Locale.of("pt")).format(date);
+// => dezembro 3, 2007
+```
+
 [iso-8601]: https://en.wikipedia.org/wiki/ISO_8601
 [localdate-docs]: https://docs.oracle.com/javase/8/docs/api/java/time/LocalDate.html
 [localdatetime-docs]: https://docs.oracle.com/javase/8/docs/api/java/time/LocalDateTime.html
+[locale]: https://docs.oracle.com/javase/8/docs/api/java/util/Locale.html
 [datetimeformatter-docs]: https://docs.oracle.com/javase/8/docs/api/java/time/format/DateTimeFormatter.html

concepts/datetime/introduction.md

@@ -86,4 +86,14 @@ printer.format(date);
 // => "December 3, 2007"
 ```
 
+A locale can also be specified when creating the custom format to format and parse for different regions:
+
+```java
+DateTimeFormatter.ofPattern("MMMM d, yyyy", Locale.FRENCH).format(date);
+// => décembre 3, 2007
+
+DateTimeFormatter.ofPattern("MMMM d, yyyy", Locale.of("pt")).format(date);
+// => dezembro 3, 2007
+```
+
 [iso-8601]: https://en.wikipedia.org/wiki/ISO_8601

config.json

@@ -1211,6 +1211,20 @@
         ],
         "difficulty": 6
       },
+      {
+        "slug": "baffling-birthdays",
+        "name": "Baffling Birthdays",
+        "uuid": "b534049a-5920-4906-9091-0fa6d81a3636",
+        "practices": [],
+        "prerequisites": [
+          "for-loops",
+          "lists",
+          "sets",
+          "randomness",
+          "datetime"
+        ],
+        "difficulty": 6
+      },
       {
         "slug": "bank-account",
         "name": "Bank Account",

exercises/build.gradle

@@ -78,8 +78,8 @@ subprojects {
 
     // configuration of the linter
     checkstyle {
-      toolVersion '10.7.0'
-      configFile file("checkstyle.xml")
+      toolVersion = '10.7.0'
+      configFile = file("$rootDir/checkstyle.xml")
       sourceSets = [project.sourceSets.main, project.sourceSets.test]
     }