Bugzilla: Add some tests for regressed coverage pieces

crobinso · crobinso · commit 5ab112338535 · 2020-01-11T19:26:08.000-05:00
Signed-off-by: Cole Robinson &lt;crobinso@redhat.com&gt;
diff --git a/tests/data/mockargs/test_api_getbugs2.txt b/tests/data/mockargs/test_api_getbugs2.txt
@@ -0,0 +1 @@
+(['123456'], ['CVE-1234-FAKE'], {'permissive': 1})
diff --git a/tests/data/mockargs/test_api_login2.txt b/tests/data/mockargs/test_api_login2.txt
@@ -0,0 +1 @@
+{'login': None, 'password': None}
diff --git a/tests/test_api_bug.py b/tests/test_api_bug.py
@@ -92,6 +92,11 @@ def test_api_getbugs():
     assert bug.alias == ["CVE-1234-5678"]
     assert bug.autorefresh is True
 
+    fakebz = tests.mockbackend.make_bz(
+        bug_get_args="data/mockargs/test_api_getbugs2.txt",
+        bug_get_return={"bugs": [{}, {}]})
+    assert fakebz.getbugs(["123456", "CVE-1234-FAKE"]) == []
+
 
 def test_bug_getattr():
     fakebz = tests.mockbackend.make_bz(
diff --git a/tests/test_api_misc.py b/tests/test_api_misc.py
@@ -198,6 +198,12 @@ def test_api_login():
     bz.connect("https:///fake/bugzilla.redhat.com")
     bz.connect()
 
+    # Test auto login if user/password is set
+    bz = tests.mockbackend.make_bz(
+        bz_kwargs={"user": "FOO", "password": "BAR"},
+        user_login_args="data/mockargs/test_api_login2.txt",
+        user_login_return={})
+
 
 def test_interactive_login(capsys, monkeypatch):
     bz = tests.mockbackend.make_bz(

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+(['123456'], ['CVE-1234-FAKE'], {'permissive': 1})`