diff options
| author | Trevor Gamblin <tgamblin@baylibre.com> | 2023-10-19 16:49:27 -0400 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2023-10-19 22:25:23 +0100 |
| commit | 9b8b9a9fa3dce59ac6f5ff0e0526b109768f796e (patch) | |
| tree | afe0a75508a6cc19c2b3eb187df2b9fd8248f2e1 /meta/lib/patchtest | |
| parent | 751e61e45c2de3f52e9fd57144c3a6c0a5bff316 (diff) | |
| download | poky-9b8b9a9fa3dce59ac6f5ff0e0526b109768f796e.tar.gz | |
patchtest: test regardless of mergeability
(From OE-Core rev: dc089073eb120de76c8907e476c341ed3e97c164)
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib/patchtest')
4 files changed, 0 insertions, 17 deletions
diff --git a/meta/lib/patchtest/tests/test_metadata_lic_files_chksum.py b/meta/lib/patchtest/tests/test_metadata_lic_files_chksum.py index b2c32507ff..cb3e7c9d34 100644 --- a/meta/lib/patchtest/tests/test_metadata_lic_files_chksum.py +++ b/meta/lib/patchtest/tests/test_metadata_lic_files_chksum.py | |||
| @@ -15,11 +15,6 @@ class LicFilesChkSum(base.Metadata): | |||
| 15 | lictag = 'License-Update' | 15 | lictag = 'License-Update' |
| 16 | lictag_re = pyparsing.Regex("^%s:" % lictag) | 16 | lictag_re = pyparsing.Regex("^%s:" % lictag) |
| 17 | 17 | ||
| 18 | def setUp(self): | ||
| 19 | # these tests just make sense on patches that can be merged | ||
| 20 | if not PatchTestInput.repo.canbemerged: | ||
| 21 | self.skip('Patch cannot be merged') | ||
| 22 | |||
| 23 | def test_lic_files_chksum_presence(self): | 18 | def test_lic_files_chksum_presence(self): |
| 24 | if not self.added: | 19 | if not self.added: |
| 25 | self.skip('No added recipes, skipping test') | 20 | self.skip('No added recipes, skipping test') |
diff --git a/meta/lib/patchtest/tests/test_metadata_license.py b/meta/lib/patchtest/tests/test_metadata_license.py index a5bc03b83f..1a7f09b747 100644 --- a/meta/lib/patchtest/tests/test_metadata_license.py +++ b/meta/lib/patchtest/tests/test_metadata_license.py | |||
| @@ -12,11 +12,6 @@ class License(base.Metadata): | |||
| 12 | metadata = 'LICENSE' | 12 | metadata = 'LICENSE' |
| 13 | invalid_license = 'PATCHTESTINVALID' | 13 | invalid_license = 'PATCHTESTINVALID' |
| 14 | 14 | ||
| 15 | def setUp(self): | ||
| 16 | # these tests just make sense on patches that can be merged | ||
| 17 | if not PatchTestInput.repo.canbemerged: | ||
| 18 | self.skip('Patch cannot be merged') | ||
| 19 | |||
| 20 | def test_license_presence(self): | 15 | def test_license_presence(self): |
| 21 | if not self.added: | 16 | if not self.added: |
| 22 | self.skip('No added recipes, skipping test') | 17 | self.skip('No added recipes, skipping test') |
diff --git a/meta/lib/patchtest/tests/test_metadata_summary.py b/meta/lib/patchtest/tests/test_metadata_summary.py index 1502863df0..170e79eb4b 100644 --- a/meta/lib/patchtest/tests/test_metadata_summary.py +++ b/meta/lib/patchtest/tests/test_metadata_summary.py | |||
| @@ -10,11 +10,6 @@ from data import PatchTestInput | |||
| 10 | class Summary(base.Metadata): | 10 | class Summary(base.Metadata): |
| 11 | metadata = 'SUMMARY' | 11 | metadata = 'SUMMARY' |
| 12 | 12 | ||
| 13 | def setUp(self): | ||
| 14 | # these tests just make sense on patches that can be merged | ||
| 15 | if not PatchTestInput.repo.canbemerged: | ||
| 16 | self.skip('Patch cannot be merged') | ||
| 17 | |||
| 18 | def test_summary_presence(self): | 13 | def test_summary_presence(self): |
| 19 | if not self.added: | 14 | if not self.added: |
| 20 | self.skip('No added recipes, skipping test') | 15 | self.skip('No added recipes, skipping test') |
diff --git a/meta/lib/patchtest/tests/test_python_pylint.py b/meta/lib/patchtest/tests/test_python_pylint.py index 9cfc491a13..304b2d5ee9 100644 --- a/meta/lib/patchtest/tests/test_python_pylint.py +++ b/meta/lib/patchtest/tests/test_python_pylint.py | |||
| @@ -26,8 +26,6 @@ class PyLint(base.Base): | |||
| 26 | def setUp(self): | 26 | def setUp(self): |
| 27 | if self.unidiff_parse_error: | 27 | if self.unidiff_parse_error: |
| 28 | self.skip('Python-unidiff parse error') | 28 | self.skip('Python-unidiff parse error') |
| 29 | if not PatchTestInput.repo.canbemerged: | ||
| 30 | self.skip('Patch cannot be merged, no reason to execute the test method') | ||
| 31 | if not PyLint.pythonpatches: | 29 | if not PyLint.pythonpatches: |
| 32 | self.skip('No python related patches, skipping test') | 30 | self.skip('No python related patches, skipping test') |
| 33 | 31 | ||
