fix: address CRs

flawiddsouza · Oct 8, 2024 · 16cabe3 · 16cabe3
1 parent 38f949b
commit 16cabe3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/packages/ui/src/components/RequestPanel.vue b/packages/ui/src/components/RequestPanel.vue
@@ -1131,7 +1131,7 @@ export default {
             this.urlPreview = url !== '' && url.trim() !== '' ? url : 'No URL'
         },
         async generateTests()  {
-            let generatedTestScripts
+            const generatedTestScripts = await generateTestScripts()
 
             const pluginData = this.$store.state.plugins.workspace.find(plugin =>
                 plugin.collectionId === this.activeTab._id && plugin.type === 'script'
@@ -1140,8 +1140,6 @@ export default {
             const { pre_request = '', post_request = '' } = pluginData?.code || {}
 
             try {
-                generatedTestScripts = await generateTestScripts()
-
                 const updatedPostRequest = `${post_request}\n${generatedTestScripts}`.trim()
 
                 const pluginPayload = {
@@ -1159,15 +1157,13 @@ export default {
                 } else {
                     this.$store.commit('updatePlugin', {
                         _id: pluginData._id,
-                        ...pluginPayload
+                        code: pluginPayload.code
                     })
                 }
 
                 this.$toast.success('Test scripts are generated successfully.')
             } catch (e) {
                 this.$toast.error(`Failed to generate test scripts: ${e.message}`)
-            } finally {
-                generatedTestScripts = null
             }
         }
     },

diff --git a/packages/ui/src/components/Sidebar.vue b/packages/ui/src/components/Sidebar.vue
@@ -92,7 +92,6 @@ export default {
             pluginManagerShow: false,
             generateCodeModalCollectionItem: null,
             generateCodeModalShow: false,
-            generateTestsModalCollectionItem: null,
             createNewList: [
                 {
                     'type': 'option',