Exercise-Tracker need help

Backend Development
#1

Testcases related to logs are failing.

here is my live app link:
https://fcc-health-tracker.onrender.com

here is my code for logs

app.post("/api/users/:_id/exercises", async function (req, res) {
  let { description, duration, date } = req.body;
  duration = Number(duration);
  const { _id } = req.params;

  if (date == "") date = new Date();
  else if (/^ +$/.test(date)) return res.send("invalid date");
  else date = new Date(date);

  let username = await checkUserById(_id);
  if (username && !isNaN(duration) && !isNaN(date.getTime())) {
    await findUserByIdAndUpdate(_id, duration, description, date.toDateString());
    res.send({ "username": username, "description": description, "duration": duration, "date": date.toDateString(), "_id": _id })
  }
  else {
    // console.log(duration, date, username);
    res.send("invalid id")
  }
})


app.get("/api/users/:_id/logs", async function(req, res){
  const {_id} = req.params;
  let {from, to, limit} = req.query;

  if(_id.length != 24) res.send("invalid id");

  from = new Date(from);
  to = new Date(to);
  limit = Number(limit);

  let result = await getLogs(_id, from, to, limit);

  
  if(result && (from == "Invalid Date" || to == "Invalid Date")){
    let {username, log} = result;
    res.send({username: username, count: log.length, _id: _id, log: log});
  }
  else if(result && from != "Invalid Date" && to != "Invalid Date"){
    let {username, log} = result;
    res.send({username: username, count: log.length, _id: _id, from: from.toDateString(), to: to.toDateString(), log: log});
  }
  else{
    res.send("invalid user")
  }
})

async function checkUserById(id) {
  let finalRes;
  try {
    await client.connect();
    let filterDoc = { _id: new ObjectId(id) };
    let result = await usersCollection.findOne(filterDoc);
    if (result) finalRes = result.username;
    else finalRes = false;
    // console.log(result);
  }
  catch (error) {
    console.log(error);
  }
  finally {
    client.close();
    return finalRes;
  }
}

async function findUserByIdAndUpdate(id, dur, des, date) {
  let finalRes;
  try {
    await client.connect();
    let filterDoc = { _id: new ObjectId(id) };
    let updateDoc = {
      $inc: { count: 1 },
      $push: {
        log: {
          description: des,
          duration: dur,
          date: date
        }
      }
    };
    let optionsObj = { returnDocument: 'after' }
    let result = await usersCollection.findOneAndUpdate(filterDoc, updateDoc, optionsObj);
    if (result) {
      finalRes = true;
    }
    else {
      finalRes = false;
    }
  }
  catch (error) {
    console.log(error);
  }
  finally {
    await client.close();
    return finalRes;
  }
}

async function getLogs(id, from, to, limit){
  let finalRes;
  try{
    await client.connect();
    let filterDoc = {_id: new ObjectId(id)};
    let options = {projection: {username: 1, log: 1, _id: 0}};
    let result = await usersCollection.findOne(filterDoc, options);

    result.log.sort((a,b) => {
      let x = new Date(a.date);
      let y = new Date(b.date);
      if(x >= y) return -1;
      else return 1;
    });//console.log(result);

    if(from != "Invalid Date" && to != "Invalid Date"){
      result.log = result.log.filter((eachLog) => {
        let date = new Date(eachLog.date);
        if(date >= from && date <= to) return eachLog;
      })
    }

    if(!isNaN(limit)){
      result.log = result.log.slice(0, limit);
    }

    if(result) finalRes = result;
    else finalRes = false;

  }
  catch(err) {console.error(err)}
  finally{
    await client.close();
    return finalRes;
  }
}
#2

We need to see the code.

If you submit the code and look at the network logs in the browser you can see the POST to exercises that comes before the GET to logs are all returning invalid id

#3

run the link 2 times, then only it works. Because for first time the link is not activated fully, but second time link will activate fully.

#4

You are not handling the optional date correctly. Log out req.body.date when you submit, it isn’t what you think it is.

#5 
if(date == undefined) date = new Date();
else date = new Date(date);

After handled the date correctly, all tests are passing, Thank you lasjorg for help.