more fixes for test maker (#4973)

cBioPortal · Aug 22, 2024 · 8e59189 · 8e59189
1 parent 69dde84
commit 8e59189
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/shared/api/cbioportalInternalClientInstance.ts b/src/shared/api/cbioportalInternalClientInstance.ts
@@ -1,6 +1,6 @@
 import { CBioPortalAPIInternal } from 'cbioportal-ts-api-client';
 import { getLoadConfig } from 'config/config';
-import { getBrowserWindow } from 'cbioportal-frontend-commons';
+import { getBrowserWindow, hashString } from 'cbioportal-frontend-commons';
 import { toJS } from 'mobx';
 import { reportValidationResult, validate } from 'shared/api/validation';
 import _ from 'lodash';
@@ -52,8 +52,8 @@ function proxyColumnStore(client: any, endpoint: string) {
                 setTimeout(() => {
                     makeTest(params, url, matchedMethod[0]);
                 }, 1000);
-
-                validate(url, params, matchedMethod[0], 0).then(
+                const hash = hashString(JSON.stringify({ data: params, url }));
+                validate(url, params, matchedMethod[0], hash).then(
                     (result: any) => {
                         reportValidationResult(result);
                     }

diff --git a/src/shared/api/testMaker.ts b/src/shared/api/testMaker.ts
@@ -52,6 +52,8 @@ export async function makeTest(data: any, url: string, label: string) {
 
     if (getBrowserWindow().localStorage.getItem(SAVE_TEST_KEY))
         saveTest(hash, entry);
+
+    return entry;
 }
 
 function saveTest(hash: number, entry: any) {