List all unsupported file names in v2 push error

[harminius]

Resolves https://github.com/MerginMaps/server-private/issues/3529

POST /v2/projects/{id}/versions now rejects unsupported file names with 422 UnsupportedFilesDetected and lists every offending path, diff paths included, in unsupported_files. Before, it returned UploadError naming only the first file. This also applies to check_only dry runs.

The name check now runs over all files before the other per-file checks. v1 push keeps its 400 response, and its message now lists all files too.

[coveralls]

Coverage Report for CI Build 36997656905

Coverage decreased (-0.03%) to 92.709%

Details

• Coverage decreased (-0.03%) from the base build.
• Patch coverage: 32 of 32 lines across 4 files are fully covered (100%).
• 32 coverage regressions across 6 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

32 previously-covered lines in 6 files lost coverage.

File	Lines Losing Coverage	Coverage
server/mergin/auth/controller.py	19	93.21%
server/mergin/auth/models.py	7	96.85%
server/mergin/tests/test_audit_events.py	2	99.34%
server/mergin/tests/test_cli.py	2	98.92%
server/mergin/auth/app.py	1	97.01%
server/mergin/auth/listeners.py	1	94.29%

---

Coverage Stats

Relevant Lines:	11246
Covered Lines:	10426
Line Coverage:	92.71%
Coverage Strength:	0.93 hits per line

---

💛 - Coveralls

[MarcelGeo]

Leave here that code and just raise specific error. Do not need specific loops. That can simplify this code.

like if diff and not is_valid_path():
raise UnsupportedFileName.

[harminius]

Ok, I put the validation in the existing loop. But I need to raise the specific error only when the loop finishes.