Add Lint in TravisCI and Update iOS TravisCI

.travis.yml

@@ -4,6 +4,9 @@ node_js: 7.6
 matrix:
     fast_finish: true
     include:
+      # format code using clang-format
+      - env: FORMAT_CODE=true
+        language: ruby
       - env: TEST_SUITE=danger 
       - env: TEST_SUITE=jsfm
       - env: TEST_SUITE=ios
@@ -25,27 +28,51 @@ matrix:
             - android-26
             - extra-android-m2repository
       - env: TEST_SUITE=android ABI=x86
+        language: android
+        dist: trusty
+        jdk: oraclejdk8
+        android:
+          components:
+            - android-26
+            - extra-android-m2repository 
+      # static check
+      - env: STATIC_CODE_ANALYSIS=true CHECK_C=true
+        language: objective-c
+        osx_image: xcode7.2
+      - env: STATIC_CODE_ANALYSIS=true CHECK_ANDROID=true
         language: android
         dist: trusty
         jdk: oraclejdk8
         android:
           components:
             - android-26
             - extra-android-m2repository
+
 cache:
   directories:
     - npm
     - $HOME/.gradle/caches/
     - $HOME/.gradle/wrapper/
     - $HOME/.android/build-cache
     - $HOME/android-ndk-r18b
