Skip to content

Commit a72188e

Browse files
committed
test: drop hardware-dependent tests from meson test target
Currently, the tests under test/ require intractions with actual hardware, which isn't available under CI. [Additionally, the test() functions are mostly incorrect, as they'll all just repeat the same test.c test] This test drops the test() defitinitons, but leaves the executable()s available for developer use, and adds a comment to indicate what we're doing. Signed-off-by: Jeremy Kerr <[email protected]>
1 parent 6a2d2ba commit a72188e

1 file changed

Lines changed: 5 additions & 7 deletions

File tree

test/meson.build

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,11 @@
44
# Copyright (c) 2021 Dell Inc.
55
#
66
# Authors: Martin Belanger <[email protected]>
7-
#
7+
8+
# These tests all require interaction with a real NVMe device, so we don't
9+
# define as meson unit-tests, and therefore get run as part of the 'test'
10+
# target. However, they're available for developer use, when hardware is
11+
# available.
812
main = executable(
913
'main-test',
1014
['test.c'],
@@ -33,9 +37,3 @@ zns = executable(
3337
link_with: libnvme,
3438
include_directories: [incdir, internal_incdir]
3539
)
36-
37-
test('main', main)
38-
test('main', main, args: ['nvme10'])
39-
test('cpp', main)
40-
test('register', main, args: ['nvme10'])
41-
test('zns', main)

0 commit comments

Comments
 (0)