Disable OnServiceProtocolGetSkSLsWorks in Fuchsia

[liyuqian]

That unit test depends on fml::WriteAtomically. Such file operations
does not seem to be well supported as FileTest is also disabled in
Fuchsia.

[chinmaygarde]

I didn't see this pending PR and landed #17367 instead to unblock LUCI. @liyuqian: For future reference, the "waiting for tree to go green" tag cannot be used to unblock LUCI and you need to force a push ignoring presubmits to break the deadlock.

That other patch disables the test entirely BTW. If you think this is the more precise way of handling the failure. Feel free to revert that and land this instead (the tree will be green when that lands).

[liyuqian]

@chinmaygarde : I rebased this PR. Now this PR reverts your PR so the test is only skipped on Fuchsia and still gets run on non-Fuchsia platforms.

[chinmaygarde]

Sounds good. But the "Linux Fuchsia" presubmit is failing on the same check now. Are you sure the flag is working?

[liyuqian]

@chinmaygarde : I see a different failure

Infra Failure
recipe infra failure: Swarming task failed: (…) [03682.737324][12214][12216][pkg-resolver] ERROR: ../../src/sys/pkg/bin/pkg-resolver/src/repository/updating_tuf_client.rs(288): AutoClient for "http://192.168.42.1:33633/auto" event stream read error: NextChunk(Error { kind: Body, cause: Custom { kind: UnexpectedEof, error: "Unexpected eof during chunk size line" } }) [03682.756256][12214][12216][pkg-resolver] ERROR: ../../src/sys/pkg/bin/pkg-resolver/src/repository_manager.rs(258): while fetching package fuchsia-pkg://eb4c071e-62aa-425d-aef5-883e925d50ae/log_listener_shell/0 using rust tuf: while looking up merkle root for package: the package was not found in the repository

==================================== STDERR ====================================

==================================== STDOUT ====================================

==================================== STDERR ====================================

Will rerun and check it again. Rerun in https://ci.chromium.org/p/flutter/builders/try/Linux%20Fuchsia/6115

[chinmaygarde]

It looks like an infra failure but its not. Unfortunately, you have to click through to the task UI for the unit-test to see the failure. It is https://chromium-swarm.appspot.com/task?id=4b3614488e2c1610

[liyuqian]

@chinmaygarde : you're right. The gtest_filter doesn't seem to work as expected. Let me try to spell the full name of the test and see if that helps.

[liyuqian]

Finally I figured out that I put the filter on the wrong test... It should be applied to shell_tests instead of fml_tests.

[gw280]

Finally I figured out that I put the filter on the wrong test... It should be applied to shell_tests instead of fml_tests.

We've all been there :) Thanks for sorting this out!