use safer path for stub tests

While these stubs shouldn't make actual changes, it's better to not use `/etc` in the tests, as they may be run as `root`. Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
opencontainers · Nov 14, 2022 · bc94efc · bc94efc
1 parent 41ff4c2
commit bc94efc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/go-selinux/label/label_stub_test.go b/go-selinux/label/label_stub_test.go
@@ -79,11 +79,12 @@ func TestCheckLabelCompile(t *testing.T) {
 		t.Fatal(err)
 	}
 
-	if _, err := FileLabel("/etc"); err != nil {
+	tmpDir := t.TempDir()
+	if _, err := FileLabel(tmpDir); err != nil {
 		t.Fatal(err)
 	}
 
-	if err := SetFileLabel("/etc", "foobar"); err != nil {
+	if err := SetFileLabel(tmpDir, "foobar"); err != nil {
 		t.Fatal(err)
 	}
 

diff --git a/go-selinux/selinux_stub_test.go b/go-selinux/selinux_stub_test.go
@@ -14,11 +14,12 @@ func TestSELinuxStubs(t *testing.T) {
 		t.Error("SELinux enabled on non-linux.")
 	}
 
-	if _, err := FileLabel("/etc"); err != nil {
+	tmpDir := t.TempDir()
+	if _, err := FileLabel(tmpDir); err != nil {
 		t.Error(err)
 	}
 
-	if err := SetFileLabel("/etc", testLabel); err != nil {
+	if err := SetFileLabel(tmpDir, testLabel); err != nil {
 		t.Error(err)
 	}