Openembedded Core Discussions
 help / color / mirror / Atom feed
From: "Aníbal Limón" <anibal.limon@linux.intel.com>
To: Paul Eggleton <paul.eggleton@linux.intel.com>
Cc: openembedded-core@lists.openembedded.org,
	benjamin.esquivel@intel.com,
	"Aníbal Limón" <limon.anibal@gmail.com>
Subject: Re: [PATCH 19/20] oeqa/sdkext: Add devtool basic tests for eSDK.
Date: Tue, 2 Feb 2016 16:14:47 -0600	[thread overview]
Message-ID: <56B12A57.5050402@linux.intel.com> (raw)
In-Reply-To: <3666142.Q8On7yAzEZ@peggleto-mobl.ger.corp.intel.com>

[-- Attachment #1: Type: text/plain, Size: 3294 bytes --]



On 02/02/2016 04:03 PM, Paul Eggleton wrote:
> On Tue, 02 Feb 2016 09:14:22 Aníbal Limón wrote:
>> From: Aníbal Limón <limon.anibal@gmail.com>
>>
>> Add simple myapp application is a C app that prints hello world
>> and exit.
>>
>> Add devtool test for that this app to the workspace, build and
>> reset it.
>>
>> Signed-off-by: Aníbal Limón <limon.anibal@gmail.com>
>> ---
>>  meta/lib/oeqa/sdkext/devtool.py           | 25 +++++++++++++++++++++++++
>>  meta/lib/oeqa/sdkext/files/myapp/Makefile | 10 ++++++++++
>>  meta/lib/oeqa/sdkext/files/myapp/myapp.c  |  9 +++++++++
>>  3 files changed, 44 insertions(+)
>>  create mode 100644 meta/lib/oeqa/sdkext/devtool.py
>>  create mode 100644 meta/lib/oeqa/sdkext/files/myapp/Makefile
>>  create mode 100644 meta/lib/oeqa/sdkext/files/myapp/myapp.c
>>
>> diff --git a/meta/lib/oeqa/sdkext/devtool.py
>> b/meta/lib/oeqa/sdkext/devtool.py new file mode 100644
>> index 0000000..0262ed3
>> --- /dev/null
>> +++ b/meta/lib/oeqa/sdkext/devtool.py
>> @@ -0,0 +1,25 @@
>> +import shutil
>> +
>> +from oeqa.oetest import oeSDKExtTest
>> +from oeqa.utils.decorators import *
>> +
>> +class DevtoolTest(oeSDKExtTest):
>> +
>> +    @classmethod
>> +    def setUpClass(self):
>> +        self.myapp_src = os.path.join(self.tc.sdkextfilesdir, "myapp")
>> +        self.myapp_dst = os.path.join(self.tc.sdktestdir, "myapp")
>> +        shutil.copytree(self.myapp_src, self.myapp_dst)
>> +
>> +    def test_devtool_add_reset(self):
>> +        self._run('devtool add myapp %s' % self.myapp_dst)
>> +        self._run('devtool reset myapp')
>> +
>> +    def test_devtool_build(self):
>> +        self._run('devtool add myapp %s' % self.myapp_dst)
>> +        self._run('devtool build myapp')
>> +        self._run('devtool reset myapp')
>> +
>> +    @classmethod
>> +    def tearDownClass(self):
>> +        shutil.rmtree(self.myapp_dst)
>> diff --git a/meta/lib/oeqa/sdkext/files/myapp/Makefile
>> b/meta/lib/oeqa/sdkext/files/myapp/Makefile new file mode 100644
>> index 0000000..abd91be
>> --- /dev/null
>> +++ b/meta/lib/oeqa/sdkext/files/myapp/Makefile
>> @@ -0,0 +1,10 @@
>> +all: myapp
>> +
>> +myapp: myapp.o
>> +	$(CC) $(LDFLAGS) $< -o $@
>> +
>> +myapp.o: myapp.c
>> +	$(CC) $(CFLAGS) -c $< -o $@
>> +
>> +clean:
>> +	rm -rf myapp.o myapp
>> diff --git a/meta/lib/oeqa/sdkext/files/myapp/myapp.c
>> b/meta/lib/oeqa/sdkext/files/myapp/myapp.c new file mode 100644
>> index 0000000..f0b63f0
>> --- /dev/null
>> +++ b/meta/lib/oeqa/sdkext/files/myapp/myapp.c
>> @@ -0,0 +1,9 @@
>> +#include <stdio.h>
>> +
>> +int
>> +main(int argc, char *argv[])
>> +{
>> +	printf("Hello world\n");
>> +
>> +	return 0;
>> +}
> 
> I guess we'll probably want to extend this test so it also looks at the 
> binaries produced by the build to see if they're reasonable, but we can do 
> that later.
> 
> One thing though - could you add a check to see that the 'devtool' command 
> we're executing is definitely from the SDK and not the outer build environment 
> (or elsewhere)? Presumably you could just do "which devtool" and check the 
> output.

Sure good idea... i'll add which devtool and search for path inside the
eSDK folder.

	alimon

> 
> Cheers,
> Paul
> 


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

  reply	other threads:[~2016-02-02 22:12 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-02 15:14 [PATCH 00/20] Add Extensible SDK test suite Aníbal Limón
2016-02-02 15:14 ` [PATCH 01/20] testimage: Modularize helper functions for get test lists Aníbal Limón
2016-02-02 15:14 ` [PATCH 02/20] classes/testsdk: Add new class testsdk Aníbal Limón
2016-02-02 15:14 ` [PATCH 03/20] classes/testimage: Add defeault inherit for testsdk Aníbal Limón
2016-02-02 15:14 ` [PATCH 04/20] populate_sdk_ext: Set TOOLCHAINEXT_OUTPUTNAME Aníbal Limón
2016-02-02 15:14 ` [PATCH 05/20] get_test_suites: Add sdkext type for load test suites Aníbal Limón
2016-02-02 15:14 ` [PATCH 06/20] classes/testsdk: Add testsdkext task only install Aníbal Limón
2016-02-02 15:14 ` [PATCH 07/20] bb/fetch2: Move export_proxies function from wget to utils Aníbal Limón
2016-02-02 15:14 ` [PATCH 08/20] classes/testsdk: Add call to export_proxies on testsdkext Aníbal Limón
2016-02-02 15:14 ` [PATCH 09/20] toolchain-shar-extract.sh: Add proxy variable to new env Aníbal Limón
2016-02-02 15:14 ` [PATCH 10/20] testimage/testsdk: Modularize TestContext Aníbal Limón
2016-02-02 15:14 ` [PATCH 11/20] testimage/testsdk: Move get test suites routine inside TestContext Aníbal Limón
2016-02-02 15:14 ` [PATCH 12/20] oetest.py/TestContext: Move loadTests and runTests inside it Aníbal Limón
2016-02-02 15:14 ` [PATCH 13/20] oeqa/oetest.py: Fix missing oeqa.runtime import Aníbal Limón
2016-02-02 21:25   ` Paul Eggleton
2016-02-02 21:31     ` Aníbal Limón
2016-02-02 21:32       ` Paul Eggleton
2016-02-02 15:14 ` [PATCH 14/20] classes/testsdk: Add function run_test_context Aníbal Limón
2016-02-02 15:14 ` [PATCH 15/20] classes/populate_sdk_ext: Add OE_SDK_EXT_SILENT env variable Aníbal Limón
2016-02-02 21:19   ` Paul Eggleton
2016-02-02 21:23     ` Aníbal Limón
2016-02-02 21:25       ` Paul Eggleton
2016-02-02 21:30         ` Aníbal Limón
2016-02-02 21:31           ` Paul Eggleton
2016-02-02 21:38             ` Aníbal Limón
2016-02-02 21:40               ` Paul Eggleton
2016-02-02 21:47                 ` Aníbal Limón
2016-02-02 21:49                   ` Paul Eggleton
2016-02-02 15:14 ` [PATCH 16/20] classes/testsdk: Add compatibility SDK testsuite to eSDK Aníbal Limón
2016-02-02 15:14 ` [PATCH 17/20] testsdkext: Add skeleton for support Extensible SDK tests Aníbal Limón
2016-02-02 15:14 ` [PATCH 18/20] classes/populate_sdk_ext: Add SDK_EXT_TARGET_MANIFEST and SDK_EXT_HOST_MANIFEST Aníbal Limón
2016-02-02 21:52   ` Paul Eggleton
2016-02-02 22:05     ` Aníbal Limón
2016-02-02 22:06       ` Paul Eggleton
2016-02-02 22:13         ` Aníbal Limón
2016-02-02 22:15           ` Paul Eggleton
2016-02-02 15:14 ` [PATCH 19/20] oeqa/sdkext: Add devtool basic tests for eSDK Aníbal Limón
2016-02-02 22:03   ` Paul Eggleton
2016-02-02 22:14     ` Aníbal Limón [this message]
2016-02-02 15:14 ` [PATCH 20/20] classes/testsdk: Add help information on how to run tests Aníbal Limón

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=56B12A57.5050402@linux.intel.com \
    --to=anibal.limon@linux.intel.com \
    --cc=benjamin.esquivel@intel.com \
    --cc=limon.anibal@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=paul.eggleton@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox