Use Promise.all() to better handle asynchronous operations

Signed-off-by: Lan Xia <[email protected]>

Author: llxia

test-result-summary-client/src/Build/AllTestsInfo.jsx

@@ -90,11 +90,15 @@ const Build = () => {
     };
 
     const getTestResult = async (buildId, limitParam) => {
-        const builds = await fetchData(
+        const buildsRes = fetchData(
             `/api/getAllTestsWithHistory?buildId=${buildId}${limitParam}`
         );
 
-        const buildData = await fetchData(`/api/getData?_id=${buildId} `);
+        const buildDataRes = fetchData(`/api/getData?_id=${buildId} `);
+        const [builds, buildData] = await Promise.all([
+            buildsRes,
+            buildDataRes,
+        ]);
         const error = buildData[0].error
             ? `${buildData[0].buildUrl}: ${buildData[0].error}`
             : '';

test-result-summary-client/src/Build/BuildDetail.jsx

@@ -16,9 +16,9 @@ export default function BuildDetail() {
 
     useEffect(() => {
         const updateData = async () => {
-            let buildsResults;
+            let buildsResultsRes;
             if (testSummaryResult || buildNameRegex || buildResult) {
-                buildsResults = await fetchData(
+                buildsResultsRes = fetchData(
                     `/api/getAllChildBuilds${params({
                         buildResult,
                         testSummaryResult,
@@ -27,14 +27,16 @@ export default function BuildDetail() {
                     })}`
                 );
             } else {
-                buildsResults = await fetchData(
+                buildsResultsRes = fetchData(
                     `/api/getChildBuilds?parentId=${parentId}`
                 );
             }
 
-            const parentResults = await fetchData(
-                `/api/getData?_id=${parentId}`
-            );
+            const parentResultsRes = fetchData(`/api/getData?_id=${parentId}`);
+            const [buildsResults, parentResults] = await Promise.all([
+                buildsResultsRes,
+                parentResultsRes,
+            ]);
 
             setBuilds(buildsResults);
             setParent(parentResults);

test-result-summary-client/src/Build/GitNewIssue.jsx

@@ -25,9 +25,16 @@ const GitNewissue = () => {
         async function updateData() {
             const { testId, buildId } = getParams(location.search);
             const originUrl = window.location.origin;
-
+            // fetch build data
+            const buildDataRes = fetchData(`/api/getData?_id=${buildId}`);
             // fetch test data
-            const testData = await fetchData(`/api/getTestById?id=${testId}`);
+            const testDataRes = fetchData(`/api/getTestById?id=${testId}`);
+
+            const [buildData, testData] = await Promise.all([
+                buildDataRes,
+                testDataRes,
+            ]);
+
             const { testName, duration, testResult } = testData;
 
             // fetch error in test output
@@ -36,8 +43,6 @@ const GitNewissue = () => {
             );
             const failureOutput = errorInOutput.output;
 
-            // fetch build data
-            const buildData = await fetchData(`/api/getData?_id=${buildId}`);
             const {
                 artifactory,
                 buildName,

test-result-summary-client/src/Build/ReleaseSummary.jsx

@@ -40,19 +40,25 @@ const ReleaseSummary = () => {
                 report += `${nl} --- ${nl}`;
 
                 const buildResult = '!SUCCESS';
-                const failedBuilds = await fetchData(
+                const failedBuildsRes = fetchData(
                     `/api/getAllChildBuilds${params({
                         buildResult,
                         parentId,
                         buildNameRegex: '^((?!(_rerun)).)*$',
                     })}`
                 );
-                const rerunBuilds = await fetchData(
+                const rerunBuildsRes = fetchData(
                     `/api/getAllChildBuilds${params({
                         parentId,
                         buildNameRegex: 'Test_openjdk.*_rerun',
                     })}`
                 );
+
+                const [failedBuilds, rerunBuilds] = await Promise.all([
+                    failedBuildsRes,
+                    rerunBuildsRes,
+                ]);
+
                 let failedBuildSummary = {};
                 let failedTestSummary = {};
                 // concat failedBuilds and rerunBuilds into allBuilds

test-result-summary-client/src/Build/Summary/ResultSummary.jsx

@@ -48,14 +48,13 @@ export default function ResultSummary() {
             const { parentId } = getParams(location.search);
 
             // get test summary (i.e., passed, failed, total numbers)
-            const summary = await fetchData(`/api/getTotals?id=${parentId}`);
+            const summaryRes = fetchData(`/api/getTotals?id=${parentId}`);
 
             // get build information
-            const buildInfo = await fetchData(`/api/getData?_id=${parentId}`);
-            const parentBuildInfo = buildInfo[0] || {};
+            const buildInfoRes = fetchData(`/api/getData?_id=${parentId}`);
 
             // get all SDK builds info
-            const sdkBuilds = await fetchData(
+            const sdkBuildsRes = fetchData(
                 `/api/getAllChildBuilds${params({
                     buildNameRegex: '^(jdk[0-9]{1,2}|Build_)',
                     parentId,
@@ -66,14 +65,21 @@ export default function ResultSummary() {
             const testSummaryResult = undefined;
             const buildResult = undefined;
 
-            const builds = await fetchData(
+            const buildsRes = fetchData(
                 `/api/getAllChildBuilds${params({
                     buildResult,
                     testSummaryResult,
                     buildNameRegex,
                     parentId,
                 })}`
             );
+            const [summary, buildInfo, sdkBuilds, builds] = await Promise.all([
+                summaryRes,
+                buildInfoRes,
+                sdkBuildsRes,
+                buildsRes,
+            ]);
+            const parentBuildInfo = buildInfo[0] || {};
             let childBuildsResult = 'UNSTABLE';
             let javaVersion = null;
             const buildMap = {};