-
Notifications
You must be signed in to change notification settings - Fork 6.8k
test(table): add tests for MatTableDataSource #18767
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
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.
LGTM
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.
Accidental premature approval
|
||
/** Randomly shuffles the array of data. */ | ||
function shuffle(data: any[]) { | ||
return data.slice().sort(() => (Math.random() - 0.5)); |
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.
No random()
in tests!
1f3a0f5
to
143e58b
Compare
Updated to replace shuffle with just a plain reverse |
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.
LGTM
* test(table): add tests for MatTableDataSource * switch shuffle to reverse
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Starts up a testing spec just for the data source - will be useful for adding tests over the sorting and pagination functions