- cleaned up objc generation

- added ability to generate mock handlers
- added pigeon.dart

Author: gaaclarke

packages/pigeon/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 0.1.0
+
+* Added pigeon.dart.
+* Fixed some Obj-C linter problems.
+* Added the ability to generate a mock handler in Dart.
+
 ## 0.1.0-experimental.11
 
 * Fixed setting an api to null in Java.

packages/pigeon/README.md

@@ -44,14 +44,14 @@ doesn't need to worry about conflicting versions of Pigeon.
 
 ### Rules for defining your communication interface
 
-1) The file should contain no methods or function definitions.
+1) The file should contain no method or function definitions.
 1) Datatypes are defined as classes with fields of the supported datatypes (see
    the supported Datatypes section).
 1) Api's should be defined as an `abstract class` with either `HostApi()` or
    `FlutterApi()` as metadata.  The former being for procedures that are defined
    on the host platform and the latter for procedures that are defined in Dart.
 1) Method declarations on the Api classes should have one argument and a return
-   value whose types are defined in the file.
+   value whose types are defined in the file or be `void`.
 
 ## Example
 

packages/pigeon/e2e_tests/test_objc/android/.settings/org.eclipse.buildship.core.prefs

@@ -1,2 +1,13 @@
+arguments=
+auto.sync=false
+build.scans.enabled=false
+connection.gradle.distribution=GRADLE_DISTRIBUTION(WRAPPER)
 connection.project.dir=
 eclipse.preferences.version=1
+gradle.user.home=
+java.home=/Library/Java/JavaVirtualMachines/jdk-11.0.4.jdk/Contents/Home
+jvm.arguments=
+offline.mode=false
+override.workspace.settings=true
+show.console.view=true
+show.executions.view=true

packages/pigeon/e2e_tests/test_objc/android/app/.classpath

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <classpath>
-	<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-10/"/>
+	<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-11/"/>
 	<classpathentry kind="con" path="org.eclipse.buildship.core.gradleclasspathcontainer"/>
 	<classpathentry kind="output" path="bin/default"/>
 </classpath>

packages/pigeon/e2e_tests/test_objc/ios/Runner/dartle.h

@@ -1,4 +1,4 @@
-// Autogenerated from Pigeon (v0.1.0-experimental.11), do not edit directly.
+// Autogenerated from Pigeon (v0.1.0), do not edit directly.
 // See also: https://pub.dev/packages/pigeon
 #import <Foundation/Foundation.h>
 @protocol FlutterBinaryMessenger;
@@ -31,15 +31,17 @@ NS_ASSUME_NONNULL_BEGIN
 - (void)search:(ACSearchRequest *)input completion:(void (^)(ACSearchReply *, NSError *))completion;
 @end
 @protocol ACNestedApi
-- (ACSearchReply *)search:(ACNested *)input error:(FlutterError *_Nullable *_Nonnull)error;
+- (nullable ACSearchReply *)search:(ACNested *)input error:(FlutterError *_Nullable *_Nonnull)error;
 @end
 
-extern void ACNestedApiSetup(id<FlutterBinaryMessenger> binaryMessenger, id<ACNestedApi> api);
+extern void ACNestedApiSetup(id<FlutterBinaryMessenger> binaryMessenger,
+                             id<ACNestedApi> _Nullable api);
 
 @protocol ACApi
-- (ACSearchReply *)search:(ACSearchRequest *)input error:(FlutterError *_Nullable *_Nonnull)error;
+- (nullable ACSearchReply *)search:(ACSearchRequest *)input
+                             error:(FlutterError *_Nullable *_Nonnull)error;
 @end
 
-extern void ACApiSetup(id<FlutterBinaryMessenger> binaryMessenger, id<ACApi> api);
+extern void ACApiSetup(id<FlutterBinaryMessenger> binaryMessenger, id<ACApi> _Nullable api);
 
 NS_ASSUME_NONNULL_END

packages/pigeon/e2e_tests/test_objc/ios/Runner/dartle.m

@@ -1,4 +1,4 @@
-// Autogenerated from Pigeon (v0.1.0-experimental.11), do not edit directly.
+// Autogenerated from Pigeon (v0.1.0), do not edit directly.
 // See also: https://pub.dev/packages/pigeon
 #import "dartle.h"
 #import <Flutter/Flutter.h>

packages/pigeon/e2e_tests/test_objc/lib/dartle.dart

@@ -1,4 +1,4 @@
-// Autogenerated from Pigeon (v0.1.0-experimental.11), do not edit directly.
+// Autogenerated from Pigeon (v0.1.0), do not edit directly.
 // See also: https://pub.dev/packages/pigeon
 // ignore_for_file: public_member_api_docs, non_constant_identifier_names, avoid_as, unused_import
 import 'dart:async';
