Skip to content

Commit

Permalink
tests/qapi-schema: Fix test 'QAPI rST doc'
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Markus Armbruster committed Feb 26, 2024
1 parent 3cee17e commit 976474f
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 16 deletions.
21 changes: 9 additions & 12 deletions tests/qapi-schema/doc-good.txt
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ Values
~~~~~~

"one" (**If: **"IFONE")
The _one_ {and only}
The _one_ {and only}, description on the same line

"two"
Not documented
Expand Down Expand Up @@ -76,7 +76,7 @@ Members
~~~~~~~

"base1": "Enum"
the first member
description starts on a new line, minimally indented


If
Expand All @@ -90,7 +90,9 @@ If

A paragraph

Another paragraph (but no "var": line)
Another paragraph

"var1" is undocumented


Members
Expand Down Expand Up @@ -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
Expand Down Expand Up @@ -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
Expand Down Expand Up @@ -203,12 +206,6 @@ Returns
"Object"


TODO
~~~~

frobnicate


Notes
~~~~~

Expand Down
10 changes: 6 additions & 4 deletions tests/qapi-schema/meson.build
Original file line number Diff line number Diff line change
Expand Up @@ -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'])
Expand Down

0 comments on commit 976474f

Please sign in to comment.