Updated cypress tests for when search query doesn't have any results …

[devcer]

…and placeholder text check

Fixes: #339

[bidoubiwa]

Thanks for you contribution 🔥:) Inside the results box could you test the rendering of all the sub parts? At least of one result

[devcer]

Thanks for you contribution 🔥:) Inside the results box could you test the rendering of all the sub parts? At least of one result

There is an existing test that does this already.

[bidoubiwa]

It only test one heading! It does not tests if the other sub-headings render correctly

    cy.get('div.dsb-suggestion:nth-child(1)') // the first entry should be "Quick start"
      .contains('Quick start')

[bidoubiwa]

Could you remove package-lock.json ?

[devcer]

@bidoubiwa I have added a test to check remaining subheadings! I don't see any package-lock.json in my commit. where do you see it?

[bidoubiwa]

Oh I just saw that cypress tests are not appearing in the checks of this CI but do appear in all the other bumps CI we have.
There seem to be a permission issue with the test run.
I'm fixing this

[bidoubiwa]

Could you update your branch with main? Cypress CI tests are not triggered automatically

[devcer]

@bidoubiwa I have updated my branch and updated the comments on the tests.

[bidoubiwa]

LGTM 🔥 Thanks a lot

[bidoubiwa]

If you are participating in Hacktoberfest, and you would like to receive a small gift from MeiliSearch too, please complete this form.

[bidoubiwa]

bors merge

[bors]

Build succeeded:

• cypress-run
• linter-check
• Tests (Node.js 12)
• Tests (Node.js 14)