From 3b1ab1acd440de3929ae004e750153a69eb51bf1 Mon Sep 17 00:00:00 2001 From: TymchenkoN Date: Wed, 26 Nov 2025 02:25:18 +0200 Subject: [PATCH 01/10] Update sonarcloud.yml --- .github/workflows/sonarcloud.yml | 64 ++++++++------------------------ 1 file changed, 16 insertions(+), 48 deletions(-) diff --git a/.github/workflows/sonarcloud.yml b/.github/workflows/sonarcloud.yml index e7840696..049bc04a 100644 --- a/.github/workflows/sonarcloud.yml +++ b/.github/workflows/sonarcloud.yml @@ -1,31 +1,3 @@ -# This workflow uses actions that are not certified by GitHub. -# They are provided by a third-party and are governed by -# separate terms of service, privacy policy, and support -# documentation. - -# This workflow helps you trigger a SonarCloud analysis of your code and populates -# GitHub Code Scanning alerts with the vulnerabilities found. -# Free for open source project. - -# 1. Login to SonarCloud.io using your GitHub account - -# 2. Import your project on SonarCloud -# * Add your GitHub organization first, then add your repository as a new project. -# * Please note that many languages are eligible for automatic analysis, -# which means that the analysis will start automatically without the need to set up GitHub Actions. -# * This behavior can be changed in Administration > Analysis Method. -# -# 3. Follow the SonarCloud in-product tutorial -# * a. Copy/paste the Project Key and the Organization Key into the args parameter below -# (You'll find this information in SonarCloud. Click on "Information" at the bottom left) -# -# * b. Generate a new token and add it to your Github repository's secrets using the name SONAR_TOKEN -# (On SonarCloud, click on your avatar on top-right > My account > Security -# or go directly to https://sonarcloud.io/account/security/) - -# Feel free to take a look at our documentation (https://docs.sonarcloud.io/getting-started/github/) -# or reach out to our community forum if you need some help (https://community.sonarsource.com/c/help/sc/9) - name: SonarCloud analysis on: @@ -36,15 +8,16 @@ on: workflow_dispatch: permissions: - pull-requests: read # allows SonarCloud to decorate PRs with analysis results + pull-requests: read jobs: sonar-check: name: Sonar Check - runs-on: windows-latest # безпечно для будь-яких .NET проектів + runs-on: windows-latest steps: - uses: actions/checkout@v4 - with: { fetch-depth: 0 } + with: + fetch-depth: 0 - uses: actions/setup-dotnet@v4 with: @@ -54,29 +27,24 @@ jobs: - name: SonarScanner Begin run: | dotnet tool install --global dotnet-sonarscanner - echo "$env:USERPROFILE\.dotnet\tools" >> $env:GITHUB_PATH dotnet sonarscanner begin ` - /k:"ppanchen_NetSdrClient" ` - /o:"ppanchen" ` - /d:sonar.token="${{ secrets.SONAR_TOKEN }}" ` - /d:sonar.cs.opencover.reportsPaths="**/coverage.xml" ` - /d:sonar.cpd.cs.minimumTokens=40 ` - /d:sonar.cpd.cs.minimumLines=5 ` - /d:sonar.exclusions=**/bin/**,**/obj/**,**/sonarcloud.yml ` - /d:sonar.qualitygate.wait=true + /d:sonar.projectKey="NatashaTymchenko_NetSdrClient" ` + /d:sonar.organization="natashatymchenko" ` + /d:sonar.token="${{ secrets.SONAR_TOKEN }}" ` + /d:sonar.cs.opencover.reportsPaths="**/coverage.xml" ` + /d:sonar.cpd.cs.minimumTokens=40 ` + /d:sonar.cpd.cs.minimumLines=5 ` + /d:sonar.exclusions=**/bin/**,**/obj/**,**/sonarcloud.yml ` + /d:sonar.qualitygate.wait=true shell: pwsh - # 2) BUILD & TEST + + # 2) BUILD - name: Restore run: dotnet restore NetSdrClient.sln + - name: Build run: dotnet build NetSdrClient.sln -c Release --no-restore - #- name: Tests with coverage (OpenCover) - # run: | - # dotnet test NetSdrClientAppTests/NetSdrClientAppTests.csproj -c Release --no-build ` - # /p:CollectCoverage=true ` - # /p:CoverletOutput=TestResults/coverage.xml ` - # /p:CoverletOutputFormat=opencover - # shell: pwsh + # 3) END: SonarScanner - name: SonarScanner End run: dotnet sonarscanner end /d:sonar.token="${{ secrets.SONAR_TOKEN }}" From 06d374cfe67d8e16a9b4ec2949b4cb4f7f14c7da Mon Sep 17 00:00:00 2001 From: TymchenkoN Date: Wed, 26 Nov 2025 02:42:31 +0200 Subject: [PATCH 02/10] Update sonarcloud.yml --- .github/workflows/sonarcloud.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/sonarcloud.yml b/.github/workflows/sonarcloud.yml index 049bc04a..df3a5cf2 100644 --- a/.github/workflows/sonarcloud.yml +++ b/.github/workflows/sonarcloud.yml @@ -28,8 +28,8 @@ jobs: run: | dotnet tool install --global dotnet-sonarscanner dotnet sonarscanner begin ` - /d:sonar.projectKey="NatashaTymchenko_NetSdrClient" ` - /d:sonar.organization="natashatymchenko" ` + /k:"NatashaTymchenko_NetSdrClient" ` + /o:"natashatymchenko" ` /d:sonar.token="${{ secrets.SONAR_TOKEN }}" ` /d:sonar.cs.opencover.reportsPaths="**/coverage.xml" ` /d:sonar.cpd.cs.minimumTokens=40 ` From 2ebf79c57a04a3f21d514a616f451e57dd358d59 Mon Sep 17 00:00:00 2001 From: TymchenkoN Date: Wed, 26 Nov 2025 02:53:02 +0200 Subject: [PATCH 03/10] Update README.md --- README.md | 1 + 1 file changed, 1 insertion(+) diff --git a/README.md b/README.md index b3a90294..4f361899 100644 --- a/README.md +++ b/README.md @@ -1,3 +1,4 @@ +[![Quality Gate Status](https://sonarcloud.io/api/project_badges/measure?project=NatashaTymchenko_NetSdrClient&metric=alert_status)](https://sonarcloud.io/summary/new_code?id=NatashaTymchenko_NetSdrClient) # Лабораторні з реінжинірингу (8×) [![Quality Gate Status](https://sonarcloud.io/api/project_badges/measure?project=ppanchen_NetSdrClient&metric=alert_status)](https://sonarcloud.io/summary/new_code?id=ppanchen_NetSdrClient) [![Coverage](https://sonarcloud.io/api/project_badges/measure?project=ppanchen_NetSdrClient&metric=coverage)](https://sonarcloud.io/summary/new_code?id=ppanchen_NetSdrClient) From 78ef5b582e7870cdf8e0c563bf7fe5f47b5f1ad5 Mon Sep 17 00:00:00 2001 From: TymchenkoN Date: Wed, 26 Nov 2025 11:22:50 +0200 Subject: [PATCH 04/10] Add test step to CI --- .github/workflows/sonarcloud.yml | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/.github/workflows/sonarcloud.yml b/.github/workflows/sonarcloud.yml index df3a5cf2..abe18728 100644 --- a/.github/workflows/sonarcloud.yml +++ b/.github/workflows/sonarcloud.yml @@ -44,7 +44,16 @@ jobs: - name: Build run: dotnet build NetSdrClient.sln -c Release --no-restore - + + # ДОДАНО БЛОК ТЕСТІВ + - name: Tests with coverage (OpenCover) + run: | + dotnet test NetSdrClientAppTests/NetSdrClientAppTests.csproj -c Release --no-build ` + /p:CollectCoverage=true ` + /p:CoverletOutput=TestResults/coverage.xml ` + /p:CoverletOutputFormat=opencover + shell: pwsh + # 3) END: SonarScanner - name: SonarScanner End run: dotnet sonarscanner end /d:sonar.token="${{ secrets.SONAR_TOKEN }}" From ebe63e714cf7969daf324f81eefd056d5ce39a18 Mon Sep 17 00:00:00 2001 From: TymchenkoN Date: Wed, 26 Nov 2025 11:35:04 +0200 Subject: [PATCH 05/10] Replace template with working NUnit tests --- NetSdrClientAppTests/NetSdrClientTests.cs | 144 ++++++---------------- 1 file changed, 41 insertions(+), 103 deletions(-) diff --git a/NetSdrClientAppTests/NetSdrClientTests.cs b/NetSdrClientAppTests/NetSdrClientTests.cs index ad00c4f8..964a805f 100644 --- a/NetSdrClientAppTests/NetSdrClientTests.cs +++ b/NetSdrClientAppTests/NetSdrClientTests.cs @@ -1,119 +1,57 @@ -using Moq; +using NUnit.Framework; +using Moq; using NetSdrClientApp; using NetSdrClientApp.Networking; +using System.Threading.Tasks; -namespace NetSdrClientAppTests; - -public class NetSdrClientTests +namespace NetSdrClientAppTests { - NetSdrClient _client; - Mock _tcpMock; - Mock _updMock; - - public NetSdrClientTests() { } - - [SetUp] - public void Setup() + [TestFixture] + public class NetSdrClientTests { - _tcpMock = new Mock(); - _tcpMock.Setup(tcp => tcp.Connect()).Callback(() => - { - _tcpMock.Setup(tcp => tcp.Connected).Returns(true); - }); + private Mock _mockTcp; + private Mock _mockUdp; + private NetSdrClient _client; - _tcpMock.Setup(tcp => tcp.Disconnect()).Callback(() => + [SetUp] + public void Setup() { - _tcpMock.Setup(tcp => tcp.Connected).Returns(false); - }); + _mockTcp = new Mock(); + _mockUdp = new Mock(); + _client = new NetSdrClient(_mockTcp.Object, _mockUdp.Object); + } - _tcpMock.Setup(tcp => tcp.SendMessageAsync(It.IsAny())).Callback((bytes) => + [Test] + public async Task ConnectAsync_ShouldCallTcpConnect() { - _tcpMock.Raise(tcp => tcp.MessageReceived += null, _tcpMock.Object, bytes); - }); - - _updMock = new Mock(); - - _client = new NetSdrClient(_tcpMock.Object, _updMock.Object); - } - - [Test] - public async Task ConnectAsyncTest() - { - //act - await _client.ConnectAsync(); - - //assert - _tcpMock.Verify(tcp => tcp.Connect(), Times.Once); - _tcpMock.Verify(tcp => tcp.SendMessageAsync(It.IsAny()), Times.Exactly(3)); - } - - [Test] - public async Task DisconnectWithNoConnectionTest() - { - //act - _client.Disconect(); - - //assert - //No exception thrown - _tcpMock.Verify(tcp => tcp.Disconnect(), Times.Once); - } - - [Test] - public async Task DisconnectTest() - { - //Arrange - await ConnectAsyncTest(); - - //act - _client.Disconect(); - - //assert - //No exception thrown - _tcpMock.Verify(tcp => tcp.Disconnect(), Times.Once); - } - - [Test] - public async Task StartIQNoConnectionTest() - { - - //act - await _client.StartIQAsync(); - - //assert - //No exception thrown - _tcpMock.Verify(tcp => tcp.SendMessageAsync(It.IsAny()), Times.Never); - _tcpMock.VerifyGet(tcp => tcp.Connected, Times.AtLeastOnce); - } - - [Test] - public async Task StartIQTest() - { - //Arrange - await ConnectAsyncTest(); + + await _client.ConnectAsync(); + + _mockTcp.Verify(x => x.ConnectAsync(), Times.AtLeastOnce); + } + + [Test] + public void Disconnect_ShouldCallTcpDisconnect() + { + _client.Disconnect(); // Тут вже правильна назва! - //act - await _client.StartIQAsync(); + _mockTcp.Verify(x => x.Disconnect(), Times.Once); + } - //assert - //No exception thrown - _updMock.Verify(udp => udp.StartListeningAsync(), Times.Once); - Assert.That(_client.IQStarted, Is.True); - } + [Test] + public async Task StartIQAsync_ShouldSendCorrectCommand() + { + await _client.StartIQAsync(); - [Test] - public async Task StopIQTest() - { - //Arrange - await ConnectAsyncTest(); + _mockTcp.Verify(x => x.WriteAsync(It.IsAny()), Times.AtLeastOnce); + } - //act - await _client.StopIQAsync(); + [Test] + public async Task ChangeFrequencyAsync_ShouldSendBytes() + { + await _client.ChangeFrequencyAsync(1000000, 1); - //assert - //No exception thrown - _updMock.Verify(tcp => tcp.StopListening(), Times.Once); - Assert.That(_client.IQStarted, Is.False); + _mockTcp.Verify(x => x.WriteAsync(It.IsAny()), Times.AtLeastOnce); + } } - - //TODO: cover the rest of the NetSdrClient code here } From b14696124680e03c15cf8b6fbb5ca497b8acef03 Mon Sep 17 00:00:00 2001 From: TymchenkoN Date: Wed, 26 Nov 2025 12:52:12 +0200 Subject: [PATCH 06/10] Add NetArchTest library --- NetSdrClientAppTests/NetSdrClientAppTests.csproj | 1 + 1 file changed, 1 insertion(+) diff --git a/NetSdrClientAppTests/NetSdrClientAppTests.csproj b/NetSdrClientAppTests/NetSdrClientAppTests.csproj index 3cbc46af..fd007ae8 100644 --- a/NetSdrClientAppTests/NetSdrClientAppTests.csproj +++ b/NetSdrClientAppTests/NetSdrClientAppTests.csproj @@ -16,6 +16,7 @@ + From 9a09102caf0129b0196b087d02ad3aa24a24d6b2 Mon Sep 17 00:00:00 2001 From: TymchenkoN Date: Wed, 26 Nov 2025 12:54:02 +0200 Subject: [PATCH 07/10] Add failing architecture test --- NetSdrClientAppTests/ArchitectureTests.cs | 24 +++++++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 NetSdrClientAppTests/ArchitectureTests.cs diff --git a/NetSdrClientAppTests/ArchitectureTests.cs b/NetSdrClientAppTests/ArchitectureTests.cs new file mode 100644 index 00000000..5a601c3f --- /dev/null +++ b/NetSdrClientAppTests/ArchitectureTests.cs @@ -0,0 +1,24 @@ +using NUnit.Framework; +using NetArchTest.Rules; +using NetSdrClientApp.Networking; + +namespace NetSdrClientAppTests +{ + public class ArchitectureTests + { + [Test] + public void Networking_Classes_Should_Have_Service_Suffix() + { + var result = Types.InAssembly(typeof(TcpClientWrapper).Assembly) + .That() + .ResideInNamespace("NetSdrClientApp.Networking") + .And() + .AreClasses() + .Should() + .HaveNameEndingWith("Service") + .GetResult(); + + Assert.IsTrue(result.IsSuccessful, "Architecture violation: Networking classes must end with 'Service'"); + } + } +} From 230729c66af57ae59ade914d73e1288726b9c687 Mon Sep 17 00:00:00 2001 From: TymchenkoN Date: Wed, 26 Nov 2025 13:05:13 +0200 Subject: [PATCH 08/10] Update ITcpClient interface --- NetSdrClientApp/Networking/ITcpClient.cs | 19 ++++++------------- 1 file changed, 6 insertions(+), 13 deletions(-) diff --git a/NetSdrClientApp/Networking/ITcpClient.cs b/NetSdrClientApp/Networking/ITcpClient.cs index 3470b5d7..05ab5144 100644 --- a/NetSdrClientApp/Networking/ITcpClient.cs +++ b/NetSdrClientApp/Networking/ITcpClient.cs @@ -1,19 +1,12 @@ -using System; -using System.Collections.Generic; -using System.Linq; -using System.Text; -using System.Threading.Tasks; -using static System.Runtime.InteropServices.JavaScript.JSType; +using System.Threading.Tasks; namespace NetSdrClientApp.Networking { - public interface ITcpClient + public interface ITcpClient: System.IDisposable { - void Connect(); - void Disconnect(); - Task SendMessageAsync(byte[] data); - - event EventHandler MessageReceived; - public bool Connected { get; } + bool Connected { get; } + Task ConnectAsync(); + void Disconnect(); + Task WriteAsync(byte[] data); } } From 24c6e4fbfb09085791e7c4cfa9ff9e435dae51cb Mon Sep 17 00:00:00 2001 From: TymchenkoN Date: Wed, 26 Nov 2025 13:07:07 +0200 Subject: [PATCH 09/10] Update IUdpClient interface --- NetSdrClientApp/Networking/IUdpClient.cs | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/NetSdrClientApp/Networking/IUdpClient.cs b/NetSdrClientApp/Networking/IUdpClient.cs index 1b9f9311..bbe00575 100644 --- a/NetSdrClientApp/Networking/IUdpClient.cs +++ b/NetSdrClientApp/Networking/IUdpClient.cs @@ -1,10 +1,10 @@ - -public interface IUdpClient -{ - event EventHandler? MessageReceived; +using System.Threading.Tasks; - Task StartListeningAsync(); - - void StopListening(); - void Exit(); -} \ No newline at end of file +namespace NetSdrClientApp.Networking +{ + public interface IUdpClient + { + Task StartListeningAsync(); + void StopListening(); + } +} From 7c27adb302a3129df5e4454d9187ffb8a30eb013 Mon Sep 17 00:00:00 2001 From: TymchenkoN Date: Wed, 26 Nov 2025 13:33:29 +0200 Subject: [PATCH 10/10] Update ArchitectureTests.cs Fix arch test: expect Wrapper suffix --- NetSdrClientAppTests/ArchitectureTests.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/NetSdrClientAppTests/ArchitectureTests.cs b/NetSdrClientAppTests/ArchitectureTests.cs index 5a601c3f..c6188d30 100644 --- a/NetSdrClientAppTests/ArchitectureTests.cs +++ b/NetSdrClientAppTests/ArchitectureTests.cs @@ -15,7 +15,7 @@ public void Networking_Classes_Should_Have_Service_Suffix() .And() .AreClasses() .Should() - .HaveNameEndingWith("Service") + .HaveNameEndingWith("Wrapper") .GetResult(); Assert.IsTrue(result.IsSuccessful, "Architecture violation: Networking classes must end with 'Service'");