Skip to content
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

Add unit tests that increase code coverage (Go/Golang/SenML) #3

Open
x448 opened this issue Nov 27, 2019 · 0 comments
Open

Add unit tests that increase code coverage (Go/Golang/SenML) #3

x448 opened this issue Nov 27, 2019 · 0 comments
Labels
good first issue Good for newcomers

Comments

@x448
Copy link
Owner

x448 commented Nov 27, 2019

A "good first issue" with two programs at 0% coverage (inherited from cisco/senml).

Code coverage using go test -cover shows:

  • 80.7% for senml.go
  • 0% for senmlCat.go
  • 0% for senmlServer.go

Add unit tests that improve code coverage. If possible, use example data directly from standards docs like RFC 8428 or gathered from sensors (as long as data doesn't violate privacy or laws).

Hint: Tests involving io.Reader or io.Writer can use iotest to increase coverage after the normal path is covered.

Before submitting a pull request, please reply here with your proposed changes.

@x448 x448 added the good first issue Good for newcomers label Nov 27, 2019
@x448 x448 changed the title Add unit tests that increase code coverage Add unit tests that increase code coverage (Go/Golang/SenML) Nov 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant