meson: Use consistent style for tests
authorDylan Baker <dylan@pnwbakers.com>
Tue, 12 Dec 2017 00:13:31 +0000 (16:13 -0800)
committerDylan Baker <dylan@pnwbakers.com>
Thu, 11 Jan 2018 23:40:02 +0000 (15:40 -0800)
Don't use intermediate variables, use consistent whitespace.

Acked-by: Eric Engestrom <eric.engestrom@imgtec.com>
Signed-off-by: Dylan Baker <dylan.c.baker@intel.com>
20 files changed:
src/broadcom/qpu/meson.build
src/compiler/glsl/tests/meson.build
src/compiler/nir/meson.build
src/egl/wayland/wayland-egl/meson.build
src/gallium/drivers/llvmpipe/meson.build
src/gallium/drivers/r300/meson.build
src/gallium/state_trackers/xvmc/meson.build
src/gbm/meson.build
src/glx/tests/meson.build
src/intel/compiler/meson.build
src/intel/isl/meson.build
src/intel/vulkan/meson.build
src/mapi/es1api/meson.build
src/mapi/es2api/meson.build
src/mapi/glapi/meson.build
src/mapi/shared-glapi/meson.build
src/mesa/main/tests/meson.build
src/util/meson.build
src/util/tests/hash_table/meson.build
src/util/tests/string_buffer/meson.build

index 8178ddbd53b91103f45fe4693bc3c85b9d882b03..5521a80ceeca9ca39bfa9b96854901566e800558 100644 (file)
@@ -33,7 +33,11 @@ libbroadcom_qpu = static_library(
   build_by_default : false,
 )
 
-test('qpu_disasm',
-  executable('qpu_disasm', 'tests/qpu_disasm.c',
-  link_with: [libbroadcom_qpu, libmesa_util],
-  include_directories: inc_common))
+test(
+  'qpu_disasm',
+  executable(
+    'qpu_disasm', 'tests/qpu_disasm.c',
+    link_with: [libbroadcom_qpu, libmesa_util],
+    include_directories: inc_common
+  )
+)
index 27f34075b4babfbd0d62b04acdbd96d1d48f42b8..146647a850ce0784fba2dbf402da4750cbcd3fd1 100644 (file)
 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 # SOFTWARE.
 
