-
Notifications
You must be signed in to change notification settings - Fork 18k
cmd/go: go test stale code #12074
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Without details it is hard to advise, but one common case of this stake A simple way to verify this is to delete $GOPATH/pkg. Thanks Dave On Sat, 8 Aug 2015 10:01 Vellocet [email protected] wrote:
|
Good to know about deleting files. Both of the structs in question were generated by a protocol buffer compiler. |
CL https://golang.org/cl/22432 mentions this issue. |
It comes up every few months that we can't understand why the go command is rebuilding some package. Add diagnostics so that the go command can explain itself if asked. For golang#2775, golang#3506, golang#12074. Change-Id: I1c73b492589b49886bf31a8f9d05514adbd6ed70 Reviewed-on: https://go-review.googlesource.com/22432 Reviewed-by: Rob Pike <[email protected]>
Fairly certain this bug is fixed. |
I am running go 1.4.1
My os is ubuntu server 14.04
When running two different unit tests with "go test", I get unexpected results, as if it has not rebuilding packages that have changed.
This has only happened twice so far.
The first time go test complained about an import cycle that no longer existed. After poking at the code for a bit, it suddenly starts running again. I revert all of my changes and it continues to work fine.
The second time I had a test which json marshals a struct, one of the elements was missing in the marshaled string. When I changed the order of the elements of the struct, the element appeared in the string. When I reverted it, it continued to work.
I haven't been able to reproduce this bug as the code currently works fine, and only breaks temporarily. This has also only happened with those two tests. All of my other tests work as expected when I change the code.
The text was updated successfully, but these errors were encountered: