Функция увеличивает ЗП, если ошибка то уведомление админу?

Question

[syb2024]

функция не проходит одну проверку

✓ функция должна возвращать промис (5ms)
✓ должна корректно увеличивать ЗП сотрудника с наименьшей ЗП (3ms)
✓ должна отправлять уведомление сотруднику
✕ в случае ошибки увеличения ЗП должна отправлять уведомление администратору, но не пользователю (3ms)

function increaseSalary() {
 return api.getEmployees()
  .then(employeeData => {
    const [minSalaryEmployee] = employeeData.reduce(([minEmployee, minSalary], employee) => {
      const {salary} = employee;
      return (salary < minSalary
        ? [employee, salary]
        : [minEmployee, minSalary]
      );
    }, [null, Infinity]);
    const {id, salary: oldSalary} = minSalaryEmployee;
    const newSalary = oldSalary * 1.2;
    return {id, salary: newSalary};
  })
   .then(({id, salary}) => api.setEmployeeSalary(id, salary))
   .then(({name, id, salary}) => api.notifyEmployee(id, `Hello, ${name}! Congratulations, your new salary is ${salary}!`))
   .catch(e => api.notifyAdmin(e));
}

const api = {
  _employees: [
    { id: 1, name: 'Alex', salary: 120000 },
    { id: 2, name: 'Fred', salary: 110000 },
    { id: 3, name: 'Bob', salary: 80000 },
  ],

  getEmployees() {
    return new Promise((resolve) => {
      resolve(this._employees.slice());
    });
  },

  setEmployeeSalary(employeeId, newSalary) {
    return new Promise((resolve) => {
      this._employees = this._employees.map((employee) =>
        employee.id !== employeeId
          ? employee
          : {
            ...employee,
            salary: newSalary,
          }
      );
      resolve(this._employees.find(({ id }) => id === employeeId));
    });
  },

  notifyEmployee(employeeId, text) {
    return new Promise((resolve) => {
      resolve(true);
    });
  },

  notifyAdmin(error) {
    return new Promise((resolve) => {
      resolve(true);
    });
  },

  setEmployees(newEmployees) {
    return new Promise((resolve) => {
      this._employees = newEmployees;
      resolve();
    });
  },
};

Answer 1

[Данила]

Не проще отделить шаг уведомления сотрудника и отправлять его только в случае успешного завершения всех предыдущих операций?

function increaseSalary() {
  return api.getEmployees()
    .then(employeeData => {
      const [minSalaryEmployee] = employeeData.reduce(([minEmployee, minSalary], employee) => {
        const { salary } = employee;
        return (salary < minSalary
          ? [employee, salary]
          : [minEmployee, minSalary]
        );
      }, [null, Infinity]);
      const { id, salary: oldSalary } = minSalaryEmployee;
      const newSalary = oldSalary * 1.2;
      return { id, salary: newSalary };
    })
    .then(({ id, salary }) => api.setEmployeeSalary(id, salary)
      .then(updatedEmployee => {
        return api.notifyEmployee(updatedEmployee.id, `Hello, ${updatedEmployee.name}! Congratulations, your new salary is ${salary}!`);
      })
    )
    .catch(e => {
      return api.notifyAdmin(e);
    });
}

Comments

[syb2024]

все равно не проходит проверку

[Данила]

syb2024,

function increaseSalary() {
  let employeeWithNewSalary;
  
  return api.getEmployees()
    .then(employeeData => {
      const [minSalaryEmployee] = employeeData.reduce(([minEmployee, minSalary], employee) => {
        const { salary } = employee;
        return (salary < minSalary
          ? [employee, salary]
          : [minEmployee, minSalary]
        );
      }, [null, Infinity]);
      const { id, salary: oldSalary } = minSalaryEmployee;
      const newSalary = oldSalary * 1.2;
      return { id, salary: newSalary };
    })
    .then(({ id, salary }) => {
      return api.setEmployeeSalary(id, salary).then(updatedEmployee => {
        employeeWithNewSalary = updatedEmployee;
        return updatedEmployee;
      });
    })
    .then(() => {
      return api.notifyEmployee(employeeWithNewSalary.id, `Hello, ${employeeWithNewSalary.name}! Congratulations, your new salary is ${employeeWithNewSalary.salary}!`);
    })
    .catch(e => {
      return api.notifyAdmin(e);
    });
}

[syb2024]

Данила, все без изменения , также не проходит

[Данила]

syb2024, а какой результат вообще вы хотите получить?

[syb2024]

Данила, ✓ функция должна возвращать промис (4ms)
✓ должна корректно увеличивать ЗП сотрудника с наименьшей ЗП (3ms)
✓ должна отправлять уведомление сотруднику (1ms)
✕ в случае ошибки увеличения ЗП должна отправлять уведомление администратору, но не пользователю (2ms)

● increaseSalaryPromises › в случае ошибки увеличения ЗП должна отправлять уведомление администратору, но не пользователю

No more money!

130 |
131 | it('в случае ошибки увеличения ЗП должна отправлять уведомление администратору, но не пользователю', async () => {
> 132 | const error = new Error('No more money!');
| ^
133 | api.setEmployeeSalary.mockRejectedValueOnce(error);
134 | const res = await increaseSalary();
135 | expect(res).toBe(false);

я особо не понимаю , нужно чтоб прошел последний тест , если я убираю эту строчку return api.notifyEmployee
проходит последний тест а первый не проходит

[Данила]

syb2024,

function increaseSalary() {
  let employeeWithNewSalary;

  return api.getEmployees()
    .then(employeeData => {
      const [minSalaryEmployee] = employeeData.reduce(([minEmployee, minSalary], employee) => {
        const { salary } = employee;
        return (salary < minSalary
          ? [employee, salary]
          : [minEmployee, minSalary]
        );
      }, [null, Infinity]);
      const { id, salary: oldSalary } = minSalaryEmployee;
      const newSalary = oldSalary * 1.2;
      return { id, salary: newSalary };
    })
    .then(({ id, salary }) => {
      return api.setEmployeeSalary(id, salary)
        .then(updatedEmployee => {
          employeeWithNewSalary = updatedEmployee;
          return updatedEmployee;
        });
    })
    .then(() => {
      return api.notifyEmployee(employeeWithNewSalary.id, `Hello, ${employeeWithNewSalary.name}! Congratulations, your new salary is ${employeeWithNewSalary.salary}!`);
    })
    .then(() => {
      return true;
    })
    .catch(e => {
      return api.notifyAdmin(e).then(() => false);
    });
}

[syb2024]

Данила, все пройдено , но не понял как и почему

[Данила]

syb2024, функция должна вернуть правильное значение в зависимости от того, произошла ошибка или нет. Если зарплата успешно увеличена, функция должна отправить уведомление сотруднику и вернуть успешный результат, а в случае ошибки - уведомить администратора и вернуть false.