Hyper-optimized Telemetry: Two unit tests are not following specifications in instructions for encoding
- Dominant language
- C#
- Stars
- 427
- Forks
- 383
- Avg merge
- 2d 2h
- Merged PRs (30d)
- 5
Description
On the Hyper-optimized Telemetry exercise, the instructions specify a table of ranges and target types for determining which type should be used when converting a long to the smallest integral type. Of particular interest in this issue is the range 0 - 65535 which should be represented with a `ushort`:

However, two unit tests are not following this specification. First one is `ToBuffer_upper_short`:

Since the supplied value 32767 is between 0 and 65353, this should be an unsigned short. Specifically `0x2, 0xff, 0x7f`.
The second unit tests not following the instructions is `ToBuffer_Zero`. It has the exact same problem, expecting 0 to be a signed short, while the instructions specify that it should be unsigned.

Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.