feat(patch): fix #758, patch cordova success/error with zone.wrap

Author: JiaLiPassion

karma-build.conf.js

@@ -9,7 +9,7 @@
 module.exports = function (config) {
   require('./karma-base.conf.js')(config);
   config.files.push('build/test/wtf_mock.js');
-  config.files.push('build/test/custom_error.js');
+  config.files.push('build/test/test_fake_polyfill.js');
   config.files.push('build/lib/zone.js');
   config.files.push('build/lib/common/promise.js');
   config.files.push('build/lib/common/error-rewrite.js');

lib/browser/browser.ts

@@ -193,4 +193,5 @@ Zone.__load_patch('PromiseRejectionEvent', (global: any, Zone: ZoneType, api: _Z
 Zone.__load_patch('util', (global: any, Zone: ZoneType, api: _ZonePrivate) => {
   api.patchEventTargetMethods = patchEventTargetMethods;
   api.patchOnProperties = patchOnProperties;
+  api.patchMethod = patchMethod;
 });

lib/extra/cordova.ts

@@ -0,0 +1,21 @@
+/**
+ * @license
+ * Copyright Google Inc. All Rights Reserved.
+ *
+ * Use of this source code is governed by an MIT-style license that can be
+ * found in the LICENSE file at https://angular.io/license
+ */
+Zone.__load_patch('cordova', (global: any, Zone: ZoneType, api: _ZonePrivate) => {
+  if (global.cordova) {
+    const nativeExec: Function = api.patchMethod(
+        global.cordova, 'exec', (delegate: Function) => function(self: any, args: any[]) {
+          if (args.length > 0 && typeof args[0] === 'function') {
+            args[0] = Zone.current.wrap(args[0], 'cordova.exec.success');
+          }
+          if (args.length > 1 && typeof args[1] === 'function') {
+            args[1] = Zone.current.wrap(args[1], 'cordova.exec.error');
+          }
+          return nativeExec.apply(self, args);
+        });
+  }
+});

lib/zone.ts

@@ -323,6 +323,10 @@ interface _ZonePrivate {
   patchEventTargetMethods:
       (obj: any, addFnName?: string, removeFnName?: string, metaCreator?: any) => boolean;
   patchOnProperties: (obj: any, properties: string[]) => void;
+  patchMethod:
+      (target: any, name: string,
+       patchFn: (delegate: Function, delegateName: string, name: string) =>
+           (self: any, args: any[]) => any) => Function;
 }
 
 /** @internal */
@@ -1294,7 +1298,8 @@ const Zone: ZoneType = (function(global: any) {
     scheduleMicroTask: scheduleMicroTask,
     showUncaughtError: () => !(Zone as any)[__symbol__('ignoreConsoleErrorUncaughtError')],
     patchEventTargetMethods: () => false,
-    patchOnProperties: noop
+    patchOnProperties: noop,
+    patchMethod: () => noop
   };
   let _currentZoneFrame: _ZoneFrame = {parent: null, zone: new Zone(null, null)};
   let _currentTask: Task = null;

test/browser-zone-setup.ts

@@ -14,4 +14,5 @@ import '../lib/zone-spec/long-stack-trace';
 import '../lib/zone-spec/proxy';
 import '../lib/zone-spec/sync-test';
 import '../lib/zone-spec/task-tracking';
-import '../lib/zone-spec/wtf';
+import '../lib/zone-spec/wtf';
+import '../lib/extra/cordova';

test/browser_entry_point.ts

@@ -22,4 +22,5 @@ import './browser/XMLHttpRequest.spec';
 import './browser/MediaQuery.spec';
 import './browser/Notification.spec';
 import './mocha-patch.spec';
-import './jasmine-patch.spec';
+import './jasmine-patch.spec';
+import './extra/cordova.spec';

test/extra/cordova.spec.ts

@@ -0,0 +1,37 @@
+/**
+ * @license
+ * Copyright Google Inc. All Rights Reserved.
+ *
+ * Use of this source code is governed by an MIT-style license that can be
+ * found in the LICENSE file at https://angular.io/license
+ */
+describe('cordova test', () => {
+  it('cordova.exec() should be patched as macroTask', (done) => {
+    const scheduleSpy = jasmine.createSpy('scheduleTask');
+    const cordova = (window as any).cordova;
+
+    const zone = Zone.current.fork({name: 'cordova'});
+
+    zone.run(() => {
+      cordova.exec(
+          () => {
+            expect(Zone.current.name).toEqual('cordova');
+            done();
+          },
+          () => {
+            fail('should not fail');
+          },
+          'service', 'successAction', ['arg0', 'arg1']);
+
+      cordova.exec(
+          () => {
+            fail('should not success');
+          },
+          () => {
+            expect(Zone.current.name).toEqual('cordova');
+            done();
+          },
+          'service', 'failAction', ['arg0', 'arg1']);
+    });
+  });
+});

test/custom_error.ts renamed to test/test_fake_polyfill.ts

@@ -8,7 +8,22 @@
 
 'use strict';
 (function(global: any) {
+  // add custom properties to Native Error
   const NativeError = global['Error'];
   NativeError.customProperty = 'customProperty';
   NativeError.customFunction = function() {};
+
+  // add fake cordova polyfill for test
+  const fakeCordova = function() {};
+
+  (fakeCordova as any).exec = function(
+      success: Function, error: Function, service: string, action: string, args: any[]) {
+    if (action === 'successAction') {
+      success();
+    } else {
+      error();
+    }
+  };
+
+  global.cordova = fakeCordova;
 })(typeof window === 'object' && window || typeof self === 'object' && self || global);