It would be nice if the uuid.Invalid documentation or the (uuid.UUID).Variant() documentation mentioned this in addition to the readme, as it was confusing until I saw 3943dc54a8bab7acdb1584b25ee8bd4fb22750b6. I had read through the documentation, code, and tests before the readme of the repository, so there was a little wasted effort and confusion.
It would be nice if the
uuid.Invalid
documentation or the(uuid.UUID).Variant()
documentation mentioned this in addition to the readme, as it was confusing until I saw 3943dc54a8bab7acdb1584b25ee8bd4fb22750b6. I had read through the documentation, code, and tests before the readme of the repository, so there was a little wasted effort and confusion.Specifically, the unit tests are a little misleading, as
uuid.Parse()
returns an error in the Invalid cases: https://github.com/google/uuid/blob/44b5fee7c49cf3bcdf723f106b36d56ef13ccc88/uuid_test.go#L62-L67The variant is never checked in the unit tests in an error scenario: https://github.com/google/uuid/blob/44b5fee7c49cf3bcdf723f106b36d56ef13ccc88/uuid_test.go#L99-L101