Open jpiesing opened 6 days ago
@afrawei
If I comment out the check in hbbtv.js in the test runner then the test runs successfully in this context.
Or indeed this from a different TV set. 390ec70c-cfhd_12.5_25_50-local-Xx01.json
I pushed a fix for this, however, I am unable to test if the fix actually works: https://github.com/cta-wave/dpctf-tests/pull/197
@afrawei the fixes @FritzHeiden pushed are not tagged yet as a release in dpctf-tests repo in order to test these changes please follow these steps (if everything works for your, we can provide a release).
./build.sh master hbbtvhotfix --tests-branch master
(this will create a docker image tagged as hbbtvhotfix
)image: dpctf:hbbtvhotfix
docker-compose up --force-recreate
@FritzHeiden @louaybassbouss I got below error with the hotfix ctvwave@EUBENB190005:~/dpctf-deploy$ docker image ls REPOSITORY TAG IMAGE ID CREATED SIZE dpctf v2.0.0 217163d985e4 3 minutes ago 1.6GB dpctf hbbtvhotfix dde982a2ec9e 9 minutes ago 1.6GB ctvwave@EUBENB190005:~/dpctf-deploy$ docker-compose up --force-recreate Recreating 0433d8cc88e2_dpctf ...
ERROR: for 0433d8cc88e2_dpctf 'ContainerConfig'
ERROR: for dpctf 'ContainerConfig'
Traceback (most recent call last):
File "/usr/bin/docker-compose", line 33, in
@louaybassbouss after I followed your instruction, how do I rebuild the original image? I got the same error as above when using "./build.sh"
@FritzHeiden @louaybassbouss After removing all images and redo the commands, issue is still seen.
663ffc78_results_json.zip
@afrawei Did you use this command to build the image? ./build.sh master hbbtvhotfix --tests-branch master
The fix is not included if you just use ./build.sh
@FritzHeiden yes, I did the command to build the image. ./build.sh master hbbtvhotfix --tests-branch master
This looks good so far. The command builds an image with latest changes in all repository and puts the tag hbbtvhotfix
on it. So you also need to tell docker to use that tag in the docker-compose.yml
. Did you change image to dpctf:hbbtvhotfix
?
@FritzHeiden Yes, I also did the change to yml.
The generated
directory did not contain the fix, so it didn't appear in your build. I pushed changes to fix this. Please rebuild the image using ./build.sh master hbbtvhotfix --tests-branch master --reload-tests
The test runner fails on HbbTV if the app has been launched as broadcast-independent, e.g. by a manufacturer back-door into the system.
The failure is reported in the .json reports as a security error when attempting to stop the broadcast.
hbbtv.js needs to be modified to catch the following error and just continue.