Skip to content
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.

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

Add test_package_query_with_no_name #125

Merged

Conversation

msabramo
Copy link
Contributor

No description provided.

@@ -65,6 +65,12 @@ def test_repo_show
let(:repo_api) { AptlyCli::AptlyRepo.new(config) }
let(:file_api) { AptlyCli::AptlyFile.new(config) }

def test_package_query_with_no_name
assert_raises ArgumentError do
res = repo_api.repo_package_query(query: 'geoipupdate')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Useless assignment to variable - res.

@msabramo msabramo force-pushed the test_package_query_with_no_name branch from ccfe850 to e303082 Compare July 23, 2016 07:31
@coveralls
Copy link

coveralls commented Jul 23, 2016

Coverage Status

Coverage increased (+0.2%) to 97.372% when pulling e303082 on msabramo:test_package_query_with_no_name into e0b3638 on sepulworld:master.

@coveralls
Copy link

coveralls commented Jul 23, 2016

Coverage Status

Coverage increased (+0.2%) to 97.372% when pulling e303082 on msabramo:test_package_query_with_no_name into e0b3638 on sepulworld:master.

@sepulworld sepulworld merged commit dd13ae6 into sepulworld:master Jul 23, 2016
@msabramo msabramo deleted the test_package_query_with_no_name branch July 23, 2016 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants