* Use assert.NoError instead of assert.Nil
This gives nicer error messages.
* Swap arguments to assert.Equal
The expectation comes first. Otherwise, error messages will be
misleading.
An example:
=== RUN TestDeleteWithIncorrectRepoForDeleteCaches
Error: authentication token not found for host github.com
delete_test.go:56:
Error Trace: /build/source/cmd/delete_test.go:56
Error: Should be true
Test: TestDeleteWithIncorrectRepoForDeleteCaches
Messages: 1 unmatched mocks: https://api.github.com/repos/testOrg/testRepo/actions/caches?key=cacheName
delete_test.go:59:
Error Trace: /build/source/cmd/delete_test.go:59
Error: Not equal:
expected: "authentication token not found for host github.com"
actual : "The given repo does not exist."
Diff:
--- Expected
+++ Actual
@@ -1 +1 @@
-authentication token not found for host github.com
+The given repo does not exist.
Test: TestDeleteWithIncorrectRepoForDeleteCaches
--- FAIL: TestDeleteWithIncorrectRepoForDeleteCaches (0.00s)
* Use assert.Error instead of assert.NotNil
This gives nicer error messages.
* Use assert.ErrorAs and assert.ErrorContains
This simplifies the assertions and potentially gives better error
messages.
* Use require instead of assert and use assert.NotNil as guard
This is to prevent panics in tests, when things get accessed which
shouldn't be nil.
* Completed List cmd and added API calls
* Minor comments and add delete code to pass linting
* Typo in descriptions
* Minor comments
* Validations
* Validations-1
* improved branch flag validation
* removed build
* working after refactory with bad names
* Command working, test not working
* Corrected creation of service
* Finalized structure using service
* Deleted tests
* cleanup
* cleanup
* cleanup
* removed space with tab
* aligned types in model.go
* Update model.go
* resolved comments
* Refactor
* removed long descriptions
* Working incomplete tests
* Completed tests
* cleanup
* checks
* PR comments
* PR comments
* minor comment issue
* minor comment issue
* Added test cases for Delete
* updated tests to work with workflow
* Added missing error condition
* Updated tests to support new option service
* Improved eror handling for list
* Fixed test case
* Improved error handling
* Error handling and test cases for delete API calls
* Added test case for user confirmation delete.
* Removed unused import from test
* Fixed test case for error scenario
* Upgraded go-gh
* reusing rest client error
* Fix for failing windows test cases
* help cmd removed when cache isnt present on delete
* Pretty print ratio and space between cols modified
* Error handling wrapping
* Reverted back error message after silencing help
Co-authored-by: t-dedah <t-dedah@github.com>
Co-authored-by: Deepak Dahiya <59823596+t-dedah@users.noreply.github.com>
