Merge pull request #48 from sahil143/backport-release-fix

fix(KFLUXBUGS-1751): release status replies on status and reason of the release type

Author: sahil143

src/components/Releases/__tests__/ReleaseOverviewTab.spec.tsx

@@ -1,6 +1,6 @@
 import { render, screen } from '@testing-library/react';
 import { createK8sWatchResourceMock, createUseWorkspaceInfoMock } from '../../../utils/test-utils';
-import { mockReleases, mockReleaseWithManagedProcessing } from '../__data__/mock-release-data';
+import { mockReleases } from '../__data__/mock-release-data';
 import ReleaseOverviewTab from '../ReleaseOverviewTab';
 
 jest.mock('react-router-dom', () => ({
@@ -57,7 +57,7 @@ describe('ReleaseOverviewTab', () => {
   });
 
   it('should render correct details if managedProcessing', () => {
-    render(<ReleaseOverviewTab release={mockReleaseWithManagedProcessing} />);
+    render(<ReleaseOverviewTab />);
     expect(screen.getByText('Pipeline Run')).toBeVisible();
     expect(screen.getByRole('link', { name: 'test-pipelinerun' }).getAttribute('href')).toBe(
       '/workspaces/target-ws/applications/test-app/pipelineruns/test-pipelinerun',

src/components/Releases/__tests__/ReleasesListRow.spec.tsx

@@ -1,4 +1,5 @@
 import { render } from '@testing-library/react';
+import { ReleaseCondition } from '../../../types';
 import ReleasesListRow from '../ReleasesListRow';
 
 jest.mock('react-router-dom', () => ({
@@ -21,6 +22,7 @@ const mockRelease = {
       {
         reason: 'Succeeded',
         status: 'True',
+        type: ReleaseCondition.Released,
       },
     ],
   },

src/hooks/__tests__/useReleaseStatus.spec.ts

@@ -1,4 +1,5 @@
 import { renderHook } from '@testing-library/react-hooks';
+import { ReleaseCondition } from '../../types';
 import { useReleaseStatus } from '../useReleaseStatus';
 
 const mockRelease = {
@@ -19,46 +20,45 @@ describe('useApplicationSnapshots', () => {
     expect(result.current).toEqual('Unknown');
   });
 
-  it('should return in progress if any of the conditions is progressing', () => {
+  it('should return in progress if release condition is progressing', () => {
     const { result } = renderHook(() =>
       useReleaseStatus({
         ...mockRelease,
         status: {
           conditions: [
-            { reason: 'Progressing' },
-            { reason: 'Succeeded', status: 'True' },
-            { reason: 'Failed', status: 'False' },
+            { reason: 'Succeeded', status: 'True', type: ReleaseCondition.Validated },
+            { reason: 'Progressing', status: 'True', type: ReleaseCondition.Released },
           ],
         },
       }),
     );
     expect(result.current).toEqual('In Progress');
   });
 
-  it('should return in succeeded if all of the conditions pass', () => {
+  it('should return in succeeded if release condition pass', () => {
     const { result } = renderHook(() =>
       useReleaseStatus({
         ...mockRelease,
         status: {
           conditions: [
-            { reason: 'Succeeded', status: 'True' },
-            { reason: 'Succeeded', status: 'True' },
+            { reason: 'Succeeded', status: 'True', type: ReleaseCondition.Released },
+            { reason: 'Progressing', status: 'True', type: ReleaseCondition.Validated },
           ],
         },
       }),
     );
     expect(result.current).toEqual('Succeeded');
   });
 
-  it('should return in failed if any of the conditions fail', () => {
+  it('should return in failed if release condition is fail', () => {
     const { result } = renderHook(() =>
       useReleaseStatus({
         ...mockRelease,
         status: {
           conditions: [
-            { reason: 'Succeeded', status: 'True' },
-            { reason: 'Succeeded', status: 'True' },
-            { reason: 'Failed', status: 'False' },
+            { reason: 'Succeeded', status: 'True', type: ReleaseCondition.Processed },
+            { reason: 'Succeeded', status: 'True', type: ReleaseCondition.Validated },
+            { reason: 'Failed', status: 'False', type: ReleaseCondition.Released },
           ],
         },
       }),

src/hooks/useReleaseStatus.ts

@@ -1,5 +1,5 @@
 import * as React from 'react';
-import { ReleaseKind } from '../types';
+import { ReleaseCondition, ReleaseKind } from '../types';
 import { runStatus } from '../utils/pipeline-utils';
 
 export const useReleaseStatus = (release: ReleaseKind) => {
@@ -8,21 +8,21 @@ export const useReleaseStatus = (release: ReleaseKind) => {
       return runStatus.Unknown;
     }
 
-    const progressing = release.status.conditions.some((c) => c.reason === 'Progressing');
-    if (progressing) {
-      return runStatus['In Progress'];
-    }
-
-    const succeeded = release.status.conditions.every(
-      (c) => c.reason === 'Succeeded' && c.status === 'True',
+    const releasedCondition = release.status.conditions.find(
+      (c) => c.type === ReleaseCondition.Released,
     );
+
+    const succeeded =
+      releasedCondition.status === 'True' && releasedCondition.reason === 'Succeeded';
     if (succeeded) {
       return runStatus.Succeeded;
     }
+    const progressing = releasedCondition.reason === 'Progressing';
+    if (progressing) {
+      return runStatus['In Progress'];
+    }
 
-    const failed = release.status.conditions.some(
-      (c) => c.reason === 'Failed' && c.status === 'False',
-    );
+    const failed = releasedCondition.reason === 'Failed' && releasedCondition.status === 'False';
     if (failed) {
       return runStatus.Failed;
     }