-
-
Notifications
You must be signed in to change notification settings - Fork 112
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
tests: add unit test to percent input #2489
tests: add unit test to percent input #2489
Conversation
…xtway/react-rainbow into tests-add-unit-test-to-percent-input
TestsBug Fixes
Contributorsyvmunayev, LeandroTorresSicilia Commit-Lint commandsYou can trigger Commit-Lint actions by commenting on this PR:
|
…to tests-add-unit-test-to-percent-input
return <CurrencyInput onChange={setValue} value={value} />; | ||
}; | ||
|
||
describe('<CurrencyInput />', () => { |
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.
Similar blocks of code found in 2 locations. Consider refactoring.
return <PercentInput onChange={setValue} value={value} />; | ||
}; | ||
|
||
describe('<PercentInput />', () => { |
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.
Similar blocks of code found in 2 locations. Consider refactoring.
|
||
import CurrencyInput from '..'; | ||
|
||
const TestCurrencyInput = props => { |
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.
Similar blocks of code found in 2 locations. Consider refactoring.
|
||
import PercentInput from '..'; | ||
|
||
const TestPercentInput = props => { |
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.
Similar blocks of code found in 2 locations. Consider refactoring.
Visit the preview URL for this PR (updated for commit 9f6f485): https://react-rainbow--pr2489-tests-add-unit-test-1f0x4vi1.web.app (expires Sat, 29 Oct 2022 16:21:52 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 2de6d670498a0a515484c5637b13d0215372df83 |
…to tests-add-unit-test-to-percent-input
Code Climate has analyzed commit 9f6f485 and detected 4 issues on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
fix: #2487
Changes proposed in this PR:
tests: add unit test to percent input
I have followed (at least) the PR section of the contributing guide.
@nexxtway/react-rainbow