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

Upgrade testing performance and simplify #975

Merged
merged 1 commit into from
Oct 3, 2019

Conversation

gabrielliwerant
Copy link
Collaborator

This test sometimes times out when the test suite is run due to the way it is building new array data. It turns out that we don't need a new data set for this test, as we can use smaller rows per page number to get the same result on the existing data set in the test suite.

This test sometimes times out when the test suite is run due to the way it is building new array data. It turns out that we don't need a new data set for this test, as we can use smaller rows per page number to get the same result on the existing data set in the test suite.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 76.565% when pulling 778d2d4 on enhancement/improve-rows-expand-test into f0ea9e5 on master.

@gabrielliwerant gabrielliwerant merged commit 33ba428 into master Oct 3, 2019
@gabrielliwerant gabrielliwerant deleted the enhancement/improve-rows-expand-test branch October 3, 2019 06:16
waqasajaz pushed a commit to waqasajaz/mui-datatables that referenced this pull request Oct 31, 2019
This test sometimes times out when the test suite is run due to the way it is building new array data. It turns out that we don't need a new data set for this test, as we can use smaller rows per page number to get the same result on the existing data set in the test suite.
lalong13 pushed a commit to lalong13/mui-datatables that referenced this pull request Jan 15, 2020
This test sometimes times out when the test suite is run due to the way it is building new array data. It turns out that we don't need a new data set for this test, as we can use smaller rows per page number to get the same result on the existing data set in the test suite.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants