uuid icon indicating copy to clipboard operation
uuid copied to clipboard

Update documentation and code for Invalid Variant

Open toothrot opened this issue 4 years ago • 0 comments

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-L67

The variant is never checked in the unit tests in an error scenario: https://github.com/google/uuid/blob/44b5fee7c49cf3bcdf723f106b36d56ef13ccc88/uuid_test.go#L99-L101

toothrot avatar Sep 21 '21 22:09 toothrot