Skip to content

Aaryaneil - Unit Tests for userProjectsReducer component#2996

Merged
one-community merged 1 commit into
developmentfrom
aaryaneil-Unit-Test-userProjectsReducer
Apr 3, 2025
Merged

Aaryaneil - Unit Tests for userProjectsReducer component#2996
one-community merged 1 commit into
developmentfrom
aaryaneil-Unit-Test-userProjectsReducer

Conversation

@aaryaneil
Copy link
Copy Markdown
Contributor

Description

Unit test for src/reducers/userProjectsReducer.js

Main changes explained:

Added test cases for the following:

  1. should return the initial state when no action is provided
  2. should handle GET_USER_PROJECTS action type
  3. should return the current state for unknown action types
  4. should handle GET_USER_PROJECTS with empty payload

How to test:

  1. check into current branch
  2. do npm install and npm test userProjectsReducer.test.js to run this PR locally

Screenshots or videos of changes:

Screenshot 2024-12-31 at 3 24 20 PM

@aaryaneil aaryaneil added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Dec 31, 2024
@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 31, 2024

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 88e7c16
🔍 Latest deploy log https://app.netlify.com/sites/highestgoodnetwork-dev/deploys/677461a4d6d4bc0008733605
😎 Deploy Preview https://deploy-preview-2996--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Copy Markdown
Contributor

@ddshah878 ddshah878 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have tested the PR and all the test cases have passed.

image

@beblicarl beblicarl self-requested a review January 2, 2025 09:56
@beblicarl
Copy link
Copy Markdown
Contributor

All tests passed successfully and the code is great

image

Copy link
Copy Markdown
Contributor

@kobakvantrishvili kobakvantrishvili left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works well.

image

Copy link
Copy Markdown
Contributor

@nikhilgiri0226 nikhilgiri0226 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have tested the PR and every test case passed successfully.

image

Copy link
Copy Markdown

@nikhilpittala16 nikhilpittala16 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have tested the PR and all the test cases have passed.
image

Copy link
Copy Markdown
Contributor

@jindaxz jindaxz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok
image

Copy link
Copy Markdown
Contributor

@sundarmachani sundarmachani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test cases are running successfully

Screenshot 2025-02-27 at 9 22 27 PM

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit 8e6e8ba into development Apr 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible Moved to Final Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants