cete icon indicating copy to clipboard operation
cete copied to clipboard

warning with `table.CountBetween`

Open SamuelMarks opened this issue 6 years ago • 2 comments

Many warnings appear, from use of this function: https://github.com/1lann/cete/blob/717790f29e5891f0a69604a1f3cdb16de8cce35d/count_test.go#L28-L152

E.g.:

=== RUN   TestCounting
2019/04/01 16:51:38 table.go:627: cete: warning: lower and upper bounds of table.CountBetween must be a string or Bounds. A count of 0 has been returned instead
2019/04/01 16:51:38 table.go:618: cete: warning: lower and upper bounds of table.CountBetween must be a string or Bounds. A count of 0 has been returned instead

SamuelMarks avatar Apr 01 '19 05:04 SamuelMarks

This is intentional and part of these tests https://github.com/1lann/cete/blob/717790f29e5891f0a69604a1f3cdb16de8cce35d/count_test.go#L94-L100

1lann avatar Apr 01 '19 09:04 1lann

Okay, maybe hide the warnings on test runs? - Because to the random developer it looks like failure, when it's an expected response.

You can also capture the stdout and stderr output, and assert that it equals what you expect.

SamuelMarks avatar Apr 02 '19 00:04 SamuelMarks