Added unit tests to TechnitiumLibrary.Net.Firewall#36
Added unit tests to TechnitiumLibrary.Net.Firewall#36zbalkan wants to merge 8 commits intoTechnitiumSoftware:masterfrom
Conversation
Signed-off-by: Zafer Balkan <zafer@zaferbalkan.com>
There was a problem hiding this comment.
Pull request overview
This PR introduces unit testing infrastructure to the TechnitiumLibrary project by adding a new test project targeting the TechnitiumLibrary.Net.Firewall library. It also adds a GitHub Actions workflow to run these tests automatically.
Changes:
- Added a new MSTest-based unit test project (TechnitiumLibrary.UnitTests) with tests for WindowsFirewall class
- Configured GitHub Actions workflow for automated testing on Windows using MSBuild
- Updated README with a unit testing status badge
Reviewed changes
Copilot reviewed 6 out of 8 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| TechnitiumLibrary.sln | Added the new UnitTests project to the solution with appropriate build configurations |
| TechnitiumLibrary.UnitTests/TechnitiumLibrary.UnitTests.csproj | New test project file configured with MSTest.Sdk 4.0.1 targeting .NET 9.0 |
| TechnitiumLibrary.UnitTests/TechnitiumLibrary.Net.Firewall/WindowsFirewallTests.cs | Unit tests for WindowsFirewall class methods, testing error cases and basic functionality |
| TechnitiumLibrary.UnitTests/MSTestSettings.cs | Configures MSTest to run tests in parallel at method level |
| .github/workflows/unit-testing.yml | GitHub Actions workflow to build and test the solution on Windows |
| README.md | Added Quality Assurance section with unit testing badge |
| TechnitiumLibrary/CollectionExtensions.cs | Added missing newline at end of file |
| TechnitiumLibrary/Base32.cs | Added missing newline at end of file |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
TechnitiumLibrary.UnitTests/TechnitiumLibrary.Net.Firewall/WindowsFirewallTests.cs
Show resolved
Hide resolved
TechnitiumLibrary.UnitTests/TechnitiumLibrary.Net.Firewall/WindowsFirewallTests.cs
Show resolved
Hide resolved
TechnitiumLibrary.UnitTests/TechnitiumLibrary.Net.Firewall/WindowsFirewallTests.cs
Show resolved
Hide resolved
TechnitiumLibrary.UnitTests/TechnitiumLibrary.Net.Firewall/WindowsFirewallTests.cs
Outdated
Show resolved
Hide resolved
TechnitiumLibrary.UnitTests/TechnitiumLibrary.Net.Firewall/WindowsFirewallTests.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 8 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
TechnitiumLibrary.UnitTests/TechnitiumLibrary.Net.Firewall/WindowsFirewallTests.cs
Show resolved
Hide resolved
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 8 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Relies on #29 getting merged.