Skip to content

Aaryaneil Unit Tests for userProjectMembersReducer component#2998

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

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

Conversation

@aaryaneil
Copy link
Copy Markdown
Contributor

Description

Unit test for src/reducers/userProjectMembersReducer.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_PROJECT_MEMBERS
  3. should return the current state when an unknown action is provided

How to test:

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

Screenshots or videos of changes:

Screenshot 2024-12-31 at 3 51 51 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 2a740da
🔍 Latest deploy log https://app.netlify.com/sites/highestgoodnetwork-dev/deploys/6774678f85440e000822a64b
😎 Deploy Preview https://deploy-preview-2998--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.

@beblicarl beblicarl self-requested a review January 2, 2025 10:10
@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

@srikanthpusthem srikanthpusthem left a comment

Choose a reason for hiding this comment

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

image test cases passed as expected

Copy link
Copy Markdown
Contributor

@anirudhsk2107 anirudhsk2107 left a comment

Choose a reason for hiding this comment

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

All tests have passed
Screen Shot 2025-01-17 at 8 01 41 PM

Copy link
Copy Markdown

@sabithanazareth sabithanazareth 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 have passed successfully
2998

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 executing successfully

Screenshot 2025-02-27 at 9 19 19 PM

@koushi1906
Copy link
Copy Markdown

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

Screenshot 2025-03-01 at 8 58 29 PM

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit af37284 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.