[ https://issues.apache.org/jira/browse/CB-14052?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jan Piotrowski (Sujan) updated CB-14052: ---------------------------------------- Description: I'm using cordova-paramedic with --justbuild. The build of the app is failing, but cordova paramedic is reporting success. You can check out one of my ci builds to see it. In this case the requirements check is failing. I would expect this to return a non-zero and fail the build. {code:java} #!/bin/bash -eo pipefail npx cordova-paramedic --cleanUpAfterRun --verbose --platform android@6.x --plugin "." --justbuild cordova-paramedic: Will use the following cli: cordova(node:1495) [DEP0022] DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead.cordova-paramedic: creating temp project at /tmp/tmp-1495jQi7eTMc8Hh6Creating a new cordova project. /tmp/tmp-1495jQi7eTMc8Hh6 /home/circleci/projectcordova-paramedic: installing pluginscordova-paramedic: installing /home/circleci/project --no-telemetry --no-update-notifier Adding com.unarin.cordova.beacon to package.json Saved plugin info for "com.unarin.cordova.beacon" to config.xml com.unarin.cordova.beacon 3.6.1 "Proximity Beacon Plugin" cordova-paramedic: installing cordova-plugin-test-framework --no-telemetry --no-update-notifier Adding cordova-plugin-test-framework to package.json Saved plugin info for "cordova-plugin-test-framework" to config.xml cordova-paramedic: installing /home/circleci/project/node_modules/cordova-paramedic/paramedic-plugin --no-telemetry --no-update-notifier Adding cordova-plugin-paramedic to package.json Saved plugin info for "cordova-plugin-paramedic" to config.xml cordova-paramedic: setting app start page to test pagecordova-paramedic: adding platform android@6.x( --no-telemetry --no-update-notifier)Using cordova-fetch for cordova-android@6.x Adding android project... Creating Cordova project for the Android platform: Path: platforms/android Package: io.cordova.hellocordova Name: HelloCordova Activity: MainActivity Android target: android-26 Subproject Path: CordovaLib Android project created with cordova-android@6.4.0 Installing "com.unarin.cordova.beacon" for android Installing "cordova-plugin-device" for android Installing "cordova-plugin-paramedic" for android Installing "cordova-plugin-test-framework" for android Discovered plugin "cordova-plugin-whitelist" in config.xml. Adding it to the project Installing "cordova-plugin-whitelist" for androidThis plugin is only applicable for versions of cordova-android greater than 4.0. If you have a previous platform version, you do not need this plugin since the whitelist will be built in.Adding cordova-plugin-whitelist to package.json Saved plugin info for "cordova-plugin-whitelist" to config.xml --save flag or autosave detected Saving android@~6.4.0 into config.xml file ...cordova-paramedic: successfully finished adding platform android@6.xcordova-paramedic: checking requirements for platform androidRequirements check results for android: Java JDK: installed 1.8.0 Android SDK: installed true {{Android target: not installed }} Please install Android target / API level: "android-26".Hint: Open the SDK manager by running: /opt/android/sdk/tools/android You will require: 1. "SDK Platform" for API level android-26 2. "Android SDK Platform-tools (latest) 3. "Android SDK Build-tools" (latest) Gradle: installed /usr/share/gradle/bin/gradle (node:1822) UnhandledPromiseRejectionWarning: CordovaError: Some of requirements check failed at /home/circleci/project/node_modules/cordova/src/cli.js:414:27 at _fulfilled (/home/circleci/project/node_modules/cordova-lib/node_modules/q/q.js:787:54) at self.promiseDispatch.done (/home/circleci/project/node_modules/cordova-lib/node_modules/q/q.js:816:30) at Promise.promise.promiseDispatch (/home/circleci/project/node_modules/cordova-lib/node_modules/q/q.js:749:13) at /home/circleci/project/node_modules/cordova-lib/node_modules/q/q.js:557:44 at flush (/home/circleci/project/node_modules/cordova-lib/node_modules/q/q.js:108:17) at _combinedTickCallback (internal/process/next_tick.js:131:7) at process._tickCallback (internal/process/next_tick.js:180:9) (node:1822) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1) (node:1822) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.cordova-paramedic: successfully finished checking requirements for platform androidlocal-server: scanning ports from 7008 to 7208 local-server: port 7028 is availablelocal-server: starting local medic servercordova-paramedic: writing medic log url to project http://10.0.2.2:7028 Start running tests at 04:45:42 cordova-paramedic: running command cordova build android --no-telemetry --no-update-notifier Skipping Appium tests... Completed tests at 04:45:43 local-server: killing local file transfer server if it's up...cordova-paramedic: Deleting the application: /tmp/tmp-1495jQi7eTMc8Hh6/home/circleci/projectFinished with exit code 0{code} look at the cordova-android step in any of these builds * [https://circleci.com/gh/dopry/cordova-plugin-ibeacon/129] * [https://circleci.com/gh/dopry/cordova-plugin-ibeacon/130] * [https://circleci.com/gh/dopry/cordova-plugin-ibeacon/132] * [https://circleci.com/gh/dopry/cordova-plugin-ibeacon/133] was: I'm using cordova-paramedic with --justbuild. The build of the app is failing, but cordova paramedic is reporting success. You can check out one of my ci builds to see it. {{In this case the requirements check is failing. I would expect this to return a non-zero and fail the build.}} {{ }} {{#!/bin/bash -eo pipefail npx cordova-paramedic --cleanUpAfterRun --verbose --platform android@6.x --plugin "." --justbuild}} {{cordova-paramedic: Will use the following cli: cordova(node:1495) [DEP0022] DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead.cordova-paramedic: creating temp project at /tmp/tmp-1495jQi7eTMc8Hh6Creating a new cordova project.}} {{/tmp/tmp-1495jQi7eTMc8Hh6 /home/circleci/projectcordova-paramedic: installing pluginscordova-paramedic: installing /home/circleci/project --no-telemetry --no-update-notifier}} {{Adding com.unarin.cordova.beacon to package.json}} {{Saved plugin info for "com.unarin.cordova.beacon" to config.xml}} {{com.unarin.cordova.beacon 3.6.1 "Proximity Beacon Plugin"}} {{cordova-paramedic: installing cordova-plugin-test-framework --no-telemetry --no-update-notifier}} {{Adding cordova-plugin-test-framework to package.json}} {{Saved plugin info for "cordova-plugin-test-framework" to config.xml}} {{cordova-paramedic: installing /home/circleci/project/node_modules/cordova-paramedic/paramedic-plugin --no-telemetry --no-update-notifier}} {{Adding cordova-plugin-paramedic to package.json}} {{Saved plugin info for "cordova-plugin-paramedic" to config.xml}} {{cordova-paramedic: setting app start page to test pagecordova-paramedic: adding platform android@6.x( --no-telemetry --no-update-notifier)Using cordova-fetch for cordova-android@6.x}} {{Adding android project...}} {{Creating Cordova project for the Android platform:}} {{Path: platforms/android}} {{Package: io.cordova.hellocordova}} {{Name: HelloCordova}} {{Activity: MainActivity}} {{Android target: android-26}} {{Subproject Path: CordovaLib}} {{Android project created with cordova-android@6.4.0}} {{Installing "com.unarin.cordova.beacon" for android}} {{Installing "cordova-plugin-device" for android}} {{Installing "cordova-plugin-paramedic" for android}} {{Installing "cordova-plugin-test-framework" for android}} {{Discovered plugin "cordova-plugin-whitelist" in config.xml. Adding it to the project}} {{Installing "cordova-plugin-whitelist" for android}}{{This plugin is only applicable for versions of cordova-android greater than 4.0. If you have a previous platform version, you do *not* need this plugin since the whitelist will be built in.}}{{Adding cordova-plugin-whitelist to package.json}} {{Saved plugin info for "cordova-plugin-whitelist" to config.xml}} {{--save flag or autosave detected}} {{Saving android@~6.4.0 into config.xml file ...cordova-paramedic: successfully finished adding platform android@6.xcordova-paramedic: checking requirements for platform android}}{{Requirements check results for android:}} {{Java JDK: installed 1.8.0}} {{Android SDK: installed true}} {{Android target: not installed }} {{Please install Android target / API level: "android-26".}}{{Hint: Open the SDK manager by running: /opt/android/sdk/tools/android}} {{You will require:}} {{1. "SDK Platform" for API level android-26}} {{2. "Android SDK Platform-tools (latest)}} {{3. "Android SDK Build-tools" (latest)}} {{Gradle: installed /usr/share/gradle/bin/gradle}} {{(node:1822) UnhandledPromiseRejectionWarning: CordovaError: Some of requirements check failed}} {{at /home/circleci/project/node_modules/cordova/src/cli.js:414:27}} {{at _fulfilled (/home/circleci/project/node_modules/cordova-lib/node_modules/q/q.js:787:54)}} {{at self.promiseDispatch.done (/home/circleci/project/node_modules/cordova-lib/node_modules/q/q.js:816:30)}} {{at Promise.promise.promiseDispatch (/home/circleci/project/node_modules/cordova-lib/node_modules/q/q.js:749:13)}} {{at /home/circleci/project/node_modules/cordova-lib/node_modules/q/q.js:557:44}} {{at flush (/home/circleci/project/node_modules/cordova-lib/node_modules/q/q.js:108:17)}} {{at _combinedTickCallback (internal/process/next_tick.js:131:7)}} {{at process._tickCallback (internal/process/next_tick.js:180:9)}} {{(node:1822) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)}} {{(node:1822) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.cordova-paramedic: successfully finished checking requirements for platform androidlocal-server: scanning ports from 7008 to 7208}} {{local-server: port 7028 is availablelocal-server: starting local medic servercordova-paramedic: writing medic log url to project [http://10.0.2.2:7028|http://10.0.2.2:7028/]}} {{Start running tests at 04:45:42}} {{cordova-paramedic: running command cordova build android --no-telemetry --no-update-notifier}} {{Skipping Appium tests...}} {{Completed tests at 04:45:43}} {{local-server: killing local file transfer server if it's up...cordova-paramedic: Deleting the application: /tmp/tmp-1495jQi7eTMc8Hh6/home/circleci/projectFinished with exit code 0}} look at the cordova-android step in any of these builds [https://circleci.com/gh/dopry/cordova-plugin-ibeacon/129] [https://circleci.com/gh/dopry/cordova-plugin-ibeacon/130] [https://circleci.com/gh/dopry/cordova-plugin-ibeacon/132] [https://circleci.com/gh/dopry/cordova-plugin-ibeacon/133] > Exits with 0 when build fails > ----------------------------- > > Key: CB-14052 > URL: https://issues.apache.org/jira/browse/CB-14052 > Project: Apache Cordova > Issue Type: Bug > Components: cordova-paramedic > Affects Versions: Master > Reporter: Darrel O'Pry > Assignee: Jesse MacFadyen > Priority: Major > > I'm using cordova-paramedic with --justbuild. The build of the app is > failing, but cordova paramedic is reporting success. You can check out one of > my ci builds to see it. > In this case the requirements check is failing. I would expect this to return > a non-zero and fail the build. > {code:java} > #!/bin/bash -eo pipefail npx cordova-paramedic --cleanUpAfterRun --verbose > --platform android@6.x --plugin "." --justbuild > cordova-paramedic: Will use the following cli: cordova(node:1495) [DEP0022] > DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() > instead.cordova-paramedic: creating temp project at > /tmp/tmp-1495jQi7eTMc8Hh6Creating a new cordova project. > /tmp/tmp-1495jQi7eTMc8Hh6 /home/circleci/projectcordova-paramedic: installing > pluginscordova-paramedic: installing /home/circleci/project --no-telemetry > --no-update-notifier > Adding com.unarin.cordova.beacon to package.json > Saved plugin info for "com.unarin.cordova.beacon" to config.xml > com.unarin.cordova.beacon 3.6.1 "Proximity Beacon Plugin" > cordova-paramedic: installing cordova-plugin-test-framework --no-telemetry > --no-update-notifier > Adding cordova-plugin-test-framework to package.json > Saved plugin info for "cordova-plugin-test-framework" to config.xml > cordova-paramedic: installing > /home/circleci/project/node_modules/cordova-paramedic/paramedic-plugin > --no-telemetry --no-update-notifier > Adding cordova-plugin-paramedic to package.json > Saved plugin info for "cordova-plugin-paramedic" to config.xml > cordova-paramedic: setting app start page to test pagecordova-paramedic: > adding platform android@6.x( --no-telemetry --no-update-notifier)Using > cordova-fetch for cordova-android@6.x > Adding android project... > Creating Cordova project for the Android platform: > Path: platforms/android > Package: io.cordova.hellocordova > Name: HelloCordova > Activity: MainActivity > Android target: android-26 > Subproject Path: CordovaLib > Android project created with cordova-android@6.4.0 > Installing "com.unarin.cordova.beacon" for android > Installing "cordova-plugin-device" for android > Installing "cordova-plugin-paramedic" for android > Installing "cordova-plugin-test-framework" for android > Discovered plugin "cordova-plugin-whitelist" in config.xml. Adding it to the > project > Installing "cordova-plugin-whitelist" for androidThis plugin is only > applicable for versions of cordova-android greater than 4.0. If you have a > previous platform version, you do not need this plugin since the whitelist > will be built in.Adding cordova-plugin-whitelist to package.json > Saved plugin info for "cordova-plugin-whitelist" to config.xml > --save flag or autosave detected > Saving android@~6.4.0 into config.xml file ...cordova-paramedic: successfully > finished adding platform android@6.xcordova-paramedic: checking requirements > for platform androidRequirements check results for android: > Java JDK: installed 1.8.0 > Android SDK: installed true > {{Android target: not installed }} > Please install Android target / API level: "android-26".Hint: Open the SDK > manager by running: /opt/android/sdk/tools/android > You will require: > 1. "SDK Platform" for API level android-26 > 2. "Android SDK Platform-tools (latest) > 3. "Android SDK Build-tools" (latest) > Gradle: installed /usr/share/gradle/bin/gradle > (node:1822) UnhandledPromiseRejectionWarning: CordovaError: Some of > requirements check failed > at /home/circleci/project/node_modules/cordova/src/cli.js:414:27 > at _fulfilled > (/home/circleci/project/node_modules/cordova-lib/node_modules/q/q.js:787:54) > at self.promiseDispatch.done > (/home/circleci/project/node_modules/cordova-lib/node_modules/q/q.js:816:30) > at Promise.promise.promiseDispatch > (/home/circleci/project/node_modules/cordova-lib/node_modules/q/q.js:749:13) > at /home/circleci/project/node_modules/cordova-lib/node_modules/q/q.js:557:44 > at flush > (/home/circleci/project/node_modules/cordova-lib/node_modules/q/q.js:108:17) > at _combinedTickCallback (internal/process/next_tick.js:131:7) > at process._tickCallback (internal/process/next_tick.js:180:9) > (node:1822) UnhandledPromiseRejectionWarning: Unhandled promise rejection. > This error originated either by throwing inside of an async function without > a catch block, or by rejecting a promise which was not handled with .catch(). > (rejection id: 1) > (node:1822) [DEP0018] DeprecationWarning: Unhandled promise rejections are > deprecated. In the future, promise rejections that are not handled will > terminate the Node.js process with a non-zero exit code.cordova-paramedic: > successfully finished checking requirements for platform androidlocal-server: > scanning ports from 7008 to 7208 > local-server: port 7028 is availablelocal-server: starting local medic > servercordova-paramedic: writing medic log url to project http://10.0.2.2:7028 > Start running tests at 04:45:42 > cordova-paramedic: running command cordova build android --no-telemetry > --no-update-notifier > Skipping Appium tests... > Completed tests at 04:45:43 > local-server: killing local file transfer server if it's > up...cordova-paramedic: Deleting the application: > /tmp/tmp-1495jQi7eTMc8Hh6/home/circleci/projectFinished with exit code 0{code} > > look at the cordova-android step in any of these builds > * [https://circleci.com/gh/dopry/cordova-plugin-ibeacon/129] > * [https://circleci.com/gh/dopry/cordova-plugin-ibeacon/130] > * [https://circleci.com/gh/dopry/cordova-plugin-ibeacon/132] > * [https://circleci.com/gh/dopry/cordova-plugin-ibeacon/133] -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org For additional commands, e-mail: issues-h...@cordova.apache.org