-
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
#160928262 Test files in Home module #16
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)
06d8177
to
2732bc7
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)
2732bc7
to
a915019
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)
a915019
to
14b6dfb
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)
14b6dfb
to
23b1205
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)
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)
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.
Nice work, I like the concept of shallow rendering the components as function.
- adds tests for the "modules/Home" files - ensures that all the tests have 100% coverage [Finishes #160928262]
23b1205
to
8fe1019
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?
CardItem
componentCards
componentHome
containerDescription of Task to be completed?
How should this be manually tested?
git pull
commandnpm test
modules/Home
has 100% coverageAny background context you want to provide?
modules/Home
folder have not yet been tested. ThisPR
adds tests for themWhat are the relevant pivotal tracker stories?
#160928262
Screenshots or gifs (if appropriate)
Questions: