* [PATCH v2] oeqa devtool: Add tests to cover devtool handling of various git URL styles
@ 2022-08-04 13:49 Thomas Roos
2022-08-08 7:18 ` [OE-core] " Luca Ceresoli
0 siblings, 1 reply; 5+ messages in thread
From: Thomas Roos @ 2022-08-04 13:49 UTC (permalink / raw)
To: openembedded-core
Add two test cases for git URL styles that trigger reformat_git_url.
[YOCTO #11394]
Signed-off-by: Thomas Roos <throos@amazon.de>
---
meta/lib/oeqa/selftest/cases/devtool.py | 34 ++++++++++++++++++++++++-
1 file changed, 33 insertions(+), 1 deletion(-)
diff --git a/meta/lib/oeqa/selftest/cases/devtool.py b/meta/lib/oeqa/selftest/cases/devtool.py
index 34fc791f3a..5a4b0d375c 100644
--- a/meta/lib/oeqa/selftest/cases/devtool.py
+++ b/meta/lib/oeqa/selftest/cases/devtool.py
@@ -246,6 +246,22 @@ class DevtoolTestCase(OESelftestTestCase):
if remaining_removelines:
self.fail('Expected removed lines not found: %s' % remaining_removelines)
+ def _test_devtool_add_git_url(self, git_url, version, pn, resulting_src_uri):
+ self.track_for_cleanup(self.workspacedir)
+ self.add_command_to_tearDown('bitbake-layers remove-layer */workspace')
+ result = runCmd('devtool add --version %s %s %s' % (version, pn, git_url))
+ self.assertExists(os.path.join(self.workspacedir, 'conf', 'layer.conf'), 'Workspace directory not created')
+ # Check the recipe name is correct
+ recipefile = get_bb_var('FILE', pn)
+ self.assertIn('%s_git.bb' % pn, recipefile, 'Recipe file incorrectly named')
+ self.assertIn(recipefile, result.output)
+ # Test devtool status
+ result = runCmd('devtool status')
+ self.assertIn(pn, result.output)
+ self.assertIn(recipefile, result.output)
+ checkvars = {}
+ checkvars['SRC_URI'] = resulting_src_uri
+ self._test_recipe_contents(recipefile, checkvars, [])
class DevtoolBase(DevtoolTestCase):
@@ -379,6 +395,22 @@ class DevtoolAddTests(DevtoolBase):
checkvars['SRCREV'] = srcrev
checkvars['DEPENDS'] = set(['dbus'])
self._test_recipe_contents(recipefile, checkvars, [])
+
+ def test_devtool_add_git_style1(self):
+ version = 'v3.1.0'
+ pn = 'mbedtls'
+ # this will trigger reformat_git_uri with branch parameter in url
+ git_url = "'git://git@github.com/ARMmbed/mbedtls.git;branch=mbedtls-2.28;protocol=https'"
+ resulting_src_uri = "git://git@github.com/ARMmbed/mbedtls.git;branch=mbedtls-2.28;protocol=https"
+ self._test_devtool_add_git_url(git_url, version, pn, resulting_src_uri)
+
+ def test_devtool_add_git_style2(self):
+ version = 'v3.1.0'
+ pn = 'mbedtls'
+ # this will trigger reformat_git_uri with branch parameter in url
+ git_url = "'git://git@github.com/ARMmbed/mbedtls.git;protocol=https'"
+ resulting_src_uri = "git://git@github.com/ARMmbed/mbedtls.git;protocol=https;branch=master"
+ self._test_devtool_add_git_url(git_url, version, pn, resulting_src_uri)
def test_devtool_add_library(self):
# Fetch source
@@ -540,7 +572,7 @@ class DevtoolAddTests(DevtoolBase):
result = runCmd('devtool status')
self.assertIn(testrecipe, result.output)
self.assertIn(srcdir, result.output)
- # Check recipe
+ # Check recipedevtool add
recipefile = get_bb_var('FILE', testrecipe)
self.assertIn('%s_%s.bb' % (testrecipe, testver), recipefile, 'Recipe file incorrectly named')
checkvars = {}
--
2.25.1
Amazon Web Services EMEA SARL
38 avenue John F. Kennedy, L-1855 Luxembourg
Sitz der Gesellschaft: L-1855 Luxemburg
eingetragen im Luxemburgischen Handelsregister unter R.C.S. B186284
Amazon Web Services EMEA SARL, Niederlassung Deutschland
Marcel-Breuer-Str. 12, D-80807 Muenchen
Sitz der Zweigniederlassung: Muenchen
eingetragen im Handelsregister des Amtsgerichts Muenchen unter HRB 242240, USt-ID DE317013094
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [OE-core] [PATCH v2] oeqa devtool: Add tests to cover devtool handling of various git URL styles
2022-08-04 13:49 [PATCH v2] oeqa devtool: Add tests to cover devtool handling of various git URL styles Thomas Roos
@ 2022-08-08 7:18 ` Luca Ceresoli
[not found] ` <1659947998632.6878@amazon.de>
0 siblings, 1 reply; 5+ messages in thread
From: Luca Ceresoli @ 2022-08-08 7:18 UTC (permalink / raw)
To: Thomas Roos via lists.openembedded.org; +Cc: throos, openembedded-core
Hello Thomas,
On Thu, 4 Aug 2022 15:49:20 +0200
"Thomas Roos via lists.openembedded.org" <throos=amazon.de@lists.openembedded.org> wrote:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
As you can see above, your sender address is getting mangled. This is
not your fault, it is done by DMARC for anti-phishing, but it makes
applying your patches annoying.
Can you please try to work around that by setting the sendemail.from
parameter in your git config?
You can read the details in this discussion:
https://lists.openembedded.org/g/openembedded-core/message/166515?p=%2C%2C%2C20%2C0%2C0%2C0%3A%3Acreated%2C0%2Crootfs.py%3A+find+.ko.zst+kernel+modules%2C20%2C2%2C0%2C91453338
Thank you very much!
See below for some comments on your patch.
> Add two test cases for git URL styles that trigger reformat_git_url.
>
> [YOCTO #11394]
>
> Signed-off-by: Thomas Roos <throos@amazon.de>
> ---
> meta/lib/oeqa/selftest/cases/devtool.py | 34 ++++++++++++++++++++++++-
> 1 file changed, 33 insertions(+), 1 deletion(-)
>
> diff --git a/meta/lib/oeqa/selftest/cases/devtool.py b/meta/lib/oeqa/selftest/cases/devtool.py
> index 34fc791f3a..5a4b0d375c 100644
> --- a/meta/lib/oeqa/selftest/cases/devtool.py
> +++ b/meta/lib/oeqa/selftest/cases/devtool.py
> @@ -246,6 +246,22 @@ class DevtoolTestCase(OESelftestTestCase):
> if remaining_removelines:
> self.fail('Expected removed lines not found: %s' % remaining_removelines)
>
> + def _test_devtool_add_git_url(self, git_url, version, pn, resulting_src_uri):
> + self.track_for_cleanup(self.workspacedir)
> + self.add_command_to_tearDown('bitbake-layers remove-layer */workspace')
Trailing whitespace here.
> + result = runCmd('devtool add --version %s %s %s' % (version, pn, git_url))
> + self.assertExists(os.path.join(self.workspacedir, 'conf', 'layer.conf'), 'Workspace directory not created')
> + # Check the recipe name is correct
> + recipefile = get_bb_var('FILE', pn)
> + self.assertIn('%s_git.bb' % pn, recipefile, 'Recipe file incorrectly named')
> + self.assertIn(recipefile, result.output)
> + # Test devtool status
> + result = runCmd('devtool status')
> + self.assertIn(pn, result.output)
> + self.assertIn(recipefile, result.output)
> + checkvars = {}
> + checkvars['SRC_URI'] = resulting_src_uri
> + self._test_recipe_contents(recipefile, checkvars, [])
>
> class DevtoolBase(DevtoolTestCase):
>
> @@ -379,6 +395,22 @@ class DevtoolAddTests(DevtoolBase):
> checkvars['SRCREV'] = srcrev
> checkvars['DEPENDS'] = set(['dbus'])
> self._test_recipe_contents(recipefile, checkvars, [])
> +
Trailing whitespace here too.
> + def test_devtool_add_git_style1(self):
> + version = 'v3.1.0'
> + pn = 'mbedtls'
> + # this will trigger reformat_git_uri with branch parameter in url
> + git_url = "'git://git@github.com/ARMmbed/mbedtls.git;branch=mbedtls-2.28;protocol=https'"
> + resulting_src_uri = "git://git@github.com/ARMmbed/mbedtls.git;branch=mbedtls-2.28;protocol=https"
> + self._test_devtool_add_git_url(git_url, version, pn, resulting_src_uri)
> +
> + def test_devtool_add_git_style2(self):
> + version = 'v3.1.0'
> + pn = 'mbedtls'
> + # this will trigger reformat_git_uri with branch parameter in url
> + git_url = "'git://git@github.com/ARMmbed/mbedtls.git;protocol=https'"
> + resulting_src_uri = "git://git@github.com/ARMmbed/mbedtls.git;protocol=https;branch=master"
> + self._test_devtool_add_git_url(git_url, version, pn, resulting_src_uri)
>
> def test_devtool_add_library(self):
> # Fetch source
> @@ -540,7 +572,7 @@ class DevtoolAddTests(DevtoolBase):
> result = runCmd('devtool status')
> self.assertIn(testrecipe, result.output)
> self.assertIn(srcdir, result.output)
> - # Check recipe
> + # Check recipedevtool add
This change looks wrong...
--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-08-09 7:00 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-08-04 13:49 [PATCH v2] oeqa devtool: Add tests to cover devtool handling of various git URL styles Thomas Roos
2022-08-08 7:18 ` [OE-core] " Luca Ceresoli
[not found] ` <1659947998632.6878@amazon.de>
2022-08-08 13:19 ` Luca Ceresoli
2022-08-08 15:11 ` Richard Purdie
2022-08-09 7:00 ` AW: " Roos, Thomas
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox