search

glycerine / goconvey

Other
1 stars 1 forks source link

go test failures #1

Open anacrolix opened 4 years ago

anacrolix commented 4 years ago 
# github.com/glycerine/goconvey/convey/reporting
../../../../pkg/mod/github.com/glycerine/goconvey@v0.0.0-20190410193231-58a59202ab31/convey/reporting/printer_test.go:35:2: Print format %3D has unknown verb D
../../../../pkg/mod/github.com/glycerine/goconvey@v0.0.0-20190410193231-58a59202ab31/convey/reporting/printer_test.go:72:2: Println format %3D has unknown verb D
../../../../pkg/mod/github.com/glycerine/goconvey@v0.0.0-20190410193231-58a59202ab31/convey/reporting/reporter_test.go:59:3: Errorf call needs 1 arg but has 2 args
# github.com/glycerine/goconvey/convey/assertions/oglematchers_test
../../../../pkg/mod/github.com/glycerine/goconvey@v0.0.0-20190410193231-58a59202ab31/convey/assertions/oglematchers/equals_test.go:3334:11: Sprintf format %v arg func1 is a func value, not called
../../../../pkg/mod/github.com/glycerine/goconvey@v0.0.0-20190410193231-58a59202ab31/convey/assertions/oglematchers/identical_to_test.go:167:11: Sprintf format %v arg o1 is a func value, not called
glycerine commented 4 years ago

happy to accept a pull request. These are trivial fixes. Also it doesn't really have any module support at the moment, so if you want to fix that too while you are at, please go ahead.

anacrolix commented 4 years ago

I'm finding that go test all fails with downstream projects, but I'm not invested enough to fix it myself.