tree b2a1ed2506d7ef58a59c5fcd7e908493d48ae2b2
parent bda55230c457d8271c577c0da94d4b94e3543dd4
author Rob Findley <rfindley@google.com> 1715603438 +0000
committer Robert Findley <rfindley@google.com> 1716420938 +0000

start,internal/upload: add two tests for concurrent upload

Add two tests, one for telemetry.Start and an other for upload.Run,
which execute the upload concurrently.

The test for telemetry.Start succeeds, due to the concurrency safety
from the exclusive acquisition of the upload.token file. The test for
upload.Run results in incorrect upload counts and occasional invalid
report json (due to write shearing). Despite the upload.token guard,
upload.Run should be more concurrency safe, since there is still a race
condition when the upload.token is released. A subsequent CL will add
more safeguards.

For golang/go#65970

Change-Id: Ic7e57b1ee794a58340901289930250bf5114fdf6
Reviewed-on: https://go-review.googlesource.com/c/telemetry/+/586141
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>