fix: Follow Codecov documentation for test analytics

Separate coverage and test results uploads as recommended:
- Use codecov-action@v5 for coverage
- Use test-results-action@v1 for test results
- Add junit_family=legacy option to pytest command
- Remove explicit file parameter from test-results-action

Author: github-actions

.github/workflows/tests.yml

@@ -62,13 +62,19 @@ jobs:
         run: uv sync --group dev --all-extras
 
       - name: Run tests
-        run: uv run pytest tests/ -v --tb=short --cov=src/ryandata_address_utils --cov-report=xml --cov-report=term --junitxml=junit.xml
+        run: uv run pytest tests/ --cov --junitxml=junit.xml -o junit_family=legacy --cov=src/ryandata_address_utils --cov-report=xml --cov-report=term
 
-      - name: Upload coverage and test results to Codecov
-        uses: codecov/codecov-action@v4
+      - name: Upload coverage to Codecov
+        uses: codecov/codecov-action@v5
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
-          file: ./coverage.xml,./junit.xml
+          file: ./coverage.xml
           flags: unittests
           name: codecov-umbrella
           fail_ci_if_error: false
+
+      - name: Upload test results to Codecov
+        if: ${{ !cancelled() }}
+        uses: codecov/test-results-action@v1
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}