From 64671e5625a7c599795166353ee0d41d6844ba0f Mon Sep 17 00:00:00 2001
From: 18alantom <2.alan.tom@gmail.com>
Date: Tue, 27 Jun 2023 13:00:17 +0530
Subject: [PATCH] test(ui): test instance setup - fix auto complete choice on
 blur - fix button disabled

---
 .gitignore                               |  1 +
 src/components/Button.vue                |  1 +
 src/components/Controls/AutoComplete.vue | 10 +++---
 src/components/Sidebar.vue               |  1 +
 src/pages/SetupWizard/SetupWizard.vue    |  1 +
 uitest/index.mjs                         | 39 ++++++++++++++++++++++--
 6 files changed, 46 insertions(+), 7 deletions(-)

diff --git a/.gitignore b/.gitignore
index d886c570..c56e8d3e 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 .DS_Store
 node_modules
+/dbs
 /dist
 /notebook
 
diff --git a/src/components/Button.vue b/src/components/Button.vue
index f13b135c..78f0b738 100644
--- a/src/components/Button.vue
+++ b/src/components/Button.vue
@@ -1,6 +1,7 @@
 <template>
   <button
     class="rounded-md flex justify-center items-center text-sm"
+    :disabled="disabled"
     :class="_class"
     v-bind="$attrs"
   >
diff --git a/src/components/Controls/AutoComplete.vue b/src/components/Controls/AutoComplete.vue
index 00c0ff8a..daaf21bd 100644
--- a/src/components/Controls/AutoComplete.vue
+++ b/src/components/Controls/AutoComplete.vue
@@ -262,15 +262,15 @@ export default {
         return;
       }
 
-      if (label && this.suggestions.length === 0) {
+      if (this.suggestions.length === 0) {
         this.triggerChange(label);
         return;
       }
 
-      if (
-        label &&
-        !this.suggestions.map(({ label }) => label).includes(label)
-      ) {
+      const suggestion = this.suggestions.find((s) => s.label === label);
+      if (suggestion) {
+        this.setSuggestion(suggestion);
+      } else {
         const suggestions = await this.getSuggestions(label);
         this.setSuggestion(suggestions[0]);
       }
diff --git a/src/components/Sidebar.vue b/src/components/Sidebar.vue
index 14945443..05919561 100644
--- a/src/components/Sidebar.vue
+++ b/src/components/Sidebar.vue
@@ -14,6 +14,7 @@
         "
       >
         <h6
+          data-testid="company-name"
           class="
             font-semibold
             whitespace-nowrap
diff --git a/src/pages/SetupWizard/SetupWizard.vue b/src/pages/SetupWizard/SetupWizard.vue
index 1635c22f..a0dd6715 100644
--- a/src/pages/SetupWizard/SetupWizard.vue
+++ b/src/pages/SetupWizard/SetupWizard.vue
@@ -60,6 +60,7 @@
         <Button
           type="primary"
           class="w-24"
+          data-testid="submit-button"
           :disabled="!areAllValuesFilled || loading"
           @click="submit"
           >{{ t`Submit` }}</Button
diff --git a/uitest/index.mjs b/uitest/index.mjs
index b7469750..22742740 100644
--- a/uitest/index.mjs
+++ b/uitest/index.mjs
@@ -10,6 +10,7 @@ const appSourcePath = path.join(root, 'dist_electron', 'build', 'main.js');
 (async function run() {
   const electronApp = await _electron.launch({ args: [appSourcePath] });
   const window = await electronApp.firstWindow();
+  window.setDefaultTimeout(10_000);
 
   test('load app', async (t) => {
     t.equal(await window.title(), 'Frappe Books', 'title matches');
@@ -42,9 +43,43 @@ const appSourcePath = path.join(root, 'dist_electron', 'build', 'main.js');
     t.ok(await createNew.isVisible(), 'create new is visible');
   });
 
+  test('fill setup form', async (t) => {
+    const filePath = path.join(root, 'dbs', 'books_uitest.db');
+    await electronApp.evaluate(({ dialog }, filePath) => {
+      dialog.showSaveDialog = () =>
+        Promise.resolve({ canceled: false, filePath });
+    }, filePath);
+    await window.getByTestId('create-new-file').click();
+    await window.getByTestId('submit-button').waitFor();
+
+    t.equal(
+      await window.getByTestId('submit-button').isDisabled(),
+      true,
+      'submit button is disabled before form fill'
+    );
+
+    await window.getByPlaceholder('Company Name').fill('Test Company');
+    await window.getByPlaceholder('John Doe').fill('Test Owner');
+    await window.getByPlaceholder('john@doe.com').fill('test@example.com');
+    await window.getByPlaceholder('Select Country').fill('India');
+    await window.getByPlaceholder('Select Country').blur();
+    await window.getByPlaceholder('Prime Bank').fill('Test Bank');
+    await window.getByPlaceholder('Prime Bank').blur();
+
+    t.equal(
+      await window.getByTestId('submit-button').isDisabled(),
+      false,
+      'submit button enabled after form fill'
+    );
+  });
+
   test('create new instance', async (t) => {
-    // await window.getByTestId('create-new-file').click();
-    t.ok(true, '...');
+    await window.getByTestId('submit-button').click();
+    t.equal(
+      await window.getByTestId('company-name').innerText(),
+      'Test Company',
+      'new instance created, company name found in sidebar'
+    );
   });
 
   test('close app', async (t) => {