search

linkedin / goavro

Apache License 2.0
971 stars 216 forks source link

go test fails #39

Closed leoneu closed 8 years ago

leoneu commented 8 years ago 
$ go test
--- FAIL: TestCodecDecoderUnionErrorYieldsName (0.00s)
        helpers_test.go:70: Actual: "cannot encode union (union): index must be between 0 and 1; read index: 2"; Expected to contain: "union (flubber)"
--- FAIL: TestFuzz_Panics (0.00s)
        fuzz_test.go:76: Error returned during fuzzing crasher[0]: cannot decode record (com.miguno.avro.twitter_schema): cannot decode string: cannot decode long: EOF
--- FAIL: TestNameWithDots (0.00s)
        name_test.go:60: The name portion of a fullname, record field names, and enum symbols must have second and remaining characters contain only [A-Za-z0-9_]
--- FAIL: TestRecordGetFieldSchema (0.00s)
        helpers_test.go:70: Actual: "The name portion of a fullname, record field names, and enum symbols must have second and remaining characters contain only [A-Za-z0-9_]"; Expected to contain: "no such field: no-such-field"
FAIL
exit status 1
FAIL    github.com/linkedin/goavro      0.015s
actgardner commented 8 years ago

A PR to fix this is here: https://github.com/linkedin/goavro/pull/48

karrick commented 8 years ago

I greatly appreciate the PR. I have been swamped on stuff at work, and I am glad you took the time to knock this out.