From f6fede86e7c81f2605e17dadc2e72a24ae4e413c Mon Sep 17 00:00:00 2001
From: Shubham Sharma <68867418+skv93-coder@users.noreply.github.com>
Date: Tue, 20 Feb 2024 17:07:12 +0530
Subject: [PATCH] hidden not needed popups (#577)

* Removed fetch test green button

* Hidden green pop up

* Hide find task button

---------

Co-authored-by: Shubham Sharma <shbh541@gmail.com>
---
 app/components/task/modal.hbs                 | 57 ++++++++++++-------
 app/components/tasks.hbs                      |  2 +-
 app/constants/tasks.js                        |  3 +-
 app/styles/tasks.css                          |  3 +
 tests/integration/components/tasks-test.js    |  4 +-
 .../components/tasks/modal-test.js            |  9 +--
 6 files changed, 45 insertions(+), 33 deletions(-)

diff --git a/app/components/task/modal.hbs b/app/components/task/modal.hbs
index 63814ce2..b7a2ce67 100644
--- a/app/components/task/modal.hbs
+++ b/app/components/task/modal.hbs
@@ -1,24 +1,39 @@
 {{#if @showModal}}
-<div class="overlay">
-  <div class="modal" data-test-modal>
-    <button type="button" class="close" {{on "click" @goBack}} disabled={{@closeDisabled}} data-test-closeBtn>
-      <i class="fa fa-close"></i>
-    </button>
-    <p class="modal-title" data-test-title>{{@message}}</p>
-    {{#if @isUpdating}}
-    <Spinner data-test-spinner />
-    {{/if}}
-    {{#if @buttonRequired}}
-    <button type="button" class={{if (eq @dev true) "not-assign" "proceed-btn" }} {{on "click" @markComplete}}
-      data-test-notAssignBtn>
-      {{if (eq @dev true) "Don't assign task" "Proceed" }}
-    </button>
-    {{#if @dev}}
-    <button type="button" class="proceed-btn" {{on "click" @markCompleteAndAssignTask}} data-test-assignBtn>
-      Assign task
-    </button>
-    {{/if}}
-    {{/if}}
+  <div class='overlay'>
+    <div class='modal' data-test-modal>
+      <button
+        type='button'
+        class='close'
+        {{on 'click' @goBack}}
+        disabled={{@closeDisabled}}
+        data-test-closeBtn
+      >
+        <i class='fa fa-close'></i>
+      </button>
+      <p class='modal-title' data-test-title>{{@message}}</p>
+      {{#if @isUpdating}}
+        <Spinner data-test-spinner />
+      {{/if}}
+      {{#if @buttonRequired}}
+        <button
+          type='button'
+          class='proceed-btn'
+          {{on 'click' @markComplete}}
+          data-test-notAssignBtn
+        >
+          Proceed
+        </button>
+        {{#if @dev}}
+          <button
+            type='button'
+            class='proceed-btn hidden'
+            {{on 'click' @markCompleteAndAssignTask}}
+            data-test-assignBtn
+          >
+            Assign task
+          </button>
+        {{/if}}
+      {{/if}}
+    </div>
   </div>
-</div>
 {{/if}}
\ No newline at end of file
diff --git a/app/components/tasks.hbs b/app/components/tasks.hbs
index f7329a3b..43bd28e1 100644
--- a/app/components/tasks.hbs
+++ b/app/components/tasks.hbs
@@ -1,7 +1,7 @@
 <div class='tasks-container'>
   {{#if @dev}}
     {{#if @noInProgressTask}}
-      <div class='fetch-task' data-test-fetchSection>
+      <div class='fetch-task hidden' data-test-fetchSection>
         <p>You have no task in progress! click below to fetch tasks</p>
         <Button
           @onClick={{@handleAssignment}}
diff --git a/app/constants/tasks.js b/app/constants/tasks.js
index c9e1d843..289e64bf 100644
--- a/app/constants/tasks.js
+++ b/app/constants/tasks.js
@@ -138,8 +138,7 @@ const TABS_TASK_STATUS_LIST = [
   },
 ];
 export const TASK_MESSAGES = {
-  MARK_DONE:
-    'This task will be marked as done and a new task will be assigned to you',
+  MARK_DONE: 'This task will be marked as Done',
   UPDATE_TASK: 'Updating task',
   FIND_TASK: 'Finding new task for you!',
 };
diff --git a/app/styles/tasks.css b/app/styles/tasks.css
index 09a86c65..7c23929a 100644
--- a/app/styles/tasks.css
+++ b/app/styles/tasks.css
@@ -736,3 +736,6 @@ div.buttons {
   align-content: center;
   justify-content: center;
 }
+.hidden {
+  display: none;
+}
diff --git a/tests/integration/components/tasks-test.js b/tests/integration/components/tasks-test.js
index 89922d1e..9a76cd46 100644
--- a/tests/integration/components/tasks-test.js
+++ b/tests/integration/components/tasks-test.js
@@ -1,4 +1,4 @@
-import { module, test } from 'qunit';
+import { module, skip, test } from 'qunit';
 import { setupRenderingTest } from 'ember-qunit';
 import { find, render, waitUntil, fillIn } from '@ember/test-helpers';
 import { hbs } from 'ember-cli-htmlbars';
@@ -17,7 +17,7 @@ module('Integration | Component | tasks', function (hooks) {
     assert.ok(this.element.textContent.trim().includes('tasks'));
   });
 
-  test('there is a fetch task button if no in progress task are there', async function (assert) {
+  skip('there is a fetch task button if no in progress task are there', async function (assert) {
     this.setProperties({
       onTaskChange: () => {},
       onTaskUpdate: () => {},
diff --git a/tests/integration/components/tasks/modal-test.js b/tests/integration/components/tasks/modal-test.js
index f4b62a6f..655eed7a 100644
--- a/tests/integration/components/tasks/modal-test.js
+++ b/tests/integration/components/tasks/modal-test.js
@@ -200,8 +200,7 @@ module('Integration | Component | tasks/modal', function (hooks) {
       showModal: true,
       buttonRequired: true,
       dev: false,
-      message:
-        'This task will be marked as done and a new task will be assigned to you',
+      message: 'This task will be marked as Done',
     });
 
     await render(hbs`
@@ -219,11 +218,7 @@ module('Integration | Component | tasks/modal', function (hooks) {
 
     assert.dom('[data-test-notAssignBtn]').exists();
     assert.dom('[data-test-notAssignBtn]').hasProperty('button');
-    assert
-      .dom('[data-test-title]')
-      .hasText(
-        'This task will be marked as done and a new task will be assigned to you'
-      );
+    assert.dom('[data-test-title]').hasText('This task will be marked as Done');
     assert.dom('[data-test-notAssignBtn]').hasText('Proceed');
   });
 });