build: Drop the install-tests option
authorMatthias Clasen <mclasen@redhat.com>
Tue, 20 Dec 2022 04:11:20 +0000 (23:11 -0500)
committerMatthias Clasen <mclasen@redhat.com>
Tue, 20 Dec 2022 04:12:47 +0000 (23:12 -0500)
We no longer use installed tests in CI,
so lets reap the benefit and simplify
our build setup for tests.

13 files changed:
docs/reference/gtk/building.md
meson.build
meson_options.txt
testsuite/a11y/meson.build
testsuite/css/change/meson.build
testsuite/css/meson.build
testsuite/css/nodes/meson.build
testsuite/css/parser/meson.build
testsuite/css/style/meson.build
testsuite/gdk/meson.build
testsuite/gsk/meson.build
testsuite/gtk/meson.build
testsuite/tools/meson.build

index 31073d1d049a944aba33dcd06f2b3f5964d32c6a..2be7da9916d6729c67fa6b9f74d963a2be8585eb 100644 (file)
@@ -294,7 +294,7 @@ is mainly useful for shortening turnaround times on developer
 systems. Installed builds of GTK should always have introspection
 support.
 
-### `build-tests`, `install-tests`, `demos`
+### `build-tests`, `demos`
 
 By default, GTK will build quite a few tests and demos.
 While these are useful on a developer system, they are not
index 80304b0edd34fca9ff7cc596615de61c33fb5d7f..b7419c62a874fd4126f98e5337c40127640a5620 100644 (file)
@@ -885,7 +885,6 @@ summary('Introspection', build_gir, section: 'Build')
 summary('Documentation', get_option('gtk_doc'), section: 'Build')
 summary('Man pages', get_option('man-pages'), section: 'Build')
 summary('Tests', get_option('build-tests'), section: 'Build')
-summary('Install tests', get_option('install-tests'), section: 'Build')
 summary('Demos', get_option('demos'), section: 'Build')
 summary('Examples', get_option('build-examples'), section: 'Build')
 
index c1df7478eb71c8ddf8032a8e5646a19f0a69183c..18f6c134c119a4278d97139f1888898a1c84d502 100644 (file)
@@ -123,8 +123,3 @@ option('build-tests',
        type: 'boolean',
        value: true,
        description : 'Build tests')
-
-option('install-tests',
-       type: 'boolean',
-       value: false,
-       description : 'Install tests')
index 4547b21ba269d734bb1449fad81ce7316bbfab49..c43465efb26b5f33b3308c7888c61367bdebcc40 100644 (file)
@@ -1,6 +1,3 @@
-testexecdir = join_paths(installed_test_bindir, 'gtk')
-testdatadir = join_paths(installed_test_datadir, 'gtk')
-
 # Available keys for each test:
 #
 #  - 'name': the test name; used for the test and to determine the base
@@ -64,12 +61,11 @@ foreach t : tests
   test_extra_suites = t.get('suites', [])
   test_timeout = 60
 
