From 1be0bfaf7b676cd9381ceee12dd2923620c8e90e Mon Sep 17 00:00:00 2001
From: Ruben Fiszel <ruben@rubenfiszel.com>
Date: Tue, 11 Jun 2024 13:57:02 +0200
Subject: [PATCH] fix: flow args should always render in test flow

---
 backend/windmill-worker/src/bun_executor.rs           | 5 +++--
 frontend/src/lib/components/FlowPreviewContent.svelte | 3 ++-
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/backend/windmill-worker/src/bun_executor.rs b/backend/windmill-worker/src/bun_executor.rs
index 7858210748..4a6cd7bc34 100644
--- a/backend/windmill-worker/src/bun_executor.rs
+++ b/backend/windmill-worker/src/bun_executor.rs
@@ -192,7 +192,7 @@ pub async fn gen_lockfile(
 async fn gen_bunfig(job_dir: &str) -> Result<()> {
     let registry = NPM_CONFIG_REGISTRY.read().await.clone();
     let bunfig_install_scopes = BUNFIG_INSTALL_SCOPES.read().await.clone();
-    Ok(if registry.is_some() || bunfig_install_scopes.is_some() {
+    if registry.is_some() || bunfig_install_scopes.is_some() {
         let (url, token_opt) = if let Some(ref s) = registry {
             let url = s.trim();
             if url.is_empty() {
@@ -222,7 +222,8 @@ registry = {}
         );
         tracing::debug!("Writing following bunfig.toml: {bunfig_toml}");
         let _ = write_file(&job_dir, "bunfig.toml", &bunfig_toml).await?;
-    })
+    }
+    Ok(())
 }
 
 pub async fn install_lockfile(
diff --git a/frontend/src/lib/components/FlowPreviewContent.svelte b/frontend/src/lib/components/FlowPreviewContent.svelte
index b095671818..ad85c3d7f5 100644
--- a/frontend/src/lib/components/FlowPreviewContent.svelte
+++ b/frontend/src/lib/components/FlowPreviewContent.svelte
@@ -308,8 +308,9 @@
 		{/if}
 		<FlowProgressBar {job} bind:reset={jobProgressReset} />
 	</div>
+
 	<div class="overflow-y-auto grow flex flex-col pr-4">
-		<div class="max-h-1/2 overflow-auto border-b">
+		<div class="border-b">
 			<SchemaForm
 				noVariablePicker
 				compact