[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 02/17] tests/qapi-schema: Fix test 'QAPI rST doc'
From: |
Markus Armbruster |
Subject: |
[PULL 02/17] tests/qapi-schema: Fix test 'QAPI rST doc' |
Date: |
Mon, 26 Feb 2024 11:49:39 +0100 |
The test compares Sphinx plain-text output against a golden reference.
To work on Windows hosts, it filters out carriage returns in both
files. Unfortunately, the filter doesn't work: it creates an empty
file. Comparing empty files always succeeds.
Fix the filter, and update the golden reference to current Sphinx
output.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-ID: <20240216145841.2099240-2-armbru@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
---
tests/qapi-schema/doc-good.txt | 21 +++++++++------------
tests/qapi-schema/meson.build | 10 ++++++----
2 files changed, 15 insertions(+), 16 deletions(-)
diff --git a/tests/qapi-schema/doc-good.txt b/tests/qapi-schema/doc-good.txt
index b3b76bd43f..879f6ff50a 100644
--- a/tests/qapi-schema/doc-good.txt
+++ b/tests/qapi-schema/doc-good.txt
@@ -44,7 +44,7 @@ Values
~~~~~~
"one" (**If: **"IFONE")
- The _one_ {and only}
+ The _one_ {and only}, description on the same line
"two"
Not documented
@@ -76,7 +76,7 @@ Members
~~~~~~~
"base1": "Enum"
- the first member
+ description starts on a new line, minimally indented
If
@@ -90,7 +90,9 @@ If
A paragraph
-Another paragraph (but no "var": line)
+Another paragraph
+
+"var1" is undocumented
Members
@@ -141,7 +143,8 @@ Members
~~~~~~~
"i": "int"
- an integer "b" is undocumented
+ description starts on the same line remainder indented the same "b"
+ is undocumented
"b": "boolean"
Not documented
@@ -172,10 +175,10 @@ Arguments
~~~~~~~~~
"arg1": "int"
- the first argument
+ description starts on a new line, indented
"arg2": "string" (optional)
- the second argument
+ description starts on the same line remainder indented differently
"arg3": "boolean"
Not documented
@@ -203,12 +206,6 @@ Returns
"Object"
-TODO
-~~~~
-
-frobnicate
-
-
Notes
~~~~~
diff --git a/tests/qapi-schema/meson.build b/tests/qapi-schema/meson.build
index af085f745d..44a9385563 100644
--- a/tests/qapi-schema/meson.build
+++ b/tests/qapi-schema/meson.build
@@ -273,15 +273,17 @@ if build_docs
output: ['doc-good.txt.nocr'],
input: qapi_doc_out[0],
build_by_default: true,
- command: [remove_cr, '@INPUT@'],
- capture: true)
+ command: [remove_cr],
+ capture: true,
+ feed: true)
qapi_doc_ref_nocr = custom_target('QAPI rST doc reference newline-sanitized',
output: ['doc-good.ref.nocr'],
input: files('doc-good.txt'),
build_by_default: true,
- command: [remove_cr, '@INPUT@'],
- capture: true)
+ command: [remove_cr],
+ capture: true,
+ feed: true)
test('QAPI rST doc', diff, args: ['-u', qapi_doc_ref_nocr[0],
qapi_doc_out_nocr[0]],
suite: ['qapi-schema', 'qapi-doc'])
--
2.43.0
- [PULL 00/17] QAPI patches patches for 2024-02-26, Markus Armbruster, 2024/02/26
- [PULL 14/17] qapi: Merge adjacent untagged sections, Markus Armbruster, 2024/02/26
- [PULL 17/17] qapi: Divorce QAPIDoc from QAPIParseError, Markus Armbruster, 2024/02/26
- [PULL 12/17] qapi: Recognize section tags and 'Features:' only after blank line, Markus Armbruster, 2024/02/26
- [PULL 01/17] qapi: Misc cleanups to migrate QAPIs, Markus Armbruster, 2024/02/26
- [PULL 15/17] qapi: Rewrite doc comment parser, Markus Armbruster, 2024/02/26
- [PULL 02/17] tests/qapi-schema: Fix test 'QAPI rST doc',
Markus Armbruster <=
- [PULL 09/17] qapi: Rename QAPIDoc.Section.name to .tag, Markus Armbruster, 2024/02/26
- [PULL 04/17] tests/qapi-schema: Cover 'Features:' not followed by descriptions, Markus Armbruster, 2024/02/26
- [PULL 05/17] sphinx/qapidoc: Drop code to generate doc for simple union branch, Markus Armbruster, 2024/02/26
- [PULL 16/17] qapi: Reject multiple and empty feature descriptions, Markus Armbruster, 2024/02/26
- [PULL 13/17] qapi: Call QAPIDoc.check() always, Markus Armbruster, 2024/02/26
- [PULL 06/17] qapi: Improve error position for bogus argument descriptions, Markus Armbruster, 2024/02/26
- [PULL 11/17] qapi: Require descriptions and tagged sections to be indented, Markus Armbruster, 2024/02/26
- [PULL 08/17] qapi: Improve error message for empty doc sections, Markus Armbruster, 2024/02/26
- [PULL 03/17] tests/qapi-schema: Cover duplicate 'Features:' line, Markus Armbruster, 2024/02/26
- [PULL 07/17] qapi: Improve error position for bogus invalid "Returns" section, Markus Armbruster, 2024/02/26