Fix-20459 Wrong sbom status displayed in UI

Signed-off-by: xuelichao <xuel@vmware.com>
goharbor · May 21, 2024 · 83c2093 · 83c2093
1 parent be839e6
commit 83c2093
Show file tree

Hide file tree

Showing 6 changed files with 18 additions and 19 deletions.
diff --git a/...rtifact/artifact-list-page/artifact-list/artifact-list-tab/artifact-list-tab.component.ts b/...rtifact/artifact-list-page/artifact-list/artifact-list-tab/artifact-list-tab.component.ts
@@ -997,7 +997,10 @@ export class ArtifactListTabComponent implements OnInit, OnDestroy {
                         this.selectedRow[i].sbomDigest =
                             artifact.sbom_overview.sbom_digest;
                     }
-                    if (artifact.accessories !== undefined) {
+                    if (
+                        artifact.accessories !== undefined &&
+                        artifact.accessories.length > 0
+                    ) {
                         this.selectedRow[i].accessories = artifact.accessories;
                     }
                     this.selectedRow.splice(i, 1);

diff --git a/...portal/src/app/base/project/repository/artifact/sbom-scanning/sbom-scan.component.spec.ts b/...portal/src/app/base/project/repository/artifact/sbom-scanning/sbom-scan.component.spec.ts
@@ -185,7 +185,7 @@ describe('ResultSbomComponent (inline template)', () => {
             expect(component.sbomOverview.scan_status).toBe(
                 SBOM_SCAN_STATUS.SUCCESS
             );
-            expect(textContent).toBe('SBOM Detail');
+            expect(textContent.trim()).toBe('SBOM.Details');
         });
     });
     it('Test ResultSbomComponent getScanner', () => {

diff --git a/src/portal/src/app/base/project/repository/artifact/sbom-scanning/sbom-scan.component.ts b/src/portal/src/app/base/project/repository/artifact/sbom-scanning/sbom-scan.component.ts
@@ -221,6 +221,7 @@ export class ResultSbomComponent implements OnInit, OnDestroy {
                 repositoryName: dbEncodeURIComponent(this.repoName),
                 reference: this.artifactDigest,
                 withSbomOverview: true,
+                withAccessory: true,
                 XAcceptVulnerabilities: DEFAULT_SUPPORTED_MIME_TYPES,
             })
             .subscribe(

diff --git a/...ct/repository/artifact/sbom-scanning/sbom-tip-histogram/sbom-tip-histogram.component.html b/...ct/repository/artifact/sbom-scanning/sbom-tip-histogram/sbom-tip-histogram.component.html
@@ -1,24 +1,24 @@
-<div class="tip-wrapper width-215">
+<div class="tip-wrapper width-120">
     <clr-tooltip>
         <div clrTooltipTrigger class="tip-block">
             <div *ngIf="showSbomDetailLink()" class="circle-block">
                 <a
                     href="javascript:void(0)"
-                    class="digest margin-left-5"
+                    class="digest"
                     (click)="goIntoArtifactSbomSummaryPage()"
                     title="{{ 'SBOM.Details' | translate }">
                     {{ 'SBOM.Details' | translate }}</a
                 >
             </div>
             <div
                 *ngIf="showNoSbom()"
-                class="pl-1 margin-left-5 tip-wrapper bar-block-none shadow-none width-150">
+                class="tip-wrapper bar-block-none shadow-none width-120">
                 {{ 'SBOM.NO_SBOM' | translate }}
             </div>
             <div
                 *ngIf="!showSbomDetailLink() && !showNoSbom()"
-                class="pl-1 margin-left-5 tip-wrapper bar-block-none shadow-none width-150">
-                {{ 'SBOM.COMPLETED,' | translate }}
+                class="tip-wrapper width-120">
+                {{ 'SBOM.COMPLETED' | translate }}
             </div>
         </div>
         <clr-tooltip-content

diff --git a/...ct/repository/artifact/sbom-scanning/sbom-tip-histogram/sbom-tip-histogram.component.scss b/...ct/repository/artifact/sbom-scanning/sbom-tip-histogram/sbom-tip-histogram.component.scss
@@ -1,9 +1,8 @@
 .tip-wrapper {
     display: flex;
-    align-items: center;
-    color: #fff;
-    text-align: center;
-    font-size: 10px;
+    align-items: left;
+    text-align: left;
+    font-size: .65rem;
     height: 15px;
     line-height: 15px;
 }
@@ -33,12 +32,8 @@ hr {
     margin-left: 5px;
 }
 
-.width-215 {
-    width: 215px;
-}
-
-.width-150 {
-    width: 150px;
+.width-120 {
+    width: 120px;
 }
 
 .level-border>div{

diff --git a/...ject/repository/artifact/sbom-scanning/sbom-tip-histogram/sbom-tip-histogram.component.ts b/...ject/repository/artifact/sbom-scanning/sbom-tip-histogram/sbom-tip-histogram.component.ts
@@ -79,11 +79,11 @@ export class SbomTipHistogramComponent {
     }
 
     showSbomDetailLink(): boolean {
-        return this.sbomDigest && this.getSbomAccessories.length > 0;
+        return this.sbomDigest && this.getSbomAccessories().length > 0;
     }
 
     showNoSbom(): boolean {
-        return !this.sbomDigest && this.getSbomAccessories.length === 0;
+        return !this.sbomDigest && this.getSbomAccessories().length === 0;
     }
 
     isThemeLight() {