Disallow backslash in urls
MEMBER
Looks like the test doesn't detect all the problems fixed by #907 , I'll fix that
2020-11-27T20:30:37Z
https://github.com/huggingface/datasets/pull/905#issuecomment-734982806
https://api.github.com/repos/huggingface/datasets/issues/905
MDEyOklzc3VlQ29tbWVudDczNDk4MjgwNg==
https://api.github.com/repos/huggingface/datasets/issues/comments/734982806/reactions
2020-11-27T20:30:37Z
https://api.github.com/repos/huggingface/datasets/issues/comments/734982806
https://avatars.githubusercontent.com/u/42851186?v=4
https://api.github.com/users/lhoestq/events{/privacy}
https://api.github.com/users/lhoestq/followers
https://api.github.com/users/lhoestq/following{/other_user}
https://api.github.com/users/lhoestq/gists{/gist_id}
https://github.com/lhoestq
lhoestq
MDQ6VXNlcjQyODUxMTg2
https://api.github.com/users/lhoestq/orgs
https://api.github.com/users/lhoestq/received_events
https://api.github.com/users/lhoestq/repos
https://api.github.com/users/lhoestq/starred{/owner}{/repo}
https://api.github.com/users/lhoestq/subscriptions
User
https://api.github.com/users/lhoestq
public
MEMBER
Ok found why it doesn't detect the problems fixed by #907 . That's because for all those datasets the urls are actually fine (no backslash) on windows, even if it uses `os.path.join`.

This is because of the behavior of `os.path.join` on windows when the first path ends with a slash : 

```python
import os
os.path.join("https://test.com/foo", "bar.txt")
# 'https://test.com/foo\\bar.txt'
os.path.join("https://test.com/foo/", "bar.txt")
# 'https://test.com/foo/bar.txt'
```

However even though the urls are correct, this is definitely bad practice and we should never use `os.path.join` for urls
2020-11-29T22:43:37Z
https://github.com/huggingface/datasets/pull/905#issuecomment-735469686
https://api.github.com/repos/huggingface/datasets/issues/905
MDEyOklzc3VlQ29tbWVudDczNTQ2OTY4Ng==
https://api.github.com/repos/huggingface/datasets/issues/comments/735469686/reactions
2020-11-29T22:43:37Z
https://api.github.com/repos/huggingface/datasets/issues/comments/735469686
https://avatars.githubusercontent.com/u/42851186?v=4
https://api.github.com/users/lhoestq/events{/privacy}
https://api.github.com/users/lhoestq/followers
https://api.github.com/users/lhoestq/following{/other_user}
https://api.github.com/users/lhoestq/gists{/gist_id}
https://github.com/lhoestq
lhoestq
MDQ6VXNlcjQyODUxMTg2
https://api.github.com/users/lhoestq/orgs
https://api.github.com/users/lhoestq/received_events
https://api.github.com/users/lhoestq/repos
https://api.github.com/users/lhoestq/starred{/owner}{/repo}
https://api.github.com/users/lhoestq/subscriptions
User
https://api.github.com/users/lhoestq
public
Following #903 @albertvillanova noticed that there are sometimes bad usage of `os.path.join` in datasets scripts to create URLS. However this should be avoided since it doesn't work on windows.

I'm suggesting a test to make sure we that all the urls don't have backslashes in them in the datasets scripts.
The tests works by adding a callback feature to the MockDownloadManager used to test the dataset scripts. In a download callback I just make sure that the url is valid.