aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJakob Unterwurzacher2017-03-07 20:53:58 +0100
committerJakob Unterwurzacher2017-03-07 20:53:58 +0100
commita80d798c2deab44e2abc37eeae59546f0d7eec40 (patch)
treeb8bb6cce783a20b92843e41059723e25127cf865
parentd48ccb3ddab71773a991b8f1b062901ff5b435b0 (diff)
tests: reverse: don't run tests that ignore "-plaintextnames" twice
TestMain() runs all tests twice, once with plaintextnames=true and once with false. Several tests mount their own filesystem and ignore the plaintextnames variable. It makes no sense to run them twice, so skip execution when plaintextnames is set.
-rw-r--r--tests/reverse/correctness_test.go5
-rw-r--r--tests/reverse/ctlsock_test.go6
2 files changed, 10 insertions, 1 deletions
diff --git a/tests/reverse/correctness_test.go b/tests/reverse/correctness_test.go
index bebf341..bfbab3d 100644
--- a/tests/reverse/correctness_test.go
+++ b/tests/reverse/correctness_test.go
@@ -55,6 +55,9 @@ func TestSymlinks(t *testing.T) {
// Symbolic link dentry sizes should be set to the length of the string
// that contains the target path.
func TestSymlinkDentrySize(t *testing.T) {
+ if plaintextnames {
+ t.Skip("this only tests encrypted names")
+ }
symlink := "a_symlink"
mnt, err := ioutil.TempDir(test_helpers.TmpDir, "reverse_mnt_")
@@ -108,7 +111,7 @@ func TestConfigMapping(t *testing.T) {
// Check that the access() syscall works on virtual files
func TestAccessVirtual(t *testing.T) {
if plaintextnames {
- t.Skip()
+ t.Skip("test makes no sense for plaintextnames")
}
var R_OK uint32 = 4
var W_OK uint32 = 2
diff --git a/tests/reverse/ctlsock_test.go b/tests/reverse/ctlsock_test.go
index 8a7d462..c3ee121 100644
--- a/tests/reverse/ctlsock_test.go
+++ b/tests/reverse/ctlsock_test.go
@@ -24,6 +24,9 @@ var ctlSockTestCases = [][]string{
// Test DecryptPath and EncryptPath
func TestCtlSockPathOps(t *testing.T) {
+ if plaintextnames {
+ t.Skip("this only tests encrypted names")
+ }
mnt, err := ioutil.TempDir(test_helpers.TmpDir, "reverse_mnt_")
if err != nil {
t.Fatal(err)
@@ -54,6 +57,9 @@ func TestCtlSockPathOps(t *testing.T) {
// We should not panic when somebody feeds requests that make no sense
func TestCtlSockCrash(t *testing.T) {
+ if plaintextnames {
+ t.Skip("this only tests encrypted names")
+ }
mnt, err := ioutil.TempDir(test_helpers.TmpDir, "reverse_mnt_")
if err != nil {
t.Fatal(err)