mirror of
https://github.com/flutter/flutter.git
synced 2026-02-20 02:29:02 +08:00
Sources under `packages/flutter_tools/` aren't accessible to the average Flutter user by navigating through sources from their projects, so it doesn't need to be as explicitly verbose with types for readability purposes. The `always_specify_types` lint results in extremely verbose code within the tool which adds little value. This change disables `always_specify_types` in favor of a new set of lints that aim to reduce verbosity by removing obvious types while also maintaining readability in cases where variable types otherwise wouldn't be obvious: - `omit_obvious_local_variable_types` - `omit_obvious_property_types` - `specify_nonobvious_local_variable_types` - `specify_nonobvious_property_types`
162 lines
5.1 KiB
Dart
162 lines
5.1 KiB
Dart
// Copyright 2014 The Flutter Authors. All rights reserved.
|
|
// Use of this source code is governed by a BSD-style license that can be
|
|
// found in the LICENSE file.
|
|
|
|
import 'package:file/file.dart';
|
|
import 'package:file/memory.dart';
|
|
import 'package:flutter_tools/src/platform_plugins.dart';
|
|
|
|
import '../src/common.dart';
|
|
|
|
void main() {
|
|
testWithoutContext(
|
|
'AndroidPlugin throws tool exit if the plugin main class can not be found',
|
|
() {
|
|
final FileSystem fileSystem = MemoryFileSystem.test();
|
|
final androidPlugin = AndroidPlugin(
|
|
name: 'pluginA',
|
|
package: 'com.company',
|
|
pluginClass: 'PluginA',
|
|
pluginPath: '.pub_cache/plugin_a',
|
|
fileSystem: fileSystem,
|
|
);
|
|
|
|
expect(
|
|
() => androidPlugin.toMap(),
|
|
throwsToolExit(
|
|
message:
|
|
"The plugin `pluginA` doesn't have a main class defined in "
|
|
'.pub_cache/plugin_a/android/src/main/java/com/company/PluginA.java '
|
|
'or .pub_cache/plugin_a/android/src/main/kotlin/com/company/PluginA.kt',
|
|
),
|
|
);
|
|
},
|
|
);
|
|
|
|
testWithoutContext('AndroidPlugin does not validate the main class for Dart-only plugins', () {
|
|
final FileSystem fileSystem = MemoryFileSystem.test();
|
|
final androidPlugin = AndroidPlugin(
|
|
name: 'pluginA',
|
|
dartPluginClass: 'PluginA',
|
|
pluginPath: '.pub_cache/plugin_a',
|
|
fileSystem: fileSystem,
|
|
);
|
|
|
|
expect(androidPlugin.toMap(), <String, Object>{
|
|
'name': 'pluginA',
|
|
'dartPluginClass': 'PluginA',
|
|
'supportsEmbeddingV1': false,
|
|
'supportsEmbeddingV2': false,
|
|
});
|
|
});
|
|
|
|
testWithoutContext('AndroidPlugin does not validate the main class for default_package', () {
|
|
final FileSystem fileSystem = MemoryFileSystem.test();
|
|
final androidPlugin = AndroidPlugin(
|
|
name: 'pluginA',
|
|
defaultPackage: 'plugin_a_android',
|
|
pluginPath: '.pub_cache/plugin_a',
|
|
fileSystem: fileSystem,
|
|
);
|
|
|
|
expect(androidPlugin.toMap(), <String, Object>{
|
|
'name': 'pluginA',
|
|
'default_package': 'plugin_a_android',
|
|
'supportsEmbeddingV1': false,
|
|
'supportsEmbeddingV2': false,
|
|
});
|
|
});
|
|
|
|
testWithoutContext('AndroidPlugin parses embedding version 2 from the Java search path', () {
|
|
final FileSystem fileSystem = MemoryFileSystem.test();
|
|
final androidPlugin = AndroidPlugin(
|
|
name: 'pluginA',
|
|
package: 'com.company',
|
|
pluginClass: 'PluginA',
|
|
pluginPath: '.pub_cache/plugin_a',
|
|
fileSystem: fileSystem,
|
|
);
|
|
|
|
fileSystem.file('.pub_cache/plugin_a/android/src/main/java/com/company/PluginA.java')
|
|
..createSync(recursive: true)
|
|
..writeAsStringSync('io.flutter.embedding.engine.plugins.FlutterPlugin');
|
|
|
|
expect(androidPlugin.toMap(), <String, Object>{
|
|
'name': 'pluginA',
|
|
'package': 'com.company',
|
|
'class': 'PluginA',
|
|
'supportsEmbeddingV1': false,
|
|
'supportsEmbeddingV2': true,
|
|
});
|
|
});
|
|
|
|
testWithoutContext('AndroidPlugin parses embedding version 1 from the Java search path', () {
|
|
final FileSystem fileSystem = MemoryFileSystem.test();
|
|
final androidPlugin = AndroidPlugin(
|
|
name: 'pluginA',
|
|
package: 'com.company',
|
|
pluginClass: 'PluginA',
|
|
pluginPath: '.pub_cache/plugin_a',
|
|
fileSystem: fileSystem,
|
|
);
|
|
|
|
fileSystem.file('.pub_cache/plugin_a/android/src/main/java/com/company/PluginA.java')
|
|
..createSync(recursive: true)
|
|
..writeAsStringSync('some.other.string');
|
|
|
|
expect(androidPlugin.toMap(), <String, Object>{
|
|
'name': 'pluginA',
|
|
'package': 'com.company',
|
|
'class': 'PluginA',
|
|
'supportsEmbeddingV1': true,
|
|
'supportsEmbeddingV2': false,
|
|
});
|
|
});
|
|
|
|
testWithoutContext('AndroidPlugin parses embedding version 2 from the Kotlin search path', () {
|
|
final FileSystem fileSystem = MemoryFileSystem.test();
|
|
final androidPlugin = AndroidPlugin(
|
|
name: 'pluginA',
|
|
package: 'com.company',
|
|
pluginClass: 'PluginA',
|
|
pluginPath: '.pub_cache/plugin_a',
|
|
fileSystem: fileSystem,
|
|
);
|
|
|
|
fileSystem.file('.pub_cache/plugin_a/android/src/main/kotlin/com/company/PluginA.kt')
|
|
..createSync(recursive: true)
|
|
..writeAsStringSync('io.flutter.embedding.engine.plugins.FlutterPlugin');
|
|
|
|
expect(androidPlugin.toMap(), <String, Object>{
|
|
'name': 'pluginA',
|
|
'package': 'com.company',
|
|
'class': 'PluginA',
|
|
'supportsEmbeddingV1': false,
|
|
'supportsEmbeddingV2': true,
|
|
});
|
|
});
|
|
|
|
testWithoutContext('AndroidPlugin parses embedding version 1 from the Kotlin search path', () {
|
|
final FileSystem fileSystem = MemoryFileSystem.test();
|
|
final androidPlugin = AndroidPlugin(
|
|
name: 'pluginA',
|
|
package: 'com.company',
|
|
pluginClass: 'PluginA',
|
|
pluginPath: '.pub_cache/plugin_a',
|
|
fileSystem: fileSystem,
|
|
);
|
|
|
|
fileSystem.file('.pub_cache/plugin_a/android/src/main/kotlin/com/company/PluginA.kt')
|
|
..createSync(recursive: true)
|
|
..writeAsStringSync('some.other.string');
|
|
|
|
expect(androidPlugin.toMap(), <String, Object>{
|
|
'name': 'pluginA',
|
|
'package': 'com.company',
|
|
'class': 'PluginA',
|
|
'supportsEmbeddingV1': true,
|
|
'supportsEmbeddingV2': false,
|
|
});
|
|
});
|
|
}
|