Skip to content

Commit

Permalink
Fix tests to access S3 (#896)
Browse files Browse the repository at this point in the history
Summary:
Fixes failed tests as the open-sourced Dataset has changed.

Pull Request resolved: #896

Reviewed By: NivekT

Differential Revision: D41309257

Pulled By: ejguan

fbshipit-source-id: 9c106a4dc5d4541c7fe402b24b68eeb8149fdd11
  • Loading branch information
ejguan committed Dec 14, 2022
1 parent 68c24d3 commit 117e9e9
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/test_remote_io.py
Original file line number Diff line number Diff line change
Expand Up @@ -192,7 +192,7 @@ def _filepath_fn(url):
@skipIfNoFSSpecS3
def test_fsspec_io_iterdatapipe(self):
input_list = [
(["s3://ai2-public-datasets"], 39), # bucket without '/'
(["s3://ai2-public-datasets"], 40), # bucket without '/'
(["s3://ai2-public-datasets/charades/"], 18), # bucket with '/'
(
[
Expand Down Expand Up @@ -226,8 +226,8 @@ def test_disabled_s3_io_iterdatapipe(self):
def test_s3_io_iterdatapipe(self):
# S3FileLister: different inputs
input_list = [
[["s3://ai2-public-datasets"], 77], # bucket without '/'
[["s3://ai2-public-datasets/"], 77], # bucket with '/'
[["s3://ai2-public-datasets"], 78], # bucket without '/'
[["s3://ai2-public-datasets/"], 78], # bucket with '/'
[["s3://ai2-public-datasets/charades"], 18], # folder without '/'
[["s3://ai2-public-datasets/charades/"], 18], # folder without '/'
[["s3://ai2-public-datasets/charad"], 18], # prefix
Expand Down

0 comments on commit 117e9e9

Please sign in to comment.