Question

0

39

Views

Adding an early return to avoid using else and additional nesting

I got the below comment from a developer: " add an early return here to avoid using else and additional nesting". But I have a bit of difficulty with modifying.

Can you please let me know what exactly should I do in this case? Thank you!

failOnStatusCode: false,

    }).then((res) => {


      // Assertion for one by one

      if (res.status != 200) {

        cy.log(JSON.stringify(res));

      } else {

        expect(res.body.id).to.eq(vehicleIdBuyNow);

        expect(res.body.auctionStatus).contains("finished");

        expect(res.body.price).to.be.equal(minPrice);

        expect(res.body.winningBidPlatformId).contains(platformIdBuyNow);

        expect(true).to.be.true;

7 months ago · Juan Pablo Isaza

1 answers

Answer question

0

Answer question

Find remote jobs

Answer 1 · 2022-07-10T13:06:27.939Z

The recommendation is to change it to:

.then((res) => {
    if (res.status != 200) {
        cy.log(JSON.stringify(res));
        return; // exit function, no need for an `else` after this
    }
    expect(res.body.id).to.eq(vehicleIdBuyNow);
    expect(res.body.auctionStatus).contains("finished");
    expect(res.body.price).to.be.equal(minPrice);
    expect(res.body.winningBidPlatformId).contains(platformIdBuyNow);
    expect(true).to.be.true;