@@ -128,3 +128,20 @@ class Api {
     }
   }
 }
+
+abstract class MockApi {
+  SearchReply search(SearchRequest arg);
+}
+
+void MockApiSetup(MockApi api) {
+  {
+    const BasicMessageChannel<dynamic> channel = BasicMessageChannel<dynamic>(
+        'dev.flutter.pigeon.Api.search', StandardMessageCodec());
+    channel.setMockMessageHandler((dynamic message) async {
+      final Map<dynamic, dynamic> mapMessage = message as Map<dynamic, dynamic>;
+      final SearchRequest input = SearchRequest._fromMap(mapMessage);
+      final SearchReply output = api.search(input);
+      return <dynamic, dynamic>{'result': output._toMap()};
+    });
+  }
+}

packages/pigeon/example/README.md

@@ -3,7 +3,7 @@
 ## message.dart
 
 ```dart
-import 'package:pigeon/pigeon_lib.dart';
+import 'package:pigeon/pigeon.dart';
 
 class SearchRequest {
   String query;

packages/pigeon/lib/ast.dart

@@ -32,7 +32,7 @@ class Method extends Node {
 /// Represents a collection of [Method]s that are hosted ona given [location].
 class Api extends Node {
   /// Parametric constructor for [Api].
-  Api({this.name, this.location, this.methods});
+  Api({this.name, this.location, this.methods, this.mockDartHandler});
 
   /// The name of the API.
   String name;
@@ -42,6 +42,9 @@ class Api extends Node {
 
   /// List of methods inside the API.
   List<Method> methods;
+
+  /// The name of the Dart mock interface to generate to help with testing.
+  String mockDartHandler;
 }
 
 /// Represents a field on a [Class].

packages/pigeon/lib/dart_generator.dart

@@ -60,7 +60,8 @@ if (replyMap == null) {
   indent.writeln('');
 }
 
-void _writeFlutterApi(Indent indent, Api api) {
+void _writeFlutterApi(Indent indent, Api api,
+    {String Function(Method) channelNameFunc, bool isMockHandler = false}) {
   assert(api.location == ApiLocation.flutter);
   indent.write('abstract class ${api.name} ');
   indent.scoped('{', '}', () {
@@ -79,11 +80,16 @@ void _writeFlutterApi(Indent indent, Api api) {
         indent.writeln('const BasicMessageChannel<dynamic> channel =');
         indent.inc();
         indent.inc();
+        final String channelName = channelNameFunc == null
+            ? makeChannelName(api, func)
+            : channelNameFunc(func);
         indent.writeln(
-            'BasicMessageChannel<dynamic>(\'${makeChannelName(api, func)}\', StandardMessageCodec());');
+            'BasicMessageChannel<dynamic>(\'$channelName\', StandardMessageCodec());');
         indent.dec();
         indent.dec();
-        indent.write('channel.setMessageHandler((dynamic message) async ');
+        final String messageHandlerSetter =
+            isMockHandler ? 'setMockMessageHandler' : 'setMessageHandler';
+        indent.write('channel.$messageHandlerSetter((dynamic message) async ');
         indent.scoped('{', '});', () {
           final String argType = func.argType;
           final String returnType = func.returnType;
@@ -99,9 +105,16 @@ void _writeFlutterApi(Indent indent, Api api) {
           }
           if (returnType == 'void') {
             indent.writeln('$call;');
+            if (isMockHandler) {
+              indent.writeln('return <dynamic, dynamic>{};');
+            }
           } else {
             indent.writeln('final $returnType output = $call;');
-            indent.writeln('return output._toMap();');
+            const String returnExpresion = 'output._toMap()';
+            final String returnStatement = isMockHandler
+                ? 'return <dynamic, dynamic>{\'${Keys.result}\': $returnExpresion};'
+                : 'return $returnExpresion;';
+            indent.writeln(returnStatement);
           }
         });
       });
@@ -166,6 +179,15 @@ void generateDart(Root root, StringSink sink) {
   for (Api api in root.apis) {
     if (api.location == ApiLocation.host) {
       _writeHostApi(indent, api);
+      if (api.mockDartHandler != null) {
+        final Api mockApi = Api(
+            name: api.mockDartHandler,
+            methods: api.methods,
+            location: ApiLocation.flutter);
+        _writeFlutterApi(indent, mockApi,
+            channelNameFunc: (Method func) => makeChannelName(api, func),
+            isMockHandler: true);
+      }
     } else if (api.location == ApiLocation.flutter) {
       _writeFlutterApi(indent, api);
     }