-
Notifications
You must be signed in to change notification settings - Fork 0
First version Test Coverage and .NET 9 upgrade #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Closed
Changes from all commits
Commits
Show all changes
8 commits
Select commit
Hold shift + click to select a range
14a140f
First version Test Coverage and .NET 9 upgrade
PerPage 0e0415c
Fix some nuget conflicts and namespace ambiguity to make stuff compile
ruudkobes e29a12d
Add security analyzer rule for Newtonsoft typenamehandling
ruudkobes 097719c
First version Test Coverage and .NET 9 upgrade
PerPage eff2f9f
Merge branch 'Test-Coverage' of https://github.com/Xablu/wallet-frame…
PerPage ddd0e59
Should't commit the offending security risk of course ;)
ruudkobes 027b7ad
2e commit this time with more test coverage
PerPage 0f370e2
Merge branch 'Test-Coverage' of https://github.com/Xablu/wallet-frame…
PerPage File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,59 @@ | ||
| { | ||
| "documentation_registry": [ | ||
| { | ||
| "path": "docs/UserBlueprint.md", | ||
| "title": "User Blueprint", | ||
| "type": "requirements", | ||
| "status": "active", | ||
| "last_modified": "2025-05-19T00:37:55Z" | ||
| }, | ||
| { | ||
| "path": "docs/master_acceptance_test_plan.md", | ||
| "description": "Comprehensive high-level end-to-end acceptance tests.", | ||
| "type": "test plan", | ||
| "timestamp": "2025-05-19T08:20:18Z" | ||
| }, | ||
| { | ||
| "path": "docs/PRDMasterPlan.md", | ||
| "description": "Detailed Master Project Plan with AI verifiable tasks.", | ||
| "type": "project plan", | ||
| "timestamp": "2025-05-19T08:20:18Z" | ||
| }, | ||
| { | ||
| "path": "docs/architecture/HighLevelArchitecture.md", | ||
| "description": "High-Level Architecture document.", | ||
| "type": "architecture", | ||
| "timestamp": "2025-05-19T08:29:30Z" | ||
| }, | ||
| { | ||
| "path": "docs/research/github_template_research_report.md", | ||
| "description": "GitHub Template Research Report findings.", | ||
| "type": "research report", | ||
| "timestamp": "2025-05-19T08:29:30Z" | ||
| }, | ||
| { | ||
| "path": "docs/FrameworkScaffoldReport.md", | ||
| "description": "Report summarizing framework scaffolding activities.", | ||
| "type": "report", | ||
| "timestamp": "2025-05-19T08:29:30Z" | ||
| }, | ||
| { | ||
| "path": "docs/updates/package_upgrades_20250519.md", | ||
| "description": "Details of package upgrades performed on 2025-05-19.", | ||
| "type": "update", | ||
| "timestamp": "2025-05-19T14:15:51Z" | ||
| }, | ||
| { | ||
| "path": "docs/updates/package_upgrades_20250519.md", | ||
| "description": "Details of package upgrades performed on 2025-05-19.", | ||
| "type": "update", | ||
| "timestamp": "2025-05-19T15:03:29Z" | ||
| }, | ||
| { | ||
| "path": "docs/updates/refinement-analysis-20250515-190428-doc-update.md", | ||
| "description": "Analysis and refinement of project documentation.", | ||
| "type": "report", | ||
| "timestamp": "2025-05-20T01:00:00Z" | ||
| } | ||
| ] | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,95 @@ | ||
| name: .NET CI Pipeline | ||
|
|
||
| on: | ||
| push: | ||
| branches: | ||
| - main | ||
| - develop | ||
| pull_request: | ||
| branches: | ||
| - main | ||
| - develop | ||
|
|
||
| jobs: | ||
| build-and-test: | ||
| runs-on: ubuntu-latest | ||
|
|
||
| steps: | ||
| - name: Checkout code | ||
| uses: actions/checkout@v4 | ||
|
|
||
| - name: Setup .NET | ||
| uses: actions/setup-dotnet@v4 | ||
| with: | ||
| dotnet-version: | | ||
| 6.0.x | ||
| 7.0.x | ||
| 8.0.x # Assuming support for multiple .NET versions based on project structure | ||
|
|
||
| - name: Restore dependencies | ||
| run: dotnet restore src/WalletFramework.sln | ||
|
|
||
| - name: Build solution | ||
| run: dotnet build src/WalletFramework.sln --no-restore | ||
|
|
||
| - name: Run Unit Tests | ||
| run: dotnet test src/WalletFramework.sln --no-build --verbosity normal --filter "Category=Unit" --collect:"XPlat Code Coverage" | ||
|
|
||
| - name: Run Integration Tests | ||
| run: dotnet test src/WalletFramework.sln --no-build --verbosity normal --filter "Category=Integration" --collect:"XPlat Code Coverage" | ||
|
|
||
| # BDD/E2E tests might require a different setup (e.g., SpecFlow, BrowserStack) | ||
| # This step is a placeholder and needs further implementation based on the specific test framework and infrastructure | ||
| - name: Run BDD/E2E Tests | ||
| run: | | ||
| echo "Running BDD/E2E tests..." | ||
| # Placeholder for actual BDD/E2E test execution command | ||
| # dotnet test src/WalletFramework.BDDE2E.Tests/WalletFramework.BDDE2E.Tests.csproj --no-build --verbosity normal | ||
|
|
||
| - name: Run Roslyn Analyzers | ||
| run: dotnet build src/WalletFramework.sln /m /p:EnableNETAnalyzers=true /p:AnalysisMode=AllEnabledByDefault | ||
|
|
||
| # OWASP ZAP scan requires a running application instance. | ||
| # This step is a placeholder and needs setup for running the application and ZAP scan. | ||
| - name: Run OWASP ZAP Scan | ||
| run: | | ||
| echo "Running OWASP ZAP scan..." | ||
| zap-cli quickscan -r http://localhost:5000 | ||
|
|
||
| # OWASP Dependency-Check requires setup and configuration. | ||
| # This step is a placeholder and needs setup for Dependency-Check execution. | ||
| - name: Run OWASP Dependency-Check | ||
| run: | | ||
| echo "Running OWASP Dependency-Check..." | ||
| dependency-check --scan . --format JUNIT --out . | ||
|
|
||
| - name: Upload Test Coverage Report | ||
| uses: actions/upload-artifact@v4 | ||
| with: | ||
| name: test-coverage-report | ||
| path: | | ||
| **/TestResults/*/coverage.cobertura.xml | ||
| **/TestResults/*/coverage.json | ||
|
|
||
| # Performance benchmarks might require a separate job or specific setup. | ||
| # This step is a placeholder for collecting and publishing performance benchmark results. | ||
| - name: Upload Performance Benchmark Results | ||
| run: | | ||
| echo "Collecting and uploading performance benchmark results..." | ||
| # Placeholder for collecting benchmark results | ||
| # For example, if using BenchmarkDotNet, results might be in BenchmarkDotNet.Artifacts | ||
| # find . -name "*-results.json" -print0 | xargs -0 -I {} mv {} . | ||
| # uses: actions/upload-artifact@v4 | ||
| # with: | ||
| # name: performance-benchmark-results | ||
| # path: | | ||
| # *-results.json # Adjust path based on benchmark output | ||
|
|
||
| - name: Upload Security Scan Reports | ||
| uses: actions/upload-artifact@v4 | ||
| with: | ||
| name: security-scan-reports | ||
| path: | | ||
| # Adjust paths based on actual output locations of ZAP and Dependency-Check reports | ||
| # zap_report.html | ||
| # dependency-check-report.xml | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,35 @@ | ||
| { | ||
| "signals": [ | ||
| { | ||
| "id": "1716219283000", | ||
| "timestamp": "2025-05-20T01:00:00Z", | ||
| "source_orchestrator": "orchestrator-state-scribe", | ||
| "summary": "State Scribe recorded new event in .memory, updated .docsregistry with formal project documents." | ||
| }, | ||
| { | ||
| "id": "1716221692000", | ||
| "timestamp": "2025-05-20T12:20:00Z", | ||
| "source": "orchestrator-state-scribe", | ||
| "summary": "Summary: Roo Code diagnosed a missing WalletFramework.Core.Tests in test/WalletFramework.Core.Tests, and provided a five-step resolution: 1. **Diagnosis**: Confirm absence of the .csproj file. 2. **Project‐File Restoration**: Run `dotnet new xunit --name WalletFramework.Core.Tests --force` in test/WalletFramework.Core.Tests. 3. **Reference & Package Setup**: - Add `<ProjectReference Include=\"../../src/WalletFramework.Core/WalletFramework.Core.csproj\" />` - `dotnet add package Moq` - `dotnet add package coverlet.collector` 4. **Solution Integration**: `dotnet sln add test/WalletFramework.Core.Tests/WalletFramework.Core.Tests.csproj`. 5. **Verification**: `dotnet test test/WalletFramework.Core.Tests/WalletFramework.Core.Tests.csproj` should now succeed." | ||
| }, | ||
| { | ||
| "id": "1716223092000", | ||
| "timestamp": "2025-05-20T13:30:00Z", | ||
| "source_orchestrator": "orchestrator-state-scribe", | ||
| "summary": "State Scribe updated .memory and .docsregistry files based on PRDMasterPlan.md." | ||
| }, | ||
| { | ||
| "id": "1716211856000", | ||
| "timestamp": "2025-05-20T13:30:00Z", | ||
| "source_orchestrator": "orchestrator-state-scribe", | ||
| "handoff_reason_code": "task complete", | ||
| "summary": "The scaffolding activities for the Preparation phase have been completed successfully. The test harness was set up, the target framework of the `WalletFramework.Oid4Vc.Tests` project was updated to `net9.0`, and the tests were executed and passed. A Framework Scaffold Report has been created to summarize the scaffolding activities, tools used, and the initial project structure." | ||
| }, | ||
| { | ||
| "id": "1716223942000", | ||
| "timestamp": "2025-05-20T14:30:00Z", | ||
| "source_orchestrator": "orchestrator-state-scribe", | ||
| "summary": "Update .memory and .docsregistry files with test execution results for WalletFramework.Oid4Vc feature." | ||
| } | ||
| ] | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,20 @@ | ||
| { | ||
| "mcpServers": { | ||
| "github": { | ||
| "command": "docker", | ||
| "args": [ | ||
| "run", | ||
| "-i", | ||
| "--rm", | ||
| "-e", | ||
| "GITHUB_PERSONAL_ACCESS_TOKEN", | ||
| "ghcr.io/github/github-mcp-server" | ||
| ], | ||
| "env": { | ||
| "GITHUB_PERSONAL_ACCESS_TOKEN": "${input:github_token}" | ||
| }, | ||
| "disabled": true, | ||
| "alwaysAllow": [] | ||
| } | ||
| } | ||
| } |
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since the project has been upgraded to target .NET 9, include
9.0.xin thedotnet-versionmatrix to ensure CI builds and tests against .NET 9.