Move the User-URL log to info so we can easily checked that toddlers behave as expected
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
This commit is contained in:
parent
ff3ab68f0c
commit
519d73d0e4
4 changed files with 9 additions and 7 deletions
|
@ -139,7 +139,7 @@ class TestFlagCIPRToddler:
|
||||||
"artifact": {
|
"artifact": {
|
||||||
"id": 456,
|
"id": 456,
|
||||||
"commit_hash": "abcdefghijklmnopqrst",
|
"commit_hash": "abcdefghijklmnopqrst",
|
||||||
"repository": "namespace/name",
|
"repository": "https://host.c/namespace/name",
|
||||||
},
|
},
|
||||||
"run": {"url": "https://example.com/testing",},
|
"run": {"url": "https://example.com/testing",},
|
||||||
}
|
}
|
||||||
|
@ -154,6 +154,7 @@ class TestFlagCIPRToddler:
|
||||||
== None
|
== None
|
||||||
)
|
)
|
||||||
assert (
|
assert (
|
||||||
caplog.records[-2].message == "Request to https://pagure.io returned: 200"
|
caplog.records[-3].message == "Request to https://pagure.io returned: 200"
|
||||||
)
|
)
|
||||||
assert caplog.records[-1].message == "All clear"
|
assert caplog.records[-2].message == "All clear"
|
||||||
|
assert caplog.records[-1].message == "User-URL: https://host.c/namespace/namepull-request/456"
|
||||||
|
|
|
@ -286,7 +286,8 @@ class TestFlagCommitBuildToddler:
|
||||||
== None
|
== None
|
||||||
)
|
)
|
||||||
assert (
|
assert (
|
||||||
caplog.records[-2].message
|
caplog.records[-3].message
|
||||||
== "Request to https://src.fedoraproject.org returned: 200"
|
== "Request to https://src.fedoraproject.org returned: 200"
|
||||||
)
|
)
|
||||||
assert caplog.records[-1].message == "All clear"
|
assert caplog.records[-2].message == "All clear"
|
||||||
|
assert caplog.records[-1].message == "User-URL: https://src.fedoraproject.org/rpms/guake/c/commit_hash123"
|
||||||
|
|
|
@ -143,7 +143,7 @@ class FlagCIPR(ToddlerBase):
|
||||||
return 1
|
return 1
|
||||||
else:
|
else:
|
||||||
_log.info("All clear")
|
_log.info("All clear")
|
||||||
_log.debug(
|
_log.info(
|
||||||
"User-URL: %s" % msg["artifact"]["repository"]
|
"User-URL: %s" % msg["artifact"]["repository"]
|
||||||
+ "/".join(["pull-request", pr_id])
|
+ "/".join(["pull-request", pr_id])
|
||||||
)
|
)
|
||||||
|
|
|
@ -150,7 +150,7 @@ class FlagCommitBuild(ToddlerBase):
|
||||||
# raise Exception("Did not successfully flag the build in dist-git")
|
# raise Exception("Did not successfully flag the build in dist-git")
|
||||||
else:
|
else:
|
||||||
_log.info("All clear")
|
_log.info("All clear")
|
||||||
_log.debug(
|
_log.info(
|
||||||
"User-URL: %s" % pagure_url
|
"User-URL: %s" % pagure_url
|
||||||
+ "/"
|
+ "/"
|
||||||
+ "/".join(["rpms", msg["name"], "c", commit])
|
+ "/".join(["rpms", msg["name"], "c", commit])
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue