Alle these should hopefully not be needed anymore, with the workaround for the
race condition in ace engine.

Signed-off-by: Esben Haabendal <esben@geanix.com>

When starting up abilities, there is a race condition between OnSurfaceChanged
and RunPage, which both are executed as PostTask on the same task executor.
This executor works in FIFO mode, and if (when) the OnSurfaceChanged comes
first, it will fail with a nullptr check on runningPage_.  This is because the
runningPage_ variable is set by the RunPage PostTask.

This patch is obviously just a workaround, and not a proper fix.  Hopefully, 500
ms will be enough in all relevant situations, but at least in theory, the
problem can still appear.

Signed-off-by: Esben Haabendal <esben@geanix.com>

Graphics seems to be basically working now, both on QEMU and rpi4 (plus/minus a
few quirks).  This patch is now causing more problems than it is solving, so
let's revert it for now.

We need to take care that slow rendering is handled properly.  We might have
more test cases failing due to this after dropping this patch.

Signed-off-by: Esben Haabendal <esben@geanix.com>

This service doesn't seem to be needed for ACTS compliance, and consumes a lot
of CPU time when activated during some ACTS tests.

Signed-off-by: Esben Haabendal <esben@geanix.com>

Signed-off-by: Esben Haabendal <esben@geanix.com>

This avoids param service log information to end up spamming the console.

Signed-off-by: Esben Haabendal <esben@geanix.com>

Signed-off-by: Esben Haabendal <esben@geanix.com>

Signed-off-by: Esben Haabendal <esben@geanix.com>

Signed-off-by: Esben Haabendal <esben@geanix.com>

Signed-off-by: Esben Haabendal <esben@geanix.com>

With this in place, we are ready to start converting OpenHarmony system
abilities (started with sa_main) to systemd services using Type=notify.

Signed-off-by: Esben Haabendal <esben@geanix.com>

Convert hilogd into a systemd service

- Patch hilogd to use sd_notify(3) to send ready notification to systemd, and
  thus use as a Type=notify service.
- Add log and logd users and groups, with similar uid and gid as in OHOS.
- Place Unix sockets in /run/openharmony/hilog instead of /dev/unix/socket,
  enabling secure creaton and managing of these.
- Use systemd.socket units for the two Unix sockets, to allow creating with
  desired uid/gid and permissions, and allowing to use hilogd with systemd
  socket activation

Signed-off-by: Esben Haabendal <esben@geanix.com>

Signed-off-by: Esben Haabendal <esben@geanix.com>

Signed-off-by: Esben Haabendal <esben@geanix.com>

This stops placing temporary files and Unix sockets in /dev, and use /run
instead.  This aligns better with FHS and makes it easier to handle permissions
safely if/when we decide to change uid/gid of the param service.

Signed-off-by: Esben Haabendal <esben@geanix.com>

This ensures that when used in a systemd service, other units that depend on it
will only been started when the param_service is actually ready to respond to
requests.

Signed-off-by: Esben Haabendal <esben@geanix.com>

With ICU data file installed in /system/usr/ohos_icu this crash doesn't happen,
so we can safely drop this workaround.

Signed-off-by: Esben Haabendal <esben@geanix.com>

Signed-off-by: Esben Haabendal <esben@geanix.com>

add and apply actsfeatureabilitytest-wait-between-testcases.patch

test/xts/acts/ActsFeatureAbilityTest: wait 8 seconds between each testcase

sometimes feature abilities fail to start because the previous testcase
still has't completely started or terminated its feature ability.
Increasing sleep time from 1 seconds to 8 seconds between each testcase.

Upstream-Status: Pending

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

test/xts/acts/ActsGetWantTest: wait 10 seconds between each testcase

sometimes feature abilities fail to start because the previous testcase
still has't completely started or terminated its feature ability.
Adding a 10 seconds wait time between each testcase.

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- increment testcase timeout and START_ABILITY_TIMEOUT for actsfeatureabilitytest
- adjust testcase timeout from 60 seconds to 20 seconds
- adjust actsbmsmoduleusagerecordtest START_ABILITY_TIMEOUT from 30 to 15 seconds
- rename patch files

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

test/xts/acts: increase testsuite timeouts

sometimes testsuites execution are timing out because
the execution takes longer to finish.
Increasing some testsuite timeouts.

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

foundation/ace/ace_engine: disable graphics

The graphics rendering is slowing the system making some ACTS test suites to timeout.
Since we are aiming for certification for headless system, graphics stack 
should not be required.
Disabling graphics by skipping JSView::CreateComponent() execution.
In the future we will remove this patch when we the graphical part will be needed.

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Adding and apply ParagraphTxt-ComputeLineBreaks-skip-sigsegv-code.patch

ParagraphTxt::ComputeLineBreaks() is causing a segmentation fault for some 
reasons. This is causing hap applications to crash, including ACTS test suites
and status bar.
As a temporary solution we skip the part of the code that
is raising the segfault. The side effect of this is that texts are 
not properly rendered.

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

The previous implementation of HiCollieCppTest used the 'ps -A' command
to check if a process had restarted, which was not reliable.
This patch adds the creation and removal of a file
'x_collie_watchdog_test_file' in the child process that is monitored,
and the test cases now verify the presence and removal of this file to
confirm that the process was restarted.
Also, this commit updates the build recipe in
openharmony-standard_3.0.bb to include the newly created patch file.

Signed-off-by: Kareem Zarka <kareem.zarka@huawei.com>

This happens at least 60 times a second, totalling at least 120 lines per
second. We are disabling them completely, so we don't even flood the debug
log.

Signed-off-by: Esben Haabendal <esben@geanix.com>

This simplifies and improves the hack added to libdrm to handle drivers (like
virtio drm) that does not support DRM_IOCTL_WAIT_VBLANK, and tries to return at
60 Hz points in time.

Signed-off-by: Esben Haabendal <esben@geanix.com>

regenerate patch from the ${S}/base/hiviewdfx/hilog patch directory, and
apply the patch to the subdirectory.
This allows `devtool modify openharmony standard` to properly apply
the patch in the external workspace. 

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Signed-off-by: Esben Haabendal <esben@geanix.com>

Signed-off-by: Esben Haabendal <esben@geanix.com>

The patch was a hacky way to allow starting of demo hap applications from a root user.
The proper way to start the demo apps is by running `aa start ...` from the system user.
For example:
```
$ hdc shell "su system -c 'aa start -a ohos.samples.clock.MainAbility -b ohos.samples.clock'"
```

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

remove test-xts-acts-Start-tests-from-onInit.patch because now the tests are properly
executed even without it. 
Probably after fixing a segfault problem in the graphics stack, the tests execution
have been fixed.

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Fix DefPermission.hap filename typo which is causing a file not found error 
when running ActsContextTest ACTS tests. 
 
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

START_ABILITY_TIMEOUT is 3000ms which is too short in our case, causing some 
testcases in ActsBmsModuleUsageRecordTest to exceed timeout. 
Increasing the timeout to 30 seconds.

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

The default test timeout is 5000ms which is too short, causing some testcases
to exceed timeout. Increasing the timeout to 60 seconds.

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

OpenHarmony uses couple of UIDs/GIDs for running it's services.
Currently in meta-openharmony we have everything run as root, which 
causes some issues in starting abilities. As a temporary workaround
we are removing the Uid check.

Apply to foundation/aafwk/standard

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

RenderText::PerformLayout() is causing a SIGSEGV for some reasons. 
The error appears to be raised in the Measure method(). This is
causing hap applications to crash, including ACTS test suites.
As a temporary solution we are removing the part of the code that
is raising the segfault. The side effect of this is that texts are 
not properly rendered.

Apply patch to foundation/ace/ace_engine

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Currently, due to some services/graphics issues onShow callback is not being
called and therefore tests are not executed. For screen-less devices graphics
stack should not be required and therefore moving test execution to onInit

This patch allows ACTS tests output to be shown on the logs

Signed-off-by: Robert Drab <robert.drab@huawei.com>
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Robert Drab authored 2 years ago and Francesco Pham committed 2 years ago

Signed-off-by: Robert Drab <robert.drab@huawei.com>

This sets NODE_OPTIONS to pass '--openssl-legacy-provider' to node 17
and fixes the error 'digital envelope routines::unsupported'.
This was resulting in ACTS hap package not contaning the manifest and js
assets files.

Even if NODE_OPTIONS is set in the openharmony-standard recipe, it also
has to be set in the nodejs invocation environment because the recipe
environment is ignored in build_js_assets.py.

Signed-off-by: Thierry Escande <thierry.escande@huawei.com>