From 18e55fbc4966a09e6a125692b13cecf5e30bc853 Mon Sep 17 00:00:00 2001
From: Ruben Fiszel <ruben@rubenfiszel.com>
Date: Fri, 9 Jun 2023 15:40:19 +0200
Subject: [PATCH] fix: add job execution time and mem everywhere applicable

---
 frontend/src/lib/components/FlowJobResult.svelte           | 2 +-
 frontend/src/lib/components/ModulePreview.svelte           | 7 ++++++-
 .../apps/components/display/AppLogsComponent.svelte        | 7 ++++++-
 frontend/src/lib/components/jobs/JobPreview.svelte         | 7 ++++++-
 .../src/routes/(root)/(logged)/run/[...run]/+page.svelte   | 7 ++++++-
 5 files changed, 25 insertions(+), 5 deletions(-)

diff --git a/frontend/src/lib/components/FlowJobResult.svelte b/frontend/src/lib/components/FlowJobResult.svelte
index cc2d758d0d..19382d1078 100644
--- a/frontend/src/lib/components/FlowJobResult.svelte
+++ b/frontend/src/lib/components/FlowJobResult.svelte
@@ -13,7 +13,7 @@
 
 <div
 	class:border={!noBorder}
-	class="grid {!col ? 'grid-cols-2' : 'grid-rows-2'} shadow  border-gray-400 h-full max-h-screen"
+	class="grid {!col ? 'grid-cols-2' : 'grid-rows-2'} shadow border-gray-400 h-full max-h-screen"
 >
 	<div class="bg-white {col ? '' : 'max-h-80'} h-full p-1 overflow-auto relative">
 		<span class="text-gray-500">Result</span>
diff --git a/frontend/src/lib/components/ModulePreview.svelte b/frontend/src/lib/components/ModulePreview.svelte
index 8275a226fb..25b88ab0cc 100644
--- a/frontend/src/lib/components/ModulePreview.svelte
+++ b/frontend/src/lib/components/ModulePreview.svelte
@@ -101,7 +101,12 @@
 	<Pane size={50} minSize={20}>
 		<Splitpanes horizontal>
 			<Pane size={50} minSize={10}>
-				<LogViewer content={testJob?.logs} isLoading={testIsLoading} />
+				<LogViewer
+					duration={testJob?.['duration_ms']}
+					mem={testJob?.['mem_peak']}
+					content={testJob?.logs}
+					isLoading={testIsLoading}
+				/>
 			</Pane>
 			<Pane size={50} minSize={10} class="text-sm text-gray-600">
 				{#if testJob != undefined && 'result' in testJob && testJob.result != undefined}
diff --git a/frontend/src/lib/components/apps/components/display/AppLogsComponent.svelte b/frontend/src/lib/components/apps/components/display/AppLogsComponent.svelte
index f2bb55a3b2..74a4904656 100644
--- a/frontend/src/lib/components/apps/components/display/AppLogsComponent.svelte
+++ b/frontend/src/lib/components/apps/components/display/AppLogsComponent.svelte
@@ -61,7 +61,12 @@
 				customCss?.container?.class
 			)}
 		>
-			<LogViewer content={testJob?.logs} isLoading={testIsLoading} />
+			<LogViewer
+				duration={testJob?.['duration_ms']}
+				mem={testJob?.['mem_peak']}
+				content={testJob?.logs}
+				isLoading={testIsLoading}
+			/>
 		</div>
 	</div>
 </RunnableWrapper>
diff --git a/frontend/src/lib/components/jobs/JobPreview.svelte b/frontend/src/lib/components/jobs/JobPreview.svelte
index dd814fc25e..b242f4955d 100644
--- a/frontend/src/lib/components/jobs/JobPreview.svelte
+++ b/frontend/src/lib/components/jobs/JobPreview.svelte
@@ -113,7 +113,12 @@
 					<DisplayResult {result} disableExpand />
 				{:else if job && `running` in job ? job.running : false}
 					<div class="text-sm font-semibold text-gray-600 mb-1"> Job is still running </div>
-					<LogViewer content={job?.logs} isLoading />
+					<LogViewer
+						duration={job?.['duration_ms']}
+						mem={job?.['mem_peak']}
+						content={job?.logs}
+						isLoading
+					/>
 				{/if}
 			</div>
 		</div>
diff --git a/frontend/src/routes/(root)/(logged)/run/[...run]/+page.svelte b/frontend/src/routes/(root)/(logged)/run/[...run]/+page.svelte
index e3b719bfd3..033f20e891 100644
--- a/frontend/src/routes/(root)/(logged)/run/[...run]/+page.svelte
+++ b/frontend/src/routes/(root)/(logged)/run/[...run]/+page.svelte
@@ -338,7 +338,12 @@
 					<div class="flex flex-row border rounded-md p-2 mt-2 max-h-1/2 overflow-auto">
 						{#if viewTab == 'logs'}
 							<div class="w-full">
-								<LogViewer isLoading={!(job && 'logs' in job && job.logs)} content={job?.logs} />
+								<LogViewer
+									duration={job?.['duration_ms']}
+									mem={job?.['mem_peak']}
+									isLoading={!(job && 'logs' in job && job.logs)}
+									content={job?.logs}
+								/>
 							</div>
 						{:else if viewTab == 'code'}
 							{#if job && 'raw_code' in job && job.raw_code}