mirror of
https://github.com/minio/minio.git
synced 2024-12-25 22:55:54 -05:00
0a3b1ad4eb
This commit replaces `ioutil.TempDir` with `t.TempDir` in tests. The
directory created by `t.TempDir` is automatically removed when the test
and all its subtests complete.
Prior to this commit, temporary directory created using `ioutil.TempDir`
needs to be removed manually by calling `os.RemoveAll`, which is omitted
in some tests. The error handling boilerplate e.g.
defer func() {
if err := os.RemoveAll(dir); err != nil {
t.Fatal(err)
}
}
is also tedious, but `t.TempDir` handles this for us nicely.
Reference: https://pkg.go.dev/testing#T.TempDir
Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
111 lines
3.4 KiB
Go
111 lines
3.4 KiB
Go
//go:build linux || darwin || dragonfly || freebsd || netbsd || openbsd
|
|
// +build linux darwin dragonfly freebsd netbsd openbsd
|
|
|
|
// Copyright (c) 2015-2021 MinIO, Inc.
|
|
//
|
|
// This file is part of MinIO Object Storage stack
|
|
//
|
|
// This program is free software: you can redistribute it and/or modify
|
|
// it under the terms of the GNU Affero General Public License as published by
|
|
// the Free Software Foundation, either version 3 of the License, or
|
|
// (at your option) any later version.
|
|
//
|
|
// This program is distributed in the hope that it will be useful
|
|
// but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
// GNU Affero General Public License for more details.
|
|
//
|
|
// You should have received a copy of the GNU Affero General Public License
|
|
// along with this program. If not, see <http://www.gnu.org/licenses/>.
|
|
|
|
package cmd
|
|
|
|
import (
|
|
"context"
|
|
"os"
|
|
"path"
|
|
"syscall"
|
|
"testing"
|
|
)
|
|
|
|
// Based on `man getumask` a vaporware GNU extension to glibc.
|
|
// returns file mode creation mask.
|
|
func getUmask() int {
|
|
mask := syscall.Umask(0)
|
|
syscall.Umask(mask)
|
|
return mask
|
|
}
|
|
|
|
// Tests if the directory and file creations happen with proper umask.
|
|
func TestIsValidUmaskVol(t *testing.T) {
|
|
tmpPath := t.TempDir()
|
|
testCases := []struct {
|
|
volName string
|
|
expectedUmask int
|
|
}{
|
|
{"is-this-valid", getUmask()},
|
|
}
|
|
testCase := testCases[0]
|
|
|
|
// Initialize a new xlStorage layer.
|
|
disk, err := newLocalXLStorage(tmpPath)
|
|
if err != nil {
|
|
t.Fatalf("Initializing xlStorage failed with %s.", err)
|
|
}
|
|
|
|
// Attempt to create a volume to verify the permissions later.
|
|
// MakeVol creates 0777.
|
|
if err = disk.MakeVol(context.Background(), testCase.volName); err != nil {
|
|
t.Fatalf("Creating a volume failed with %s expected to pass.", err)
|
|
}
|
|
|
|
// Stat to get permissions bits.
|
|
st, err := os.Stat(path.Join(tmpPath, testCase.volName))
|
|
if err != nil {
|
|
t.Fatalf("Stat failed with %s expected to pass.", err)
|
|
}
|
|
|
|
// Get umask of the bits stored.
|
|
currentUmask := 0o777 - uint32(st.Mode().Perm())
|
|
|
|
// Verify if umask is correct.
|
|
if int(currentUmask) != testCase.expectedUmask {
|
|
t.Fatalf("Umask check failed expected %d, got %d", testCase.expectedUmask, currentUmask)
|
|
}
|
|
}
|
|
|
|
// Tests if the file creations happen with proper umask.
|
|
func TestIsValidUmaskFile(t *testing.T) {
|
|
tmpPath := t.TempDir()
|
|
testCases := []struct {
|
|
volName string
|
|
expectedUmask int
|
|
}{
|
|
{"is-this-valid", getUmask()},
|
|
}
|
|
testCase := testCases[0]
|
|
|
|
// Initialize a new xlStorage layer.
|
|
disk, err := newLocalXLStorage(tmpPath)
|
|
if err != nil {
|
|
t.Fatalf("Initializing xlStorage failed with %s.", err)
|
|
}
|
|
|
|
// Attempt to create a volume to verify the permissions later.
|
|
// MakeVol creates directory with 0777 perms.
|
|
if err = disk.MakeVol(context.Background(), testCase.volName); err != nil {
|
|
t.Fatalf("Creating a volume failed with %s expected to pass.", err)
|
|
}
|
|
|
|
// Attempt to create a file to verify the permissions later.
|
|
// AppendFile creates file with 0666 perms.
|
|
if err = disk.AppendFile(context.Background(), testCase.volName, pathJoin("hello-world.txt", xlStorageFormatFile), []byte("Hello World")); err != nil {
|
|
t.Fatalf("Create a file `test` failed with %s expected to pass.", err)
|
|
}
|
|
|
|
// CheckFile - stat the file.
|
|
if _, err := disk.StatInfoFile(context.Background(), testCase.volName, "hello-world.txt/"+xlStorageFormatFile, false); err != nil {
|
|
t.Fatalf("Stat failed with %s expected to pass.", err)
|
|
}
|
|
}
|