Skip to content

Navbar prop 'isUserOwner' is marked as required, but its value is 'undefined' - Jest Error #1724

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

Closed
rt1301 opened this issue Jan 1, 2021 · 8 comments · Fixed by #1739
Closed
Labels
Good First Issue A beginner-friendly issue, great for first-time contributors Help Wanted Would love additional input or contributions! Priority:High Should be addressed soon but not critical

Comments

@rt1301
Copy link
Contributor

rt1301 commented Jan 1, 2021

Nature of issue?

  • Found a bug

Details about the bug:

While running npm run test, the following error is shown :

Failed prop type: The prop isUserOwner is marked as required in Nav, but its value is undefined.

  • Operating System: Windows 10 (64-bit)
  • Steps to reproduce this bug: run the command npm run test in the command line

Navbar-err

@welcome
Copy link

welcome bot commented Jan 1, 2021

Welcome! 👋 Thanks for opening your first issue here! And to ensure the community is able to respond to your issue, be sure to follow the issue template if you haven't already.

@catarak
Copy link
Member

catarak commented Jan 5, 2021

Thanks for reporting! I am able to reproduce.

@catarak catarak added Good First Issue A beginner-friendly issue, great for first-time contributors Help Wanted Would love additional input or contributions! Priority:High Should be addressed soon but not critical type:bug labels Jan 5, 2021
@Swapnil-2001
Copy link
Contributor

Hi @catarak I would like to work on this please. Could you suggest where I should start looking?

@SundeepChand
Copy link
Contributor

Hi @Swapnil-2001 . The test for client/components/Nav.jsx is written in client/components/__test__/Nav.test.jsx

@Swapnil-2001
Copy link
Contributor

Thanks @SundeepChand, I'll start working from there.

@markogrady1
Copy link
Contributor

can i fix this, unless its still being worked on?

@Swapnil-2001
Copy link
Contributor

Hi @markogrady1, I intended to fix this but got caught up in other work. You can take it up if you wish.

@markogrady1
Copy link
Contributor

markogrady1 commented Jan 20, 2021

nice one. I will work on this @Swapnil-2001 if thats ok @catarak

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue A beginner-friendly issue, great for first-time contributors Help Wanted Would love additional input or contributions! Priority:High Should be addressed soon but not critical
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants