Replace `assertEqual` with `assertTupleEqual` in unit tests for verbosity
NONE
_The documentation is not available anymore as the PR was closed or merged._
2022-06-15T09:36:48Z
https://github.com/huggingface/datasets/pull/4496#issuecomment-1156238949
https://api.github.com/repos/huggingface/datasets/issues/4496
IC_kwDODunzps5E6s5l
https://api.github.com/repos/huggingface/datasets/issues/comments/1156238949/reactions
2022-07-07T17:06:51Z
https://api.github.com/repos/huggingface/datasets/issues/comments/1156238949
https://avatars.githubusercontent.com/u/99929124?v=4
https://api.github.com/users/HuggingFaceDocBuilderDev/events{/privacy}
https://api.github.com/users/HuggingFaceDocBuilderDev/followers
https://api.github.com/users/HuggingFaceDocBuilderDev/following{/other_user}
https://api.github.com/users/HuggingFaceDocBuilderDev/gists{/gist_id}
https://github.com/HuggingFaceDocBuilderDev
HuggingFaceDocBuilderDev
U_kgDOBfTMJA
https://api.github.com/users/HuggingFaceDocBuilderDev/orgs
https://api.github.com/users/HuggingFaceDocBuilderDev/received_events
https://api.github.com/users/HuggingFaceDocBuilderDev/repos
https://api.github.com/users/HuggingFaceDocBuilderDev/starred{/owner}{/repo}
https://api.github.com/users/HuggingFaceDocBuilderDev/subscriptions
User
https://api.github.com/users/HuggingFaceDocBuilderDev
public
MEMBER
FYI I used the following regex to look for the `assertEqual` statements where the assertion was being done over a Tuple: `self.assertEqual(.*, \(.*,)(\)\))$`, hope this is useful!
2022-06-15T09:44:13Z
https://github.com/huggingface/datasets/pull/4496#issuecomment-1156246575
https://api.github.com/repos/huggingface/datasets/issues/4496
IC_kwDODunzps5E6uwv
https://api.github.com/repos/huggingface/datasets/issues/comments/1156246575/reactions
2022-06-15T09:44:13Z
https://api.github.com/repos/huggingface/datasets/issues/comments/1156246575
https://avatars.githubusercontent.com/u/36760800?v=4
https://api.github.com/users/alvarobartt/events{/privacy}
https://api.github.com/users/alvarobartt/followers
https://api.github.com/users/alvarobartt/following{/other_user}
https://api.github.com/users/alvarobartt/gists{/gist_id}
https://github.com/alvarobartt
alvarobartt
MDQ6VXNlcjM2NzYwODAw
https://api.github.com/users/alvarobartt/orgs
https://api.github.com/users/alvarobartt/received_events
https://api.github.com/users/alvarobartt/repos
https://api.github.com/users/alvarobartt/starred{/owner}{/repo}
https://api.github.com/users/alvarobartt/subscriptions
User
https://api.github.com/users/alvarobartt
public
As detailed in #4419 and as suggested by @mariosasko, we could replace the `assertEqual` assertions with `assertTupleEqual` when the assertion is between Tuples, in order to make the tests more verbose.