Skip to content
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

unit test for getWeekHeaders() #1798

Merged
merged 1 commit into from Oct 4, 2019

Conversation

@v47
Copy link
Contributor

commented Sep 23, 2019

Will prevent regressions like in #1795.
Ensures that state stays immuted while executing getWeekHeaders().

test/components/DayPicker_spec.jsx Outdated Show resolved Hide resolved
test/components/DayPicker_spec.jsx Outdated Show resolved Hide resolved
test/components/DayPicker_spec.jsx Outdated Show resolved Hide resolved
test/components/DayPicker_spec.jsx Outdated Show resolved Hide resolved
@v47 v47 force-pushed the v47:getWeekHeaders-unit-test branch from fcfaf38 to 4852b31 Sep 23, 2019
@v47 v47 force-pushed the v47:getWeekHeaders-unit-test branch from 4852b31 to 0016b55 Sep 23, 2019
@v47 v47 force-pushed the v47:getWeekHeaders-unit-test branch from 0016b55 to 14108be Oct 4, 2019
@ljharb
ljharb approved these changes Oct 4, 2019
@ljharb ljharb force-pushed the v47:getWeekHeaders-unit-test branch 2 times, most recently from 0ef4175 to 5584f82 Oct 4, 2019
@ljharb ljharb merged commit 5584f82 into airbnb:master Oct 4, 2019
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 11.246%
Details
@v47

This comment has been minimized.

Copy link
Contributor Author

commented Oct 4, 2019

@ljharb Thanks! Feels like you are always watching :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.