Removed user caching and important css flag

Signed-off-by: James Cocker <james.s.earth@gmail.com>

Author: James-Cocker

galasa-ui/src/actions/runsAction.ts

@@ -9,25 +9,9 @@ import { ResultArchiveStoreAPIApi, TagsAPIApi } from '@/generated/galasaapi';
 import { createAuthenticatedApiConfiguration } from '@/utils/api';
 import { fetchRunDetailLogs } from '@/utils/testRuns';
 import { CLIENT_API_VERSION } from '@/utils/constants/common';
-import { Configuration } from '@/generated/galasaapi';
-
-// Cache for API configuration to avoid multiple cookie accesses in the same request
-let cachedApiConfig: Configuration | null = null;
-
-const getCachedApiConfiguration = (): Configuration => {
-  if (!cachedApiConfig) {
-    cachedApiConfig = createAuthenticatedApiConfiguration();
-  }
-  return cachedApiConfig;
-};
-
-// Reset the cached API configuration (useful for testing)
-export const resetApiConfigCache = (): void => {
-  cachedApiConfig = null;
-};
 
 export const downloadArtifactFromServer = async (runId: string, artifactUrl: string) => {
-  const apiConfig = getCachedApiConfiguration();
+  const apiConfig = createAuthenticatedApiConfiguration();
   const rasApiClient = new ResultArchiveStoreAPIApi(apiConfig);
 
   const artifactFile = await rasApiClient.getRasRunArtifactByPath(
@@ -69,7 +53,7 @@ export const downloadArtifactFromServer = async (runId: string, artifactUrl: str
 
 export const updateRunTags = async (runId: string, tags: string[]) => {
   try {
-    const apiConfig = getCachedApiConfiguration();
+    const apiConfig = createAuthenticatedApiConfiguration();
     const rasApiClient = new ResultArchiveStoreAPIApi(apiConfig);
 
     // Note: Tags are already unique from the Set in the frontend, but is checked again by the rest api.
@@ -89,7 +73,7 @@ export const updateRunTags = async (runId: string, tags: string[]) => {
 
 export const getExistingTagObjects = async () => {
   try {
-    const apiConfig = getCachedApiConfiguration();
+    const apiConfig = createAuthenticatedApiConfiguration();
     const tagsApiClient = new TagsAPIApi(apiConfig);
 
     const tagsResponse = await tagsApiClient.getTags();

galasa-ui/src/components/test-runs/test-run-details/3270Tab/TableOfScreenshots.tsx

@@ -287,7 +287,7 @@ export default function TableOfScreenshots({
           getRowProps: (options: any) => TableRowProps;
           getTableProps: () => TableBodyProps;
         }) => (
-          <Table stickyHeader {...getTableProps()} className={styles.innerScreenshotTable}>
+          <Table stickyHeader {...getTableProps()} id={styles.innerScreenshotTable}>
             <TableHead>
               <TableRow>
                 {headers.map((header) => (

galasa-ui/src/styles/test-runs/test-run-details/Tab3270.module.css

@@ -12,7 +12,7 @@
   cursor: pointer;
 }
 
-.innerScreenshotTable {
+#innerScreenshotTable {
   /* Override Carbon's max-height for the Table component */
   max-height: min(calc(100vh - 450px), 550px) !important;
   overflow-y: auto;

galasa-ui/src/tests/actions/runActions.test.ts

@@ -3,11 +3,7 @@
  *
  * SPDX-License-Identifier: EPL-2.0
  */
-import {
-  downloadArtifactFromServer,
-  updateRunTags,
-  resetApiConfigCache,
-} from '@/actions/runsAction';
+import { downloadArtifactFromServer, updateRunTags } from '@/actions/runsAction';
 import * as apiUtils from '@/utils/api';
 import * as galasaapi from '@/generated/galasaapi';
 import { CLIENT_API_VERSION } from '@/utils/constants/common';
@@ -34,7 +30,6 @@ describe('downloadArtifactFromServer', () => {
 
   afterEach(() => {
     jest.resetAllMocks();
-    resetApiConfigCache();
   });
 
   it('parses valid JSON payload', async () => {
@@ -127,7 +122,6 @@ describe('updateRunTags', () => {
 
   afterEach(() => {
     jest.resetAllMocks();
-    resetApiConfigCache();
   });
 
   it('successfully updates tags', async () => {