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

START_ABILITY_TIMEOUT is too short, causing some 
testcases in ActsBmsModuleUsageRecordTest to exceed timeout. 
Increasing the timeout to 15 seconds.

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

The HDF peripherals drivers for wlan had been patched out
before. Enable them again in the build and package the
needed files.

They do not end up being installed in the normal libdir so
we pick them from the build dir instead.

Signed-off-by: Stefan Schmidt <stefan.schmidt@huawei.com>

add InnerBundleInfo-initialize-installationFree.patch
apply to foundation/appexecfwk/standard

```
InnerModuleInfo: initialize installationFree to false

uninitialized installationFree property of InnerModuleInfo causes 
ActsBmsModuleUsageRecordTest to sometimes fail on 
`expect(data.installationFreeSupported).assertEqual(false);`
Initializing installationFree property to false.
```

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

add appdatamgr-IsColumnNull-initialize-result.patch and apply to
foundation/distributeddatamgr/appdatamgr

```
testIsColumnNull0003 ACTS testcase sometimes fail because the
result value of IsColumnNull is unpredictable when an error is returned.
This is because AbsSharedResultSet::IsColumnNull does not set any
value to &isNull when there is an error. The result value is
not initialized, producing an unpredictable output of IsColumnNull.
```

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

adding test-xts-acts-testRdbStoreUpdate0001-await-updatePromise.patch and apply to test/xts/acts .

```
The testRdbStoreUpdate0001 testcase which is part of ActsAppdatamagrJsTest
sometimes fails (in our case on raspberrypi4-64). The reason is that
the values in rdbStore are inspected before the update is completed.
The `await updatePromise` command that is clearly needed was commented out
for some unknown reason. Reintroducing the await command to avoid race
conditions.
```

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

On raspberry pi with VC4 drm graphics driver DRM_IOCTL_WAIT_VBLANK
is failing returning EINVAL for some reason, applying  drmWaitVBlank hack
also on EINVAL until a proper solution is provided.

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

This reverts commit 312c51f4.

We are seeing multiple different race conditions causing random failures in
ACTS, many of them (maybe all) goes away when we stub out the mostly stub out
JSView::CreateComponent()

As before, this should be dropped again later on when the problems have been
solved.  And is required in order to get graphics support to work.

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

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>