Added test for os minor version

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

Author: victormlg

libenv/sysinfo.c

@@ -3816,7 +3816,7 @@ static void SysOSVersionMinor(EvalContext *ctx)
     const char *version_id = OSReleaseGet(ctx, "VERSION_ID");
     const char *name = OSReleaseGet(ctx, "NAME");
 
-    char *minor;
+    char *minor = NULL;
     if (version_id != NULL)
     {
         Item *version_tuple = SplitString(version_id, '.');
@@ -3831,11 +3831,11 @@ static void SysOSVersionMinor(EvalContext *ctx)
             {
                 minor = version_tuple->next->name;
             }
+            free(version_tuple);
         }
-        free(version_tuple);
     }
 
-    if (NULL_OR_EMPTY(minor))
+    if (minor == NULL)
     {
         EvalContextVariablePutSpecial(ctx, SPECIAL_SCOPE_SYS,
                                       "os_version_minor", "Unknown",

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";
+}