Erhöhe die Testabdeckung für ETL-Services und füge neue Tests für den…

[christopherdukart]

… Übersetzungsbatchkontext, den Confluence-Renderer, den URL-Builder und den Bildoptimierer hinzu. Setze die Coverage-Schwelle auf 70-80 und aktualisiere die CI-Konfiguration für die Coverage-Berichterstattung.

Summary

Describe what this PR changes and why.

Related Issues

• Closes #
• Related #

Type of change

• Bug fix
• New feature
• Refactor
• Documentation update
• CI / tooling

Validation

• dotnet build ConfluenceSynkMD.slnx --configuration Release
• dotnet test ConfluenceSynkMD.slnx --configuration Release --no-build
• Added/updated tests where appropriate

Breaking changes

• No
• Yes (describe below)

Checklist

• I followed the contribution guidelines
• I updated relevant docs
• I did not include secrets or private data

[Copilot]

Pull request overview

Diese PR erhöht die Testabdeckung rund um zentrale ETL-/Rendering-/Service-Komponenten und passt die CI-Pipeline an, um Code-Coverage zu erzeugen und per Threshold zu erzwingen.

Changes:

• Neue Unit-Tests für TranslationBatchContext, ConfluenceRenderer, ConfluenceUrlBuilder und ImageOptimizer.
• Anpassung der Coverlet-Threshold-Konfiguration im Testprojekt.
• CI-Workflow-Update: Coverage-Erzeugung via Coverlet MSBuild + Upload der Coverage-Artefakte/Codecov-Upload.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
tests/ConfluenceSynkMD.Tests/Services/ImageOptimizerTests.cs	Neue Tests für Bildoptimierung (Disabled/Nonexistent/Resize/Load-Failure).
tests/ConfluenceSynkMD.Tests/Services/ConfluenceUrlBuilderTests.cs	Neue Tests für URL-Strategie/Encoding/Fallbacks.
tests/ConfluenceSynkMD.Tests/Markdig/ConfluenceRendererTests.cs	Neue Tests für Default-State und registrierte Renderer.
tests/ConfluenceSynkMD.Tests/ETL/Core/TranslationBatchContextTests.cs	Neue Tests für Initialzustand und Mutierbarkeit des Batch-Kontexts.
tests/ConfluenceSynkMD.Tests/ConfluenceSynkMD.Tests.csproj	Threshold-Anhebung und ThresholdStat=total.
.github/workflows/ci.yml	dotnet test mit Coverlet MSBuild, Threshold-Enforcement, Artifact-/Codecov-Upload.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

PR-Beschreibung erwähnt eine Coverage-Schwelle von 70–80, hier wird jedoch 65 gesetzt. Bitte entweder die PR-Beschreibung anpassen oder den Threshold konsistent auf den intendierten Wert erhöhen (und CI/Tests entsprechend).

[Copilot]

Die Threshold-Parameter (ThresholdType/Stat/Value) werden sowohl im Testprojekt (.csproj) als auch hier in der CI gesetzt. Das ist fehleranfällig (Drift bei zukünftigen Änderungen). Besser: Thresholds nur an einer Stelle definieren (z.B. im .csproj/Directory.Build.props) und in der CI nur CollectCoverage/Output/Format konfigurieren.

Suggested change

	- name: Test with coverage (enforced threshold)
	run: |
	dotnet test ConfluenceSynkMD.slnx \
	--no-build \
	--configuration Release \
	/p:CollectCoverage=true \
	/p:CoverletOutput=./TestResults/Coverage/ \
	/p:CoverletOutputFormat="cobertura,json" \
	/p:ThresholdType=line \
	/p:ThresholdStat=total \
	/p:Threshold=65
	- name: Test with coverage
	run: |
	dotnet test ConfluenceSynkMD.slnx \
	--no-build \
	--configuration Release \
	/p:CollectCoverage=true \
	/p:CoverletOutput=./TestResults/Coverage/ \
	/p:CoverletOutputFormat="cobertura,json"

[Copilot]

Die Assertions prüfen Renderer über GetType().Name == "…", was bei Umbenennungen/Refactors unnötig fragil ist. Da die Renderer-Typen im Projekt verfügbar sind, besser direkt auf die Typen prüfen (z.B. r is HeadingRenderer) statt auf String-Namen.

Suggested change

	sut.ObjectRenderers.Should().Contain(r => r.GetType().Name == "HeadingRenderer");
	sut.ObjectRenderers.Should().Contain(r => r.GetType().Name == "CodeBlockRenderer");
	sut.ObjectRenderers.Should().Contain(r => r.GetType().Name == "LinkInlineRenderer");
	sut.ObjectRenderers.Should().Contain(r => r.GetType().Name == "MathInlineRenderer");
	sut.ObjectRenderers.Should().Contain(r => r.GetType().Name == "MathBlockRenderer");
	sut.ObjectRenderers.Should().Contain(r => r is HeadingRenderer);
	sut.ObjectRenderers.Should().Contain(r => r is CodeBlockRenderer);
	sut.ObjectRenderers.Should().Contain(r => r is LinkInlineRenderer);
	sut.ObjectRenderers.Should().Contain(r => r is MathInlineRenderer);
	sut.ObjectRenderers.Should().Contain(r => r is MathBlockRenderer);

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!