fga model test --verbose output should be easy to parse

[jmickeyd]

Currently when running tests with the --verbose flag, a nicely parsable json is immediately followed by the plain text summary, making the whole thing harder to parse.

Example:

$ fga model test --verbose --tests tests.fga.yaml | jq .
[ json blob ... ]
jq: parse error: Invalid numeric literal at line 2, column 2

Either one of the blobs should be send to stderr, the pretty print text should be suppressed, or at a minimum, a flag be added to optionally suppress the pretty print.

[jmickeyd]

I'm willing to send a patch, it should be trivially small, but I'm not sure what you consider a breaking change for things like this. I.e. is the last option (adding a new flag) the preferable one?

[jmickeyd]

Actually, it looks like due to an effort to set the exit code correctly, the pretty print is already suppressed when there are test failures,

cli/cmd/model/test.go

Lines 77 to 79 in 36091f0

	if !passing {
	os.Exit(1)
	}