Skip to content

Commit

Permalink
removed flaky test
Browse files Browse the repository at this point in the history
  • Loading branch information
dashpole committed Oct 14, 2016
1 parent e972272 commit 835fd2e
Showing 1 changed file with 0 additions and 34 deletions.
34 changes: 0 additions & 34 deletions fs/fs_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -106,23 +106,6 @@ func TestDirDiskUsage(t *testing.T) {
as.True(expectedSize <= size, "expected dir size to be at-least %d; got size: %d", expectedSize, size)
}

// Make sure that the timeout is actually being triggered (found this bug in PR#1489)
func TestDirDiskUsageTimeout(t *testing.T) {
as := assert.New(t)
fsInfo, err := NewFsInfo(Context{})
as.NoError(err)
dir, err := ioutil.TempDir(os.TempDir(), "")
as.NoError(err)
defer os.RemoveAll(dir)
dataSize := 1024 * 10000 //10000 KB. It is large to make sure it triggers the timeout
b := make([]byte, dataSize)
f, err := ioutil.TempFile(dir, "")
as.NoError(err)
as.NoError(ioutil.WriteFile(f.Name(), b, 0700))
_, err = fsInfo.GetDirDiskUsage(dir, time.Nanosecond)
as.Error(err)
}

func TestDirInodeUsage(t *testing.T) {
as := assert.New(t)
fsInfo, err := NewFsInfo(Context{})
Expand All @@ -141,23 +124,6 @@ func TestDirInodeUsage(t *testing.T) {
as.True(uint64(numFiles+1) == inodes, "expected inodes in dir to be %d; got inodes: %d", numFiles+1, inodes)
}

//make sure that the timeout is actually being triggered
func TestDirInodeUsageTimeout(t *testing.T) {
as := assert.New(t)
fsInfo, err := NewFsInfo(Context{})
as.NoError(err)
dir, err := ioutil.TempDir(os.TempDir(), "")
as.NoError(err)
defer os.RemoveAll(dir)
numFiles := 100000 //make sure we actually trigger the timeout
for i := 0; i < numFiles; i++ {
_, err := ioutil.TempFile(dir, "")
require.NoError(t, err)
}
_, err = fsInfo.GetDirInodeUsage(dir, time.Nanosecond)
as.Error(err)
}

var dmStatusTests = []struct {
dmStatus string
used uint64
Expand Down

0 comments on commit 835fd2e

Please sign in to comment.