Bug report: flow environment list doesnt output anything

[brugh]

Priority

(Medium) I'm annoyed but I'll live

Description

the v7.0.0 of m365 doesn't output any information when running

m365 flow environment list

When adding output -o text it outputs the correct information but no output option or -o json doesn't work.

changing line 21 in dist/m365/flow/commands/environment/environment-list.js from

            if (args.options.output !== 'json' && res.length > 0) {

to

            if (res.length > 0) {

fixes the issue

Steps to reproduce

m365 login -t browser
m365 flow environment list

Expected results

a list of json formatted environments

Actual results

no output at all

Diagnostics

No response

CLI for Microsoft 365 version

v7.0.0

nodejs version

v18.17.1

Operating system (environment)

Windows

Shell

PowerShell

cli doctor

No response

Additional Info

No response

[martinlingstuyl]

Hi @brugh, thanks for bringing this to our attention, and for researching a fix to it as well. Would you like to submit a Pull Request for it?

[milanholemans]

The correct resolution should be to move the logger.log (line 33) out of the if condition to line 34.
Let's make sure that we add an extra test so this doesn't happen again.

cli-microsoft365/src/m365/flow/commands/environment/environment-list.ts

Lines 25 to 35 in 129c6c7

	try {
	const res = await odata.getAllItems<{ name: string, displayName: string; properties: { displayName: string } }>(`${this.resource}providers/Microsoft.ProcessSimple/environments?api-version=2016-11-01`);
	if (args.options.output !== 'json' && res.length > 0) {
	res.forEach(e => {
	e.displayName = e.properties.displayName;
	});
	await logger.log(res);
	}
	}