Add gitreview file for Prickly Pike branch

Update the .gitreview file.

Bug-AGL: SPEC-4867
Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I2136f2415ce7da647e3fd8a59dd6640f9a962e27

Adapt screenshot tests after session removal

Adapt the scripts.

Bug-AGL: SPEC-4813
Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: Ief1ca6d35ed4f891082660500c0ab08b307e7674

Use full path to reference image

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I664757283777cc79f3b40bc043b07294a724b785

Work in /home/agl-driver

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I364ee1531fbd6444166b3404e3402d5321b1afa9

Update screenshot test for multiple images.

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: Ifa5952e05b463fecf50d9e6a4f84a324cb0f8f76

Disable color output in pytest

Disable the color output to avoid escape sequences in LAVA logs.

Bug-AGL: SPEC-4817

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: Iff38a79b72952416792c4c66d1e482736d308544

SPEC-4816: Fix upload in agl-extend-test.sh

agl-extend-test.sh fail to upload a zip file, since it give the
directory instead of the file to upload.
While fixing this, add some extra test and debug around upload of files.

Bug-AGL: SPEC-4816
Fixes: 2e5067e302bc ("qa-testdefinitions: add yaml and script for agl extend test")
Change-Id: Ieeefa239398a82b2156391dfe8f58f654a6a29b3
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Enable multi-screen and list screens

Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Change-Id: I315657ac59d0a2aee20974215e426810b412985c

Adapt path

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: Ib9d15959a4ac1de4642ea2070f325dd6cd02ac08

debug

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: Icc2938efb7568ff65630a316a624b3372409d4d6

Add image upload to qa-testdefinition.

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: Id5902477f8b946757427d344f47082b673031ce0

Use fallback IPs for pyartiproxy.

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: Ia31e5a29e46ee4fca45b4d8ddd8e8cd8e7347133

Add extended test with redirect

try with a logfile

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: Ia6124760784614e9965770ea688bcc87eee5c5f2

Revert "For master do not check connman.service"

With systemd-network-generator masked, we can start up connman again.

This reverts commit bd2c66954f5f0139cfd7610aeb3989965f4b3f7d.

Change-Id: I80edf1a036c1fd418375f8e4cb9b8a77fd354ca8

Update skiplists in qa-testdefinitions

This updates the skiplists. Only pi4 does currently skip the screenshooter.

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I12d276b4268d80392dee3464e4ec5280c2e950ea

For master do not check connman.service

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: Id05a2e1e78bad13b53edd77feff5c10068ba3208

qa-testdefinitions: add yaml and script for agl extend test

In qa-testdefinitions, add lava testdefinition (yaml+script)
for agl extend test

Bug-AGL: SPEC-4345

Signed-off-by: duerpei <duep.fnst@fujitsu.com>
Change-Id: I2df480b5e880c62868c17192c827e8f7ccc2a749

The launcher is no longer needed for window activation

Instead we rely now on the homescreen. So remove the launcher.

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I77d8adea7080e19fb3bffe63a029632c4f4d195a

agl-screenshot-test.sh: Save the contents of the screenshooter

We're getting a checksum error when comparing the reference image
against the one done at boot, so let's have to store it, for later
retrieval and investigation.

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: I7a532d78fc840064d74c6f385ddbf2537390b1ad

agl-screenshot-test.sh: Redo the compositor restart

Stopping services and the client shell before restarting the compositor,
would avoid some large delays until the new compositor instance
is brought up.

With this approach, were we stop explicitly the client shell and the
launcher, we're basically have a hard stop, and should avoid those huge
delays for qemu-arm. Until we can confirm this is the case, we still
maintain that delay.

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: Ifa067ac40de162291d7d6cfb70035e87add07220

Fix screenshot test and user session

The user session has its own daemon that needs to be reloaded separately.
Thus execute a separate reload call.

Bug-AGL: SPEC-4165

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: Ie58423dfb29002660eea69462623d3f37614079f

Adapt screenshot test to new framework

Update the restart command and adapt to the user session.

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I92ee001ded6d49e83078ce13e31e29cebe69d967

Extend delay before taking screenshot

On slow emulated targets, we do need more time to spin up.
Extend the timeout.

Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Change-Id: I90231ad23846e77191626ccaebd657b53ee4b897

screenshot-test: Extend startup delay and use no pager

We need more time on e.g. qemuarm to finish processing
and also have no pager enabled.

