HomePhorge

(#12349) Order stderr output to fix acceptance test for `module list`
3365c854b959Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

(#12349) Order stderr output to fix acceptance test for module list

The stdout was ordered, but not the stderr, so the acceptance tests that
passed on my system failed when run by CI.

To order the stderr output I changed the format of the method that gets
unmet dependencies to be a hash that contains the dependency module name
in addition to the error message so that I could sort by that name.

There's also some additional documentation updates to the list command
to show example error output for unmet dependencies.

I changed the unsatisfied_dependencies method to unmet_dependencies
since in future changes that's how dependency problems will be described
to the user, and it's shorter.

Details

Provenance
Matt Robinson <matt@puppetlabs.com>Authored on
vanmeeuwenPushed on Jun 2 2015, 2:22 PM
Parents
rPUe80e514b7883: Merge pull request #232 from kscherer/ticket/2.7.x/7485-zypper-0.6-support
Branches
Unknown
Tags
Unknown

Event Timeline

Matt Robinson <matt@puppetlabs.com> committed rPU3365c854b959: (#12349) Order stderr output to fix acceptance test for `module list` (authored by Matt Robinson <matt@puppetlabs.com>).Feb 3 2012, 10:37 PM