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

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


The following commit(s) were added to refs/heads/master by this push:
     new dfed2c7  Fix inappbrowser on Android 7 (#50)
dfed2c7 is described below

commit dfed2c716bfaf1c9bfde2fba5c073d8e0685d7af
Author: Jan Piotrowski <[email protected]>
AuthorDate: Tue Oct 2 21:27:34 2018 +0200

    Fix inappbrowser on Android 7 (#50)
    
    Android 7.0 tests were failing for `cordova-plugin-inappbrowser`: 
https://github.com/apache/cordova-plugin-inappbrowser/issues/307 Long research 
showed that this had to do with the permission buster hanging.
    
    Lo and behold, here we have code for not make a plugin hang on Android 7.
    Let's add the inappbrowser plugin to this check and see if this fixes 
things.
    
    Closes https://github.com/apache/cordova-plugin-inappbrowser/issues/307
---
 lib/paramedic.js | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/lib/paramedic.js b/lib/paramedic.js
index 1c4d642..a386bb7 100644
--- a/lib/paramedic.js
+++ b/lib/paramedic.js
@@ -984,17 +984,20 @@ ParamedicRunner.prototype.runSauceTests = function () {
 
         var platform = self.config.getPlatformId();
         var plugins = self.config.getPlugins();
+
         var skipBuster = false;
-        // skip permission buster for splashscreen plugin
+        // skip permission buster for splashscreen and inappbrowser plugins
         // it hangs the test run on Android 7 for some reason
         for (var i = 0; i < plugins.length; i++) {
-            if (plugins[i].indexOf('cordova-plugin-splashscreen') >= 0) {
+            if (plugins[i].indexOf('cordova-plugin-splashscreen') >= 0 || 
plugins[i].indexOf('cordova-plugin-inappbrowser') >= 0) {
                 skipBuster = true;
             }
         }
+        // always skip buster for browser platform
         if (platform === util.BROWSER) {
             skipBuster = true;
         }
+
         if (!self.config.getUseTunnel()) {
             var polling = false;
             pollForResults = setInterval(function () {


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to