Skip to content

Disable download tests for Python2 #1269

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.

Sign up for GitHub

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

Merged
merged 1 commit into from
Aug 28, 2019
Merged

Disable download tests for Python2 #1269

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions test/test_datasets_utils.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
import tarfile
import gzip
import warnings
from torch._six import PY2
from torch._utils_internal import get_file_path_2

from common_utils import get_tmp_dir
Expand Down Expand Up @@ -40,6 +41,7 @@ def test_check_integrity(self):
self.assertTrue(utils.check_integrity(existing_fpath))
self.assertFalse(utils.check_integrity(nonexisting_fpath))

@unittest.skipIf(PY2, "https://github.com/pytorch/vision/issues/1268")
def test_download_url(self):
with get_tmp_dir() as temp_dir:
url = "http://github.com/pytorch/vision/archive/master.zip"
Expand All @@ -51,6 +53,7 @@ def test_download_url(self):
warnings.warn(msg, RuntimeWarning)
raise unittest.SkipTest(msg)

@unittest.skipIf(PY2, "https://github.com/pytorch/vision/issues/1268")
def test_download_url_retry_http(self):
with get_tmp_dir() as temp_dir:
url = "https://github.com/pytorch/vision/archive/master.zip"
Expand Down