-
Notifications
You must be signed in to change notification settings - Fork 0
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
#161116965 Create tests for create team module #18
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some files could not be reviewed due to errors:
Cannot find module 'eslint-config-react'
Cannot find module 'eslint-config-react' Referenced from: .eslintrc Error: Cannot find module 'eslint-config-react' at ModuleResolver.resolve (/home/linters/node_modules/eslint/lib/util/module-resolver.js:74:19) at resolve (/home/linters/node_modules/eslint/lib/config/config-file.js:471:25) at load (/home/linters/node_modules/eslint/lib/config/config-file.js:542:26) at configExtends.reduceRight (/home/linters/node_modules/eslint/lib/config/config-file.js:421:36) at Array.reduceRight () at applyExtends (/home/linters/node_modules/eslint/lib/config/config-file.js:403:28) at loadFromDisk (/home/linters/node_modules/eslint/lib/config/config-file.js:514:22) at Object.load (/home/linters/node_modules/eslint/lib/config/config-file.js:550:20) at Config.getLocalConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:228:44) at Config.getConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:180:43)
0cea258
to
c60fc69
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some files could not be reviewed due to errors:
Cannot find module 'eslint-config-react'
Cannot find module 'eslint-config-react' Referenced from: .eslintrc Error: Cannot find module 'eslint-config-react' at ModuleResolver.resolve (/home/linters/node_modules/eslint/lib/util/module-resolver.js:74:19) at resolve (/home/linters/node_modules/eslint/lib/config/config-file.js:471:25) at load (/home/linters/node_modules/eslint/lib/config/config-file.js:542:26) at configExtends.reduceRight (/home/linters/node_modules/eslint/lib/config/config-file.js:421:36) at Array.reduceRight () at applyExtends (/home/linters/node_modules/eslint/lib/config/config-file.js:403:28) at loadFromDisk (/home/linters/node_modules/eslint/lib/config/config-file.js:514:22) at Object.load (/home/linters/node_modules/eslint/lib/config/config-file.js:550:20) at Config.getLocalConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:228:44) at Config.getConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:180:43)
c60fc69
to
5ae03f1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some files could not be reviewed due to errors:
Cannot find module 'eslint-config-react'
Cannot find module 'eslint-config-react' Referenced from: .eslintrc Error: Cannot find module 'eslint-config-react' at ModuleResolver.resolve (/home/linters/node_modules/eslint/lib/util/module-resolver.js:74:19) at resolve (/home/linters/node_modules/eslint/lib/config/config-file.js:471:25) at load (/home/linters/node_modules/eslint/lib/config/config-file.js:542:26) at configExtends.reduceRight (/home/linters/node_modules/eslint/lib/config/config-file.js:421:36) at Array.reduceRight () at applyExtends (/home/linters/node_modules/eslint/lib/config/config-file.js:403:28) at loadFromDisk (/home/linters/node_modules/eslint/lib/config/config-file.js:514:22) at Object.load (/home/linters/node_modules/eslint/lib/config/config-file.js:550:20) at Config.getLocalConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:228:44) at Config.getConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:180:43)
5ae03f1
to
12510e8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some files could not be reviewed due to errors:
Cannot find module 'eslint-config-react'
Cannot find module 'eslint-config-react' Referenced from: .eslintrc Error: Cannot find module 'eslint-config-react' at ModuleResolver.resolve (/home/linters/node_modules/eslint/lib/util/module-resolver.js:74:19) at resolve (/home/linters/node_modules/eslint/lib/config/config-file.js:471:25) at load (/home/linters/node_modules/eslint/lib/config/config-file.js:542:26) at configExtends.reduceRight (/home/linters/node_modules/eslint/lib/config/config-file.js:421:36) at Array.reduceRight () at applyExtends (/home/linters/node_modules/eslint/lib/config/config-file.js:403:28) at loadFromDisk (/home/linters/node_modules/eslint/lib/config/config-file.js:514:22) at Object.load (/home/linters/node_modules/eslint/lib/config/config-file.js:550:20) at Config.getLocalConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:228:44) at Config.getConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:180:43)
12510e8
to
70a3d02
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some files could not be reviewed due to errors:
Cannot find module 'eslint-config-react'
Cannot find module 'eslint-config-react' Referenced from: .eslintrc Error: Cannot find module 'eslint-config-react' at ModuleResolver.resolve (/home/linters/node_modules/eslint/lib/util/module-resolver.js:74:19) at resolve (/home/linters/node_modules/eslint/lib/config/config-file.js:471:25) at load (/home/linters/node_modules/eslint/lib/config/config-file.js:542:26) at configExtends.reduceRight (/home/linters/node_modules/eslint/lib/config/config-file.js:421:36) at Array.reduceRight () at applyExtends (/home/linters/node_modules/eslint/lib/config/config-file.js:403:28) at loadFromDisk (/home/linters/node_modules/eslint/lib/config/config-file.js:514:22) at Object.load (/home/linters/node_modules/eslint/lib/config/config-file.js:550:20) at Config.getLocalConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:228:44) at Config.getConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:180:43)
70a3d02
to
b7822e0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some files could not be reviewed due to errors:
Cannot find module 'eslint-config-react'
Cannot find module 'eslint-config-react' Referenced from: .eslintrc Error: Cannot find module 'eslint-config-react' at ModuleResolver.resolve (/home/linters/node_modules/eslint/lib/util/module-resolver.js:74:19) at resolve (/home/linters/node_modules/eslint/lib/config/config-file.js:471:25) at load (/home/linters/node_modules/eslint/lib/config/config-file.js:542:26) at configExtends.reduceRight (/home/linters/node_modules/eslint/lib/config/config-file.js:421:36) at Array.reduceRight () at applyExtends (/home/linters/node_modules/eslint/lib/config/config-file.js:403:28) at loadFromDisk (/home/linters/node_modules/eslint/lib/config/config-file.js:514:22) at Object.load (/home/linters/node_modules/eslint/lib/config/config-file.js:550:20) at Config.getLocalConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:228:44) at Config.getConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:180:43)
b7822e0
to
4f97650
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some files could not be reviewed due to errors:
Cannot find module 'eslint-config-react'
Cannot find module 'eslint-config-react' Referenced from: .eslintrc Error: Cannot find module 'eslint-config-react' at ModuleResolver.resolve (/home/linters/node_modules/eslint/lib/util/module-resolver.js:74:19) at resolve (/home/linters/node_modules/eslint/lib/config/config-file.js:471:25) at load (/home/linters/node_modules/eslint/lib/config/config-file.js:542:26) at configExtends.reduceRight (/home/linters/node_modules/eslint/lib/config/config-file.js:421:36) at Array.reduceRight () at applyExtends (/home/linters/node_modules/eslint/lib/config/config-file.js:403:28) at loadFromDisk (/home/linters/node_modules/eslint/lib/config/config-file.js:514:22) at Object.load (/home/linters/node_modules/eslint/lib/config/config-file.js:550:20) at Config.getLocalConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:228:44) at Config.getConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:180:43)
4f97650
to
abf2b9e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some files could not be reviewed due to errors:
Cannot find module 'eslint-config-react'
Cannot find module 'eslint-config-react' Referenced from: .eslintrc Error: Cannot find module 'eslint-config-react' at ModuleResolver.resolve (/home/linters/node_modules/eslint/lib/util/module-resolver.js:74:19) at resolve (/home/linters/node_modules/eslint/lib/config/config-file.js:471:25) at load (/home/linters/node_modules/eslint/lib/config/config-file.js:542:26) at configExtends.reduceRight (/home/linters/node_modules/eslint/lib/config/config-file.js:421:36) at Array.reduceRight () at applyExtends (/home/linters/node_modules/eslint/lib/config/config-file.js:403:28) at loadFromDisk (/home/linters/node_modules/eslint/lib/config/config-file.js:514:22) at Object.load (/home/linters/node_modules/eslint/lib/config/config-file.js:550:20) at Config.getLocalConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:228:44) at Config.getConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:180:43)
- add test for Form component - add test for create team container component [Finishes #161116965]
abf2b9e
to
14cecb8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some files could not be reviewed due to errors:
Cannot find module 'eslint-config-react'
Cannot find module 'eslint-config-react' Referenced from: .eslintrc Error: Cannot find module 'eslint-config-react' at ModuleResolver.resolve (/home/linters/node_modules/eslint/lib/util/module-resolver.js:74:19) at resolve (/home/linters/node_modules/eslint/lib/config/config-file.js:471:25) at load (/home/linters/node_modules/eslint/lib/config/config-file.js:542:26) at configExtends.reduceRight (/home/linters/node_modules/eslint/lib/config/config-file.js:421:36) at Array.reduceRight () at applyExtends (/home/linters/node_modules/eslint/lib/config/config-file.js:403:28) at loadFromDisk (/home/linters/node_modules/eslint/lib/config/config-file.js:514:22) at Object.load (/home/linters/node_modules/eslint/lib/config/config-file.js:550:20) at Config.getLocalConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:228:44) at Config.getConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:180:43)
What does this PR do?
Description of Task to be completed?
Add tests for the files in Create Team modules
It is important to write tests for components to ensure that when the components are updated, breaking changes do not occur to the app.
Also, tests make it easy for new members of the team to onboard to the codebase and also serves as a way to document our application
How should this be manually tested?
Any background context you want to provide?
N/A
What are the relevant pivotal tracker stories?
Story #161116965
Screenshots or gifs (if appropriate)
N/A
Questions: None