3 commits

Author	SHA1	Message	Date
Adam Leff	f7c8c646a9	Extract Compliance::API version parsing to separate method (#1931) ... For cleanliness and ease of testing, I've moved the logic that parses the server version from the compliance config to a separate method. Signed-off-by: Adam Leff <adam@leff.co>	2017-06-23 08:29:50 -07:00
Adam Leff	6668bf15ea	Fix detection of Automate pre-0.8.x in Compliance::API (#1922) ... The is_automate_server_pre_080? and is_automate_server_080_and_later? methods needed some fixing. The Compliance configuration could have a "version" key that was not nil but was an empty hash, indicating that it came from a pre-0.8.x Automate server. What we really need to look for is config['version']['version'] being nil?. Signed-off-by: Adam Leff <adam@leff.co>	2017-06-13 10:05:09 +02:00
Adam Leff	dda24b9f98	Fix compliance uploads when version is not present ... The Compliance::API.version method could potentially return a hash containing no "version" key but would return an empty hash upon any expected failure. Downstream callers of the Compliance::API.version method were looking for a "version" key to always be present when, in some cases, it would not be. This change ensures that if a version is not available, there is no "version" key in the hash, and downstream callers of this method have been changed to check for nil instead of empty. Signed-off-by: Adam Leff <adam@leff.co>	2017-06-06 14:49:25 +02:00