From cc4dd5a0211d107e39188b9425fe1b77a94ba3d8 Mon Sep 17 00:00:00 2001
From: Shubham Sharma <68867418+skv93-coder@users.noreply.github.com>
Date: Tue, 4 Jun 2024 23:21:30 +0530
Subject: [PATCH] Removed completed label (#597)

* Removed completed label

* Removed the completed status label

---------

Co-authored-by: Shubham Sharma <shbh541@gmail.com>
---
 app/components/task/holder.js                 | 18 +++++++--------
 .../components/tasks/holder-test.js           | 22 -------------------
 2 files changed, 8 insertions(+), 32 deletions(-)

diff --git a/app/components/task/holder.js b/app/components/task/holder.js
index d862a8b0..4fea4e77 100644
--- a/app/components/task/holder.js
+++ b/app/components/task/holder.js
@@ -19,7 +19,12 @@ export default class TasksHolderComponent extends Component {
 
   constructor() {
     super(...arguments);
-    if (!this.args.dev) {
+    if (this.args.dev) {
+      this.status =
+        this.args.task.status === TASK_KEYS.COMPLETED
+          ? TASK_KEYS.DONE
+          : this.args.task.status;
+    } else {
       this.status =
         this.args.task.status === TASK_KEYS.DONE
           ? TASK_KEYS.COMPLETED
@@ -30,11 +35,7 @@ export default class TasksHolderComponent extends Component {
     const statusToDisplay = this.availabletaskStatusList.filter(
       (taskStatus) => {
         if (this.args.dev) {
-          return (
-            taskStatus.key !== 'ALL' &&
-            (taskStatus.key !== TASK_KEYS.COMPLETED ||
-              this.args.task.status === TASK_KEYS.COMPLETED)
-          );
+          return taskStatus.key !== 'ALL' && taskStatus.key !== 'COMPLETED';
         } else {
           return taskStatus.key !== 'ALL' && taskStatus.key !== 'DONE';
         }
@@ -45,13 +46,10 @@ export default class TasksHolderComponent extends Component {
 
   get taskStyleClass() {
     const statusNotOverDueList = [
-      TASK_KEYS.COMPLETED,
+      this.args.dev ? TASK_KEYS.DONE : TASK_KEYS.COMPLETED,
       TASK_KEYS.VERIFIED,
       TASK_KEYS.AVAILABLE,
     ];
-    if (this.args.dev) {
-      statusNotOverDueList.push(TASK_KEYS.DONE);
-    }
     if (
       this.args.task.endsOn * 1000 < Date.now() &&
       !statusNotOverDueList.includes(this.status)
diff --git a/tests/integration/components/tasks/holder-test.js b/tests/integration/components/tasks/holder-test.js
index 2c09130b..c600fdd6 100644
--- a/tests/integration/components/tasks/holder-test.js
+++ b/tests/integration/components/tasks/holder-test.js
@@ -385,28 +385,6 @@ module('Integration | Component | Tasks Holder', function (hooks) {
     this.set('defaultType', DEFAULT_TASK_TYPE);
     this.set('dev', false);
 
-    await render(hbs`<Task::Holder
-    @task={{this.task}} 
-    @onTaskChange={{this.mock}} 
-    @onStausChange={{this.mock}} 
-    @onTaskUpdate={{this.mock}} 
-    @isLoading={{this.isLoading}} 
-    @userSelectedTask={{this.defaultType}} 
-    @disabled={{this.disabled}}
-    @dev={{this.dev}}
-  />`);
-
-    assert.dom('[data-test-task-status-select]').exists();
-    assert.dom('[data-test-dropdown-option=Completed]').hasText('Completed');
-  });
-  test('Render Task holder and check if task status is COMPLETED it renders COMPLETED in dev mode', async function (assert) {
-    this.set('task', { ...tasksData[5], status: 'COMPLETED' });
-    this.set('mock', () => {});
-    this.set('isLoading', false);
-    this.set('disabled', false);
-    this.set('defaultType', DEFAULT_TASK_TYPE);
-    this.set('dev', true);
-
     await render(hbs`<Task::Holder
     @task={{this.task}} 
     @onTaskChange={{this.mock}}