-  test_exe = executable(test_name, test_srcs,
+  test_exe = executable(test_name,
+    sources: test_srcs,
     c_args: test_cargs + test_extra_cargs,
     link_args: test_extra_ldflags,
     dependencies: libgtk_dep,
-    install: get_option('install-tests'),
-    install_dir: testexecdir,
   )
 
   if test_extra_suites.contains('slow')
index 9409f483c1799f45bb49ee179752cd2a1ca5b63f..a75b530731ddb758312979e00ae93583c9726c0c 100644 (file)
@@ -7,17 +7,10 @@ changetest_env.set('GIO_USE_VFS', 'local')
 changetest_env.set('GSETTINGS_BACKEND', 'memory')
 changetest_env.set('G_ENABLE_DIAGNOSTIC', '0')
 
-testexecdir = join_paths(installed_test_bindir, 'css', 'change')
-testdatadir = join_paths(installed_test_datadir, 'css')
-
-test_change = executable(
-  'test-css-change',
-  'test-css-change.c',
-  '../../testutils.c',
+test_change = executable('test-css-change',
+  sources: ['test-css-change.c', '../../testutils.c'],
   c_args: common_cflags,
   dependencies: libgtk_dep,
-  install: get_option('install-tests'),
-  install_dir: testexecdir,
 )
 
 test('change', test_change,
@@ -26,22 +19,3 @@ test('change', test_change,
   env: changetest_env,
   suite: 'css',
 )
-
-test_data = [
-  'test1.css', 'test1.ui', 'test1.nodes',
-  'test2.css', 'test2.ui', 'test2.nodes',
-  'test3.css', 'test3.ui', 'test3.nodes',
-  'test4.css', 'test4.ui', 'test4.nodes',
-]
-
-if get_option('install-tests')
-  conf = configuration_data()
-  conf.set('libexecdir', gtk_libexecdir)
-  configure_file(input: 'change.test.in',
-                 output: 'change.test',
-                 configuration: conf,
-                 install_dir: testdatadir)
-
-  install_data(test_data, install_dir: testexecdir)
-
-endif
index 20c031410dc645cc8f3b765cad8dccdecb9ff040..24084d6b05e26d8bde4996a5b6470baab08e0408 100644 (file)
@@ -13,14 +13,10 @@ subdir('nodes')
 subdir('style')
 subdir('change')
 
-testexecdir = join_paths(installed_test_bindir, 'css')
-testdatadir = join_paths(installed_test_datadir, 'css')
-
-test_api = executable('api', 'api.c',
+test_api = executable('api',
+  sources: ['api.c'],
   c_args: common_cflags,
   dependencies: libgtk_dep,
-  install: get_option('install-tests'),
-  install_dir: testexecdir,
 )
 
 test('api', test_api,
@@ -30,12 +26,11 @@ test('api', test_api,
   suite: 'css',
 )
 
-test_data = executable('data', 'data.c',
+test_data = executable('data',
+  sources: ['data.c'],
   c_args: common_cflags,
   include_directories: [confinc, ],
   dependencies: libgtk_static_dep,
-  install: get_option('install-tests'),
-  install_dir: testexecdir,
 )
 
 test('data', test_data,
@@ -45,11 +40,10 @@ test('data', test_data,
   suite: 'css',
 )
 
-transition = executable('transition', 'transition.c',
+transition = executable('transition',
+  sources: ['transition.c'],
   c_args: common_cflags,
   dependencies: libgtk_static_dep,
-  install: get_option('install-tests'),
-  install_dir: testexecdir,
 )
 
 test('transition', transition,
@@ -59,24 +53,6 @@ test('transition', transition,
      suite: 'css'
 )
 
-if get_option('install-tests')
-  conf = configuration_data()
-  conf.set('libexecdir', gtk_libexecdir)
-  configure_file(input: 'api.test.in',
-    output: 'api.test',
-    configuration: conf,
-    install_dir: testdatadir,
-  )
-
-  conf = configuration_data()
-  conf.set('libexecdir', gtk_libexecdir)
-  configure_file(input: 'data.test.in',
-    output: 'data.test',
-    configuration: conf,
-    install_dir: testdatadir,
-  )
-endif
-
 if false and get_option ('profiler')
 
   adwaita_env = csstest_env
index eb619cf29ce0c9c6775a61cc5d1ac2a6e4afcf53..576b4d8456a947ef98668c7f4d4eb7e82c4b7aef 100644 (file)
@@ -7,81 +7,15 @@ nodetest_env.set('GIO_USE_VFS', 'local')
 nodetest_env.set('GSETTINGS_BACKEND', 'memory')
 nodetest_env.set('G_ENABLE_DIAGNOSTIC', '0')
 
-testexecdir = join_paths(installed_test_bindir, 'css', 'nodes')
-testdatadir = join_paths(installed_test_datadir, 'css')
+test_nodes = executable('test-css-nodes',
+  sources: ['test-css-nodes.c', '../../testutils.c'],
+  c_args: common_cflags,
+  dependencies: libgtk_dep,
+)
 
-test_nodes = executable('test-css-nodes', 'test-css-nodes.c', '../../testutils.c',
-                        c_args: common_cflags,
-                        install: get_option('install-tests'),
-                        install_dir: testexecdir,
-                        dependencies: libgtk_dep)
 test('nodes', test_nodes,
   args: [ '--tap', '-k' ],
   protocol: 'tap',
   env: nodetest_env,
   suite: 'css',
 )
-
-test_data = [
-  'box.ltr.nodes',
-  'box.ltr.ui',
-  'box.rtl.nodes',
-  'box.rtl.ui',
-  'buttons.nodes',
-  'buttons.ui',
-  'checkbutton.ltr.nodes',
-  'checkbutton.ltr.ui',
-  'checkbutton.rtl.nodes',
-  'checkbutton.rtl.ui',
-  'combobox.nodes',
-  'combobox.ui',
-  'entries.nodes',
-  'entries.ui',
-  'expander.ltr.nodes',
-  'expander.ltr.ui',
-  'expander.rtl.nodes',
-  'expander.rtl.ui',
-  'levelbar.ltr.nodes',
-  'levelbar.ltr.ui',
-  'levelbar.rtl.nodes',
-  'levelbar.rtl.ui',
-  'notebook-arrows2.nodes',
-  'notebook-arrows2.ui',
-  'notebook-arrows.nodes',
-  'notebook-arrows.ui',
-  'notebook.bottom.ltr.nodes',
-  'notebook.bottom.ltr.ui',
-  'notebook.bottom.rtl.nodes',
-  'notebook.bottom.rtl.ui',
-  'notebook.left.ltr.nodes',
-  'notebook.left.ltr.ui',
-  'notebook.left.rtl.nodes',
-  'notebook.left.rtl.ui',
-  'notebook.right.ltr.nodes',
-  'notebook.right.ltr.ui',
-  'notebook.right.rtl.nodes',
-  'notebook.right.rtl.ui',
-  'notebook.top.ltr.nodes',
-  'notebook.top.ltr.ui',
-  'notebook.top.rtl.nodes',
-  'notebook.top.rtl.ui',
-  'paned.ltr.nodes',
-  'paned.ltr.ui',
-  'paned.rtl.nodes',
-  'paned.rtl.ui',
-  'progressbar.nodes',
-  'progressbar.ui',
-  'scale.nodes',
-  'scale.ui',
-]
-
-if get_option('install-tests')
-  conf = configuration_data()
-  conf.set('libexecdir', gtk_libexecdir)
-  configure_file(input: 'nodes.test.in',
-                 output: 'nodes.test',
-                 configuration: conf,
-                 install_dir: testdatadir)
-
-  install_data(test_data, install_dir: testexecdir)
-endif
index d0fdd0d9a98037377fcba880877dd7ec5dbf2bd7..b419c29865e534190d44aa6d8f7c3eb9a460d7d2 100644 (file)
@@ -1,11 +1,8 @@
-testexecdir = join_paths(installed_test_bindir, 'css', 'parser')
-testdatadir = join_paths(installed_test_datadir, 'css')
-
-test_parser = executable('test-css-parser', 'test-css-parser.c', '../../testutils.c',
-                         c_args: common_cflags,
-                         install: get_option('install-tests'),
-                         install_dir: testexecdir,
-                         dependencies: libgtk_dep)
+test_parser = executable('test-css-parser',
+  sources: ['test-css-parser.c', '../../testutils.c'],
+  c_args: common_cflags,
+  dependencies: libgtk_dep,
+)
 
 test_data = [
   'animation-crash-3.12.css',
@@ -537,15 +534,3 @@ foreach testname : test_data
          suite: 'css')
   endif
 endforeach
-
-if get_option('install-tests')
-  conf = configuration_data()
-  conf.set('libexecdir', gtk_libexecdir)
-  configure_file(input: 'parser.test.in',
-                 output: 'parser.test',
-                 configuration: conf,
-                 install_dir: testdatadir)
-
-  install_data(test_data, install_dir: testexecdir)
-
-endif
index 32280494990e4317f87dde524ca688a078542d47..ef703d79949342803aa72012f19a185e5e32d9db 100644 (file)
@@ -7,73 +7,21 @@ styletest_env.set('GIO_USE_VFS', 'local')
 styletest_env.set('GSETTINGS_BACKEND', 'memory')
 styletest_env.set('G_ENABLE_DIAGNOSTIC', '0')
 
-testexecdir = join_paths(installed_test_bindir, 'css', 'style')
-testdatadir = join_paths(installed_test_datadir, 'css')
-
 cssresources = gnome.compile_resources(
   'cssresources',
   'test-css-style.gresource.xml',
   source_dir: '.',
 )
 
-test_style = executable(
-  'test-css-style',
-  'test-css-style.c',
-  '../../testutils.c',
-  cssresources,
+test_style = executable('test-css-style',
+  sources: ['test-css-style.c', '../../testutils.c', cssresources],
   c_args: common_cflags,
   dependencies: libgtk_dep,
-  install: get_option('install-tests'),
-  install_dir: testexecdir,
 )
+
 test('style', test_style,
   args: [ '--tap', '-k' ],
   protocol: 'tap',
   env: styletest_env,
   suite: 'css',
 )
-
-test_data = [
-  'adjacent-states.css',
-  'adjacent-states.nodes',
-  'adjacent-states.ui',
-  'bloomfilter-not.css',
-  'bloomfilter-not.nodes',
-  'bloomfilter-not.ui',
-  'colornames.css',
-  'colornames.nodes',
-  'colornames.ui',
-  'currentcolor.css',
-  'currentcolor.nodes',
-  'currentcolor.ui',
-  'font.css',
-  'font.nodes',
-  'font.ui',
-  'gradient.css',
-  'gradient.nodes',
-  'gradient.ui',
-  'gtk.css',
-  'inherit.css',
-  'inherit.nodes',
-  'inherit.ui',
-  'label.css',
-  'label.nodes',
-  'label.ui',
-  'misc.css',
-  'misc.nodes',
-  'misc.ui',
-  'nth-child.css',
-  'nth-child.nodes',
-  'nth-child.ui',
-]
-
-if get_option('install-tests')
-  conf = configuration_data()
-  conf.set('libexecdir', gtk_libexecdir)
-  configure_file(input: 'style.test.in',
-                 output: 'style.test',
-                 configuration: conf,
-                 install_dir: testdatadir)
-
-  install_data(test_data, install_dir: testexecdir)
-endif
index 3236ce7d5dbaa9d22eaec9d48be2789a192891ad..f3d3a1ba4a6aeffafc339c41992268cc5b64a327 100644 (file)
@@ -1,13 +1,10 @@
-testexecdir = join_paths(installed_test_bindir, 'gdk')
-testdatadir = join_paths(installed_test_datadir, 'gdk')
-
 clipboard_client = executable('clipboard-client',
-                              sources: ['clipboard-client.c'],
-                              include_directories: [confinc],
-                              c_args: common_cflags,
-                              dependencies: [ libgtk_dep ],
-                              install: get_option('install-tests'),
-                              install_dir: testexecdir)
+  sources: ['clipboard-client.c'],
+  include_directories: [confinc],
+  c_args: common_cflags,
+  dependencies: [ libgtk_dep ],
+  install: false,
+)
 
 tests = [
   { 'name': 'array' },
@@ -35,8 +32,7 @@ foreach t : tests
     sources: '@0@.c'.format(test_name),
     c_args: common_cflags,
     dependencies: libgtk_dep,
-    install: get_option('install-tests'),
-    install_dir: testexecdir,
+    install: false,
   )
 
   suites = ['gdk'] + t.get('suites', [])
@@ -63,8 +59,7 @@ foreach t : internal_tests
   test_exe = executable(t, '@0@.c'.format(t),
     c_args: common_cflags,
     dependencies: libgtk_static_dep,
-    install: get_option('install-tests'),
-    install_dir: testexecdir,
+    install: false,
   )
 
   test(t, test_exe,
@@ -78,28 +73,3 @@ foreach t : internal_tests
     suite: 'gdk',
   )
 endforeach
-
-
-if get_option('install-tests')
-  foreach t : tests
-    test_name = t.get('name')
-    suites = t.get('suites', [])
-
-    if suites.contains('flaky') or suites.contains('failing')
-      continue
-    endif
-
-    test_cdata = configuration_data()
-    test_cdata.set('testexecdir', testexecdir)
-    test_cdata.set('test', test_name)
-    configure_file(input: 'gdk.test.in',
-      output: '@0@.test'.format(test_name),
-      configuration: test_cdata,
-      install: true,
-      install_dir: testdatadir,
-    )
-  endforeach
-
-  install_subdir('clipboard-data', install_dir: testexecdir)
-  install_subdir('image-data', install_dir: testexecdir)
-endif
index 67514ebf8c0b10b052e932302f57a99179abe306..1897562465996bb97b56e078fbfb26875c595db7 100644 (file)
@@ -1,22 +1,12 @@
-testexecdir = join_paths(installed_test_bindir, 'gsk')
-testdatadir = join_paths(installed_test_datadir, 'gsk')
-
-compare_render = executable(
-  'compare-render',
+compare_render = executable('compare-render',
   ['compare-render.c', '../reftests/reftest-compare.c'],
   dependencies: libgtk_dep,
   c_args: common_cflags,
-  install: get_option('install-tests'),
-  install_dir: testexecdir
 )
 
-node_parser = executable(
-  'node-parser',
-  ['node-parser.c'],
+node_parser = executable('node-parser', 'node-parser.c',
   dependencies: libgtk_dep,
   c_args: common_cflags,
-  install: get_option('install-tests'),
-  install_dir: testexecdir
 )
 
 compare_render_tests = [
@@ -266,11 +256,9 @@ foreach t : tests
   test_extra_ldflags = t.get(3, [])
 
   test_exe = executable(test_name, test_srcs,
+    dependencies : libgtk_dep,
     c_args : test_cargs + test_extra_cargs + common_cflags,
     link_args : test_extra_ldflags,
-    dependencies : libgtk_dep,
-    install: get_option('install-tests'),
-    install_dir: testexecdir,
   )
 
   test(test_name, test_exe,
@@ -299,12 +287,11 @@ foreach t : internal_tests
   test_extra_cargs = t.get(2, [])
   test_extra_ldflags = t.get(3, [])
 
-  test_exe = executable(test_name, test_srcs,
+  test_exe = executable(test_name,
+    sources: test_srcs,
+    dependencies : libgtk_static_dep,
     c_args : test_cargs + test_extra_cargs + common_cflags,
     link_args : test_extra_ldflags,
-    dependencies : libgtk_static_dep,
-    install: get_option('install-tests'),
-    install_dir: testexecdir,
   )
 
   test(test_name, test_exe,
index 3c107efd2a76bb4cca15e2c22e1f9fcd5bdef7b3..73dd1369c399b69c1d660c2bf64e59d917c69ec6 100644 (file)
@@ -1,6 +1,3 @@
-testexecdir = join_paths(installed_test_bindir, 'gtk')
-testdatadir = join_paths(installed_test_datadir, 'gtk')
-
 gtk_tests_export_dynamic_ldflag = []
 
 if cc.get_id() != 'msvc'
@@ -172,8 +169,6 @@ foreach t : tests
     c_args: test_cargs + test_extra_cargs,
     link_args: test_extra_ldflags,
     dependencies: libgtk_dep,
-    install: get_option('install-tests'),
-    install_dir: testexecdir,
   )
 
   if test_extra_suites.contains('slow')
@@ -202,8 +197,6 @@ foreach t : internal_tests
     c_args: test_cargs + test_extra_cargs,
     link_args: test_extra_ldflags,
     dependencies: libgtk_static_dep,
-    install: get_option('install-tests'),
-    install_dir: testexecdir,
   )
 
   if test_extra_suites.contains('slow')
@@ -222,10 +215,8 @@ endforeach
 # FIXME: if objc autotestkeywords_CPPFLAGS += -DHAVE_OBJC=1 -x objective-c++
 if add_languages('cpp', required: false, native: false)
   test_exe = executable('autotestkeywords',
-    sources: 'autotestkeywords.cc',
+    sources: ['autotestkeywords.cc'],
     dependencies: libgtk_dep,
-    install: get_option('install-tests'),
-    install_dir: testexecdir,
   )
   test('c++ keywords', test_exe,
     args: [ '--tap', '-k' ],
@@ -233,16 +224,6 @@ if add_languages('cpp', required: false, native: false)
     env: test_env,
     suite: 'gtk',
   )
-  if get_option('install-tests')
-    conf = configuration_data()
-    conf.set('testexecdir', testexecdir)
-    conf.set('test', 'autotestkeywords')
-    configure_file(input: 'gtk.test.in',
-      output: 'autotestkeywords.test',
-      configuration: conf,
-      install_dir: testdatadir,
-    )
-  endif
 endif
 
 
@@ -268,13 +249,10 @@ focus_chain_tests = [
   #[ 'widget-factory3', 'tab-backward' ],
 ]
 
-focus_chain = executable(
-  'test-focus-chain',
+focus_chain = executable('test-focus-chain',
   sources: ['test-focus-chain.c', '../testutils.c'],
   dependencies: libgtk_dep,
   c_args: common_cflags,
-  install: get_option('install-tests'),
-  install_dir: testexecdir
 )
 
 foreach test : focus_chain_tests
@@ -296,30 +274,6 @@ test('potfiles', make_pot,
      workdir: meson.project_source_root(),
      suite: ['gtk', 'translations' ])
 
-if get_option('install-tests')
-  foreach t : tests
-    test_name = t.get('name')
-    suites = t.get('suites', [])
-
-    if suites.contains('flaky') or suites.contains('failing')
-      continue
-    endif
-
-    conf = configuration_data()
-    conf.set('testexecdir', testexecdir)
-    conf.set('test', test_name)
-    configure_file(input: 'gtk.test.in',
-      output: '@0@.test'.format(test_name),
-      configuration: conf,
-      install_dir: testdatadir,
-    )
-  endforeach
-
-  install_subdir('icons', install_dir: testexecdir)
-  install_subdir('icons2', install_dir: testexecdir)
-  install_subdir('ui', install_dir: testexecdir)
-endif
-
 if false and get_option ('profiler')
 
   performance_env = test_env
index 582bce6fb808a69f39fb76c9ad6f2523ac0c26b5..986c2d6dcc94b35b4d39bf33db1a23dac9b8528b 100644 (file)
@@ -1,23 +1,12 @@
-testexecdir = join_paths(installed_test_bindir, 'tools')
-testdatadir = join_paths(installed_test_datadir, 'tools')
-
 bash = find_program('bash', required : false)
 if bash.found()
   test_env = environment()
 
   foreach t : ['simplify', 'simplify-3to4', 'validate', 'settings']
-    if get_option('install-tests')
-      configure_file(output: t,
-        input: '@0@.in'.format(t),
-        copy: true,
-        install_dir: testexecdir,
-      )
-    else
-      configure_file(output: t,
-        input: '@0@.in'.format(t),
-        copy: true,
-      )
-    endif
+    configure_file(output: t,
+      input: '@0@.in'.format(t),
+      copy: true,
+    )
     test(t, bash,
       args: t,
       workdir: meson.current_build_dir(),
@@ -34,20 +23,3 @@ if bash.found()
     )
   endforeach
 endif
-
-if get_option('install-tests')
-  foreach t : ['simplify', 'settings']
-    test_conf = configuration_data()
-    test_conf.set('testexecdir', testexecdir)
-    test_conf.set('test', t)
-    configure_file(output: '@0@.test'.format(t),
-      input: 'tools.test.in',
-      configuration: test_conf,
-      install_dir: testdatadir,
-    )
-  endforeach
-
-  install_subdir('simplify-data', install_dir: testexecdir)
-  install_subdir('simplify-data-3to4', install_dir: testexecdir)
-  install_subdir('validate-data', install_dir: testexecdir)
-endif