From 5dd198a38ec878fae20e0ffc7d9dd9efd0bc401e Mon Sep 17 00:00:00 2001 From: Yann Dirson Date: Wed, 26 Jun 2024 16:02:37 +0200 Subject: [PATCH] examples: be more precise about TestReport outcome * testing only for "failed" should not be reported as "or skipped" * test for "skipped" explicitly instead --- changelog/12535.doc.rst | 5 +++++ doc/en/example/simple.rst | 4 +++- 2 files changed, 8 insertions(+), 1 deletion(-) create mode 100644 changelog/12535.doc.rst diff --git a/changelog/12535.doc.rst b/changelog/12535.doc.rst new file mode 100644 index 000000000..6ce37ac12 --- /dev/null +++ b/changelog/12535.doc.rst @@ -0,0 +1,5 @@ +`This +example` +showed ``print`` statements that do not exactly reflect what the +different branches actually do. The fix makes the example more precise. + diff --git a/doc/en/example/simple.rst b/doc/en/example/simple.rst index d4ace3f04..64b54c7bf 100644 --- a/doc/en/example/simple.rst +++ b/doc/en/example/simple.rst @@ -929,7 +929,9 @@ here is a little example implemented via a local plugin: # "function" scope report = request.node.stash[phase_report_key] if report["setup"].failed: - print("setting up a test failed or skipped", request.node.nodeid) + print("setting up a test failed", request.node.nodeid) + elif report["setup"].skipped: + print("setting up a test skipped", request.node.nodeid) elif ("call" not in report) or report["call"].failed: print("executing test failed or skipped", request.node.nodeid)