New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 732007 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Xcode 9 chrome egtest integration doesn't find any tests

Project Member Reported by justincohen@chromium.org, Jun 10 2017

Issue description

./out/Debug-iphonesimulator/iossim out/Debug-iphonesimulator/ios_chrome_integration_egtests.app/ out/Debug-iphonesimulator/ios_chrome_integration_egtests.app/PlugIns/ios_chrome_integration_egtests_module.xctest/

Test Suite 'Selected tests' passed at 2017-06-09 22:14:50.267.
	 Executed 0 tests, with 0 failures (0 unexpected) in 0.000 (0.002) seconds
** TEST EXECUTE SUCCEEDED **
 
hmm, looks like Apple has closed the hole that tests in application's namespace can be found by Xcode.
Xcode seems to run our tests fine, presumably there's something wrong with iossim's wrapping of the xcodebuild test cmd line.
Owner: liaoyuke@chromium.org
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 14 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d40236c61af27c3a50474415203c9ac26e2d725e

commit d40236c61af27c3a50474415203c9ac26e2d725e
Author: Yuke Liao <liaoyuke@chromium.org>
Date: Wed Jun 14 18:36:02 2017

Fix an "only testing tests" bug in iossim.

In iossim.mm, following code is buggy:

if (only_testing_tests) {
  [testTargetName setObject:only_testing_tests
      forKey:@"OnlyTestIdentifiers"];
}

because only_testing_tests is an initialized NSMutableArray, which will
always be evaluated to true, so changing the condition to:

if ([only_testing_tests count] > 0)

Bug:  732007 
Change-Id: I48014956c971fdf744ff38cf3975008f40061133
Reviewed-on: https://chromium-review.googlesource.com/534944
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Reviewed-by: Justin Cohen <justincohen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#479452}
[modify] https://crrev.com/d40236c61af27c3a50474415203c9ac26e2d725e/testing/iossim/iossim.mm

Status: Fixed (was: Assigned)
Fixed, this is caused by a bug in iossim.mm, please refer to the CL for details.

Sign in to add a comment