search

googleinterns / ocsp-response-linter

Apache License 2.0
6 stars 1 forks source link

Added tests for ocsp_status.go #12

Closed tnarg999 closed 4 years ago

tnarg999 commented 4 years ago

Following Arinze's suggestion, this is a branch of a branch so that PRs can be smaller and I don't run into merge conflicts.

The main change here is in the ocsp_status_test.go file.

tnarg999 commented 4 years ago

Oh the go tests don't run because this isn't a PR into master. So I could change the go.yml file to make it so that it does run the test, but then I would have to remove it later, and doing that each time I want to do a PR into a non-master branch seems messy.

Please trust that I have run the tests and they do work :). Also when this branch gets merged into ocsp_status_tests, it will trigger the tests there.