ManageIQ / httpd_configmap_generator

Scripts used in httpd containers for configuring authentication
Apache License 2.0
3 stars 19 forks source link

[Ruby 2.5] Sort the auth types so tests can predict the order #39

Closed jrafanie closed 5 years ago

jrafanie commented 5 years ago

The code walks constants to build the list of available auth types and apparently this list is in a different sequence on ruby 2.5.

We can sort the types for the CLI's display so you can reliably test it.

jrafanie commented 5 years ago

cc @juliancheal

jvlcek commented 5 years ago

LGTM @jrafanie 👍

jrafanie commented 5 years ago

Fixes the test failure on 2.5 seen in PR #40

jrafanie commented 5 years ago

@abellotti Can you review/merge?

abellotti commented 5 years ago

Thanks @jrafanie for fixing this. LGTM!! 👍