BugFix IPv6 prefix manifest search & unit tests#7
Merged
Conversation
🚀 PR Status Dashboard📊 Workflow Status⏳ CI: pending 🌐 Preview EnvironmentsSPA Preview: https://bug-ipv6-prefix-search.albatross-5kt.pages.dev 📝 PR DetailsBranch: This comment is automatically updated by the PR Update workflow |
Deploying albatross with
|
| Latest commit: |
fa74927
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://370266be.albatross-5kt.pages.dev |
| Branch Preview URL: | https://bug-ipv6-prefix-search.albatross-5kt.pages.dev |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
This pull request introduces comprehensive improvements to testing, build automation, and IPv6 support, along with updates to cloud provider IP manifests and documentation. The most significant changes include adding a full unit test suite for IP range matching, enhancing the build process to run tests automatically, extending the application to support IPv6 cloud IP range matching, and updating documentation to reflect these enhancements.
Testing and Build Automation
Tests/Albatross.Tests.csproj) with 41 unit tests covering IPv4 and IPv6 matching, edge cases, and real-world cloud provider ranges.build.sh) to detect and use the appropriate .NET SDK, run all tests before building, and abort the build if any tests fail..vscode/tasks.jsonto add "test" and "build-and-test" tasks for easier local development and validation.Albatross.csprojto exclude theTestsfolder from the main build to prevent duplicate compilation.IPv6 Support and Application Logic
Pages/Home.razorto support IPv6 cloud provider ranges, including robust validation for prefix lengths and improved error handling. [1] [2] [3] [4]Documentation Updates
README.mdwith detailed instructions on running tests, test coverage breakdown, CI/CD integration, and an updated project structure reflecting the new test suite. [1] [2]Cloud Provider Manifest Updates
manifest-backup/GCP.json) with new and expanded IPv4/IPv6 prefixes, ensuring the application's data is current. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16]These changes collectively improve the reliability, maintainability, and functionality of the project, especially regarding cloud IP range detection and automated quality assurance.