Skip to content
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

Add test coverage #14

Open
4 tasks
ackzell opened this issue Apr 26, 2020 · 2 comments
Open
4 tasks

Add test coverage #14

ackzell opened this issue Apr 26, 2020 · 2 comments
Labels
help wanted Extra attention is needed
Milestone

Comments

@ackzell
Copy link
Owner

ackzell commented Apr 26, 2020

We want coverage for this repo, namely:

  • Unit tests for the components and their logic (Jest)
  • E2E testing for the whole app (Cypress.io)
  • Visual tests that indicate the different states of the app (Storybook)
  • Manual tests

Any type of test that makes sense for this app.

@ackzell ackzell added this to the MVP 1.0 milestone Apr 26, 2020
@diego-zco
Copy link
Collaborator

diego-zco commented May 29, 2020

Observation 1:
Env: Chrome/Android
Description: The user can record and remove multiple messages before actually saving one. It seems if the user removes nine messages, and then attempts to save a tenth one, nothing is recorded (empty message with reproduction button but no content)

@diego-zco
Copy link
Collaborator

diego-zco commented May 30, 2020

Obersvation 2:
Env: Chrome/Desktop
Description: When trying to record and remove multiple messages, the 23rd is saved and from that one, any new message is saved without allowing to remove them. The 23rd message seems to be complete, but the rest of messages are "empty instances."

@ackzell ackzell added the help wanted Extra attention is needed label Jun 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants