|
Lukáš Hrázký |
4e70f6 |
From b2b80e3b57615aee491d2e384d55e73045b27cca Mon Sep 17 00:00:00 2001
|
|
Lukáš Hrázký |
4e70f6 |
From: =?UTF-8?q?Ale=C5=A1=20Mat=C4=9Bj?= <amatej@redhat.com>
|
|
Lukáš Hrázký |
4e70f6 |
Date: Wed, 9 Mar 2022 15:23:40 +0100
|
|
Lukáš Hrázký |
4e70f6 |
Subject: [PATCH] Revert "Add python tests for parsing metadata together"
|
|
Lukáš Hrázký |
4e70f6 |
|
|
Lukáš Hrázký |
4e70f6 |
This reverts commit 0c050128f7e24222011cdfcc67ba58da744affa9.
|
|
Lukáš Hrázký |
4e70f6 |
---
|
|
Lukáš Hrázký |
4e70f6 |
tests/python/tests/test_xml_parser.py | 206 --------------------------
|
|
Lukáš Hrázký |
4e70f6 |
1 file changed, 206 deletions(-)
|
|
Lukáš Hrázký |
4e70f6 |
|
|
Lukáš Hrázký |
4e70f6 |
diff --git a/tests/python/tests/test_xml_parser.py b/tests/python/tests/test_xml_parser.py
|
|
Lukáš Hrázký |
4e70f6 |
index ed13cac..ddc089f 100644
|
|
Lukáš Hrázký |
4e70f6 |
--- a/tests/python/tests/test_xml_parser.py
|
|
Lukáš Hrázký |
4e70f6 |
+++ b/tests/python/tests/test_xml_parser.py
|
|
Lukáš Hrázký |
4e70f6 |
@@ -950,209 +950,3 @@ class TestCaseXmlParserRepomd(unittest.TestCase):
|
|
Lukáš Hrázký |
4e70f6 |
self.assertEqual(repomd.content_tags, [])
|
|
Lukáš Hrázký |
4e70f6 |
self.assertEqual(len(repomd.records), 3)
|
|
Lukáš Hrázký |
4e70f6 |
|
|
Lukáš Hrázký |
4e70f6 |
-class TestCaseXmlParserMainMetadataTogether(unittest.TestCase):
|
|
Lukáš Hrázký |
4e70f6 |
- def test_xml_parser_main_metadata_together_repo01(self):
|
|
Lukáš Hrázký |
4e70f6 |
- userdata = {
|
|
Lukáš Hrázký |
4e70f6 |
- "pkgs": [],
|
|
Lukáš Hrázký |
4e70f6 |
- "pkgcb_calls": 0,
|
|
Lukáš Hrázký |
4e70f6 |
- "warnings": []
|
|
Lukáš Hrázký |
4e70f6 |
- }
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def newpkgcb(pkgId, name, arch):
|
|
Lukáš Hrázký |
4e70f6 |
- pkg = cr.Package()
|
|
Lukáš Hrázký |
4e70f6 |
- userdata["pkgs"].append(pkg)
|
|
Lukáš Hrázký |
4e70f6 |
- return pkg
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def pkgcb(pkg):
|
|
Lukáš Hrázký |
4e70f6 |
- userdata["pkgcb_calls"] += 1
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def warningcb(warn_type, msg):
|
|
Lukáš Hrázký |
4e70f6 |
- userdata["warnings"].append((warn_type, msg))
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- cr.xml_parse_main_metadata_together(primary=REPO_01_PRIXML, filelists=REPO_01_FILXML, other=REPO_01_OTHXML,
|
|
Lukáš Hrázký |
4e70f6 |
- newpkgcb=newpkgcb, pkgcb=pkgcb, warningcb=warningcb)
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual([pkg.name for pkg in userdata["pkgs"]], ['super_kernel'])
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(userdata["pkgcb_calls"], 1)
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(userdata["warnings"], [])
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- pkg = userdata["pkgs"][0]
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.pkgId, "152824bff2aa6d54f429d43e87a3ff3a0286505c6d93ec87692b5e3a9e3b97bf")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.name, "super_kernel")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.arch, "x86_64")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.version, "6.0.1")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.epoch, "0")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.release, "2")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.summary, "Test package")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.description, "This package has provides, requires, obsoletes, conflicts options.")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.url, "http://so_super_kernel.com/it_is_awesome/yep_it_really_is")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.time_file, 1334667003)
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.time_build, 1334667003)
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.rpm_license, "LGPLv2")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.rpm_vendor, None)
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.rpm_group, "Applications/System")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.rpm_buildhost, "localhost.localdomain")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.rpm_sourcerpm, "super_kernel-6.0.1-2.src.rpm")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.rpm_header_start, 280)
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.rpm_header_end, 2637)
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.rpm_packager, None)
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.size_package, 2845)
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.size_installed, 0)
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.size_archive, 404)
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.location_href, "super_kernel-6.0.1-2.x86_64.rpm")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.location_base, None)
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.checksum_type, "sha256")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.requires,
|
|
Lukáš Hrázký |
4e70f6 |
- [('bzip2', 'GE', '0', '1.0.0', None, True),
|
|
Lukáš Hrázký |
4e70f6 |
- ('expat', None, None, None, None, True),
|
|
Lukáš Hrázký |
4e70f6 |
- ('glib', 'GE', '0', '2.26.0', None, False),
|
|
Lukáš Hrázký |
4e70f6 |
- ('zlib', None, None, None, None, False)])
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.provides,
|
|
Lukáš Hrázký |
4e70f6 |
- [('not_so_super_kernel', 'LT', '0', '5.8.0', None, False),
|
|
Lukáš Hrázký |
4e70f6 |
- ('super_kernel', 'EQ', '0', '6.0.0', None, False),
|
|
Lukáš Hrázký |
4e70f6 |
- ('super_kernel', 'EQ', '0', '6.0.1', '2', False),
|
|
Lukáš Hrázký |
4e70f6 |
- ('super_kernel(x86-64)', 'EQ', '0', '6.0.1', '2', False)])
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.conflicts,
|
|
Lukáš Hrázký |
4e70f6 |
- [('kernel', None, None, None, None, False),
|
|
Lukáš Hrázký |
4e70f6 |
- ('super_kernel', 'EQ', '0', '5.0.0', None, False),
|
|
Lukáš Hrázký |
4e70f6 |
- ('super_kernel', 'LT', '0', '4.0.0', None, False)])
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.obsoletes,
|
|
Lukáš Hrázký |
4e70f6 |
- [('kernel', None, None, None, None, False),
|
|
Lukáš Hrázký |
4e70f6 |
- ('super_kernel', 'EQ', '0', '5.9.0', None, False)])
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.files,
|
|
Lukáš Hrázký |
4e70f6 |
- [(None, '/usr/bin/', 'super_kernel'),
|
|
Lukáš Hrázký |
4e70f6 |
- (None, '/usr/share/man/', 'super_kernel.8.gz')])
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(pkg.changelogs,
|
|
Lukáš Hrázký |
4e70f6 |
- [('Tomas Mlcoch <tmlcoch@redhat.com> - 6.0.1-1',
|
|
Lukáš Hrázký |
4e70f6 |
- 1334664000,
|
|
Lukáš Hrázký |
4e70f6 |
- '- First release'),
|
|
Lukáš Hrázký |
4e70f6 |
- ('Tomas Mlcoch <tmlcoch@redhat.com> - 6.0.1-2',
|
|
Lukáš Hrázký |
4e70f6 |
- 1334664001,
|
|
Lukáš Hrázký |
4e70f6 |
- '- Second release')])
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def test_xml_parser_main_metadata_together_repo02(self):
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- userdata = {
|
|
Lukáš Hrázký |
4e70f6 |
- "pkgs": [],
|
|
Lukáš Hrázký |
4e70f6 |
- "pkgcb_calls": 0,
|
|
Lukáš Hrázký |
4e70f6 |
- "warnings": []
|
|
Lukáš Hrázký |
4e70f6 |
- }
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def newpkgcb(pkgId, name, arch):
|
|
Lukáš Hrázký |
4e70f6 |
- pkg = cr.Package()
|
|
Lukáš Hrázký |
4e70f6 |
- userdata["pkgs"].append(pkg)
|
|
Lukáš Hrázký |
4e70f6 |
- return pkg
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def pkgcb(pkg):
|
|
Lukáš Hrázký |
4e70f6 |
- userdata["pkgcb_calls"] += 1
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def warningcb(warn_type, msg):
|
|
Lukáš Hrázký |
4e70f6 |
- userdata["warnings"].append((warn_type, msg))
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- cr.xml_parse_main_metadata_together(REPO_02_PRIXML, REPO_02_FILXML, REPO_02_OTHXML, newpkgcb, pkgcb, warningcb,
|
|
Lukáš Hrázký |
4e70f6 |
- allow_out_of_order=True)
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual([pkg.name for pkg in userdata["pkgs"]],
|
|
Lukáš Hrázký |
4e70f6 |
- ['fake_bash', 'super_kernel'])
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(userdata["pkgcb_calls"], 2)
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(userdata["warnings"], [])
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def test_xml_parser_main_metadata_together_repo02_only_pkgcb(self):
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- pkgs = []
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def pkgcb(pkg):
|
|
Lukáš Hrázký |
4e70f6 |
- pkgs.append(pkg)
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- cr.xml_parse_main_metadata_together(REPO_02_PRIXML, REPO_02_FILXML, REPO_02_OTHXML,
|
|
Lukáš Hrázký |
4e70f6 |
- None, pkgcb, None)
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual([pkg.name for pkg in pkgs],
|
|
Lukáš Hrázký |
4e70f6 |
- ['fake_bash', 'super_kernel'])
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def test_xml_parser_main_metadata_together_repo02_no_cbs(self):
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertRaises(ValueError,
|
|
Lukáš Hrázký |
4e70f6 |
- cr.xml_parse_main_metadata_together,
|
|
Lukáš Hrázký |
4e70f6 |
- REPO_02_PRIXML, REPO_02_FILXML, REPO_02_OTHXML,
|
|
Lukáš Hrázký |
4e70f6 |
- None, None, None)
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def test_xml_parser_main_metadata_together_warnings(self):
|
|
Lukáš Hrázký |
4e70f6 |
- userdata = {
|
|
Lukáš Hrázký |
4e70f6 |
- "pkgs": [],
|
|
Lukáš Hrázký |
4e70f6 |
- "warnings": []
|
|
Lukáš Hrázký |
4e70f6 |
- }
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def newpkgcb(pkgId, name, arch):
|
|
Lukáš Hrázký |
4e70f6 |
- pkg = cr.Package()
|
|
Lukáš Hrázký |
4e70f6 |
- userdata["pkgs"].append(pkg)
|
|
Lukáš Hrázký |
4e70f6 |
- return pkg
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def warningcb(warn_type, msg):
|
|
Lukáš Hrázký |
4e70f6 |
- userdata["warnings"].append((warn_type, msg))
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- cr.xml_parse_main_metadata_together(PRIMARY_MULTI_WARN_00_PATH,
|
|
Lukáš Hrázký |
4e70f6 |
- FILELISTS_MULTI_WARN_00_PATH,
|
|
Lukáš Hrázký |
4e70f6 |
- OTHER_MULTI_WARN_00_PATH,
|
|
Lukáš Hrázký |
4e70f6 |
- newpkgcb, None, warningcb)
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual([pkg.name for pkg in userdata["pkgs"]],
|
|
Lukáš Hrázký |
4e70f6 |
- ['fake_bash', 'super_kernel'])
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(userdata["warnings"],
|
|
Lukáš Hrázký |
4e70f6 |
- [(0, 'Unknown element "fooelement"'),
|
|
Lukáš Hrázký |
4e70f6 |
- (1, 'Missing attribute "type" of a package element'),
|
|
Lukáš Hrázký |
4e70f6 |
- (0, 'Unknown element "foo"'),
|
|
Lukáš Hrázký |
4e70f6 |
- (3, 'Conversion of "foobar" to integer failed'),
|
|
Lukáš Hrázký |
4e70f6 |
- (0, 'Unknown element "bar"'),
|
|
Lukáš Hrázký |
4e70f6 |
- (1, 'Missing attribute "arch" of a package element'),
|
|
Lukáš Hrázký |
4e70f6 |
- (2, 'Unknown file type "xxx"'),
|
|
Lukáš Hrázký |
4e70f6 |
- (0, 'Unknown element "bar"'),
|
|
Lukáš Hrázký |
4e70f6 |
- (1, 'Missing attribute "name" of a package element'),
|
|
Lukáš Hrázký |
4e70f6 |
- (0, 'Unknown element "bar"'),
|
|
Lukáš Hrázký |
4e70f6 |
- (3, 'Conversion of "xxx" to integer failed')])
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def test_xml_parser_main_metadata_together_error(self):
|
|
Lukáš Hrázký |
4e70f6 |
- userdata = { "pkgs": [] }
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def newpkgcb(pkgId, name, arch):
|
|
Lukáš Hrázký |
4e70f6 |
- pkg = cr.Package()
|
|
Lukáš Hrázký |
4e70f6 |
- userdata["pkgs"].append(pkg)
|
|
Lukáš Hrázký |
4e70f6 |
- return pkg
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertRaises(cr.CreaterepoCError, cr.xml_parse_main_metadata_together,
|
|
Lukáš Hrázký |
4e70f6 |
- PRIMARY_ERROR_00_PATH, FILELISTS_ERROR_00_PATH,
|
|
Lukáš Hrázký |
4e70f6 |
- OTHER_ERROR_00_PATH, newpkgcb, None, None)
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- # unlike parsing just primary, filelists or other separately when parsing together primary is parsed first fully
|
|
Lukáš Hrázký |
4e70f6 |
- # before newpkgcb is called so the error is detected before any user package is created
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertEqual(len(userdata["pkgs"]), 0)
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def test_xml_parser_main_metadata_together_newpkgcb_abort(self):
|
|
Lukáš Hrázký |
4e70f6 |
- def newpkgcb(pkgId, name, arch):
|
|
Lukáš Hrázký |
4e70f6 |
- raise Error("Foo error")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertRaises(cr.CreaterepoCError,
|
|
Lukáš Hrázký |
4e70f6 |
- cr.xml_parse_main_metadata_together,
|
|
Lukáš Hrázký |
4e70f6 |
- REPO_02_PRIXML, REPO_02_FILXML, REPO_02_OTHXML,
|
|
Lukáš Hrázký |
4e70f6 |
- newpkgcb, None, None)
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def test_xml_parser_main_metadata_together_pkgcb_abort(self):
|
|
Lukáš Hrázký |
4e70f6 |
- def newpkgcb(pkgId, name, arch):
|
|
Lukáš Hrázký |
4e70f6 |
- return cr.Package()
|
|
Lukáš Hrázký |
4e70f6 |
- def pkgcb(pkg):
|
|
Lukáš Hrázký |
4e70f6 |
- raise Error("Foo error")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertRaises(cr.CreaterepoCError,
|
|
Lukáš Hrázký |
4e70f6 |
- cr.xml_parse_main_metadata_together,
|
|
Lukáš Hrázký |
4e70f6 |
- REPO_02_PRIXML, REPO_02_FILXML, REPO_02_OTHXML,
|
|
Lukáš Hrázký |
4e70f6 |
- newpkgcb, pkgcb, None)
|
|
Lukáš Hrázký |
4e70f6 |
-
|
|
Lukáš Hrázký |
4e70f6 |
- def test_xml_parser_main_metadata_together_warningcb_abort(self):
|
|
Lukáš Hrázký |
4e70f6 |
- def newpkgcb(pkgId, name, arch):
|
|
Lukáš Hrázký |
4e70f6 |
- return cr.Package()
|
|
Lukáš Hrázký |
4e70f6 |
- def warningcb(type, msg):
|
|
Lukáš Hrázký |
4e70f6 |
- raise Error("Foo error")
|
|
Lukáš Hrázký |
4e70f6 |
- self.assertRaises(cr.CreaterepoCError,
|
|
Lukáš Hrázký |
4e70f6 |
- cr.xml_parse_main_metadata_together,
|
|
Lukáš Hrázký |
4e70f6 |
- PRIMARY_MULTI_WARN_00_PATH,
|
|
Lukáš Hrázký |
4e70f6 |
- FILELISTS_MULTI_WARN_00_PATH,
|
|
Lukáš Hrázký |
4e70f6 |
- OTHER_MULTI_WARN_00_PATH,
|
|
Lukáš Hrázký |
4e70f6 |
- newpkgcb, None, warningcb)
|
|
Lukáš Hrázký |
4e70f6 |
--
|
|
Lukáš Hrázký |
4e70f6 |
2.34.3
|
|
Lukáš Hrázký |
4e70f6 |
|