Bug-AGL: SPEC-4165

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I1f1d4adcc2b9b702b09e33005559d2d061f7472f

Update agl-screenshot-test for master

The latest changes on the master branch require an update
to the screenshot test.

The script has been changed to use the journal and use the
new settings file locations.

Bug-AGL: SPEC-4150
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Change-Id: If321c62940bb1809e38585c7dcdb08b273af8221

Limit service check after appfw rework

These services are disabled now and will re-enter as part of the appfw rework.

Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Change-Id: I716a1c437c0d50f1756bb0a058b5b1036ca95430

Ignore screenshot on arm 32 - known flakey

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I4a16b8f0449fb7413777f08bbc569d5cd76527e0

Wait for screenshot until display is up

Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Change-Id: Ie74175ab892f12a4eca439350bdd043bb282f931

Extend the waiting time for emulated platforms

Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Change-Id: I3d8cf2dd5227edabf8dc39eb50f5fa0560a5d221

Merge "SPEC-4003: Use pyartiproxy for uploading a file to artyfactory"

SPEC-4003: Use pyartiproxy for uploading a file to artyfactory

Dumpjournal will now dump to artyfactory by default.

Bug-AGL: SPEC-4003
Change-Id: I2ae89be507062bceb84a6a7cbd054dcce09b6874
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Add skiplist for bbe

bbe has no sound. skip sound tests.

Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Change-Id: Ief9254c5c856b19df063985bce3b2de7d4b6ac87

Extend wait time for compositor restart

This fixes slow (e.g. mesa) startup.

Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Change-Id: I10ef1e14ca97370c59984b5d848b90e6cbb9d40b

application-lifecycle: replace continue by return

Lot of code was moved in do_release_test(), but since it is not in a
loop anymore, continue are noop.
Replace them by returns.

Fixes: 1aee04806027c9496f5860a0dfcdbd4aafde148d ("SPEC-3484: create new application tests using new wgt types")
Change-Id: I1dcd7b389dd859d8aadb3608c6e5255d13a470b8
Bug-AGL: SPEC-3808
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Merge "SPEC-3764: add screenshoot test"

SPEC-3764: add screenshoot test

Add basic screenshooter test.

Change-Id: I8be9c6b4f621dc757c7e8ab846384c0fb32561df
Bug-AGL: SPEC-3764
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Add AGL_CAN_INTERFACE=vcan0 to application-lifecycle

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: Ic0b2e75122835786899aae2979feb8d29edefd5c

Update pyagl testdefinition to set AGL_CAN_INTERFACE

We need to use vcan0 for the canreplay.

Bug-AGL: SPEC-3756
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Change-Id: I8e3e8c7d6eadf081014e129cfd59ad217004f46d

Disable colors in journal dump

Avoid escaping characters.

Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Change-Id: I50b1895cc148b43a73e8730f76a39c29382973cb

Do not require tests that enforce internet to be available in CI

Do not enforce tests that require the internet connection to be up.
This is the case for geoclue and weather.

We might be able to revert this in the future after
fixes to the systemd dependencies and lab setup.

Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Change-Id: I1f53e6bfda5be7c9f9613b485efb4f2e25839a63

Flip order in veth setup

Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Change-Id: I2da141b438122eee2768582c5a31054cb8a68964

SPEC-3597: prevent conman to break routes

conman break routes after adding veth, prevent this.

Change-Id: Ic674a8ad609783950ac2ca95a9176d427c59b6ad
Bug-AGL: SPEC-3597
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Merge "Extend skiplists and fix typo in export"

Extend skiplists and fix typo in export

We the can test 1_pyagl-tests.can_write_wo_auth can easily fail on
the 2nd invocation. Work this around until fixed.

Bug-AGL: SPEC-3648
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Change-Id: I25af2c86604672f2e439fef19b400c83fff358d9

SPEC-3631: Add veth test

Add veth test which adds an extra virtual network card for testing.

Change-Id: I4e6fef99009830cd016b2124b45b6815192b5ced
Bug-AGL: SPEC-3631
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Add AGL_AVAILABLE_INTERFACES to application test

The standalone run did have it defined, but the application test
did mit the setting of
AGL_AVAILABLE_INTERFACES=ethernet.

If necessary it has to be redefined in the test for bluetooth and wifi
per board.

Bug-AGL: SPEC-3597
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Change-Id: I0772f9a6cc449406785fb167f575d6ca65e6267c

Disable afm-test execution

agl-service-homescreen crashes when a testwgt is executed.
Until resolved, disable the test wgt execution.

Bug-AGL: SPEC-3608

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I415b2fe80338f4090ef037c77d61af8205fbb77e

SPEC-3484: Use the new gcovr-wrapper

For using gcov, a new gcovr-wrapper was created.
Let's use it, capture its output ang generate some metrics.

Bug-AGL: SPEC-3484
Change-Id: Idbd9aa34d37d31414d18341b035440e28df5dc83
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Merge "SPEC-3484: install and basicly test coverage package"

Merge "Change regular pyagl invocation to nohw"

Change regular pyagl invocation to nohw

In CI we will execute the 'not hwrequired' flag.
Make this the default case in the test-suites.

Bug-AGL: SPEC-3572
Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I370bb3c12953d7455a1a4c06c573cbc10ee17bda

SPEC-3484: install and basicly test coverage package

Install and test coverage the same way than normal package.
Handling coverage generated files will be handled later.

Bug-AGL: SPEC-3484
Change-Id: I02d4ea1aa77954a7c85317accb6b565ece8676c0
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

SPEC-3550: fix parsing of id

The id could be not the first element, and so the parsing is wrong.

Change-Id: I325aac6de4733a166c7be8e3515b85e9edff4365
Bug-AGL: SPEC-3550
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

SPEC-3550: name could be in uppercase

In config.xml, name could be in uppercase, so let's use id which is
lowercase.

Change-Id: Ibc3e9e145692c7773f70a1308c7ff0de27e9f0bd
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Add not hwrequired to regular script case

Qemu by default has no hardware attached. Limit the run to not
execut hardware-specific tests.

Bug-AGL: SPEC-3572

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I3305d0c34737cf026989e2bde197d8ce3d96db19

Add test definitions for pyagl

We can now call pytest using the pyagl/tests module.

These expose -k regular  and -k hwrequired, thus create two caller scripts.

Also create a sample testsuite entry using the regular variant with limited AGL_AVAILABLE_INTERFACES.

Bug-AGL: SPEC-3572

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: Ic6b977da959f946ca88348715d5249748c70f736

SPEC-3550: handle package with service name different than their filename

Some package has their service name different than their filename.
This patch handle this situation

Bug-AGL: SPEC-3550

Change-Id: I99c102f411ce5a0636fa14528cd2890694d986c6
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

SPEC-3484: create new application tests using new wgt types

The wgt-test and wgt-coverage needs to be handled differently.
This patchs splits the application tests to handle all files
encountered.
First we download all files, then we use them in a specific order.

Change-Id: Idc71ecfe36496db314b288f2b8ca7c2d61887de7
Bug-AGL: SPEC-3484
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

SPEC-3482: handle new package name

Now we build all package type, we need to handle coverage/test packages.

Bug-AGL: SPEC-3482

Change-Id: Idb6dc2d453f26183094d6679dcc5c1f553780a16
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Print the syslog when the application does not start

Dump the journal when the app does not start.

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: Ia886e87599721deca57dd8664c6637c3cd1f9b3f

SPEC-3533: unzip wgt files in a dedicated directory

For finding wgtname, we download an index.html and the WGT archive could
contains a file with the same name and unzip complain about replacing a file.
The solution is to unzip in an empty dedicated directory.

Change-Id: I9680eb8909256e749a2b5177ed1d942add841af5
BUG-AGL: SPEC-3533
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

SPEC-3495: fix greping of systemd units

The grep pattern for systemd units is not enough restrictive.
So let's made grep a little more restrictive.

Change-Id: If0f75919a89c39da63e5375e3b3cdd020c6ea324
Bug-AGL: SPEC-3495
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

SPEC-3495: handle better all package name

Package could have -debug in their filename, but their service name is
without it.
Some package could exists with a basename-extension, so we need to grep
better to not catch them all.
Example: agl-service-bluetooth catch agl-service-bluetooth-map.

Change-Id: Ibc750e7893a8cbe3fc22a0d8f13a32c28a898bd1
Bug-AGL: SPEC-3495
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

SPEC-3427: check if RID is null

RID could be null if process failed to start.
In that case, we will retry to start it.
On a second fail, we will not continue the test suite and fail.

In th same time add a second status check, but after a delay to be sure
the service is still working.

Bug-AGL: SPEC-3427
Change-Id: I566f1aefa06cbb791674373340ce87be220b18a2
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Make the script runnable locally and adapt ps call

The script can be run for testing locally now and afm-util ps needs to start as the target user.

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: Ief0f785a46439531d4257c2eef9ced15be1b20fb

Remove versioning of wgts from test

This was removed thus there is no '@?.?' .

Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Update skiplists.

lava-read-feedback fail on qemu due to a sequence issues.
This is merely a timing issue on the lava side that we mask here.

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I19070931704e7b9bf0a27e9370fdffd5adb899ba

Disable the can module unload/load test

And also fix a logic bug in the test_can_modules.

The test can stay disabled. There is no need to unload the modules.

Bug-AGL: SPEC-3070
.

Change-Id: Ic94d5bb9cad3d6cdcbc6534cfc4003213ad82aaa
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

SPEC-3064: handle removal of can_bcm

The test removing can module fail on DRA7 due to can_bcm using it.
This patchs handle this case.

Bug-AGL: SPEC-3064

Change-Id: I272e0ea62786fb43c4ea68601f2fb12864a279b6
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Merge "Skip the can test"

SPEC-2970: Do not fail ofone and bluetooth services if bluetooth tag is non present

When no bluetooth dongle is present, there is a timing issue which let
the service fail.

So if no bluetooth tag is present, skip the service test instead of
failling.

Bug-AGL: SPEC-2907
Change-Id: I10755ec698ab632ffcf0eec18ebe6bc41b1b5ec1
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Skip the can test

As the behaviour of the test not determined, to avoid the fail of the
test, we will skip it if the value of no_transmission not found.

AGL-bug: SPEC-3025

Change-Id: If24e512f2cc83ee7018a4309cb64434cf3a2b938
Signed-off-by: Khouloud Touil <ktouil@baylibre.com>

SPEC-2970: Add bluetooth and ofono on qemu ARM/ARM64 skiplists

Since bluetooth is still failling on qemu ARM and ARM64 (and bring down
ofono by dependency), let's add them to the skiplist.

Change-Id: I1ac3485f339763f16d49e7527faa74f6efe71b30
Bug-AGL: SPEC-2970
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Also call kill in the right context

depending on the app.

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I7208497e81fb3fcb2636bf7559b27ff2b5ce7676

Call info ps and status in the context of the app user

We need to call the info and status call under the context of user under which the app runs.

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I25abff89db22c5effb6addfbc59ec30575aae2ed

Merge "SPEC-2996: handle rcar_can"

SPEC-2996: handle rcar_can

The test remove can_dv module fail on H3 due to rcar_can using it.
This patchs handle this case.

Change-Id: I452a61813567ad09f3f819ce73c1c11753326514
Bug-AGL: SPEC-2996
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

SPEC-2996: Fix ip check

BusyBox ip print its version stderr, so we need to redirect it to stdin.
Furthermore, using set -x reset the error code, we need to store it before.

Bug-AGL: SPEC-2996
Change-Id: I4b80475b0f9291fc55e661362a981ac14dbdb9b0
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

SPEC-2996: rework the CAN modules tests

The CAN modules tests have lots of redundant code, let's add some
function for probing/removing modules.

Furthermore, some tests cannot work with busybox ip, I adds some tests
for it.

When reworking the tests, I have removed the "lsmod |grep can" tests
which was of no value.
I have also removed all sleep which was undocumented(probably for some
log flushing), if something need them I will re-add it with comment.

Bug-AGL: SPEC-2996
Change-Id: I2e8a109d799c58726d848814fd4c21a9fbd2eef5
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Verbose output

Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Merge "test_can_loopback_mode: Add ip util version check"

test_can_loopback_mode: Add ip util version check

CAN tests fail on M3 and H3 because their build includes busybox version
of ip (and not iproute one).
This patchs adds a test for detecting if the right ip util is installed.

Change-Id: I5909bc0af23271626dec9c1ed7003bdfc9129041
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

SPEC-2970: Improove logging size

When TERM is dump, the width of logging is cropped at 80columns which
made debugging harder.
This patchs increase columns size to 1000.

Bug-AGL: SPEC-2970
Change-Id: Ia70136b6aa6242362b1286631ae4b00ab567ae2e
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Cynagora: Replace cynara with cynagora

Currently we disable ptests/socket tests
to be enabled in SPEC-2940 and SPEC-2941.