+    - bundle
+
 before_cache:
   - rm -f  $HOME/.gradle/caches/modules-2/modules-2.lock
   - rm -fr $HOME/.gradle/caches/*/plugin-resolution/
+
+before_install:
+  - |
+    # install oclint
+    if [[ ("${STATIC_CODE_ANALYSIS}" = "true") && ("${CHECK_C}" = "true") ]]; then
+      brew cask uninstall oclint
+      brew tap oclint/formulae
+      brew install oclint
+    fi
+
 install:
   - |
-    case $TEST_SUITE in
-      "android") 
+    if [[ ("$TEST_SUITE" = "android") || ("${CHECK_ANDROID}" = "true") ]]; then
         curl -o- https://github.com/raw/nvm-sh/nvm/v0.34.0/install.sh | bash
         nvm install 7.6
         npm install
@@ -64,11 +91,18 @@ install:
         export ANDROID_NDK_HOME=$HOME/android-ndk-r18b
         export PATH=$PATH:$ANDROID_NDK_HOME
         echo "ndk.dir=$ANDROID_NDK_HOME" > android/local.properties
-        ;;
-      "jsfm" | "danger" | "ios" )
+    elif [[ ("$TEST_SUITE" = "jsfm") || ("$TEST_SUITE" = "danger") ||  ("${CHECK_C}" = "true") ]]; then
         npm install
-        ;;
-    esac
+    elif [[ ("$TEST_SUITE" = "ios") ]]; then
+        git submodule update --init --remote
+        cd weex-playground/ios && bash update_podfile_for_travisci.sh
+        cd ../../ && npm install
+        cd weex-playground/ios && pod install --repo-update
+        cd ../../
+    elif [[ ("${FORMAT_CODE}" = "true" )]]; then
+        bundle install
+    fi
+
 script:
   - |
     case $TEST_SUITE in
@@ -106,10 +140,60 @@ script:
         hasIosFile=$(npm run danger -- run --dangerfile ./dangerfile-ios.js)
         echo "The value of hasIosFile is ${hasIosFile}"
         if [[ "$hasIosFile" =~ "hasIosFile" ]]; then
-          xcodebuild -project ios/sdk/WeexSDK.xcodeproj test -scheme WeexSDKTests CODE_SIGN_IDENTITY="" CODE_SIGNING_REQUIRED=NO -destination "platform=iOS Simulator,name=iPhone 6"
+          # build WeexSDK and run WeexSDKTests
+          xcodebuild -project ios/sdk/WeexSDK.xcodeproj test -scheme WeexSDKTests CODE_SIGN_IDENTITY="" CODE_SIGNING_REQUIRED=NO -destination "platform=iOS Simulator,name=iPhone 6" || exit 1
+          # build WeexDemo and run WeexDemo test
+          cd weex-playground/ios && mkdir tmp && mv * tmp;cd tmp
+          xcodebuild -workspace WeexDemo.xcworkspace test -scheme WeexDemo CODE_SIGN_IDENTITY="" CODE_SIGNING_REQUIRED=NO -destination "platform=iOS Simulator,name=iPhone 6" || exit 1
         fi
         ;;
     esac
+
+    if [[ ("${STATIC_CODE_ANALYSIS}" = "true") && ("${CHECK_C}" = "true") ]]; then
+      hasCFile=$(npm run danger -- run --dangerfile ./dangerfile-static-check.js)
+      echo "The value of hasCFile is ${hasCFile}"
+      if [[ "$hasCFile" =~ "hasCFile" ]]; then
+        echo "hasCFile"
+        cd ios/sdk && xcodebuild | xcpretty -r json-compilation-database -o compile_commands.json
+        oclint-json-compilation-database oclint_args -- \
+          -disable-rule=ShortVariableName \
+          -disable-rule=LongLine \
+          -disable-rule=LongMethod \
+          -disable-rule=HighNcssMethod \
+          -disable-rule=LongVariableName \
+          -disable-rule=HighCyclomaticComplexity \
+          -disable-rule=HighNPathComplexity \
+          -disable-rule=UnusedLocalVariable \
+          -disable-rule=DoubleNegative \
+          -disable-rule=MultipleUnaryOperator \
+          -disable-rule=DeepNestedBlock \
+          -max-priority-1=15000 \
+          -max-priority-2=15000 \
+          -max-priority-3=15000
+      fi
+    fi 
+
+    if [[ ("${STATIC_CODE_ANALYSIS}" = "true") && ("${CHECK_ANDROID}" = "true") ]]; then
+      hasAndroidFile=$(npm run danger -- run --dangerfile ./dangerfile-static-check.js)
+      echo "The value of hasAndroidFile is ${hasAndroidFile}"
+      if [[ "$hasAndroidFile" =~ "hasAndroidFile" ]]; then
+        # format code using google-java-format at first
+        # for n in `find ./android -name "*.java"`; do
+        #   java -jar ./format-tool/google-java-format-1.7-all-deps.jar --replace $n
+        # done
+        # then run android lint
+        # echo "hasAndroidFile"
+        echo "run android lint"
+        cd android
+        ./gradlew lint
+      fi
+    fi
+
+    if [[ ("${FORMAT_CODE}" = "true" )]]; then
+        echo "exec danger ruby"
+        bundle exec danger
+    fi
+
 notifications:
   webhooks:
     on_pull_requests: false

Dangerfile

@@ -0,0 +1 @@
+code_style_validation.check file_extensions: ['.cc', '.cpp', '.m', 'mm', '.h', '.java']

Gemfile

@@ -6,6 +6,9 @@ if !ENV['CIRCLECI']
     gem "danger"
     gem "danger-xcode_summary"
     gem "cocoapods"
+    # fork by 'https://github.com/flix-tech/danger-code_style_validation.git'
+    # update the message level from fail to warn
+    gem 'danger-code_style_validation', :git => 'https://github.com/wrmswindmill/danger-code_style_validation.git'
   end
 end
 # gem "danger-prose"

android/sdk/src/main/java/com/taobao/weex/adapter/ClassLoaderAdapter.java

@@ -52,4 +52,5 @@ public Class<? extends WXComponent> getComponentClass(String name, String classN
         }
     }
 
-}
+}
+

dangerfile-ios.js

@@ -90,22 +90,5 @@ if (!hasIosFile && danger.git.deleted_files) {
 }
 
 if(hasIosFile){
-  console.log('hasIosFile');
-}
-// console.log('-----------------------------hasIosFile-----------------------------:'+hasIosFile);
-// if(hasIosFile){
-//   var runTestCmd='source ~/.bash_profile; '
-//     +'xcodebuild -project ios/sdk/WeexSDK.xcodeproj test '
-//     +'-scheme WeexSDKTests CODE_SIGN_IDENTITY="" CODE_SIGNING_REQUIRED=NO '
-//     +'-destination "platform=iOS Simulator,name=iPhone 6"'
-//   runSuccess = shell.exec(runTestCmd,{ async: false, timeout: 8 * 60 * 1000, maxBuffer: 200 * 1024 * 1024 }).code == 0;
-//   if(!runSuccess){
-//     fail("ios platform run unit test failed!");
-//   }
-// }else{
-//   console.log('has no ios file changed.');
-//   message('has no ios file changed.');
-// }
-// message('ios test finished.')
-
-
+  console.log('hasIosFile!!!');
+}

dangerfile-static-check.js

@@ -0,0 +1,88 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+// Removed import
+import fs from "fs";
+import path from 'path';
+import GitHubApi from 'github';
+import parseDiff from 'parse-diff';
+import shell from "shelljs";
+
+const type_unknown = 0;
+const type_c = 1;
+const type_android = 2;
+
+const getFileType = file => {
+    if (file.match(/.+\.(m|h|mm|cpp|cc)/)) {
+        return type_c;
+    } else if (file.match(/.+\.java/)) {
+        return type_android;
+    } else {
+        return type_unknown;
+    }
+}
+
+var hasCFile = false;
+var hasAndroidFile = false;
+
+function check(file_type) {
+    var has_file_type = false;
+    if (!has_file_type && danger.git.created_files) {
+        danger.git.created_files.some(file => {
+            var f = (getFileType(file) == file_type)
+            if (f) {
+                has_file_type = f;
+            }
+            return f;
+        });
+    }
+
+    if (!has_file_type && danger.git.modified_files) {
+        danger.git.modified_files.some(file => {
+            var f = (getFileType(file) == file_type)
+            if (f) {
+                has_file_type = f;
+            }
+            return f;
+        });
+    }
+
+    if (!has_file_type && danger.git.deleted_files) {
+        danger.git.deleted_files.some(file => {
+            var f = (getFileType(file) == file_type)
+            if (f) {
+                has_file_type = f;
+            }
+            return f;
+        });
+    }
+
+    return has_file_type
+}
+
+hasCFile = check(type_c)
+hasAndroidFile = check(type_android)
+
+var output_str = ""
+if (hasCFile) {
+    output_str += 'hasCFile\n'
+}
+if (hasAndroidFile) {
+    output_str += 'hasAndroidFile\n'
+}
+console.log(output_str)

ios/sdk/.gitignore

@@ -0,0 +1 @@
+*compile_commands.json

ios/sdk/WeexSDK/Sources/Utility/WXVersion.m

@@ -20,8 +20,8 @@
 #import "WXVersion.h"
 #import "WXDefine.h"
 
-static const char* WeexSDKBuildTime = "2019-07-16 07:08:34 UTC";
-static const unsigned long WeexSDKBuildTimestamp = 1563260914;
+static const char* WeexSDKBuildTime = "2019-07-21 09:08:41 UTC";
+static const unsigned long WeexSDKBuildTimestamp = 1563700121;
 
 NSString* GetWeexSDKVersion(void)
 {