tree 1df1171e2ba29bbf19e21db92f26e657f9c9c391
parent 2756c524cc1ca1284d912b9c5ee3b0092ff4459b
author Emmanuel T Odeke <emmanuel@orijtech.com> 1564874018 -0700
committer Emmanuel Odeke <emm.odeke@gmail.com> 1565034647 +0000

internal/lsp/telemetry/ocagent: add convertAnnotation tests

* Adds tests for 'convertAnnotation'.
* Ensures that converting an empty string "" to
a truncatable string returns nil, to save bandwidth. However,
in the future, we should perhaps allow empty strings to
be serialized if say "emptyAllowed" is set.
* Caught the case where convertAttribute hadn't type
switched on "int"
* Provides 40.5% test coverage for ocagent.go

More tests for the other functions shall follow in later CLs.

Updates CL 186679

Change-Id: Ie9b46b0b320339ed79cd136fff536ccfcfbeb9e7
Reviewed-on: https://go-review.googlesource.com/c/tools/+/188877
Run-TryBot: Emmanuel Odeke <emm.odeke@gmail.com>
Reviewed-by: Ian Cottrell <iancottrell@google.com>