aztfmod / rovergo

The next version of Rover, the command line tool for Azure CAF Landingzones. Developed in Go
MIT License
10 stars 7 forks source link

bug: cobra.CheckError crashes tests #113

Closed sebastus closed 3 years ago

sebastus commented 3 years ago

When running tests (e.g. go tests ./...) if a cobra.CheckError faults, the test is terminated.

The hope is that this task results in evaluation of cobra.CheckError instances that might impact existing and planned tests. Prioritized instances to be replaced with code that results in tests running to completion with a failed outcome.

Expected Behavior

It would be better if a failure were made available to report up to the test results.

Actual Behavior

A test crashes to the O.S. - stopping any further tests from executing and leaving system state incomplete.

sebastus commented 3 years ago

we're fixing them as we come across them.