fix tests for SLES and CoreOS

* include #15230

lib/ansible/module_utils/facts.py

@@ -126,8 +126,8 @@ class Facts(object):
                     ('/etc/alpine-release', 'Alpine'),
                     ('/etc/release', 'Solaris'),
                     ('/etc/arch-release', 'Archlinux'),
-                    ('/etc/SuSE-release', 'SuSE'),
                     ('/etc/os-release', 'SuSE'),
+                    ('/etc/SuSE-release', 'SuSE'),
                     ('/etc/gentoo-release', 'Gentoo'),
                     ('/etc/os-release', 'Debian'),
                     ('/etc/lsb-release', 'Mandriva'),
@@ -459,7 +459,7 @@ class Facts(object):
                                                 self.facts['distribution_release'] = release.groups()[0]
                                         elif 'enterprise' in data.lower() and 'VERSION_ID' in line:
                                              release = re.search('^VERSION_ID="?[0-9]+\.?([0-9]*)"?', line) # SLES doesn't got funny release names
-                                             if release.group(1):
+                                             if release and release.group(1):
                                                  release = release.group(1)
                                              else:
                                                  release = "0" # no minor number, so it is the first release
@@ -521,9 +521,11 @@ class Facts(object):
 
                             if self.facts['distribution'].lower() == 'coreos':
                                 data = get_file_content('/etc/coreos/update.conf')
-                                release = re.search("^GROUP=(.*)", data)
-                                if release:
-                                    self.facts['distribution_release'] = release.group(1).strip('"')
+                                if data:
+                                    release = re.search("^GROUP=(.*)", data)
+                                    if release:
+                                        self.facts['distribution_release'] = release.group(1).strip('"')
+
                     else:
                         self.facts['distribution'] = name
         machine_id = get_file_content("/var/lib/dbus/machine-id") or get_file_content("/etc/machine-id")

test/units/module_utils/test_distribution_version.py

@@ -113,9 +113,9 @@ class TestModuleUtilsFactsDistribution(unittest.TestCase):
                 "name": "SLES 11.3",
                 "input": {
                     "/etc/SuSE-release":"""
-                    SUSE Linux Enterprise Server 11 (x86_64)
-                    VERSION = 11
-                    PATCHLEVEL = 3
+SUSE Linux Enterprise Server 11 (x86_64)
+VERSION = 11
+PATCHLEVEL = 3
                     """
                 },
                 "platform.dist": ['SuSE', '11', 'x86_64'],
@@ -130,18 +130,18 @@ class TestModuleUtilsFactsDistribution(unittest.TestCase):
                 "name": "SLES 11.4",
                 "input": {
                     "/etc/SuSE-release":"""
-                    SUSE Linux Enterprise Server 11 (x86_64)
-                    VERSION = 11
-                    PATCHLEVEL = 4
+SUSE Linux Enterprise Server 11 (x86_64)
+VERSION = 11
+PATCHLEVEL = 4
                     """,
                     "/etc/os-release":"""
-                    NAME="SLES"
-                    VERSION="11.4"
-                    VERSION_ID="11.4"
-                    PRETTY_NAME="SUSE Linux Enterprise Server 11 SP4"
-                    ID="sles"
-                    ANSI_COLOR="0;32"
-                    CPE_NAME="cpe:/o:suse:sles:11:4"
+NAME="SLES"
+VERSION="11.4"
+VERSION_ID="11.4"
+PRETTY_NAME="SUSE Linux Enterprise Server 11 SP4"
+ID="sles"
+ANSI_COLOR="0;32"
+CPE_NAME="cpe:/o:suse:sles:11:4"
                     """,
                 },
                 "platform.dist": ['SuSE', '11', 'x86_64'],