Added test for os minor version

Signed-off-by: Victor Moene <victor.moene@northern.tech>

Author: victormlg

libenv/sysinfo.c

@@ -3811,31 +3811,9 @@ static const char *OSReleaseGet(EvalContext *ctx, const char *field)
     #endif
 }
 
-static void SysOSVersionMinor(EvalContext *ctx)
+static void OSVersionMinorPut(EvalContext *ctx, char *minor)
 {
-    const char *version_id = OSReleaseGet(ctx, "VERSION_ID");
-    const char *name = OSReleaseGet(ctx, "NAME");
-
-    char *minor;
-    if (version_id != NULL)
-    {
-        Item *version_tuple = SplitString(version_id, '.');
-
-        if (version_tuple != NULL)
-        {
-            if (name != NULL && (StringStartsWith(name, "solaris") || StringStartsWith(name, "sunos")))
-            {
-                minor = version_tuple->name;
-            }
-            else if (version_tuple->next != NULL)
-            {
-                minor = version_tuple->next->name;
-            }
-        }
-        free(version_tuple);
-    }
-
-    if (NULL_OR_EMPTY(minor))
+    if (minor == NULL)
     {
         EvalContextVariablePutSpecial(ctx, SPECIAL_SCOPE_SYS,
                                       "os_version_minor", "Unknown",
@@ -3850,6 +3828,37 @@ static void SysOSVersionMinor(EvalContext *ctx)
     }
 }
 
+static void SysOSVersionMinor(EvalContext *ctx)
+{
+    const char *version_id = OSReleaseGet(ctx, "VERSION_ID");
+    const char *name = OSReleaseGet(ctx, "NAME");
+    if (version_id == NULL)
+    {
+        return OSVersionMinorPut(ctx, NULL);
+    }
+    if (name == NULL)
+    {
+        return OSVersionMinorPut(ctx, NULL);
+    }
+    Item *version_tuple = SplitString(version_id, '.');
+    if (version_tuple == NULL)
+    {
+        return OSVersionMinorPut(ctx, NULL);
+    }
+    if (name != NULL && (StringStartsWith(name, "solaris") || StringStartsWith(name, "sunos")))
+    {
+        OSVersionMinorPut(ctx, version_tuple->name);
+        return DeleteItemList(version_tuple);
+    }
+    if (version_tuple->next == NULL)
+    {
+        OSVersionMinorPut(ctx, NULL);
+        return DeleteItemList(version_tuple);
+    }
+    OSVersionMinorPut(ctx, version_tuple->next->name);
+    return DeleteItemList(version_tuple);
+}
+
 
 /*****************************************************************************/
 

tests/acceptance/01_vars/01_basic/os_version_minor.cf

@@ -0,0 +1,74 @@
+body common control
+{
+  bundlesequence => { "test", "check" };
+}
+
+bundle agent test
+{
+
+  vars:
+    # Platforms to test
+    any::
+      "platforms"
+        slist => { "debian", "ubuntu", "redhat", "rhel", "centos", "fedora",
+                   "aix", "hpux", "suse", "opensuse", "opensuse_leap", "sles",
+                   "solaris", "sunos", "windows", "freebsd", "macos" };
+
+
+    # Regex matching current platforms OS-class with version numbers
+    !solaris&!sunos::
+      "class_regex"
+        string => format("^(%s)_[0-9]+_[0-9]+$", join("|", "platforms"));
+    solaris|sunos::
+      "class_regex"
+        string => format("^(%s)_[0-9]+$", join("|", "platforms"));
+
+    # Regex to extract major version number from OS-class
+    # Edge cases:
+    #  - On Solaris/SunOS major version comes second
+    #    E.g. Solaris 11 has class "solaris_5_11"
+    any::
+      "extract_regex"
+        string => ifelse("solaris|sunos", "^[a-z]+_([0-9]+)$",
+                         "opensuse_leap", "^[a-z_]+_[0-9]+_([0-9]+$)",
+                                          "^[a-z]+_[0-9]+_([0-9]+$)");
+
+    # Find OS-class with version numbers using regex
+    any::
+      "os_class"
+        string => nth(classesmatching("$(class_regex)"), "0");
+
+    # Get extracted major version number
+    any::
+      "expected"
+        string => nth("version_number", "1");
+
+  classes:
+    any::
+      "regextract_success"
+        expression => regextract("$(extract_regex)", "$(os_class)", "version_number");
+}
+
+bundle agent check
+{
+  vars:
+    any::
+      "defined_classes"
+        slist => classesmatching(".*");
+
+  classes:
+    any::
+      "passed"
+        expression => strcmp("$(test.expected)", "$(sys.os_version_minor)");
+
+  reports:
+    DEBUG::
+      "Version number extracted from class: $(test.os_class)";
+      "Defined classes: $(defined_classes)";
+      "$(this.promise_filename) Expected: $(test.expected)";
+      "$(this.promise_filename) Found: $(sys.os_version_minor)";
+    passed::
+      "$(this.promise_filename) Pass";
+    !passed::
+      "$(this.promise_filename) FAIL";
+}