aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEnji Cooper <ngie@FreeBSD.org>2023-08-08 13:42:07 +0000
committerEnji Cooper <ngie@FreeBSD.org>2023-08-10 09:32:33 +0000
commit811e0a31acafd6ab21f89bec8ba0fb8c09f258d2 (patch)
tree92fa9a92f97240d76885ca13ccaa462869229041
parent28f6c2f292806bf31230a959bc4b19d7081669a7 (diff)
downloadsrc-811e0a31acafd6ab21f89bec8ba0fb8c09f258d2.tar.gz
src-811e0a31acafd6ab21f89bec8ba0fb8c09f258d2.zip
fusefs tests: handle -Wdeprecated* issues with GoogleTest 1.14.0
`INSTANTIATE_TEST_CASE_P` has been replaced with `INSTANTIATE_TEST_SUITE_P`. Replace all uses of the former macro with the latter macro. While here, address the fact that the latter macro doesn't permit some of the constructions that the former macro did, e.g., empty parameters, etc. MFC after: 2 weeks Reviewed by: asomers Differential Revision: https://reviews.freebsd.org/D41398
-rw-r--r--tests/sys/fs/fusefs/bmap.cc2
-rw-r--r--tests/sys/fs/fusefs/cache.cc2
-rw-r--r--tests/sys/fs/fusefs/dev_fuse_poll.cc2
-rw-r--r--tests/sys/fs/fusefs/fallocate.cc4
-rw-r--r--tests/sys/fs/fusefs/io.cc4
-rw-r--r--tests/sys/fs/fusefs/last_local_modify.cc2
-rw-r--r--tests/sys/fs/fusefs/mount.cc6
-rw-r--r--tests/sys/fs/fusefs/read.cc4
-rw-r--r--tests/sys/fs/fusefs/write.cc6
9 files changed, 16 insertions, 16 deletions
diff --git a/tests/sys/fs/fusefs/bmap.cc b/tests/sys/fs/fusefs/bmap.cc
index 9d97d1c08ca5..3191ac657de9 100644
--- a/tests/sys/fs/fusefs/bmap.cc
+++ b/tests/sys/fs/fusefs/bmap.cc
@@ -253,6 +253,6 @@ TEST_P(BmapEof, eof)
leak(fd);
}
-INSTANTIATE_TEST_CASE_P(BE, BmapEof,
+INSTANTIATE_TEST_SUITE_P(BE, BmapEof,
Values(1, 2, 3)
);
diff --git a/tests/sys/fs/fusefs/cache.cc b/tests/sys/fs/fusefs/cache.cc
index cf9476a0a853..429b54c7ba75 100644
--- a/tests/sys/fs/fusefs/cache.cc
+++ b/tests/sys/fs/fusefs/cache.cc
@@ -197,7 +197,7 @@ TEST_P(Cache, truncate_by_surprise_invalidates_cache)
leak(fd);
}
-INSTANTIATE_TEST_CASE_P(Cache, Cache,
+INSTANTIATE_TEST_SUITE_P(Cache, Cache,
Combine(
/* Test every combination that:
* - does not cache at least one of entries and attrs
diff --git a/tests/sys/fs/fusefs/dev_fuse_poll.cc b/tests/sys/fs/fusefs/dev_fuse_poll.cc
index de1b9a6b5f3f..4777f9818c60 100644
--- a/tests/sys/fs/fusefs/dev_fuse_poll.cc
+++ b/tests/sys/fs/fusefs/dev_fuse_poll.cc
@@ -98,7 +98,7 @@ TEST_P(DevFusePoll, destroy)
m_mock->unmount();
}
-INSTANTIATE_TEST_CASE_P(PM, DevFusePoll,
+INSTANTIATE_TEST_SUITE_P(PM, DevFusePoll,
::testing::Values("BLOCKING", "KQ", "POLL", "SELECT"));
static void* statter(void* arg) {
diff --git a/tests/sys/fs/fusefs/fallocate.cc b/tests/sys/fs/fusefs/fallocate.cc
index d10abfba3807..166f9ca7ff3a 100644
--- a/tests/sys/fs/fusefs/fallocate.cc
+++ b/tests/sys/fs/fusefs/fallocate.cc
@@ -493,8 +493,8 @@ TEST_P(FspacectlCache, clears_cache)
leak(fd);
}
-INSTANTIATE_TEST_CASE_P(FspacectlCache, FspacectlCache,
- Values(Uncached, Writethrough, Writeback),
+INSTANTIATE_TEST_SUITE_P(FspacectlCache, FspacectlCache,
+ Values(Uncached, Writethrough, Writeback)
);
/*
diff --git a/tests/sys/fs/fusefs/io.cc b/tests/sys/fs/fusefs/io.cc
index d08dea99ff47..fe3237d39c95 100644
--- a/tests/sys/fs/fusefs/io.cc
+++ b/tests/sys/fs/fusefs/io.cc
@@ -509,14 +509,14 @@ TEST_P(Io, resize_a_valid_buffer_while_extending)
close(m_test_fd);
}
-INSTANTIATE_TEST_CASE_P(Io, Io,
+INSTANTIATE_TEST_SUITE_P(Io, Io,
Combine(Bool(), /* async read */
Values(0x1000, 0x10000, 0x20000), /* m_maxwrite */
Values(Uncached, Writethrough, Writeback, WritebackAsync)
)
);
-INSTANTIATE_TEST_CASE_P(Io, IoCacheable,
+INSTANTIATE_TEST_SUITE_P(Io, IoCacheable,
Combine(Bool(), /* async read */
Values(0x1000, 0x10000, 0x20000), /* m_maxwrite */
Values(Writethrough, Writeback, WritebackAsync)
diff --git a/tests/sys/fs/fusefs/last_local_modify.cc b/tests/sys/fs/fusefs/last_local_modify.cc
index dd0d9c740355..537187a6bfb7 100644
--- a/tests/sys/fs/fusefs/last_local_modify.cc
+++ b/tests/sys/fs/fusefs/last_local_modify.cc
@@ -506,7 +506,7 @@ TEST_P(LastLocalModify, vfs_vget)
}
-INSTANTIATE_TEST_CASE_P(LLM, LastLocalModify,
+INSTANTIATE_TEST_SUITE_P(LLM, LastLocalModify,
Values(
"VOP_ALLOCATE",
"VOP_COPY_FILE_RANGE",
diff --git a/tests/sys/fs/fusefs/mount.cc b/tests/sys/fs/fusefs/mount.cc
index 06ebe7fedebc..db06068e4eb8 100644
--- a/tests/sys/fs/fusefs/mount.cc
+++ b/tests/sys/fs/fusefs/mount.cc
@@ -192,11 +192,11 @@ TEST_P(UpdateErr, update)
EXPECT_TRUE((newflags ^ statbuf.f_flags) & flag);
}
-INSTANTIATE_TEST_CASE_P(Mount, UpdateOk,
+INSTANTIATE_TEST_SUITE_P(Mount, UpdateOk,
::testing::Values("MNT_RDONLY", "MNT_NOEXEC", "MNT_NOSUID", "MNT_NOATIME",
"MNT_SUIDDIR")
);
-INSTANTIATE_TEST_CASE_P(Mount, UpdateErr,
- ::testing::Values( "MNT_USER");
+INSTANTIATE_TEST_SUITE_P(Mount, UpdateErr,
+ ::testing::Values( "MNT_USER")
);
diff --git a/tests/sys/fs/fusefs/read.cc b/tests/sys/fs/fusefs/read.cc
index f1f485d0b736..72e09d355ad3 100644
--- a/tests/sys/fs/fusefs/read.cc
+++ b/tests/sys/fs/fusefs/read.cc
@@ -1314,7 +1314,7 @@ TEST_P(ReadAhead, readahead) {
free(contents);
}
-INSTANTIATE_TEST_CASE_P(RA, ReadAhead,
+INSTANTIATE_TEST_SUITE_P(RA, ReadAhead,
Values(tuple<bool, int>(false, 0),
tuple<bool, int>(false, 1),
tuple<bool, int>(false, 2),
@@ -1362,4 +1362,4 @@ TEST_P(TimeGran, atime_during_setattr)
leak(fd);
}
-INSTANTIATE_TEST_CASE_P(TG, TimeGran, Range(0u, 10u));
+INSTANTIATE_TEST_SUITE_P(TG, TimeGran, Range(0u, 10u));
diff --git a/tests/sys/fs/fusefs/write.cc b/tests/sys/fs/fusefs/write.cc
index a217e285a76b..c8cb7d08f474 100644
--- a/tests/sys/fs/fusefs/write.cc
+++ b/tests/sys/fs/fusefs/write.cc
@@ -616,7 +616,7 @@ TEST_P(WriteRlimitFsize, rlimit_fsize_truncate)
leak(fd);
}
-INSTANTIATE_TEST_CASE_P(W, WriteRlimitFsize,
+INSTANTIATE_TEST_SUITE_P(W, WriteRlimitFsize,
Values(0, O_DIRECT)
);
@@ -724,7 +724,7 @@ TEST_P(WriteEofDuringVnopStrategy, eof_during_vop_strategy)
}
-INSTANTIATE_TEST_CASE_P(W, WriteEofDuringVnopStrategy,
+INSTANTIATE_TEST_SUITE_P(W, WriteEofDuringVnopStrategy,
Values(1, 2, 3)
);
@@ -1546,7 +1546,7 @@ TEST_P(TimeGran, timestamps_during_setattr)
leak(fd);
}
-INSTANTIATE_TEST_CASE_P(RA, TimeGran, Range(0u, 10u));
+INSTANTIATE_TEST_SUITE_P(RA, TimeGran, Range(0u, 10u));
/*
* Without direct_io, writes should be committed to cache