When package's description is not defined, use NULL (None) instead of empty string.
authorTomas Mlcoch <tmlcoch@redhat.com>
Tue, 29 Oct 2013 14:28:34 +0000 (15:28 +0100)
committerTomas Mlcoch <tmlcoch@redhat.com>
Tue, 29 Oct 2013 14:28:34 +0000 (15:28 +0100)
src/parsehdr.c
tests/python/tests/test_package.py

index de21a61..9e3c017 100644 (file)
@@ -147,7 +147,7 @@ cr_package_from_header(Header hdr, gint64 mtime, gint64 size,
 
     pkg->release = cr_safe_string_chunk_insert(pkg->chunk, headerGetString(hdr, RPMTAG_RELEASE));
     pkg->summary = cr_safe_string_chunk_insert(pkg->chunk, headerGetString(hdr, RPMTAG_SUMMARY));
-    pkg->description = cr_safe_string_chunk_insert(pkg->chunk, headerGetString(hdr, RPMTAG_DESCRIPTION));
+    pkg->description = cr_safe_string_chunk_insert_null(pkg->chunk, headerGetString(hdr, RPMTAG_DESCRIPTION));
     pkg->url = cr_safe_string_chunk_insert(pkg->chunk, headerGetString(hdr, RPMTAG_URL));
     pkg->time_file = mtime;
     if (headerGet(hdr, RPMTAG_BUILDTIME, td, flags)) {
index e79696d..3f2b239 100644 (file)
@@ -14,7 +14,7 @@ class TestCasePackage(unittest.TestCase):
         self.assertEqual(pkg.epoch, "0")
         self.assertEqual(pkg.release, "0")
         self.assertEqual(pkg.summary, '""')
-        self.assertEqual(pkg.description, "")
+        self.assertEqual(pkg.description, None)
         self.assertEqual(pkg.url, None)
         #self.assertEqual(pkg.time_file, 1340709886)
         self.assertEqual(pkg.time_build, 1340696582)