fix(pipeline): null check on status of taskrun

Author: JoaoPedroPP

src/components/EnterpriseContract/__tests__/useEnterpriseContractResultFromLogs.spec.tsx

@@ -54,6 +54,20 @@ describe('useEnterpriseContractResultFromLogs', () => {
     expect(result.current[0][0].warnings).toEqual(undefined);
   });
 
+  it('should call tknResults when taskRun is empty array', () => {
+    mockUseTaskRuns.mockReturnValueOnce([[], true, undefined]);
+    mockGetTaskRunLogs.mockReturnValue(`asdfcdfadsf
+      [report-json] { "components": [] }
+      `);
+    const { result } = renderHook(() => useEnterpriseContractResultFromLogs('dummy-abcd'));
+    const [, loaded] = result.current;
+    expect(mockCommmonFetchJSON).toHaveBeenCalled();
+    expect(loaded).toBe(true);
+    expect(mockCommmonFetchJSON).toHaveBeenLastCalledWith(
+      '/api/v1/namespaces/test-ns/pods/pod-acdf/log?container=step-report-json&follow=true',
+    );
+  });
+
   it('should filter out all 404 image url components from EC results', async () => {
     const { result, waitForNextUpdate } = renderHook(() =>
       useEnterpriseContractResultFromLogs('dummy-abcd'),

src/components/EnterpriseContract/useEnterpriseContractResultFromLogs.tsx

@@ -21,7 +21,7 @@ export const useEnterpriseContractResultFromLogs = (
   const [ecJson, setEcJson] = React.useState<EnterpriseContractResult>();
   const [ecLoaded, setEcLoaded] = React.useState<boolean>(false);
   const ecResultOpts = React.useMemo(() => {
-    const podName = loaded && !error ? taskRun[0].status.podName : null;
+    const podName = loaded && !error ? taskRun?.[0]?.status?.podName : null;
     return podName
       ? {
           ns: namespace,
@@ -37,6 +37,9 @@ export const useEnterpriseContractResultFromLogs = (
 
   React.useEffect(() => {
     let unmount = false;
+    if (loaded && !ecResultOpts) {
+      setFetchTknLogs(true);
+    }
     if (ecResultOpts) {
       commonFetchJSON(getK8sResourceURL(PodModel, undefined, ecResultOpts))
         .then((res: EnterpriseContractResult) => {
@@ -58,7 +61,7 @@ export const useEnterpriseContractResultFromLogs = (
     return () => {
       unmount = true;
     };
-  }, [ecResultOpts]);
+  }, [ecResultOpts, loaded]);
 
   React.useEffect(() => {
     let unmount = false;