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

chore(tasks): add e2e and integration tests #6345

Draft
wants to merge 1 commit into
base: next
Choose a base branch
from
Draft

Conversation

pedrobonamin
Copy link
Contributor

Description

Adds e2e tests and some new integration tests to tasks to cover the critical paths.

  • e2e tests added:

    • Create a task from the sidebar (create new button)
      • Test all fields are working:
        • Title
        • Description
        • Assignee
        • Deadline
        • Target document.
      • Tests the task shows in the list
    • Edit the new task.
      • Changes values in the task, the change is reflected.
    • Deletes the task
    • Creates a task from the document action button.
    • Navigates to tasks from:
      • Document footer action.
      • Navbar toggle button.
      • Visiting the studio with a task url.
  • Fixes an issue in which when editing quickly through tasks and clicking create, the task was not created correctly.

What to review

Is there something else considered critical that we are missing tests?

Testing

Notes for release

Adds e2e tests to tasks.
Fixes an issue in tasks in which in some cases tasks were not created.

Copy link

vercel bot commented Apr 12, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
performance-studio ✅ Ready (Inspect) Visit Preview Apr 12, 2024 11:56am
test-studio ✅ Ready (Inspect) Visit Preview Apr 12, 2024 11:56am
1 Ignored Deployment
Name Status Preview Updated (UTC)
studio-workshop ⬜️ Ignored (Inspect) Apr 12, 2024 11:56am

Copy link
Contributor

No changes to documentation

Copy link
Contributor

Component Testing Report Updated Apr 12, 2024 12:05 PM (UTC)

File Status Duration Passed Skipped Failed
comments/CommentInput.spec.tsx ✅ Passed (Inspect) 30s 15 0 0
formBuilder/ArrayInput.spec.tsx ✅ Passed (Inspect) 6s 3 0 0
formBuilder/inputs/PortableText/Annotations.spec.tsx ✅ Passed (Inspect) 25s 6 0 0
formBuilder/inputs/PortableText/copyPaste/CopyPaste.spec.tsx ✅ Passed (Inspect) 12s 4 2 0
formBuilder/inputs/PortableText/Decorators.spec.tsx ✅ Passed (Inspect) 12s 6 0 0
formBuilder/inputs/PortableText/FocusTracking.spec.tsx ✅ Passed (Inspect) 32s 15 0 0
formBuilder/inputs/PortableText/Input.spec.tsx ✅ Passed (Inspect) 58s 14 0 0
formBuilder/inputs/PortableText/ObjectBlock.spec.tsx ✅ Passed (Inspect) 1m 1s 18 0 0
formBuilder/inputs/PortableText/RangeDecoration.spec.tsx ✅ Passed (Inspect) 12s 6 0 0
formBuilder/inputs/PortableText/Styles.spec.tsx ✅ Passed (Inspect) 13s 6 0 0
formBuilder/inputs/PortableText/Toolbar.spec.tsx ✅ Passed (Inspect) 19s 9 0 0

@pedrobonamin pedrobonamin marked this pull request as ready for review April 12, 2024 12:55
@pedrobonamin pedrobonamin requested review from a team as code owners April 12, 2024 12:55
@pedrobonamin pedrobonamin requested review from ninaandal and jtpetty and removed request for a team April 12, 2024 12:55
@pedrobonamin pedrobonamin marked this pull request as draft April 12, 2024 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant