5c80ccacc4
The file permissions can be specified (unfortunately) as a string or an octal integer. During the normalization step, every field is unmarshalled into an interface{}. String types are kept in tact but integers are converted to decimal integers. If the raw config represented the permissions as an octal, it would be converted to decimal _before_ it was saved to RawFilePermissions. Permissions() would then try to convert it again, assuming it was an octal. The new behavior doesn't assume the radix of the number, allowing decimal and octal input.
169 lines
4.0 KiB
Go
169 lines
4.0 KiB
Go
/*
|
|
Copyright 2014 CoreOS, Inc.
|
|
|
|
Licensed under the Apache License, Version 2.0 (the "License");
|
|
you may not use this file except in compliance with the License.
|
|
You may obtain a copy of the License at
|
|
|
|
http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
Unless required by applicable law or agreed to in writing, software
|
|
distributed under the License is distributed on an "AS IS" BASIS,
|
|
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
See the License for the specific language governing permissions and
|
|
limitations under the License.
|
|
*/
|
|
|
|
package system
|
|
|
|
import (
|
|
"io/ioutil"
|
|
"os"
|
|
"path"
|
|
"testing"
|
|
|
|
"github.com/coreos/coreos-cloudinit/config"
|
|
)
|
|
|
|
func TestWriteFileUnencodedContent(t *testing.T) {
|
|
dir, err := ioutil.TempDir(os.TempDir(), "coreos-cloudinit-")
|
|
if err != nil {
|
|
t.Fatalf("Unable to create tempdir: %v", err)
|
|
}
|
|
defer os.RemoveAll(dir)
|
|
|
|
fn := "foo"
|
|
fullPath := path.Join(dir, fn)
|
|
|
|
wf := File{config.File{
|
|
Path: fn,
|
|
Content: "bar",
|
|
RawFilePermissions: "0644",
|
|
}}
|
|
|
|
path, err := WriteFile(&wf, dir)
|
|
if err != nil {
|
|
t.Fatalf("Processing of WriteFile failed: %v", err)
|
|
} else if path != fullPath {
|
|
t.Fatalf("WriteFile returned bad path: want %s, got %s", fullPath, path)
|
|
}
|
|
|
|
fi, err := os.Stat(fullPath)
|
|
if err != nil {
|
|
t.Fatalf("Unable to stat file: %v", err)
|
|
}
|
|
|
|
if fi.Mode() != os.FileMode(0644) {
|
|
t.Errorf("File has incorrect mode: %v", fi.Mode())
|
|
}
|
|
|
|
contents, err := ioutil.ReadFile(fullPath)
|
|
if err != nil {
|
|
t.Fatalf("Unable to read expected file: %v", err)
|
|
}
|
|
|
|
if string(contents) != "bar" {
|
|
t.Fatalf("File has incorrect contents")
|
|
}
|
|
}
|
|
|
|
func TestWriteFileInvalidPermission(t *testing.T) {
|
|
dir, err := ioutil.TempDir(os.TempDir(), "coreos-cloudinit-")
|
|
if err != nil {
|
|
t.Fatalf("Unable to create tempdir: %v", err)
|
|
}
|
|
defer os.RemoveAll(dir)
|
|
|
|
wf := File{config.File{
|
|
Path: path.Join(dir, "tmp", "foo"),
|
|
Content: "bar",
|
|
RawFilePermissions: "pants",
|
|
}}
|
|
|
|
if _, err := WriteFile(&wf, dir); err == nil {
|
|
t.Fatalf("Expected error to be raised when writing file with invalid permission")
|
|
}
|
|
}
|
|
|
|
func TestDecimalFilePermissions(t *testing.T) {
|
|
dir, err := ioutil.TempDir(os.TempDir(), "coreos-cloudinit-")
|
|
if err != nil {
|
|
t.Fatalf("Unable to create tempdir: %v", err)
|
|
}
|
|
defer os.RemoveAll(dir)
|
|
|
|
fn := "foo"
|
|
fullPath := path.Join(dir, fn)
|
|
|
|
wf := File{config.File{
|
|
Path: fn,
|
|
RawFilePermissions: "484", // Decimal representation of 0744
|
|
}}
|
|
|
|
path, err := WriteFile(&wf, dir)
|
|
if err != nil {
|
|
t.Fatalf("Processing of WriteFile failed: %v", err)
|
|
} else if path != fullPath {
|
|
t.Fatalf("WriteFile returned bad path: want %s, got %s", fullPath, path)
|
|
}
|
|
|
|
fi, err := os.Stat(fullPath)
|
|
if err != nil {
|
|
t.Fatalf("Unable to stat file: %v", err)
|
|
}
|
|
|
|
if fi.Mode() != os.FileMode(0744) {
|
|
t.Errorf("File has incorrect mode: %v", fi.Mode())
|
|
}
|
|
}
|
|
|
|
func TestWriteFilePermissions(t *testing.T) {
|
|
dir, err := ioutil.TempDir(os.TempDir(), "coreos-cloudinit-")
|
|
if err != nil {
|
|
t.Fatalf("Unable to create tempdir: %v", err)
|
|
}
|
|
defer os.RemoveAll(dir)
|
|
|
|
fn := "foo"
|
|
fullPath := path.Join(dir, fn)
|
|
|
|
wf := File{config.File{
|
|
Path: fn,
|
|
RawFilePermissions: "0755",
|
|
}}
|
|
|
|
path, err := WriteFile(&wf, dir)
|
|
if err != nil {
|
|
t.Fatalf("Processing of WriteFile failed: %v", err)
|
|
} else if path != fullPath {
|
|
t.Fatalf("WriteFile returned bad path: want %s, got %s", fullPath, path)
|
|
}
|
|
|
|
fi, err := os.Stat(fullPath)
|
|
if err != nil {
|
|
t.Fatalf("Unable to stat file: %v", err)
|
|
}
|
|
|
|
if fi.Mode() != os.FileMode(0755) {
|
|
t.Errorf("File has incorrect mode: %v", fi.Mode())
|
|
}
|
|
}
|
|
|
|
func TestWriteFileEncodedContent(t *testing.T) {
|
|
dir, err := ioutil.TempDir(os.TempDir(), "coreos-cloudinit-")
|
|
if err != nil {
|
|
t.Fatalf("Unable to create tempdir: %v", err)
|
|
}
|
|
defer os.RemoveAll(dir)
|
|
|
|
wf := File{config.File{
|
|
Path: path.Join(dir, "tmp", "foo"),
|
|
Content: "",
|
|
Encoding: "base64",
|
|
}}
|
|
|
|
if _, err := WriteFile(&wf, dir); err == nil {
|
|
t.Fatalf("Expected error to be raised when writing file with encoding")
|
|
}
|
|
}
|