Travis: Move frozen tools check to the end

When the scripts for the old tools are modified (e.g. when a new
target is added), the frozen tools check fails but we allow the change
if it's really needed. By moving the frozen tools check to the end,
Travis doesn't terminate early and the other tests get to run.

Author: LDong-Arm

.travis.yml

@@ -191,35 +191,6 @@ matrix:
           find  -name "*.s" | tee BUILD/badasm |
           sed -e "s/^/Bad Assembler file name found: /" && [ ! -s BUILD/badasm ]
 
-    - <<: *docs-vm
-      name: "Frozen tools check"
-      env: NAME=frozen_tools_check
-      script:
-        # Reject any changes to tools that would require a re-release of the
-        # tools for the online compiler.
-        - >-
-            frozen_files=`\
-                git diff --name-only --diff-filter=d FETCH_HEAD..HEAD \
-                    | egrep \
-                      -e "^tools/build_api*" \
-                      -e "^tools/config*" \
-                      -e "^tools/export*" \
-                      -e "^tools/notifier*" \
-                      -e "^tools/paths*" \
-                      -e "^tools/resources*" \
-                      -e "^tools/targets*" \
-                      -e "^tools/toolchains*" \
-                      -e "^tools/utils*" \
-                      -e "^$"`
-            if [ -z "$frozen_files" ]; then
-                echo "Success!";
-            else
-                echo -e "Failure: Frozen files were modified\n$frozen_files";
-                echo -e "Please see https://os.mbed.com/blog/entry/Introducing-the-new-Mbed-Tools/" \
-                        "\nfor why we've frozen the legacy tools.";
-                false;
-            fi
-
     ### Python Tests ###
     - &pytools-vm
       stage: "Pytest"
@@ -342,4 +313,35 @@ matrix:
         - echo ctest --build-and-test . build --build-generator Ninja --build-options -DBUILD_TESTING=ON -DCMAKE_BUILD_TYPE=Debug -DCOVERAGE=ON -DCMAKE_CXX_COMPILER=g++-7 -DCMAKE_C_COMPILER=gcc-7 --test-command ctest        
         - ctest --build-and-test . build --build-generator Ninja --build-options -DBUILD_TESTING=ON -DCMAKE_BUILD_TYPE=Debug -DCOVERAGE=ON -DCMAKE_CXX_COMPILER=g++-7 -DCMAKE_C_COMPILER=gcc-7 --test-command ctest        
         - gcovr --gcov-executable gcov-7 -r . ./build -s -e ".*\.h" --exclude-directories=$TRAVIS_BUILD_DIR/build/UNITTESTS --exclude-directories=$TRAVIS_BUILD_DIR/build/_deps
-        - ccache -s
+        - ccache -s
+
+    ### Frozen tools check ###
+    - &frozen-tools-vm
+      stage: "Frozen tools check"
+      name: "Frozen tools check"
+      env: NAME=frozen_tools_check
+      script:
+        # Reject any changes to tools that would require a re-release of the
+        # tools for the online compiler.
+        - >-
+            frozen_files=`\
+                git diff --name-only --diff-filter=d FETCH_HEAD..HEAD \
+                    | egrep \
+                      -e "^tools/build_api*" \
+                      -e "^tools/config*" \
+                      -e "^tools/export*" \
+                      -e "^tools/notifier*" \
+                      -e "^tools/paths*" \
+                      -e "^tools/resources*" \
+                      -e "^tools/targets*" \
+                      -e "^tools/toolchains*" \
+                      -e "^tools/utils*" \
+                      -e "^$"`
+            if [ -z "$frozen_files" ]; then
+                echo "Success!";
+            else
+                echo -e "Failure: Frozen files were modified\n$frozen_files";
+                echo -e "Please see https://os.mbed.com/blog/entry/Introducing-the-new-Mbed-Tools/" \
+                        "\nfor why we've frozen the legacy tools.";
+                false;
+            fi