Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

[flutter_plugin_tests] Split analyze out of xctest #4161

Merged
stuartmorgan-g merged 10 commits into from
Jul 21, 2021
Merged

[flutter_plugin_tests] Split analyze out of xctest #4161

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
3611f35
Add more options to control what runs
stuartmorgan-g Jul 15, 2021
47e9d90
Report skip, not fail, when analyze is off and there are no tests
stuartmorgan-g Jul 15, 2021
33940f5
Extract a helper for Xcode commands, and add tests for it
stuartmorgan-g Jul 15, 2021
aba3199
Migrate xctest to the Xcode class
stuartmorgan-g Jul 15, 2021
fd1958a
Split out xcode-analyze
stuartmorgan-g Jul 16, 2021
b27265a
Comment fix
stuartmorgan-g Jul 16, 2021
1a357e5
Remove unnecessary process mocking
stuartmorgan-g Jul 16, 2021
02945c9
Skip, rather than failing, when a package doesn't have a requested te…
stuartmorgan-g Jul 16, 2021
b4be22c
Merge branch 'master' into xctest-refactor
stuartmorgan-g Jul 20, 2021
be1596a
Filter out unavailable via simctl
stuartmorgan-g Jul 20, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Comment fix
  • Loading branch information
@stuartmorgan-g
stuartmorgan-g committed Jul 16, 2021
commit b27265ad11dc378bae18d9a3e953a6ebf8c22a25
10 changes: 3 additions & 7 deletions script/tool/lib/src/xcode_analyze_command.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,11 +11,7 @@ import 'common/plugin_utils.dart';
import 'common/process_runner.dart';
import 'common/xcode.dart';

/// The command to run XCTests (XCUnitTest and XCUITest) in plugins.
/// The tests target have to be added to the Xcode project of the example app,
/// usually at "example/{ios,macos}/Runner.xcworkspace".
///
/// The static analyzer is also run.
/// The command to run Xcode's static analyzer on plugins.
class XcodeAnalyzeCommand extends PackageLoopingCommand {
/// Creates an instance of the test command.
XcodeAnalyzeCommand(
Expand All @@ -24,8 +20,8 @@ class XcodeAnalyzeCommand extends PackageLoopingCommand {
Platform platform = const LocalPlatform(),
}) : _xcode = Xcode(processRunner: processRunner, log: true),
super(packagesDir, processRunner: processRunner, platform: platform) {
argParser.addFlag(kPlatformIos, help: 'Analyzes iOS');
argParser.addFlag(kPlatformMacos, help: 'Analyzes macOS');
argParser.addFlag(kPlatformIos, help: 'Analyze iOS');
argParser.addFlag(kPlatformMacos, help: 'Analyze macOS');
}

final Xcode _xcode;
Expand Down