cordova-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CB-12210) Specifying the --plugin flag without a value causes an exception
Date Sat, 24 Nov 2018 17:17:00 GMT

    [ https://issues.apache.org/jira/browse/CB-12210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16697894#comment-16697894
] 

ASF GitHub Bot commented on CB-12210:
-------------------------------------

janpio commented on issue #19: [CB-12210] require a string when parsing the plugin argument.
URL: https://github.com/apache/cordova-paramedic/pull/19#issuecomment-441382189
 
 
   Without change:
   
   ```
   C:\Projects\Cordova\cordova-paramedic (janpio-fix_inappbrowser -> origin) (cordova-paramedic@0.5.0)
   λ node main.js --platform android --plugin
   cordova-paramedic: Will use the following cli: cordova
   (node:76004) [DEP0022] DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead.
   cordova-paramedic: creating temp project at C:\Users\Jan\AppData\Local\Temp\tmp-76004SdXZwQ4jZHnM
   $ cordova create C:\Users\Jan\AppData\Local\Temp\tmp-76004SdXZwQ4jZHnM --no-telemetry --no-update-notifier
   C:\Users\Jan\AppData\Local\Temp\tmp-76004SdXZwQ4jZHnM C:\Projects\Cordova\cordova-paramedic
   cordova-paramedic: installing plugins
   Completed tests at 6:14:30 PM
   Collecting logs for the devices.
   It looks like there is no target to get logs from.
   Uninstalling the app.
   TypeError: plugin.indexOf is not a function
       at PluginsManager.installSinglePlugin (C:\Projects\Cordova\cordova-paramedic\lib\PluginsManager.js:70:28)
       at PluginsManager.installPlugins (C:\Projects\Cordova\cordova-paramedic\lib\PluginsManager.js:37:14)
       at ParamedicRunner.installPlugins (C:\Projects\Cordova\cordova-paramedic\lib\paramedic.js:163:25)
       at ParamedicRunner.prepareProjectToRunTests (C:\Projects\Cordova\cordova-paramedic\lib\paramedic.js:152:10)
       at C:\Projects\Cordova\cordova-paramedic\lib\paramedic.js:76:21
       at _fulfilled (C:\Projects\Cordova\cordova-paramedic\node_modules\q\q.js:854:54)
       at C:\Projects\Cordova\cordova-paramedic\node_modules\q\q.js:883:30
       at Promise.promise.promiseDispatch (C:\Projects\Cordova\cordova-paramedic\node_modules\q\q.js:816:13)
       at C:\Projects\Cordova\cordova-paramedic\node_modules\q\q.js:877:14
       at runSingle (C:\Projects\Cordova\cordova-paramedic\node_modules\q\q.js:137:13)
   ```
   
   With change:
   ```
   λ node main.js --platform android --plugin
   Error missing args.
   cordova-paramedic --platform PLATFORM --plugin PATH [--justbuild --timeout MSECS --startport
PORTNUM --endport PORTNUM --browserify --version]
   `PLATFORM` : the platform id. Currently supports 'ios', 'browser', 'windows', 'android',
'wp8'.
           Path to platform can be specified as link to git repo like:
           windows@https://github.com/apache/cordova-windows.git
           or path to local copied git repo like:
           windows@../cordova-windows/
   `PATH` : the relative or absolute path to a plugin folder
           expected to have a 'tests' folder.
           You may specify multiple --plugin flags and they will all
           be installed and tested together.
   `MSECS` : (optional) time in millisecs to wait for tests to pass|fail
           (defaults to 10 minutes)
   `PORTNUM` : (optional) ports to find available and use for posting results from emulator
back to paramedic server(default is from 8008 to 8009)
   --browserify : (optional) plugins are browserified into cordova.js
   --buildName : (optional) Build name to show in Saucelabs dashboard
   --ci : (optional) Skip tests that require user interaction
   --cleanUpAfterRun : (optional) cleans up the application after the run
   --cli : (optional) A path to Cordova CLI
   --config : (optional) read configuration from paramedic configuration file
   --fileTransferServer : (optional) (cordova-plugin-file-transfer only) A server address
tests should connect to
   --justbuild : (optional) just builds the project, without running the tests
   --logMins : (optional) Windows only - specifies number of minutes to get logs
   --outputDir : (optional) path to save Junit results file & Device logs
   --sauceAppiumVersion : (optional) Appium version to use when running on Saucelabs. For
example, "1.5.3"
   --sauceDeviceName : (optional) Name of the SauceLabs emulator/browser. For example, "iPhone
Simulator" or "firefox"
   --sauceKey : (optional) Saucelabs access key
   --saucePlatformVersion : (optional) Version of the emulator OS or version of the browser.
For example, "9.3" or "54.0"
   --sauceTunnelId : (optional) Tunnel identifier to use. Only usable if you have Sauce Connect
up
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Specifying the --plugin flag without a value causes an exception
> ----------------------------------------------------------------
>
>                 Key: CB-12210
>                 URL: https://issues.apache.org/jira/browse/CB-12210
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: cordova-paramedic
>            Reporter: sterling gerritz
>            Assignee: Jesse MacFadyen
>            Priority: Minor
>
> FYI: when I failed to add a path paramedic threw a 'type error':
>  cordova-paramedic --platform android --plugin
> cordova-paramedic: creating temp project at /var/folders/p4/m50wzcnx3_5grsq_d3z5kq5w0000gn/T/tmp-21300Vfzeii04OHd0
> cordova-paramedic: installing true
> path.js:7
>    throw new TypeError('Path must be a string. Received ' + inspect(path));
>    ^
> TypeError: Path must be a string. Received true
>    at assertPath (path.js:7:11)



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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


Mime
View raw message