• Lindsay Pasricha's avatar
    Revert "Enable iOS test runner unit tests as part of presubmit" · fdd9f7c6
    Lindsay Pasricha authored
    This reverts commit 7d6c5197.
    
    Reason for revert: This breaks on device runs for xcodebuild_runner.
    This revert will land before the revert of crrev/c/2048851.
    
    Original change's description:
    > Enable iOS test runner unit tests as part of presubmit
    > 
    > Run iOS test runner unit tests as part of presubmit. Keeping scope to
    > functional tests, and will have separate CLs for the remaining files
    > that require further fixes.
    > 
    > * PRESUBMIT.py - include unit test execution for ios/build/bots/scripts
    > * run.py - unit tests caught logic incorrectness. Running the json
    >    parsing first, before validation.
    > * wpr_runner_test.py - remove install xcode mock, fixed broken tests
    > * test_apps_test.py and xcode_log_parser_test.py - include main call
    >    to unittest.
    > 
    > Bug: 1056457,1058257
    > Change-Id: Ifc233695ba8f6120f862586a497df61e96d4954f
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2076765
    > Reviewed-by: Rohit Rao <rohitrao@chromium.org>
    > Commit-Queue: Jeff Yoon <jeffyoon@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#746102}
    
    TBR=rohitrao@chromium.org,lindsayw@chromium.org,zhaoyangli@chromium.org,jeffyoon@chromium.org
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Bug: 1056457
    Change-Id: Iaa9e1ca8a6ac76c41e067d17e13108aeeb2ac83a
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2089992Reviewed-by: default avatarRohit Rao <rohitrao@chromium.org>
    Commit-Queue: Lindsay Pasricha <lindsayw@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#747396}
    fdd9f7c6
wpr_runner_test.py 7 KB