With the upcoming support for wdspec tests on Android the following test fails because the wrong platform name is assumed: https://firefoxci.taskcluster-artifacts.net/ZudT0eAHQSKdfO1IywbDqg/1/public/logs/live_backing.log ``` [task 2022-03-29T12:46:02.701Z] 12:46:02 INFO - STDOUT: @pytest.mark.skipif(platform_name is None, reason="Unsupported platform {}".format(platform_name)) [task 2022-03-29T12:46:02.701Z] 12:46:02 INFO - STDOUT: def test_corresponds_to_local_system(new_session, add_browser_capabilities): [task 2022-03-29T12:46:02.701Z] 12:46:02 INFO - STDOUT: response, _ = new_session({"capabilities": {"alwaysMatch": add_browser_capabilities({})}}) [task 2022-03-29T12:46:02.702Z] 12:46:02 INFO - STDOUT: value = assert_success(response) [task 2022-03-29T12:46:02.702Z] 12:46:02 INFO - STDOUT: > assert value["capabilities"]["platformName"] == platform_name [task 2022-03-29T12:46:02.702Z] 12:46:02 INFO - STDOUT: E AssertionError: assert 'android' == 'linux' [task 2022-03-29T12:46:02.702Z] 12:46:02 INFO - STDOUT: E - linux [task 2022-03-29T12:46:02.702Z] 12:46:02 INFO - STDOUT: E + android ``` We should not assume that the `platformName` capability will be equal to the host system. I'm going to disable this test for now.
Bug 1762131 Comment 0 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
With the upcoming support for wdspec tests on Android the following test fails because the wrong platform name is assumed: https://firefoxci.taskcluster-artifacts.net/ZudT0eAHQSKdfO1IywbDqg/1/public/logs/live_backing.log ``` [task 2022-03-29T12:46:02.701Z] 12:46:02 INFO - STDOUT: @pytest.mark.skipif(platform_name is None, reason="Unsupported platform {}".format(platform_name)) [task 2022-03-29T12:46:02.701Z] 12:46:02 INFO - STDOUT: def test_corresponds_to_local_system(new_session, add_browser_capabilities): [task 2022-03-29T12:46:02.701Z] 12:46:02 INFO - STDOUT: response, _ = new_session({"capabilities": {"alwaysMatch": add_browser_capabilities({})}}) [task 2022-03-29T12:46:02.702Z] 12:46:02 INFO - STDOUT: value = assert_success(response) [task 2022-03-29T12:46:02.702Z] 12:46:02 INFO - STDOUT: > assert value["capabilities"]["platformName"] == platform_name [task 2022-03-29T12:46:02.702Z] 12:46:02 INFO - STDOUT: E AssertionError: assert 'android' == 'linux' [task 2022-03-29T12:46:02.702Z] 12:46:02 INFO - STDOUT: E - linux [task 2022-03-29T12:46:02.702Z] 12:46:02 INFO - STDOUT: E + android ``` We should not assume that the `platformName` capability will be equal to the host system. I'm going to disable this test for now via bug 1749444.