Skip to content

docs: describe comment messages available to test #1027

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Oct 16, 2017
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 24 additions & 14 deletions PULL_REQUEST_TESTS.md
Original file line number Diff line number Diff line change
@@ -1,14 +1,24 @@
This document describes comment messages available to execute tests on pull request.

|Comment |Description|
|:----------------------:|:---------:|
|`run ci` |Executes all described below.|
|`renderer-android` |Executes [renderer](https://github.com/NativeScript/nativescript-angular/tree/master/e2e/renderer) tests app for Android.|
|`renderer-ios` |Executes [renderer](https://github.com/NativeScript/nativescript-angular/tree/master/e2e/renderer) tests app for iOS.|
|`router-android` |Executes [router](https://github.com/NativeScript/nativescript-angular/tree/master/e2e/router) tests app for Android.|
|`router-ios` |Executes [router](https://github.com/NativeScript/nativescript-angular/tree/master/e2e/router) tests app for iOS.|
|`sdkwebpack` |Webpacks [SDK](https://github.com/NativeScript/nativescript-sdk-examples-ng) examples app for both Android and iOS.|
|`tests-android` |Executes [tests](https://github.com/NativeScript/nativescript-angular/tree/master/tests) app for Android.|
|`tests-ios` |Executes [tests](https://github.com/NativeScript/nativescript-angular/tree/master/tests) app for iOS.|
|`testsappng-android` |Executes [TestsAppNg](https://github.com/NativeScript/tests-app-ng) app for Android.|
|`testsappng-ios` |Executes [TestsAppNg](https://github.com/NativeScript/tests-app-ng) app for iOS.|
### Pull Requests Checks

Builds distributed between Travis CI and internal Jenkins CI execute on pull requests. By default, Travis CI builds trigger on community pull requests and both Travis CI and Jenkins CI builds trigger on core team pull requests. Only NativeScript members have rights to trigger internal Jenkins CI builds. In long term, we aim to migrate all the builds in a public cloud environment.

### Travis

This build compiles the `nativescript-angular` plugin and builds the [tests](https://github.com/NativeScript/nativescript-angular/tree/master/tests) app for Android. Please, refer to the [.travis.yml](https://github.com/NativeScript/nativescript-angular/blob/master/.travis.yml) config file for complete information.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"builds the nativescript-angular package" instead


### Jenkins

More extensive tests execute in Jenkins CI. The table below describes how to trigger and the location of the app/tests of all builds available to execute tests on pull requests.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a note again that these can't be triggered externally.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mention that to manually trigger a build, you need to comment inside the PR.


|Comment |Description|
|:------------------:|:---------:|
|`run ci` |Executes all described below.|
|`renderer-android` |Executes [renderer](https://github.com/NativeScript/nativescript-angular/tree/master/e2e/renderer) tests app for Android.|
|`renderer-ios` |Executes [renderer](https://github.com/NativeScript/nativescript-angular/tree/master/e2e/renderer) tests app for iOS.|
|`router-android` |Executes [router](https://github.com/NativeScript/nativescript-angular/tree/master/e2e/router) tests app for Android.|
|`router-ios` |Executes [router](https://github.com/NativeScript/nativescript-angular/tree/master/e2e/router) tests app for iOS.|
|`sdkwebpack` |Webpacks [SDK](https://github.com/NativeScript/nativescript-sdk-examples-ng) examples app for both Android and iOS.|
|`tests-android` |Executes [tests](https://github.com/NativeScript/nativescript-angular/tree/master/tests) app for Android.|
|`tests-ios` |Executes [tests](https://github.com/NativeScript/nativescript-angular/tree/master/tests) app for iOS.|
|`testsappng-android`|Executes [TestsAppNg](https://github.com/NativeScript/tests-app-ng) app for Android.|
|`testsappng-ios` |Executes [TestsAppNg](https://github.com/NativeScript/tests-app-ng) app for iOS.|