Increase test coverage for KernelBuilder

Author: major

tests/test_kernelbuilder.py

@@ -137,3 +137,36 @@ def kbuilder_mktgz_silent(self, *args, **kwargs):
         """Run self.kbuilder.mktgz with disabled output"""
         with mock.patch('sys.stdout'):
             return self.kbuilder.mktgz(*args, **kwargs)
+
+    def test_extra_make_args(self):
+        """Ensure KernelBuilder handles extra_make_args properly"""
+        extra_make_args_example = '-j10'
+        kbuilder = kernelbuilder.KernelBuilder(
+            self.tmpdir,
+            self.tmpconfig.name,
+            extra_make_args=extra_make_args_example
+        )
+        self.assertEqual(kbuilder.extra_make_args, [extra_make_args_example])
+
+    def test_getrelease(self):
+        """Ensure get_release() handles a valid kernel version string"""
+        mock_prepare = mock.patch("skt.kernelbuilder.KernelBuilder.prepare")
+
+        kernel_version = '4.17.0-rc6+\n'
+        mock_popen = self.ctx_popen
+        self.m_popen.communicate = Mock(return_value=(kernel_version, None))
+
+        with mock_popen, mock_prepare:
+            result = self.kbuilder.getrelease()
+            self.assertEqual(kernel_version.strip(), result)
+
+    def test_getrelease_regex_fail(self):
+        """Ensure get_release() fails if the regex doesn't match"""
+        mock_prepare = mock.patch("skt.kernelbuilder.KernelBuilder.prepare")
+
+        mock_popen = self.ctx_popen
+        self.m_popen.communicate = Mock(return_value=('this_is_silly', None))
+
+        with mock_popen, mock_prepare:
+            with self.assertRaises(Exception):
+                self.kbuilder.getrelease()