Fix error at build import time.

  * Defconfig errors variable name was wrong (typo).
  * Add test.

2 files changed, 29 insertions, 1 deletions

diff --git a/app/utils/docimport.py b/app/utils/docimport.py
index df70a01..08d4575 100644
--- a/app/utils/docimport.py
+++ b/app/utils/docimport.py
@@ -333,7 +333,7 @@ def _parse_build_data(data_file, job, kernel, defconfig_dir):
                 models.BUILD_PLATFORM_KEY, [])
             defconfig_doc.build_time = data_pop(models.BUILD_TIME_KEY, 0)
             defconfig_doc.dtb_dir = data_pop(models.DTB_DIR_KEY, None)
-            defconfig_doc.errros = data_pop(models.BUILD_ERRORS_KEY, 0)
+            defconfig_doc.errors = data_pop(models.BUILD_ERRORS_KEY, 0)
             defconfig_doc.file_server_resource = data_pop(
                 models.FILE_SERVER_RESOURCE_KEY, None)
             defconfig_doc.file_server_url = data_pop(
diff --git a/app/utils/tests/test_docimport.py b/app/utils/tests/test_docimport.py
index de559a7..7349f25 100644
--- a/app/utils/tests/test_docimport.py
+++ b/app/utils/tests/test_docimport.py
@@ -181,6 +181,34 @@ class TestDocImport(unittest.TestCase):
             defconf_doc.defconfig_full, "defoo_confbar+CONFIG_TEST=y")
         self.assertEqual(defconf_doc.defconfig, "defoo_confbar")
 
+    def test_parse_and_update_build_metadata_errors(self):
+        meta_content = {
+            "arch": "arm",
+            "defconfig": "defoo_confbar",
+            "job": "job",
+            "kernel": "kernel",
+            "build_errors": 3,
+            "build_warnings": 1,
+        }
+
+        try:
+            fake_meta = tempfile.NamedTemporaryFile(delete=False)
+            with open(fake_meta.name, 'w') as w_file:
+                w_file.write(json.dumps(meta_content))
+
+            defconf_doc = docimport._parse_build_data(
+                fake_meta.name,
+                "job",
+                "kernel",
+                "arm-defoo_confbar"
+            )
+        finally:
+            os.unlink(fake_meta.name)
+
+        self.assertIsInstance(defconf_doc, mdefconfig.DefconfigDocument)
+        self.assertEqual(defconf_doc.errors, 3)
+        self.assertEqual(defconf_doc.warnings, 1)
+
     def test_extrapolate_defconfig_full_non_valid(self):
         kconfig_fragments = "foo-CONFIG.bar"
         defconfig = "defconfig"