-glsl_blob_test = executable(
+test(
   'blob_test',
-  'blob_test.c',
-  c_args : [c_vis_args, c_msvc_compat_args, no_override_init_args],
-  include_directories : [inc_common, inc_compiler],
-  link_with : [libglsl],
+  executable(
+    'blob_test',
+    'blob_test.c',
+    c_args : [c_vis_args, c_msvc_compat_args, no_override_init_args],
+    include_directories : [inc_common, inc_compiler],
+    link_with : [libglsl],
+  )
 )
 
-glsl_cache_test = executable(
+test(
   'cache_test',
-  'cache_test.c',
-  c_args : [c_vis_args, c_msvc_compat_args, no_override_init_args],
-  include_directories : [inc_common, inc_glsl],
-  link_with : [libglsl],
-  dependencies : [dep_clock, dep_thread],
+  executable(
+    'cache_test',
+    'cache_test.c',
+    c_args : [c_vis_args, c_msvc_compat_args, no_override_init_args],
+    include_directories : [inc_common, inc_glsl],
+    link_with : [libglsl],
+    dependencies : [dep_clock, dep_thread],
+  )
 )
 
-glsl_general_ir_test = executable(
+
+test(
   'general_ir_test',
-  ['array_refcount_test.cpp', 'builtin_variable_test.cpp',
-   'invalidate_locations_test.cpp', 'general_ir_test.cpp',
-   'lower_int64_test.cpp', 'opt_add_neg_to_sub_test.cpp', 'varyings_test.cpp',
-   ir_expression_operation_h],
-  cpp_args : [cpp_vis_args, cpp_msvc_compat_args],
-  include_directories : [inc_common, inc_glsl],
-  link_with : [libglsl, libglsl_standalone, libglsl_util],
-  dependencies : [dep_clock, dep_thread, idep_gtest],
+  executable(
+    'general_ir_test',
+    ['array_refcount_test.cpp', 'builtin_variable_test.cpp',
+     'invalidate_locations_test.cpp', 'general_ir_test.cpp',
+     'lower_int64_test.cpp', 'opt_add_neg_to_sub_test.cpp',
+     'varyings_test.cpp', ir_expression_operation_h],
+    cpp_args : [cpp_vis_args, cpp_msvc_compat_args],
+    include_directories : [inc_common, inc_glsl],
+    link_with : [libglsl, libglsl_standalone, libglsl_util],
+    dependencies : [dep_clock, dep_thread, idep_gtest],
+  )
 )
 
-glsl_uniform_initializer_test = executable(
+test(
   'uniform_initializer_test',
-  ['copy_constant_to_storage_tests.cpp', 'set_uniform_initializer_tests.cpp',
-   'uniform_initializer_utils.cpp', 'uniform_initializer_utils.h',
-   ir_expression_operation_h],
-  cpp_args : [cpp_vis_args, cpp_msvc_compat_args],
-  include_directories : [inc_common, inc_glsl],
-  link_with : [libglsl, libglsl_util],
-  dependencies : [dep_thread, idep_gtest],
+  executable(
+    'uniform_initializer_test',
+    ['copy_constant_to_storage_tests.cpp', 'set_uniform_initializer_tests.cpp',
+     'uniform_initializer_utils.cpp', 'uniform_initializer_utils.h',
+     ir_expression_operation_h],
+    cpp_args : [cpp_vis_args, cpp_msvc_compat_args],
+    include_directories : [inc_common, inc_glsl],
+    link_with : [libglsl, libglsl_util],
+    dependencies : [dep_thread, idep_gtest],
+  )
 )
 
-glsl_sampler_types_test = executable(
+test(
   'sampler_types_test',
-  ['sampler_types_test.cpp', ir_expression_operation_h],
-  cpp_args : [cpp_vis_args, cpp_msvc_compat_args],
-  include_directories : [inc_common, inc_glsl],
-  link_with : [libglsl, libglsl_util],
-  dependencies : [dep_thread, idep_gtest],
+  executable(
+    'sampler_types_test',
+    ['sampler_types_test.cpp', ir_expression_operation_h],
+    cpp_args : [cpp_vis_args, cpp_msvc_compat_args],
+    include_directories : [inc_common, inc_glsl],
+    link_with : [libglsl, libglsl_util],
+    dependencies : [dep_thread, idep_gtest],
+  )
 )
 
-test('blob_test', glsl_blob_test)
-test('cache_test', glsl_cache_test)
-test('general_ir_test', glsl_general_ir_test)
-test('uniform_initializer_test', glsl_uniform_initializer_test)
-test('sampler_types_test', glsl_sampler_types_test)
-
 # TODO: figure out how to get the shell based tests to work?
index 5dd21e6652f0c70633d9a5353eedd2b34f7247d2..54cef67e9e01160066d7b8429bf8bcd53ea51608 100644 (file)
@@ -208,14 +208,15 @@ libnir = static_library(
 nir_algebraic_py = files('nir_algebraic.py')
 
 if with_tests
-  nir_control_flow_test = executable(
-    'nir_control_flow_test',
-    [files('tests/control_flow_tests.cpp'), nir_opcodes_h],
-    c_args : [c_vis_args, c_msvc_compat_args, no_override_init_args],
-    include_directories : [inc_common],
-    dependencies : [dep_thread, idep_gtest],
-    link_with : [libmesa_util, libnir],
+  test(
+    'nir_control_flow',
+    executable(
+      'nir_control_flow_test',
+      [files('tests/control_flow_tests.cpp'), nir_opcodes_h],
+      c_args : [c_vis_args, c_msvc_compat_args, no_override_init_args],
+      include_directories : [inc_common],
+      dependencies : [dep_thread, idep_gtest],
+      link_with : [libmesa_util, libnir],
+    )
   )
-
-  test('nir_control_flow', nir_control_flow_test)
 endif
index 14c78a3ecbc37308e45b15711654b1d138222fe5..c95d4c1d1f4175ce4754e28e72f8d46853c28b0d 100644 (file)
@@ -42,5 +42,8 @@ if with_tests
     find_program('wayland-egl-symbols-check'),
     args : libwayland_egl
   )
-  test('wayland-egl-abi-check', executable('wayland-egl-abi-check', 'wayland-egl-abi-check.c'))
+  test(
+    'wayland-egl-abi-check',
+    executable('wayland-egl-abi-check', 'wayland-egl-abi-check.c')
+  )
 endif
index 597245ee814cd81b150a77bcb8d9d37321dc3cbb..0b0ac2a3a023fb7bb12fb1b797ddff07e32266f7 100644 (file)
@@ -111,7 +111,9 @@ driver_swrast = declare_dependency(
 if with_tests and with_gallium_softpipe and with_llvm
   foreach t : ['lp_test_format', 'lp_test_arit', 'lp_test_blend',
                'lp_test_conv', 'lp_test_printf']
-    test(t, executable(
+    test(
+      t,
+      executable(
         t,
         ['@0@.c'.format(t), 'lp_test_main.c'],
         dependencies : [dep_llvm, dep_dl, dep_thread, dep_clock],
index 90fa5949fa481a2d994455168bede2483ce5965e..698e4608dedfedf91a5b1ffb3bd4fd1f47b7e9cf 100644 (file)
@@ -135,7 +135,9 @@ driver_r300 = declare_dependency(
 )
 
 if with_tests
-  test('r300_compiler_test', executable(
+  test(
+    'r300_compiler_test',
+    executable(
       'r300_compiler_test',
       files(
         'compiler/tests/r300_compiler_tests.c',
index a1022c164b1259e5a58f5d8e82e01b0c208b5ae1..2c98b0290d9993ebeb86061fa41f47588a82880b 100644 (file)
@@ -32,7 +32,9 @@ if with_tests
   dep_real_xvmc = dependency('xvmc')
   foreach x : ['context', 'surface', 'subpicture', 'blocks', 'rendering']
     _name = 'xvmc_@0@'.format(x)
-    test(_name, executable(
+    test(
+      _name,
+      executable(
         _name,
         files('tests/test_@0@.c'.format(x), 'tests/testlib.c'), 
         dependencies : [
@@ -42,7 +44,9 @@ if with_tests
     )
   endforeach
 
-  test('xbmc_bench', executable(
+  test(
+    'xvmc_bench',
+    executable(
       'xvmc_bench',
       files('tests/xvmc_bench.c', 'tests/testlib.c'), 
       dependencies : [
index 6e512996add1cfadd189c3f02dbaec5a8590ff7a..14b9e960360abd6d6983a0790204f33b5801378a 100644 (file)
@@ -71,7 +71,8 @@ pkg.generate(
 )
 
 if with_tests
-  test('gbm-symbols-check',
+  test(
+    'gbm-symbols-check',
     find_program('gbm-symbols-check'),
     args : libgbm
   )
index 51b2843fc90c883244f1a46cbf148a4f86a0c103..fe0075a69393c08426da4f802ba3caac395c80ab 100644 (file)
@@ -33,17 +33,17 @@ if with_shared_glapi
     files_glx_test += files('query_renderer_implementation_unittest.cpp')
   endif
 
-  glx_test = executable(
+  test(
     'glx-test',
-    [files_glx_test, glx_indirect_size_h, main_dispatch_h],
-    link_with : [libglx, libglapi],
-    include_directories : [
-      include_directories('..', '../../../include/GL/internal'),
-      inc_src, inc_include, inc_mesa, inc_mapi,
-    ],
-    dependencies : [dep_libdrm, dep_thread, idep_gtest]
+    executable(
+      'glx-test',
+      [files_glx_test, glx_indirect_size_h, main_dispatch_h],
+      link_with : [libglx, libglapi],
+      include_directories : [
+        include_directories('..', '../../../include/GL/internal'),
+        inc_src, inc_include, inc_mesa, inc_mapi,
+      ],
+      dependencies : [dep_libdrm, dep_thread, idep_gtest]
+    )
   )
-
-  test('glx-test', glx_test)
-  test('glx-dispatch-index-check', find_program('dispatch-index-check'))
 endif
index c54c117223c4b710f440a04b716438f16185a2c4..19e5926a0d92c6d6d079e8a985b89b192380da15 100644 (file)
@@ -145,14 +145,17 @@ if with_tests
                'fs_saturate_propagation', 'vf_float_conversions',
                'vec4_register_coalesce', 'vec4_copy_propagation',
                'vec4_cmod_propagation', 'eu_compact', 'eu_validate']
-    _exe = executable(
-      [t, nir_opcodes_h, ir_expression_operation_h],
-      'test_@0@.cpp'.format(t),
-      include_directories : [inc_common, inc_intel],
-      link_with : [libintel_compiler, libintel_common, libnir, libmesa_util,
-                   libisl],
-      dependencies : [dep_thread, dep_dl, idep_gtest],
+    test(
+      t,
+      executable(
+        [t, nir_opcodes_h, ir_expression_operation_h],
+        'test_@0@.cpp'.format(t),
+        include_directories : [inc_common, inc_intel],
+        link_with : [
+          libintel_compiler, libintel_common, libnir, libmesa_util, libisl,
+        ],
+        dependencies : [dep_thread, dep_dl, idep_gtest],
+      )
     )
-    test(t, _exe)
   endforeach
 endif
index 47fd8d97c32e9ae5d5cecebff743f7e485e3ca9f..116750d86d6659298a7fbf0d6f5c034567356c2a 100644 (file)
@@ -90,13 +90,14 @@ libisl = static_library(
 )
 
 if with_tests
-  isl_surf_get_image_offset_test = executable(
-    'isl_surf_get_image_offset_test',
-    'tests/isl_surf_get_image_offset_test.c',
-    dependencies : dep_m,
-    include_directories : [inc_common, inc_intel],
-    link_with : [libisl, libintel_common],
+  test(
+    'isl_surf_get_image_offset',
+    executable(
+      'isl_surf_get_image_offset_test',
+      'tests/isl_surf_get_image_offset_test.c',
+      dependencies : dep_m,
+      include_directories : [inc_common, inc_intel],
+      link_with : [libisl, libintel_common],
+    )
   )
-
-  test('isl_surf_get_image_offset', isl_surf_get_image_offset_test)
 endif
index b219201944bce1f09e6a3a60ae043edaac0bf60a..66266a0e95a1cb41ad5830fa72758e564b4f922c 100644 (file)
@@ -193,25 +193,28 @@ if with_tests
     link_whole : libanv_common,
     link_with : [
       libanv_gen_libs, libintel_compiler, libintel_common, libisl, libblorp,
-      libvulkan_util, libvulkan_wsi, libnir, libmesa_util,
+      libvulkan_util, libvulkan_wsi, libmesa_util,
     ],
     dependencies : [
       dep_libdrm, dep_thread, dep_dl, dep_m, anv_deps, dep_valgrind,
+      idep_nir,
     ],
     c_args : [c_vis_args, no_override_init_args, '-msse2', anv_flags],
   )
 
   foreach t : ['block_pool_no_free', 'state_pool_no_free',
                'state_pool_free_list_only', 'state_pool']
-    _exe = executable(
-      t,
-      ['tests/@0@.c'.format(t), dummy_cpp, block_entrypoints],
-      link_with : libvulkan_intel_test,
-      dependencies : [dep_libdrm, dep_thread, dep_m, dep_valgrind],
-      include_directories : [
-        inc_common, inc_intel, inc_compiler, inc_vulkan_util, inc_vulkan_wsi,
-      ],
+    test(
+      'anv_@0@'.format(t),
+      executable(
+        t,
+        ['tests/@0@.c'.format(t), dummy_cpp, block_entrypoints],
+        link_with : libvulkan_intel_test,
+        dependencies : [dep_libdrm, dep_thread, dep_m, dep_valgrind],
+        include_directories : [
+          inc_common, inc_intel, inc_compiler, inc_vulkan_util, inc_vulkan_wsi,
+        ],
+      )
     )
-    test('anv_@0@'.format(t), _exe)
   endforeach
 endif
index 53851da306bedbad2b4ffa86e119dc8ee7cc4308..ea14654d2c4b200350d7d43703c87b34d62ce2af 100644 (file)
@@ -52,7 +52,8 @@ pkg.generate(
 )
 
 if with_tests
-  test('es1-ABI-check',
+  test(
+    'es1-ABI-check',
     find_program('ABI-check'),
     args : libglesv1_cm
   )
index 8751107cb714a434704990b637c2b692fc602495..de8a29bb6beddc840ef78f2edd702dcb1385c62f 100644 (file)
@@ -52,7 +52,8 @@ pkg.generate(
 )
 
 if with_tests
-  test('es2-ABI-check',
+  test(
+    'es2-ABI-check',
     find_program('ABI-check'),
     args : libgles2
   )
index 8856a95fb66af7cfdd713fbffdc80abc75459a5a..267cfe7e7be0a4c5434da0fe6b8bd1ea47769d7b 100644 (file)
@@ -78,12 +78,13 @@ libglapi_static = static_library(
 )
 
 if not with_shared_glapi and with_tests
-  glapi_static_check_table = executable(
+  test(
     'glapi_static_check_table',
-    'tests/check_table.cpp',
-    link_with : [libglapi_static],
-    dependencies : [idep_gtest],
+    executable(
+      'glapi_static_check_table',
+      'tests/check_table.cpp',
+      link_with : [libglapi_static],
+      dependencies : [idep_gtest],
+    )
   )
-
-  test('glapi_static_check_table', glapi_static_check_table)
 endif
index fe0d458d93470c8f53b328ec5808c144b3ba708d..c7d136ced2c1cdc6c4968ccea27b285169bb5bbd 100644 (file)
@@ -51,14 +51,15 @@ libglapi = shared_library(
 )
 
 if with_tests
-  shared_glapi_test = executable(
-    ['shared-glapi-test', glapitable_h],
-    'tests/check_table.cpp',
-    cpp_args : [cpp_msvc_compat_args],
-    include_directories : [inc_src, inc_include, inc_mapi],
-    link_with : [libglapi],
-    dependencies : [dep_thread, idep_gtest],
+  test(
+    'shared-glapi-test',
+    executable(
+      ['shared-glapi-test', glapitable_h],
+      'tests/check_table.cpp',
+      cpp_args : [cpp_msvc_compat_args],
+      include_directories : [inc_src, inc_include, inc_mapi],
+      link_with : [libglapi],
+      dependencies : [dep_thread, idep_gtest],
+    )
   )
-
-  test('shared-glapi-test', shared_glapi_test)
 endif
index 2c1d8e067e8e74e5c6db4f2f2294717ee5fdd8e2..12be73939451a29aa23f9f55d624eecb45daf97c 100644 (file)
@@ -33,7 +33,9 @@ else
   files_main_test += files('stub.cpp')
 endif
 
-test('main-test', executable(
+test(
+  'main-test',
+  executable(
     'main_test',
     [files_main_test, main_dispatch_h],
     include_directories : [inc_include, inc_src, inc_mapi, inc_mesa],
index 225593f5d381b486bdb2980c8816c1869a98fb19..fa591c92e5624c5e63f49f3bcb96c803ce899394 100644 (file)
@@ -118,34 +118,39 @@ libxmlconfig = static_library(
 )
 
 if with_tests
-  u_atomic_test = executable(
-    'u_atomic_test',
-    files('u_atomic_test.c'),
-    include_directories : inc_common,
-    link_with : libmesa_util,
-    c_args : [c_msvc_compat_args],
+  test(
+    'u_atomic',
+    executable(
+      'u_atomic_test',
+      files('u_atomic_test.c'),
+      include_directories : inc_common,
+      link_with : libmesa_util,
+      c_args : [c_msvc_compat_args],
+    )
   )
 
-  roundeven_test = executable(
-    'roundeven_test',
-    files('roundeven_test.c'),
-    include_directories : inc_common,
-    c_args : [c_msvc_compat_args],
-    dependencies : [dep_m],
+  test(
+    'roundeven',
+    executable(
+      'roundeven_test',
+      files('roundeven_test.c'),
+      include_directories : inc_common,
+      c_args : [c_msvc_compat_args],
+      dependencies : [dep_m],
+    )
   )
 
-  mesa_sha1_test = executable(
-    'mesa-sha1_test',
-    files('mesa-sha1_test.c'),
-    include_directories : inc_common,
-    link_with : libmesa_util,
-    c_args : [c_msvc_compat_args],
+  test(
+    'mesa-sha1',
+    executable(
+      'mesa-sha1_test',
+      files('mesa-sha1_test.c'),
+      include_directories : inc_common,
+      link_with : libmesa_util,
+      c_args : [c_msvc_compat_args],
+    )
   )
 
-  test('u_atomic', u_atomic_test)
-  test('roundeven', roundeven_test)
-  test('mesa-sha1', mesa_sha1_test)
-
   subdir('tests/hash_table')
   subdir('tests/string_buffer')
 endif
index 43d8f43a83e43c609a70a3d810569f90c4fbb1cf..4bbc5100ea3c8836409627af0005ed9f275f5949 100644 (file)
 foreach t : ['clear', 'collision', 'delete_and_lookup', 'delete_management',
              'destroy_callback', 'insert_and_lookup', 'insert_many',
              'null_destroy', 'random_entry', 'remove_null', 'replacement']
-  _test = executable(
-    '@0@_test'.format(t),
-    files('@0@.c'.format(t)),
-    dependencies : [dep_thread, dep_dl],
-    include_directories : [inc_include, inc_util],
-    link_with : libmesa_util,
+  test(
+    t,
+    executable(
+      '@0@_test'.format(t),
+      files('@0@.c'.format(t)),
+      dependencies : [dep_thread, dep_dl],
+      include_directories : [inc_include, inc_util],
+      link_with : libmesa_util,
+    )
   )
-  test(t, _test)
 endforeach
index 14dbebca7d87585898d1c20b0544c9dd08aeb94d..9f42e3550ae15c63f2cefec0d7cffbd14bafafd5 100644 (file)
 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 # SOFTWARE.
 
-string_buffer_test = executable(
-  'string_buffer_test',
-  'string_buffer_test.cpp',
-  dependencies : [dep_thread, dep_dl, idep_gtest],
-  include_directories : inc_common,
-  link_with : [libmesa_util],
+test(
+  'string_buffer',
+  executable(
+    'string_buffer_test',
+    'string_buffer_test.cpp',
+    dependencies : [dep_thread, dep_dl, idep_gtest],
+    include_directories : inc_common,
+    link_with : [libmesa_util],
+  )
 )
-
-test('string_buffer', string_buffer_test)