-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Extend e2e code coverage #56
Labels
Comments
queleok
added a commit
that referenced
this issue
Aug 22, 2021
Couldn't find how to mock timers with puppeteer non-intrusively, so I had to make an intrusive change to support some speedup |
queleok
added a commit
that referenced
this issue
Aug 22, 2021
queleok
added a commit
that referenced
this issue
Aug 22, 2021
… omition of the overlapping scenarios
queleok
added a commit
that referenced
this issue
Aug 23, 2021
queleok
added a commit
that referenced
this issue
Aug 24, 2021
queleok
added a commit
that referenced
this issue
Aug 28, 2021
queleok
added a commit
that referenced
this issue
Aug 28, 2021
…possibility of having recoverable failures
queleok
added a commit
that referenced
this issue
Aug 28, 2021
…, added test covering most of the 5 section from the #56
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1. Styling of published words
With the styling of published words everything is simple, they should be published in the reverse order, and they are always in one of four states:
successful
,failure
,network-failure
(maybe I should rename this torecoverable-failure
), orpending
. The first two are already covered, so I only need to cover the last two, which implies:network-failure
2. Results calculation & processing
With results there are a few things to consider. First of all, the basic result calculation should be verified, i.e. the tests should:
Somewhat more complicated to cover would be the following:
3. Network issues disclaimer
There should also be tests that
4. Word publishing
Enter
from the keyboard if the input contains only alphabetical symbols in different casesEnter
from the keyboard if there are non-alphabetical symbols in the inputEnter
from the keyboard if there are less than 3 letters in the input5. Letters highlighting
This is the most complicated part to test. Essentially there are two ways to input letters: with the generated set of letters, and with the input directly. Both should lead to the button highlighting or unhighlighting given respective letter was generated for this round. Moreover, highlighted button should be strictly bind to the respective letter in the input meaning that if that letter is removed from the input, the button should be unhighlighted.
Enter
clears the input and unhighlights the letter buttons6. Replay
After the replay button is pressed, the tests need to confirm that:
The text was updated successfully, but these errors were encountered: