search

cta-wave / dpctf-tests

Repo for DPCTF Tests. We prefer to keep the Tests separated from Test Runner
Other
2 stars 5 forks source link

"message": "Script error.", #126

Closed jpiesing closed 11 months ago

jpiesing commented 1 year ago

What should be done about these?

        "message": "Script error.", 
        "message": "Script error.", 
        "message": "Script error.",

How significant are they?

louaybassbouss commented 1 year ago

This is raised on an HbbTV set and not to be related to the test. The test results are not affected by this.

jpiesing commented 1 year ago

This is raised on an HbbTV set and not to be related to the test. The test results are not affected by this.

Is it reasonable to catch this instead of generating an error in the test report? This error will be very confusing to people.

louaybassbouss commented 1 year ago

This is raised on an HbbTV set and not to be related to the test. The test results are not affected by this.

Is it reasonable to catch this instead of generating an error in the test report? This error will be very confusing to people.

It is not possible since the error is triggered by the HbbTV UA on only a TV set and not from the testing code. We know this error from the other HbbTV dev on the specific TV.

jpiesing commented 1 year ago

If you can't catch it, can you add it to the documentation somewhere?

gitwjr commented 1 year ago

The script errors cannot be caught. Needs to be documented that some implementations generate the error. @louaybassbouss Fraunhofer to document in test execution section of Deploy. @louaybassbouss

FritzHeiden commented 11 months ago

Added documentation for this (https://github.com/cta-wave/dpctf-tests/pull/141)

gitwjr commented 11 months ago

Fixed.