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

Fix test cases warnings in src/views/tests/project.test.js #5979

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Fix test cases warnings in src/views/tests/project.test.js #5979

wants to merge 2 commits into from

Commits on Jul 12, 2023

  1. Fix warnings for tests in src/views/tests/project.test.js 

    - Fixed warning in related to the usage of navigate() outside of a useEffect() hook. The warning was triggered when setting parameters only in the test case, not directly related to the navigate() function from react-router-dom. The issue specifically occurred when no parameters were passed while visiting the user projects page, resulting in the selection of a fallback filter for 'managed by me'.
- Refactored the test case for the 'Project Detail Page' to utilize the `createComponentWithMemoryRouter` function.
- Resolved the `Error: Not implemented: window.scrollTo` by assigning a Jest function to `window.scrollTo`.
    @HelNershingThapa
    HelNershingThapa committed Jul 12, 2023
    Configuration menu
    Copy the full SHA
    930545c View commit details
    Browse the repository at this point in the history

  2. Refactored src/views/tests/taskSelection.test.js to utilize the `cr… 

    …eateComponentWithMemoryRouter` function.
    @HelNershingThapa
    HelNershingThapa committed Jul 12, 2023
    Configuration menu
    Copy the full SHA
    8765cac View commit details
    Browse the repository at this point in the history