Remove assertion from HomeIntro test - Githubissues

gristlabs / grist-core

Grist is the evolution of spreadsheets.

https://www.getgrist.com

Apache License 2.0

7.27k stars 322 forks source link

Remove assertion from HomeIntro test #1230

Closed georgegevoian closed 1 month ago

georgegevoian commented 1 month ago

Context

A HomeIntro test was asserting the presence of a tutorial card, but in Docker tests, tutorials are not configured, so the assertion always failed.

Proposed solution

A different test (DocTutorial) already tests tutorial cards, so the assertion can be removed.

Has this been tested?

[ ] 👍 yes, I added tests to the test suite
[ ] 💭 no, because this PR is a draft and still needs work
[x] 🙅 no, because this is not relevant here
[ ] 🙋 no, because I need help