[plugin_platform_interface] Test that PlatformInterface and `MockPl…

…atformInterfaceMixinTest` have no instance methods (#4659)
flutter · Jan 20, 2022 · c1bdc81 · c1bdc81
1 parent 3e4483d
commit c1bdc81
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/packages/plugin_platform_interface/CHANGELOG.md b/packages/plugin_platform_interface/CHANGELOG.md
@@ -1,3 +1,7 @@
+## NEXT
+
+* Adds additional tests for `PlatformInterface` and `MockPlatformInterfaceMixin`.
+
 ## 2.1.2
 
 * Updates README to demonstrate `verify` rather than `verifyToken`, and to note

diff --git a/packages/plugin_platform_interface/test/plugin_platform_interface_test.dart b/packages/plugin_platform_interface/test/plugin_platform_interface_test.dart
@@ -78,6 +78,14 @@ class ImplementsConstVerifyTokenPluginPlatform extends PlatformInterface
   ImplementsConstVerifyTokenPluginPlatform() : super(token: const Object());
 }
 
+// Ensures that `PlatformInterface` has no instance methods. Adding an
+// instance method is discouraged and may be a breaking change if it
+// conflicts with instance methods in subclasses.
+class StaticMethodsOnlyPlatformInterfaceTest implements PlatformInterface {}
+
+class StaticMethodsOnlyMockPlatformInterfaceMixinTest
+    implements MockPlatformInterfaceMixin {}
+
 void main() {
   group('`verify`', () {
     test('prevents implementation with `implements`', () {