Skip to content

Commit 8bd0841

Browse files
authored
Merge pull request #390 from siemens/feature/warningpipeline
fix: Sonar issue fix
2 parents 6eb3d2a + fb3b03d commit 8bd0841

35 files changed

+1478
-395
lines changed

src/AritfactoryUploader.UTest/PackageUploadHelperTest.cs

Lines changed: 155 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// --------------------------------------------------------------------------------------------------------------------
1+
// --------------------------------------------------------------------------------------------------------------------
22
// SPDX-FileCopyrightText: 2025 Siemens AG
33
//
44
// SPDX-License-Identifier: MIT
@@ -573,6 +573,160 @@ public async Task JfrogFoundPackagesAsync_CargoComponentMultipleCalls_AddsMultip
573573
Assert.AreEqual("tokio", displayPackagesInfo.JfrogFoundPackagesCargo[1].Name);
574574
}
575575

576+
[Test]
577+
public async Task JfrogNotFoundPackagesAsync_WithNullComponentType_DoesNotAddComponent()
578+
{
579+
// Arrange
580+
var item = new ComponentsToArtifactory
581+
{
582+
ComponentType = null,
583+
Name = "test-package",
584+
Version = "1.0.0"
585+
};
586+
var displayPackagesInfo = new DisplayPackagesInfo
587+
{
588+
JfrogNotFoundPackagesNpm = new List<ComponentsToArtifactory>(),
589+
JfrogNotFoundPackagesNuget = new List<ComponentsToArtifactory>(),
590+
JfrogNotFoundPackagesMaven = new List<ComponentsToArtifactory>(),
591+
JfrogNotFoundPackagesPython = new List<ComponentsToArtifactory>(),
592+
JfrogNotFoundPackagesConan = new List<ComponentsToArtifactory>(),
593+
JfrogNotFoundPackagesDebian = new List<ComponentsToArtifactory>(),
594+
JfrogNotFoundPackagesCargo = new List<ComponentsToArtifactory>(),
595+
JfrogNotFoundPackagesChoco = new List<ComponentsToArtifactory>()
596+
};
576597

598+
// Act
599+
await PackageUploadHelper.JfrogNotFoundPackagesAsync(item, displayPackagesInfo);
600+
601+
// Assert - No components should be added to any list
602+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesNpm.Count);
603+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesNuget.Count);
604+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesMaven.Count);
605+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesPython.Count);
606+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesConan.Count);
607+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesDebian.Count);
608+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesCargo.Count);
609+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesChoco.Count);
610+
}
611+
612+
[Test]
613+
public async Task JfrogNotFoundPackagesAsync_WithEmptyComponentType_DoesNotAddComponent()
614+
{
615+
// Arrange
616+
var item = new ComponentsToArtifactory
617+
{
618+
ComponentType = "",
619+
Name = "test-package",
620+
Version = "1.0.0"
621+
};
622+
var displayPackagesInfo = new DisplayPackagesInfo
623+
{
624+
JfrogNotFoundPackagesNpm = new List<ComponentsToArtifactory>(),
625+
JfrogNotFoundPackagesNuget = new List<ComponentsToArtifactory>(),
626+
JfrogNotFoundPackagesMaven = new List<ComponentsToArtifactory>(),
627+
JfrogNotFoundPackagesPython = new List<ComponentsToArtifactory>(),
628+
JfrogNotFoundPackagesConan = new List<ComponentsToArtifactory>(),
629+
JfrogNotFoundPackagesDebian = new List<ComponentsToArtifactory>(),
630+
JfrogNotFoundPackagesCargo = new List<ComponentsToArtifactory>(),
631+
JfrogNotFoundPackagesChoco = new List<ComponentsToArtifactory>()
632+
};
633+
634+
// Act
635+
await PackageUploadHelper.JfrogNotFoundPackagesAsync(item, displayPackagesInfo);
636+
637+
// Assert - No components should be added to any list
638+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesNpm.Count);
639+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesNuget.Count);
640+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesMaven.Count);
641+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesPython.Count);
642+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesConan.Count);
643+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesDebian.Count);
644+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesCargo.Count);
645+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesChoco.Count);
646+
}
647+
648+
[Test]
649+
public async Task JfrogNotFoundPackagesAsync_WithWhitespaceComponentType_DoesNotAddComponent()
650+
{
651+
// Arrange
652+
var item = new ComponentsToArtifactory
653+
{
654+
ComponentType = " ",
655+
Name = "test-package",
656+
Version = "1.0.0"
657+
};
658+
var displayPackagesInfo = new DisplayPackagesInfo
659+
{
660+
JfrogNotFoundPackagesNpm = new List<ComponentsToArtifactory>(),
661+
JfrogNotFoundPackagesNuget = new List<ComponentsToArtifactory>(),
662+
JfrogNotFoundPackagesMaven = new List<ComponentsToArtifactory>(),
663+
JfrogNotFoundPackagesPython = new List<ComponentsToArtifactory>(),
664+
JfrogNotFoundPackagesConan = new List<ComponentsToArtifactory>(),
665+
JfrogNotFoundPackagesDebian = new List<ComponentsToArtifactory>(),
666+
JfrogNotFoundPackagesCargo = new List<ComponentsToArtifactory>(),
667+
JfrogNotFoundPackagesChoco = new List<ComponentsToArtifactory>()
668+
};
669+
670+
// Act
671+
await PackageUploadHelper.JfrogNotFoundPackagesAsync(item, displayPackagesInfo);
672+
673+
// Assert - No components should be added to any list
674+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesNpm.Count);
675+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesNuget.Count);
676+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesMaven.Count);
677+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesPython.Count);
678+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesConan.Count);
679+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesDebian.Count);
680+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesCargo.Count);
681+
Assert.AreEqual(0, displayPackagesInfo.JfrogNotFoundPackagesChoco.Count);
682+
}
683+
684+
[Test]
685+
public void WriteCreatorKpiDataToConsole_ValidUploaderKpiData_LogsDataSuccessfully()
686+
{
687+
// Arrange
688+
var uploaderKpiData = new UploaderKpiData
689+
{
690+
ComponentInComparisonBOM = 100,
691+
ComponentNotApproved = 10,
692+
PackagesToBeUploaded = 90,
693+
PackagesUploadedToJfrog = 85,
694+
PackagesNotUploadedToJfrog = 5,
695+
DevPackagesUploaded = 15,
696+
DevPackagesNotUploadedToJfrog = 2,
697+
InternalPackagesUploaded = 20,
698+
InternalPackagesNotUploadedToJfrog = 3,
699+
PackagesNotExistingInRemoteCache = 4,
700+
PackagesNotUploadedDueToError = 1,
701+
TimeTakenByArtifactoryUploader = 45.5
702+
};
703+
704+
// Act & Assert - Should not throw any exceptions
705+
Assert.DoesNotThrow(() => PackageUploadHelper.WriteCreatorKpiDataToConsole(uploaderKpiData));
706+
}
707+
708+
[Test]
709+
public void WriteCreatorKpiDataToConsole_WithZeroValues_LogsDataSuccessfully()
710+
{
711+
// Arrange
712+
var uploaderKpiData = new UploaderKpiData
713+
{
714+
ComponentInComparisonBOM = 0,
715+
ComponentNotApproved = 0,
716+
PackagesToBeUploaded = 0,
717+
PackagesUploadedToJfrog = 0,
718+
PackagesNotUploadedToJfrog = 0,
719+
DevPackagesUploaded = 0,
720+
DevPackagesNotUploadedToJfrog = 0,
721+
InternalPackagesUploaded = 0,
722+
InternalPackagesNotUploadedToJfrog = 0,
723+
PackagesNotExistingInRemoteCache = 0,
724+
PackagesNotUploadedDueToError = 0,
725+
TimeTakenByArtifactoryUploader = 0.0
726+
};
727+
728+
// Act & Assert - Should not throw any exceptions
729+
Assert.DoesNotThrow(() => PackageUploadHelper.WriteCreatorKpiDataToConsole(uploaderKpiData));
730+
}
577731
}
578732
}

0 commit comments

Comments
 (0)