Merge pull request #367 from fkautz/pr_out_updating_iodine

master
Frederick F. Kautz IV 10 years ago
commit d6507305ac
  1. 2
      Godeps/Godeps.json
  2. 2
      Godeps/_workspace/src/github.com/minio-io/iodine/iodine.go
  3. 2
      Godeps/_workspace/src/github.com/minio-io/iodine/iodine_test.go
  4. 2
      Godeps/_workspace/src/gopkg.in/check.v1/benchmark_test.go
  5. 145
      Godeps/_workspace/src/gopkg.in/check.v1/printer_test.go
  6. 4
      Godeps/_workspace/src/gopkg.in/check.v1/run_test.go
  7. 4
      pkg/storage/donut/donut.go

2
Godeps/Godeps.json generated vendored

@ -20,7 +20,7 @@
}, },
{ {
"ImportPath": "github.com/minio-io/iodine", "ImportPath": "github.com/minio-io/iodine",
"Rev": "f2dbd51c3b4f8530a9b8e2dbc32788175df2fa5c" "Rev": "d7c4850883ade19431e3808e9c712d3d3e69cdcc"
}, },
{ {
"ImportPath": "gopkg.in/check.v1", "ImportPath": "gopkg.in/check.v1",

@ -44,7 +44,7 @@ type StackEntry struct {
// Wrap an error, turning it into an iodine error. // Wrap an error, turning it into an iodine error.
// Adds an initial stack trace. // Adds an initial stack trace.
func Wrap(err error, data map[string]string) *Error { func New(err error, data map[string]string) *Error {
entry := createStackEntry() entry := createStackEntry()
for k, v := range data { for k, v := range data {
entry.Data[k] = v entry.Data[k] = v

@ -25,7 +25,7 @@ import (
) )
func TestIodine(t *testing.T) { func TestIodine(t *testing.T) {
iodineError := Wrap(errors.New("Hello"), nil) iodineError := New(errors.New("Hello"), nil)
iodineError.Annotate(nil) iodineError.Annotate(nil)
iodineError.Annotate(nil) iodineError.Annotate(nil)
iodineError.Annotate(nil) iodineError.Annotate(nil)

@ -3,8 +3,8 @@
package check_test package check_test
import ( import (
. "gopkg.in/check.v1"
"time" "time"
. "gopkg.in/check.v1"
) )
var benchmarkS = Suite(&BenchmarkS{}) var benchmarkS = Suite(&BenchmarkS{})

@ -1,7 +1,7 @@
package check_test package check_test
import ( import (
. "gopkg.in/check.v1" . "gopkg.in/check.v1"
) )
var _ = Suite(&PrinterS{}) var _ = Suite(&PrinterS{})
@ -9,101 +9,96 @@ var _ = Suite(&PrinterS{})
type PrinterS struct{} type PrinterS struct{}
func (s *PrinterS) TestCountSuite(c *C) { func (s *PrinterS) TestCountSuite(c *C) {
suitesRun += 1 suitesRun += 1
} }
var printTestFuncLine int var printTestFuncLine int
func init() { func init() {
printTestFuncLine = getMyLine() + 3 printTestFuncLine = getMyLine() + 3
} }
func printTestFunc() { func printTestFunc() {
println(1) // Comment1 println(1) // Comment1
if 2 == 2 { // Comment2 if 2 == 2 { // Comment2
println(3) // Comment3 println(3) // Comment3
} }
switch 5 { switch 5 {
case 6: case 6: println(6) // Comment6
println(6) // Comment6 println(7)
println(7) }
} switch interface{}(9).(type) {// Comment9
switch interface{}(9).(type) { // Comment9 case int: println(10)
case int: println(11)
println(10) }
println(11) select {
} case <-(chan bool)(nil): println(14)
select { println(15)
case <-(chan bool)(nil): default: println(16)
println(14) println(17)
println(15) }
default: println(19,
println(16) 20)
println(17) _ = func() { println(21)
} println(22)
println(19, }
20) println(24, func() {
_ = func() { println(25)
println(21) })
println(22) // Leading comment
} // with multiple lines.
println(24, func() { println(29) // Comment29
println(25)
})
// Leading comment
// with multiple lines.
println(29) // Comment29
} }
var printLineTests = []struct { var printLineTests = []struct {
line int line int
output string output string
}{ }{
{1, "println(1) // Comment1"}, {1, "println(1) // Comment1"},
{2, "if 2 == 2 { // Comment2\n ...\n}"}, {2, "if 2 == 2 { // Comment2\n ...\n}"},
{3, "println(3) // Comment3"}, {3, "println(3) // Comment3"},
{5, "switch 5 {\n...\n}"}, {5, "switch 5 {\n...\n}"},
{6, "case 6:\n println(6) // Comment6\n ..."}, {6, "case 6:\n println(6) // Comment6\n ..."},
{7, "println(7)"}, {7, "println(7)"},
{9, "switch interface{}(9).(type) { // Comment9\n...\n}"}, {9, "switch interface{}(9).(type) { // Comment9\n...\n}"},
{10, "case int:\n println(10)\n ..."}, {10, "case int:\n println(10)\n ..."},
{14, "case <-(chan bool)(nil):\n println(14)\n ..."}, {14, "case <-(chan bool)(nil):\n println(14)\n ..."},
{15, "println(15)"}, {15, "println(15)"},
{16, "default:\n println(16)\n ..."}, {16, "default:\n println(16)\n ..."},
{17, "println(17)"}, {17, "println(17)"},
{19, "println(19,\n 20)"}, {19, "println(19,\n 20)"},
{20, "println(19,\n 20)"}, {20, "println(19,\n 20)"},
{21, "_ = func() {\n println(21)\n println(22)\n}"}, {21, "_ = func() {\n println(21)\n println(22)\n}"},
{22, "println(22)"}, {22, "println(22)"},
{24, "println(24, func() {\n println(25)\n})"}, {24, "println(24, func() {\n println(25)\n})"},
{25, "println(25)"}, {25, "println(25)"},
{26, "println(24, func() {\n println(25)\n})"}, {26, "println(24, func() {\n println(25)\n})"},
{29, "// Leading comment\n// with multiple lines.\nprintln(29) // Comment29"}, {29, "// Leading comment\n// with multiple lines.\nprintln(29) // Comment29"},
} }
func (s *PrinterS) TestPrintLine(c *C) { func (s *PrinterS) TestPrintLine(c *C) {
for _, test := range printLineTests { for _, test := range printLineTests {
output, err := PrintLine("printer_test.go", printTestFuncLine+test.line) output, err := PrintLine("printer_test.go", printTestFuncLine+test.line)
c.Assert(err, IsNil) c.Assert(err, IsNil)
c.Assert(output, Equals, test.output) c.Assert(output, Equals, test.output)
} }
} }
var indentTests = []struct { var indentTests = []struct {
in, out string in, out string
}{ }{
{"", ""}, {"", ""},
{"\n", "\n"}, {"\n", "\n"},
{"a", ">>>a"}, {"a", ">>>a"},
{"a\n", ">>>a\n"}, {"a\n", ">>>a\n"},
{"a\nb", ">>>a\n>>>b"}, {"a\nb", ">>>a\n>>>b"},
{" ", ">>> "}, {" ", ">>> "},
} }
func (s *PrinterS) TestIndent(c *C) { func (s *PrinterS) TestIndent(c *C) {
for _, test := range indentTests { for _, test := range indentTests {
out := Indent(test.in, ">>>") out := Indent(test.in, ">>>")
c.Assert(out, Equals, test.out) c.Assert(out, Equals, test.out)
} }
} }

@ -400,7 +400,7 @@ func (s *RunS) TestStreamModeWithMiss(c *C) {
// ----------------------------------------------------------------------- // -----------------------------------------------------------------------
// Verify that that the keep work dir request indeed does so. // Verify that that the keep work dir request indeed does so.
type WorkDirSuite struct{} type WorkDirSuite struct {}
func (s *WorkDirSuite) Test(c *C) { func (s *WorkDirSuite) Test(c *C) {
c.MkDir() c.MkDir()
@ -411,7 +411,7 @@ func (s *RunS) TestKeepWorkDir(c *C) {
runConf := RunConf{Output: &output, Verbose: true, KeepWorkDir: true} runConf := RunConf{Output: &output, Verbose: true, KeepWorkDir: true}
result := Run(&WorkDirSuite{}, &runConf) result := Run(&WorkDirSuite{}, &runConf)
c.Assert(result.String(), Matches, ".*\nWORK="+result.WorkDir) c.Assert(result.String(), Matches, ".*\nWORK=" + result.WorkDir)
stat, err := os.Stat(result.WorkDir) stat, err := os.Stat(result.WorkDir)
c.Assert(err, IsNil) c.Assert(err, IsNil)

@ -26,7 +26,7 @@ func NewDonut(root string) (Donut, *iodine.Error) {
for nodeID, node := range nodes { for nodeID, node := range nodes {
bucketIDs, err := node.GetBuckets() bucketIDs, err := node.GetBuckets()
if err != nil { if err != nil {
return nil, iodine.Wrap(err, map[string]string{"root": root}) return nil, iodine.New(err, map[string]string{"root": root})
} }
for _, bucketID := range bucketIDs { for _, bucketID := range bucketIDs {
tokens := strings.Split(bucketID, ":") tokens := strings.Split(bucketID, ":")
@ -38,7 +38,7 @@ func NewDonut(root string) (Donut, *iodine.Error) {
driver.buckets[tokens[0]] = bucket driver.buckets[tokens[0]] = bucket
} }
if err = driver.buckets[tokens[0]].AddNode(nodeID, bucketID); err != nil { if err = driver.buckets[tokens[0]].AddNode(nodeID, bucketID); err != nil {
return nil, iodine.Wrap(err, map[string]string{"root": root}) return nil, iodine.New(err, map[string]string{"root": root})
} }
} }
} }

Loading…
Cancel
Save