Shift from the permission database cynara
to cynagora permission database with a compatibility
library.

Bug-AGL: SPEC-2844
Bug-AGL: SPEC-2941

Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>
Change-Id: I673e47ae237a83926f15d003dd2d2623a08eefb5

Mask the bluetooth test until issues are resolved

Currently the bluetooth service fails exclusively on m3ulcb-nogfx.
We need to adapt the CI infra (lava labs) either with additional hw
or adapt the dtb used conditionally. This needs further work.
Meanwhile mask the bluetooth service test on the m3ulcb-nogfx target
to avoid false-positives being reported.

Bug-AGL: SPEC-2921

.

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I592ea7f8a6b8ddc4ff10eac8428162c44a1d3ac4

remove debuging information of do_afm_util

This printing breaks 'rid' parsing, remove it.

Bug-AGL: SPEC-2903
Bug-AGL: SPEC-2925

Change-Id: I505ab9d2994f983de53849fce25d87c8af44362d
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

SPEC-2903: fix the grep test method

While inspecting if application test is working with our latest
SPEC-2903 changes I see in job 8845 the following errors:
+ grep urn:AGL:widget:provided-api config.xml
+ '<feature' 'name=\"urn:AGL:widget:provided-api\">'
./common/scripts/application-lifecycle.sh: line 74: <feature: command not found

So let's change the grep method.
Furthermore, add a debug for the do_afm_util for being sure it work as
needed.

Bug-AGL: SPEC-2903
Change-Id: I5a8a6d98cd9dc6c1b2d1626fc6841eb2c2b16d53
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

SPEC-2903: Use a wrapper instead of constructing a command

This patch change the way we ran afm-util commands by using a simple
wrapper.
Furthermore, since we are in a loop, let's reset all variable
(SERVICE_PLATFORM/etc..) to 0 at the beginning of the loop.

Extra-furthermore: restore the unzip error code test, but in non-fatal
way, since busybox unzip seems to dont like our CRC.

Bug-AGL: SPEC-2903

Change-Id: I5d7357fd74307ed69a6675beac57d8601fdcdb52
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Improve lifecycle test

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I9076278096e15829b4bca52f6b5ef9e9a3b39e9e

Merge "Differ platform and user scope in service and application"

Differ platform and user scope in service and application

When executing the lifecycle test, we need to take into account if
the service or app runs in platform or user scope.

Bug-AGL: SPEC-2903

.

Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I49a53a5a2b14666757b65b0f8203eb73ed2c7b45

Merge "Inverse the fail/pass logic"

Merge "qa-testdefinitions: Split can tests to avoid failures"

Inverse the fail/pass logic

This test should be failing, so it should be set to pass when it
fails, and should be set to fail when it passes.

AGL-bug: SPEC-2878

Change-Id: I43921717eea77d654921eca4691c936bb2901e54
Signed-off-by: Khouloud Touil <ktouil@baylibre.com>

qa-testdefinitions: Split can tests to avoid failures

Split the can tests in two, one for basic tests that doesn't need the
two can interfaces to be related, and the other one is for a board with
two related interfaces.

Modify the test_can_modules to be run on a board with only one can if at
least.

AGL-bug: SPEC-2878

Change-Id: I6f761458c3e191aa8000d19cb05b0592566cb6c9
Signed-off-by: Khouloud Touil <ktouil@baylibre.com>

Merge "Avoid some test cases failures"

Avoid some test cases failures

To make sure that the can interface could be configured, we have to make
it dowm each time before the start of the config step.

Change-Id: I7bebea65072ffd822ee30241c68a0a89123b1576
Signed-off-by: Khouloud Touil <ktouil@baylibre.com>

Merge "Use the new arg --build-tags and the arg --device-tags"

SPEC-2740: add weston in the qemuarm/qemuarm64 skiplist

weston does not start on qemuarm/qemuarm64, let's add it to the test skiplist.

Change-Id: I885b8fe0e6521a1e707c4a3298a9f7d9aa384728
Bug-AGL: SPEC-2740
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>

Use the new arg --build-tags and the arg --device-tags

Use the  new arg --build-tags along with the arg --device-tags to decide
the tests to be run and the tests to be skiped.
The test to be tested in this patch is for the weston service.

Bug-AGL: SPEC-2721

Change-Id: Ia34680c5e60e27dfe286626fedc5775efff87ba3
Signed-off-by: Khouloud Touil <ktouil@baylibre.com>