Commit 952d46de authored by Joshua Pawlicki's avatar Joshua Pawlicki Committed by Chromium LUCI CQ

Updater: fix and enable ReportsActive test on Mac 10.11.

Bug: 1163625
Change-Id: Iaf87321b3ed90fc563f991d61a8300f3c6c7edf0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2614861
Commit-Queue: Sorin Jianu <sorin@chromium.org>
Auto-Submit: Joshua Pawlicki <waffles@chromium.org>
Reviewed-by: default avatarSorin Jianu <sorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#841181}
parent 416dda7d
...@@ -18,5 +18,10 @@ ...@@ -18,5 +18,10 @@
<string>1.0</string> <string>1.0</string>
<key>LSUIElement</key> <key>LSUIElement</key>
<true/> <true/>
<key>NSAppTransportSecurity</key>
<dict>
<key>NSAllowsArbitraryLoads</key>
<true/>
</dict>
</dict> </dict>
</plist> </plist>
...@@ -239,7 +239,7 @@ TEST_F(IntegrationTest, RegisterTestApp) { ...@@ -239,7 +239,7 @@ TEST_F(IntegrationTest, RegisterTestApp) {
// TODO(crbug.com/1163524): Enable on Windows. // TODO(crbug.com/1163524): Enable on Windows.
// TODO(crbug.com/1163625): Failing on Mac 10.11. // TODO(crbug.com/1163625): Failing on Mac 10.11.
TEST_F(IntegrationTest, DISABLED_ReportsActive) { TEST_F(IntegrationTest, ReportsActive) {
// A longer than usual timeout is needed for this test because the macOS // A longer than usual timeout is needed for this test because the macOS
// UpdateServiceInternal server takes at least 10 seconds to shut down after // UpdateServiceInternal server takes at least 10 seconds to shut down after
// Install, and RegisterApp cannot make progress until it shut downs and // Install, and RegisterApp cannot make progress until it shut downs and
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment