You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by ja...@apache.org on 2019/04/12 12:36:21 UTC

[cordova-paramedic] branch janpio-saucelabs_ios_10plus updated: fix: workaround recursive object error (#122)

This is an automated email from the ASF dual-hosted git repository.

janpio pushed a commit to branch janpio-saucelabs_ios_10plus
in repository https://gitbox.apache.org/repos/asf/cordova-paramedic.git


The following commit(s) were added to refs/heads/janpio-saucelabs_ios_10plus by this push:
     new f5e4b24  fix: workaround recursive object error (#122)
f5e4b24 is described below

commit f5e4b2450833dc699a91b3db4d5c8eeb2df52880
Author: Tim Brust <ra...@gmx.de>
AuthorDate: Fri Apr 12 14:36:17 2019 +0200

    fix: workaround recursive object error (#122)
    
    <!--
    Please make sure the checklist boxes are all checked before submitting the PR. The checklist is intended as a quick reference, for complete details please see our Contributor Guidelines:
    
    http://cordova.apache.org/contribute/contribute_guidelines.html
    
    Thanks!
    -->
    
    ### Platforms affected
    
    
    
    ### Motivation and Context
    <!-- Why is this change required? What problem does it solve? -->
    <!-- If it fixes an open issue, please link to the issue here. -->
    
    
    
    ### Description
    <!-- Describe your changes in detail -->
    
    
    
    ### Testing
    <!-- Please describe in detail how you tested your changes. -->
    
    
    
    ### Checklist
    
    - [ ] I've run the tests to see all new and existing tests pass
    - [ ] I added automated test coverage as appropriate for this change
    - [ ] Commit is prefixed with `(platform)` if this change only applies to one platform (e.g. `(android)`)
    - [ ] If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct [keyword to close issues using keywords](https://help.github.com/articles/closing-issues-using-keywords/))
    - [ ] I've updated the documentation if necessary
---
 .travis.yml                    |  5 +++--
 lib/appium/helpers/wdHelper.js |  5 ++++-
 package-lock.json              | 14 +++++++-------
 package.json                   |  4 ++--
 4 files changed, 16 insertions(+), 12 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index 8cc4104..b9d3920 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -12,7 +12,8 @@ env:
     - ANDROID_BUILD_TOOLS_VERSION=28.0.3
 
 language: node_js
-node_js: 6
+node_js: 
+  - "6"
 
 # yaml anchor/alias: https://medium.com/@tommyvn/travis-yml-dry-with-anchors-8b6a3ac1b027
 
@@ -53,7 +54,7 @@ matrix:
       <<: *_ios
     - env: PLATFORM=ios-12.0
       <<: *_ios
-      
+
     - env: PLATFORM=android-4.4
       <<: *_android
     - env: PLATFORM=android-5.1
diff --git a/lib/appium/helpers/wdHelper.js b/lib/appium/helpers/wdHelper.js
index be852d4..1e6c702 100644
--- a/lib/appium/helpers/wdHelper.js
+++ b/lib/appium/helpers/wdHelper.js
@@ -234,9 +234,12 @@ module.exports.pollForEvents = function (driver, platform, skipBuster, windowOff
         // get the results and clean up the cache
         var result = window._jasmineParamedicProxyCache;
         window._jasmineParamedicProxyCache = [];
-        return result;
+        return JSON.stringify(result);
     }, [])
     .then(function (result) {
+        if (result) {
+            result = JSON.parse(result);
+        }
         // found
         if (Object.prototype.toString.call(result) === '[object Array]') {
             return result;
diff --git a/package-lock.json b/package-lock.json
index 0222c5e..c60ea5c 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -1503,11 +1503,11 @@
       }
     },
     "rimraf": {
-      "version": "2.6.2",
-      "resolved": "https://registry.npmjs.org/rimraf/-/rimraf-2.6.2.tgz",
-      "integrity": "sha512-lreewLK/BlghmxtfH36YYVg1i8IAce4TI7oao75I1g245+6BctqTVQiBP3YUJ9C6DQOXJmkYR9X9fCLtCOJc5w==",
+      "version": "2.6.3",
+      "resolved": "https://registry.npmjs.org/rimraf/-/rimraf-2.6.3.tgz",
+      "integrity": "sha512-mwqeW5XsA2qAejG46gYdENaxXjx9onRNCfn7L0duuP4hCuTIi/QO7PDK07KJfp1d+izWPrzEJDcSqBa0OZQriA==",
       "requires": {
-        "glob": "^7.0.5"
+        "glob": "^7.1.3"
       },
       "dependencies": {
         "glob": {
@@ -1536,9 +1536,9 @@
       "integrity": "sha512-YZo3K82SD7Riyi0E1EQPojLz7kpepnSQI9IyPbHHg1XXXevb5dJI7tpyN2ADxGcQbHG7vcyRHk0cbwqcQriUtg=="
     },
     "sauce-connect-launcher": {
-      "version": "1.2.4",
-      "resolved": "https://registry.npmjs.org/sauce-connect-launcher/-/sauce-connect-launcher-1.2.4.tgz",
-      "integrity": "sha512-X2vfwulR6brUGiicXKxPm1GJ7dBEeP1II450Uv4bHGrcGOapZNgzJvn9aioea5IC5BPp/7qjKdE3xbbTBIVXMA==",
+      "version": "1.2.5",
+      "resolved": "https://registry.npmjs.org/sauce-connect-launcher/-/sauce-connect-launcher-1.2.5.tgz",
+      "integrity": "sha512-q9GoMBgsBApfYC/GWfNZMgoYOls6YZ77bJ7kXiHDtltVnV+QFiz0/Ac7LSRd6n69RfP/ful8spIFWv0oFs3eOQ==",
       "requires": {
         "adm-zip": "~0.4.3",
         "async": "^2.1.2",
diff --git a/package.json b/package.json
index b0f1466..bfb1f40 100644
--- a/package.json
+++ b/package.json
@@ -50,7 +50,7 @@
     "path-extra": "^3.0.0",
     "q": "^1.4.1",
     "randomstring": "^1.1.5",
-    "sauce-connect-launcher": "^1.2.2",
+    "sauce-connect-launcher": "^1.2.5",
     "saucelabs": "^1.2.0",
     "shelljs": "~0.3.0",
     "socket.io": "^1.4.5",
@@ -58,7 +58,7 @@
     "tmp": "0.0.25",
     "tree-kill": "^1.1.0",
     "unorm": "^1.4.1",
-    "wd": "^1.2.0"
+    "wd": "^1.11.1"
   },
   "devDependencies": {
     "jshint": "^2.6.0"


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org