Rm extracted test dirs
CONTRIBUTOR
Thanks for cleaning up the extracted dummy data folders! Instead of changing the file_utils we could also just put these folders under `.gitignore` (or maybe already done?).
2020-05-20T16:32:16Z
https://github.com/huggingface/datasets/pull/173#issuecomment-631586222
https://api.github.com/repos/huggingface/datasets/issues/173
MDEyOklzc3VlQ29tbWVudDYzMTU4NjIyMg==
https://api.github.com/repos/huggingface/datasets/issues/comments/631586222/reactions
2020-05-20T16:32:16Z
https://api.github.com/repos/huggingface/datasets/issues/comments/631586222
https://avatars.githubusercontent.com/u/23423619?v=4
https://api.github.com/users/patrickvonplaten/events{/privacy}
https://api.github.com/users/patrickvonplaten/followers
https://api.github.com/users/patrickvonplaten/following{/other_user}
https://api.github.com/users/patrickvonplaten/gists{/gist_id}
https://github.com/patrickvonplaten
patrickvonplaten
MDQ6VXNlcjIzNDIzNjE5
https://api.github.com/users/patrickvonplaten/orgs
https://api.github.com/users/patrickvonplaten/received_events
https://api.github.com/users/patrickvonplaten/repos
https://api.github.com/users/patrickvonplaten/starred{/owner}{/repo}
https://api.github.com/users/patrickvonplaten/subscriptions
User
https://api.github.com/users/patrickvonplaten
public
CONTRIBUTOR
Awesome! I guess you might have to add the changes for the MockDLManager now in a different file though because of my last PR - sorry!
2020-05-22T15:22:34Z
https://github.com/huggingface/datasets/pull/173#issuecomment-632751031
https://api.github.com/repos/huggingface/datasets/issues/173
MDEyOklzc3VlQ29tbWVudDYzMjc1MTAzMQ==
https://api.github.com/repos/huggingface/datasets/issues/comments/632751031/reactions
2020-05-22T15:22:34Z
https://api.github.com/repos/huggingface/datasets/issues/comments/632751031
https://avatars.githubusercontent.com/u/23423619?v=4
https://api.github.com/users/patrickvonplaten/events{/privacy}
https://api.github.com/users/patrickvonplaten/followers
https://api.github.com/users/patrickvonplaten/following{/other_user}
https://api.github.com/users/patrickvonplaten/gists{/gist_id}
https://github.com/patrickvonplaten
patrickvonplaten
MDQ6VXNlcjIzNDIzNjE5
https://api.github.com/users/patrickvonplaten/orgs
https://api.github.com/users/patrickvonplaten/received_events
https://api.github.com/users/patrickvonplaten/repos
https://api.github.com/users/patrickvonplaten/starred{/owner}{/repo}
https://api.github.com/users/patrickvonplaten/subscriptions
User
https://api.github.com/users/patrickvonplaten
public
All the dummy data used for tests were duplicated. For each dataset, we had one zip file but also its extracted directory. I removed all these directories

Furthermore instead of extracting next to the dummy_data.zip file, we extract in the temp `cached_dir` used for tests, so that all the extracted directories get removed after testing.

Finally there was a bug in the `mock_download_manager` that would let it create directories with invalid names, as in #172. I fixed that by encoding url arguments. I had to rename the dummy data for `scientific_papers` and `cnn_dailymail` (the aws tests don't pass for those 2 in this PR, but they will once aws will be synced, as the local ones do)

Let me know if it sounds good to you @patrickvonplaten . I'm still not entirely familiar with the mock downloader