Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Replace unused tuple value with `_`

Co-authored-by: John Davis <[email protected]>
  • Loading branch information
davelopez and jdavcs authored May 21, 2024
1 parent 3ef1065 commit 8f8c885
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions test/unit/files/_util.py
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ def list_root(
):
file_source_pair = file_sources.get_file_source_path(uri)
file_source = file_source_pair.file_source
res, count = file_source.list("/", recursive=recursive, user_context=user_context)
res, _ = file_source.list("/", recursive=recursive, user_context=user_context)
return res


Expand All @@ -56,7 +56,7 @@ def list_dir(
file_source = file_source_pair.file_source
print(file_source_pair.path)
print(uri)
res, count = file_source.list(file_source_pair.path, recursive=recursive, user_context=user_context)
res, _ = file_source.list(file_source_pair.path, recursive=recursive, user_context=user_context)
return res


Expand Down Expand Up @@ -190,7 +190,7 @@ def assert_simple_file_realize(conf_file, recursive=False, filename="a", content

assert file_source_pair.path == "/"
file_source = file_source_pair.file_source
res, count = file_source.list("/", recursive=recursive, user_context=user_context)
res, _ = file_source.list("/", recursive=recursive, user_context=user_context)
a_file = find(res, class_="File", name=filename)
assert a_file

Expand Down
2 changes: 1 addition & 1 deletion test/unit/files/test_basespace.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ def test_file_source():
assert file_source_pair.path == "/"
file_source = file_source_pair.file_source
test_file = os.environ.get("GALAXY_TEST_BASESPACE_TEST_FILE_PATH", "")
res, count = file_source.list(os.path.dirname(test_file), recursive=False, user_context=user_context)
res, _ = file_source.list(os.path.dirname(test_file), recursive=False, user_context=user_context)
a_file = find(res, class_="File", name=os.path.basename(test_file))
assert a_file

Expand Down
4 changes: 2 additions & 2 deletions test/unit/files/test_webdav.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,12 +29,12 @@ def test_file_source():

assert file_source_pair.path == "/"
file_source = file_source_pair.file_source
res, count = file_source.list("/", recursive=True)
res, _ = file_source.list("/", recursive=True)
a_file = find_file_a(res)
assert a_file
assert a_file["uri"] == "gxfiles://test1/a", a_file

res, count = file_source.list("/", recursive=False)
res, _ = file_source.list("/", recursive=False)
file_a = find_file_a(res)
assert file_a
assert file_a["uri"] == "gxfiles://test1/a"
Expand Down

0 comments on commit 8f8c885

Please sign in to comment.