Skip to content

Commit

Permalink
add RNTester to ci (#19673)
Browse files Browse the repository at this point in the history
Summary:
* Current ci is missing an important part to test the whole part. With this we can make sure the js and android part compiles.
* Ensure the current android proguard rules is okay.

The `my-release-key.keystore` is just a copy of debug.keystore in `react-native/keystores`.
Pass all ci.
none
 [GENERAL] [ENHANCEMENT] [CI] - Add RNTester to ci
Closes #19673

Differential Revision: D8435419

Pulled By: hramos

fbshipit-source-id: d3d92a5d1b8477c1f298643cc96695769e5c93ea
  • Loading branch information
gengjiawen authored and facebook-github-bot committed Jun 15, 2018
1 parent e8fea87 commit 8200c98
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 4 deletions.
6 changes: 6 additions & 0 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -558,6 +558,12 @@ jobs:
- run: *run-android-unit-tests
- run: *run-android-instrumentation-tests

# Build Android RNTester
- run:
name: Build Android RNTester
command: |
./gradlew RNTester:android:app:assembleRelease
# Collect Results
- run: *collect-android-test-results
- store_test_results:
Expand Down
2 changes: 1 addition & 1 deletion RNTester/android/app/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ def enableSeparateBuildPerCPUArchitecture = false
/**
* Run Proguard to shrink the Java bytecode in release builds.
*/
def enableProguardInReleaseBuilds = false
def enableProguardInReleaseBuilds = true

android {
compileSdkVersion 26
Expand Down
6 changes: 3 additions & 3 deletions RNTester/android/app/gradle.properties
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,6 @@ android.useDeprecatedNdk=true
org.gradle.parallel=true
org.gradle.configureondemand=true
MYAPP_RELEASE_STORE_FILE=my-release-key.keystore
MYAPP_RELEASE_KEY_ALIAS=my-key-alias
MYAPP_RELEASE_STORE_PASSWORD=*****
MYAPP_RELEASE_KEY_PASSWORD=*****
MYAPP_RELEASE_KEY_ALIAS=androiddebugkey
MYAPP_RELEASE_STORE_PASSWORD=android
MYAPP_RELEASE_KEY_PASSWORD=android
Binary file added RNTester/android/app/my-release-key.keystore
Binary file not shown.

1 comment on commit 8200c98

@dulmandakh
Copy link
Contributor

@dulmandakh dulmandakh commented on 8200c98 Jun 16, 2018

Choose a reason for hiding this comment

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

Excellent. Maybe need to install apk and start activity, then check if app is running just fine

Please sign in to comment.