Skip to content

Commit

Permalink
test: disable test due to pmem#5641
Browse files Browse the repository at this point in the history
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@intel.com>
  • Loading branch information
grom72 committed May 16, 2023
1 parent 5bfd7d9 commit 4b76303
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 1 deletion.
12 changes: 12 additions & 0 deletions src/test/ex_libpmem2/TESTS.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
#
import futils
import testframework as t
from testframework import granularity as g


@t.require_build(['debug', 'release'])
Expand Down Expand Up @@ -73,6 +74,12 @@ def run(self, ctx):
ctx.exec(example_path, *args, stdout_file='out4.log')


# XXX
# Manually disabled for pmemcheck for g.PAGE until ready fix for
# https://github.com/pmem/pmdk/issues/5641
# additionall test TEST501 has been added to cover non-pmemcheck configs.
@g.require_granularity(g.CACHELINE, g.BYTE) # to be removed when fix available
@t.require_valgrind_enabled('pmemcheck') # to be removed when fix available
class TEST5(EX_LIBPMEM2):

def run(self, ctx):
Expand All @@ -81,6 +88,11 @@ def run(self, ctx):
ctx.exec(example_path, "write", file_path, "foobar")
ctx.exec(example_path, "read", file_path, stdout_file='out5.log')

# XXX see TEST5 above
@t.require_valgrind_disabled('pmemcheck')
class TEST501(TEST5): # to be removed when fix available

pass

@t.windows_exclude
# This test case would require two VALGRIND_SET_CLEAN() calls
Expand Down
1 change: 0 additions & 1 deletion src/test/unittest/granularity.py
Original file line number Diff line number Diff line change
Expand Up @@ -280,7 +280,6 @@ def wrapped(tc):
return tc
return wrapped


def no_testdir(**kwargs):
def wrapped(tc):
ctx.add_requirement(tc, 'granularity', Non, **kwargs)
Expand Down

0 comments on commit 4b76303

Please # to comment.