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.

Sign up for GitHub

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

Jump to bottom

PAF-304: Remove '.only' from file-upload unit test to run all unit tests #244

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

PAF-304: Remove '.only' from file-upload unit test to run all unit tests #244

wants to merge 1 commit into from

Conversation

Rhodine-orleans-lindsay
Copy link
Contributor

@Rhodine-orleans-lindsay Rhodine-orleans-lindsay commented Oct 24, 2024 •
edited
Loading

What?

Remove .only from describe block in file-upload unit test - PAF-304

Why?

Unit tests are currently only running one test

How?

  • remove .only from describe block in file-upload.spec.js

Testing?

test passing locally and on branch

Screenshots (optional)

Anything Else? (optional)

  • updated snyk list

Check list

  • I have reviewed my own pull request for linting issues (e.g. adding new lines)
  • I have written tests (if relevant)
  • I have created a JIRA number for my branch
  • I have created a JIRA number for my commit
  • I have followed the chris beams method for my commit https://cbea.ms/git-commit/
    here is an example commit
  • Ensure drone builds are green especially tests
  • I will squash the commits before merging

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Rhodine-orleans-lindsay