Skip to content

Commit

Permalink
bjbdbMerge branch 'fix/wrong-eta-on-task-cards' of github.com:Real-De…
Browse files Browse the repository at this point in the history
…v-Squad/website-my into fix/wrong-eta-on-task-cards
  • Loading branch information
Achintya-Chatterjee committed Jul 11, 2024
2 parents e27495c + 53834e4 commit 67199fb
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 32 deletions.
18 changes: 8 additions & 10 deletions app/components/task/holder.js
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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';
}
Expand All @@ -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)
Expand Down
22 changes: 0 additions & 22 deletions tests/integration/components/tasks/holder-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -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}}
Expand Down

0 comments on commit 67199fb

Please sign in to comment.