Don't skip dbus-codegen tests on Win32

And coincidentally on Darwin either.
This commit is contained in:
John Ralls 2024-01-28 14:31:37 -08:00
parent 8afefe963d
commit fbdc9a2d03
2 changed files with 46 additions and 68 deletions

View File

@ -24,6 +24,7 @@
import argparse
import os
import sys
from contextlib import contextmanager
from . import config
from . import dbustypes
@ -63,6 +64,14 @@ def find_prop(iface, prop):
return None
@contextmanager
def file_or_stdout(filename):
if filename is None or filename == "stdout":
yield sys.stdout
else:
with open(filename, "w") as outfile:
yield outfile
def apply_annotation(iface_list, iface, method, signal, prop, arg, key, value):
iface_obj = None
for i in iface_list:
@ -446,7 +455,7 @@ def codegen_main():
rst_gen.generate(rst, args.output_directory)
if args.header:
with open(h_file, "w") as outfile:
with file_or_stdout(h_file) as outfile:
gen = codegen.HeaderCodeGenerator(
all_ifaces,
args.c_namespace,
@ -463,7 +472,7 @@ def codegen_main():
gen.generate()
if args.body:
with open(c_file, "w") as outfile:
with file_or_stdout(c_file) as outfile:
gen = codegen.CodeGenerator(
all_ifaces,
args.c_namespace,
@ -478,7 +487,7 @@ def codegen_main():
gen.generate()
if args.interface_info_header:
with open(h_file, "w") as outfile:
with file_or_stdout(h_file) as outfile:
gen = codegen.InterfaceInfoHeaderCodeGenerator(
all_ifaces,
args.c_namespace,
@ -493,7 +502,7 @@ def codegen_main():
gen.generate()
if args.interface_info_body:
with open(c_file, "w") as outfile:
with file_or_stdout(c_file):
gen = codegen.InterfaceInfoBodyCodeGenerator(
all_ifaces,
args.c_namespace,

View File

@ -41,10 +41,6 @@ import taptestrunner
Result = collections.namedtuple("Result", ("info", "out", "err", "subs"))
def on_win32():
return sys.platform.find("win") != -1
class TestCodegen(unittest.TestCase):
"""Integration test for running gdbus-codegen.
@ -355,10 +351,9 @@ class TestCodegen(unittest.TestCase):
with self.assertRaises(subprocess.CalledProcessError):
self.runCodegen()
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_empty_interface_header(self):
"""Test generating a header with an empty interface file."""
result = self.runCodegenWithInterface("", "--output", "/dev/stdout", "--header")
result = self.runCodegenWithInterface("", "--output", "stdout", "--header")
self.assertEqual("", result.err)
self.assertEqual(
"""{standard_top_comment}
@ -379,10 +374,9 @@ G_END_DECLS
result.out.strip(),
)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_empty_interface_body(self):
"""Test generating a body with an empty interface file."""
result = self.runCodegenWithInterface("", "--output", "/dev/stdout", "--body")
result = self.runCodegenWithInterface("", "--output", "stdout", "--body")
self.assertEqual("", result.err)
self.assertEqual(
"""{standard_top_comment}
@ -401,7 +395,6 @@ G_END_DECLS
result.out.strip(),
)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_reproducible(self):
"""Test builds are reproducible regardless of file ordering."""
xml_contents1 = """
@ -444,7 +437,7 @@ G_END_DECLS
xml_file1.name,
xml_file2.name,
"--output",
"/dev/stdout",
"stdout",
header_or_body,
)
self.assertEqual("", result1.err)
@ -453,7 +446,7 @@ G_END_DECLS
xml_file2.name,
xml_file1.name,
"--output",
"/dev/stdout",
"stdout",
header_or_body,
)
self.assertEqual("", result2.err)
@ -653,35 +646,32 @@ G_END_DECLS
rst,
)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_glib_min_required_invalid(self):
"""Test running with an invalid --glib-min-required."""
with self.assertRaises(subprocess.CalledProcessError):
self.runCodegenWithInterface(
"",
"--output",
"/dev/stdout",
"stdout",
"--body",
"--glib-min-required",
"hello mum",
)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_glib_min_required_too_low(self):
"""Test running with a --glib-min-required which is too low (and hence
probably a typo)."""
with self.assertRaises(subprocess.CalledProcessError):
self.runCodegenWithInterface(
"", "--output", "/dev/stdout", "--body", "--glib-min-required", "2.6"
"", "--output", "stdout", "--body", "--glib-min-required", "2.6"
)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_glib_min_required_major_only(self):
"""Test running with a --glib-min-required which contains only a major version."""
result = self.runCodegenWithInterface(
"",
"--output",
"/dev/stdout",
"stdout",
"--header",
"--glib-min-required",
"3",
@ -691,43 +681,38 @@ G_END_DECLS
self.assertEqual("", result.err)
self.assertNotEqual("", result.out.strip())
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_glib_min_required_with_micro(self):
"""Test running with a --glib-min-required which contains a micro version."""
result = self.runCodegenWithInterface(
"", "--output", "/dev/stdout", "--header", "--glib-min-required", "2.46.2"
"", "--output", "stdout", "--header", "--glib-min-required", "2.46.2"
)
self.assertEqual("", result.err)
self.assertNotEqual("", result.out.strip())
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_glib_max_allowed_too_low(self):
"""Test running with a --glib-max-allowed which is too low (and hence
probably a typo)."""
with self.assertRaises(subprocess.CalledProcessError):
self.runCodegenWithInterface(
"", "--output", "/dev/stdout", "--body", "--glib-max-allowed", "2.6"
"", "--output", "stdout", "--body", "--glib-max-allowed", "2.6"
)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_glib_max_allowed_major_only(self):
"""Test running with a --glib-max-allowed which contains only a major version."""
result = self.runCodegenWithInterface(
"", "--output", "/dev/stdout", "--header", "--glib-max-allowed", "3"
"", "--output", "stdout", "--header", "--glib-max-allowed", "3"
)
self.assertEqual("", result.err)
self.assertNotEqual("", result.out.strip())
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_glib_max_allowed_with_micro(self):
"""Test running with a --glib-max-allowed which contains a micro version."""
result = self.runCodegenWithInterface(
"", "--output", "/dev/stdout", "--header", "--glib-max-allowed", "2.46.2"
"", "--output", "stdout", "--header", "--glib-max-allowed", "2.46.2"
)
self.assertEqual("", result.err)
self.assertNotEqual("", result.out.strip())
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_glib_max_allowed_unstable(self):
"""Test running with a --glib-max-allowed which is unstable. It should
be rounded up to the next stable version number, and hence should not
@ -735,7 +720,7 @@ G_END_DECLS
result = self.runCodegenWithInterface(
"",
"--output",
"/dev/stdout",
"stdout",
"--header",
"--glib-max-allowed",
"2.63",
@ -745,7 +730,6 @@ G_END_DECLS
self.assertEqual("", result.err)
self.assertNotEqual("", result.out.strip())
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_glib_max_allowed_less_than_min_required(self):
"""Test running with a --glib-max-allowed which is less than
--glib-min-required."""
@ -753,7 +737,7 @@ G_END_DECLS
self.runCodegenWithInterface(
"",
"--output",
"/dev/stdout",
"stdout",
"--body",
"--glib-max-allowed",
"2.62",
@ -761,7 +745,6 @@ G_END_DECLS
"2.64",
)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_dbus_types(self):
bad_types = [
"{vs}", # Bad dictionary key type
@ -798,7 +781,7 @@ G_END_DECLS
</node>"""
with self.assertRaises(subprocess.CalledProcessError):
self.runCodegenWithInterface(
interface_xml, "--output", "/dev/stdout", "--body"
interface_xml, "--output", "stdout", "--body"
)
good_types = [
"si{s{b(ybnqiuxtdh)}}{yv}{nv}{dv}",
@ -815,11 +798,10 @@ G_END_DECLS
</interface>
</node>"""
result = self.runCodegenWithInterface(
interface_xml, "--output", "/dev/stdout", "--body"
interface_xml, "--output", "stdout", "--body"
)
self.assertEqual("", result.err)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_unix_fd_types_and_annotations(self):
"""Test an interface with `h` arguments, no annotation, and GLib < 2.64.
@ -847,7 +829,7 @@ G_END_DECLS
# Try without specifying --glib-min-required.
result = self.runCodegenWithInterface(
interface_xml, "--output", "/dev/stdout", "--header"
interface_xml, "--output", "stdout", "--header"
)
self.assertEqual("", result.err)
self.assertEqual(result.out.strip().count("GUnixFDList"), 6)
@ -856,7 +838,7 @@ G_END_DECLS
result = self.runCodegenWithInterface(
interface_xml,
"--output",
"/dev/stdout",
"stdout",
"--header",
"--glib-min-required",
"2.32",
@ -870,7 +852,7 @@ G_END_DECLS
result = self.runCodegenWithInterface(
interface_xml,
"--output",
"/dev/stdout",
"stdout",
"--header",
"--glib-min-required",
"2.64",
@ -878,7 +860,6 @@ G_END_DECLS
self.assertEqual("", result.err)
self.assertEqual(result.out.strip().count("GUnixFDList"), 18)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_call_flags_and_timeout_method_args(self):
"""Test that generated method call functions have @call_flags and
@timeout_msec args if and only if GLib >= 2.64.
@ -892,7 +873,7 @@ G_END_DECLS
# Try without specifying --glib-min-required.
result = self.runCodegenWithInterface(
interface_xml, "--output", "/dev/stdout", "--header"
interface_xml, "--output", "stdout", "--header"
)
self.assertEqual("", result.err)
self.assertEqual(result.out.strip().count("GDBusCallFlags call_flags,"), 0)
@ -902,7 +883,7 @@ G_END_DECLS
result = self.runCodegenWithInterface(
interface_xml,
"--output",
"/dev/stdout",
"stdout",
"--header",
"--glib-min-required",
"2.32",
@ -916,7 +897,7 @@ G_END_DECLS
result = self.runCodegenWithInterface(
interface_xml,
"--output",
"/dev/stdout",
"stdout",
"--header",
"--glib-min-required",
"2.64",
@ -925,7 +906,6 @@ G_END_DECLS
self.assertEqual(result.out.strip().count("GDBusCallFlags call_flags,"), 2)
self.assertEqual(result.out.strip().count("gint timeout_msec,"), 2)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_generate_signal_id_simple_signal(self):
"""Test that signals IDs are used to emit signals"""
interface_xml = """
@ -939,7 +919,7 @@ G_END_DECLS
</node>"""
result = self.runCodegenWithInterface(
interface_xml, "--output", "/dev/stdout", "--body"
interface_xml, "--output", "stdout", "--body"
)
stripped_out = result.out.strip()
self.assertFalse(result.err)
@ -957,7 +937,6 @@ G_END_DECLS
1,
)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_generate_signal_id_multiple_signals_types(self):
"""Test that signals IDs are used to emit signals for all types"""
@ -978,7 +957,7 @@ G_END_DECLS
</node>"""
result = self.runCodegenWithInterface(
interface_xml, "--output", "/dev/stdout", "--body"
interface_xml, "--output", "stdout", "--body"
)
stripped_out = result.out.strip()
self.assertFalse(result.err)
@ -997,7 +976,6 @@ G_END_DECLS
1,
)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_generate_signal_id_multiple_signal_args_types(self):
"""Test that signals IDs are used to emit signals for all types"""
@ -1016,7 +994,7 @@ G_END_DECLS
</node>"""
result = self.runCodegenWithInterface(
interface_xml, "--output", "/dev/stdout", "--body"
interface_xml, "--output", "stdout", "--body"
)
stripped_out = result.out.strip()
self.assertFalse(result.err)
@ -1036,7 +1014,6 @@ G_END_DECLS
1,
)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_generate_signals_marshaller_simple_signal(self):
"""Test that signals marshaller is generated for simple signal"""
interface_xml = """
@ -1050,7 +1027,7 @@ G_END_DECLS
</node>"""
result = self.runCodegenWithInterface(
interface_xml, "--output", "/dev/stdout", "--body"
interface_xml, "--output", "stdout", "--body"
)
stripped_out = result.out.strip()
self.assertFalse(result.err)
@ -1066,7 +1043,6 @@ G_END_DECLS
self.assertIs(stripped_out.count(f"{func_name} ("), 1)
self.assertIs(stripped_out.count("g_cclosure_marshal_VOID__VOID (closure"), 2)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_generate_signals_marshaller_single_typed_args(self):
"""Test that signals marshaller is generated for each known type"""
for t, props in self.ARGUMENTS_TYPES.items():
@ -1082,7 +1058,7 @@ G_END_DECLS
</node>"""
result = self.runCodegenWithInterface(
interface_xml, "--output", "/dev/stdout", "--body"
interface_xml, "--output", "stdout", "--body"
)
stripped_out = result.out.strip()
self.assertFalse(result.err)
@ -1113,7 +1089,6 @@ G_END_DECLS
1,
)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_generate_signals_marshallers_multiple_args(self):
"""Test that signals marshallers are generated"""
generated_args = [
@ -1132,7 +1107,7 @@ G_END_DECLS
</node>"""
result = self.runCodegenWithInterface(
interface_xml, "--output", "/dev/stdout", "--body"
interface_xml, "--output", "stdout", "--body"
)
stripped_out = result.out.strip()
self.assertFalse(result.err)
@ -1157,7 +1132,6 @@ G_END_DECLS
)
index += 1
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_generate_methods_marshaller_simple_method(self):
"""Test that methods marshaller is generated for simple method"""
interface_xml = """
@ -1171,7 +1145,7 @@ G_END_DECLS
</node>"""
result = self.runCodegenWithInterface(
interface_xml, "--output", "/dev/stdout", "--body"
interface_xml, "--output", "stdout", "--body"
)
stripped_out = result.out.strip()
self.assertFalse(result.err)
@ -1202,7 +1176,6 @@ G_END_DECLS
stripped_out.count("_g_dbus_codegen_marshal_BOOLEAN__OBJECT (closure"), 2
)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_generate_methods_marshaller_single_typed_in_args(self):
"""Test that methods marshallers are generated for each known type"""
for t, props in self.ARGUMENTS_TYPES.items():
@ -1217,7 +1190,7 @@ G_END_DECLS
</node>"""
result = self.runCodegenWithInterface(
interface_xml, "--output", "/dev/stdout", "--body"
interface_xml, "--output", "stdout", "--body"
)
stripped_out = result.out.strip()
self.assertFalse(result.err)
@ -1241,7 +1214,6 @@ G_END_DECLS
1,
)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_generate_methods_marshaller_single_typed_out_args(self):
"""Test that methods marshallers are generated for each known type"""
for t, props in self.ARGUMENTS_TYPES.items():
@ -1256,7 +1228,7 @@ G_END_DECLS
</node>"""
result = self.runCodegenWithInterface(
interface_xml, "--output", "/dev/stdout", "--body"
interface_xml, "--output", "stdout", "--body"
)
stripped_out = result.out.strip()
self.assertFalse(result.err)
@ -1275,7 +1247,6 @@ G_END_DECLS
)
self.assertIs(stripped_out.count("(param_values + 2)"), 0)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_generate_methods_marshallers_multiple_in_args(self):
"""Test that methods marshallers are generated"""
generated_args = [
@ -1301,7 +1272,7 @@ G_END_DECLS
</node>"""
result = self.runCodegenWithInterface(
interface_xml, "--output", "/dev/stdout", "--body"
interface_xml, "--output", "stdout", "--body"
)
stripped_out = result.out.strip()
self.assertFalse(result.err)
@ -1344,7 +1315,6 @@ G_END_DECLS
self.assertIs(stripped_out.count(f"{func_name},"), 1)
self.assertIs(stripped_out.count(f"{func_name} ("), 1)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_generate_methods_marshallers_multiple_out_args(self):
"""Test that methods marshallers are generated"""
generated_args = [
@ -1362,7 +1332,7 @@ G_END_DECLS
</node>"""
result = self.runCodegenWithInterface(
interface_xml, "--output", "/dev/stdout", "--body"
interface_xml, "--output", "stdout", "--body"
)
stripped_out = result.out.strip()
self.assertFalse(result.err)
@ -1392,7 +1362,6 @@ G_END_DECLS
1,
)
@unittest.skipIf(on_win32(), "requires /dev/stdout")
def test_generate_methods_marshallers_with_unix_fds(self):
"""Test an interface with `h` arguments"""
interface_xml = """
@ -1407,7 +1376,7 @@ G_END_DECLS
</node>"""
result = self.runCodegenWithInterface(
interface_xml, "--output", "/dev/stdout", "--body"
interface_xml, "--output", "stdout", "--body"
)
stripped_out = result.out.strip()
self.assertFalse(result.err)