fixes #830, #880: bring back iOS tests; split OS X & iOS tests

.gitmodules

@@ -12,7 +12,7 @@
 
 [submodule "test/ios/KIF"]
 	path = test/ios/KIF
-	url = https://github.com/mapbox/KIF.git
+	url = https://github.com/kif-framework/KIF.git
 
 [submodule "platform/ios/vendor/SMCalloutView"]
 	path = platform/ios/vendor/SMCalloutView

.travis.yml

@@ -35,7 +35,10 @@ matrix:
       env: BUILDTYPE=Release JOBS=8
       compiler: gcc
     - os: osx
-      env: BUILDTYPE=Debug JOBS=8
+      env: BUILDTYPE=Debug JOBS=8 TEST_OS=osx
+      compiler: clang
+    - os: osx
+      env: BUILDTYPE=Debug JOBS=8 TEST_OS=ios
       compiler: clang
 
 env:
@@ -65,6 +68,7 @@ before_install:
 - ./scripts/travis_before_install.sh
 - if [[ ${TRAVIS_OS_NAME} == "linux" && ${MASON_PLATFORM} != "android" ]]; then export LD_LIBRARY_PATH=`mason prefix mesa 10.4.3`/lib; fi
 - if [[ ${TRAVIS_OS_NAME} == "linux" && ${MASON_PLATFORM} != "android" ]]; then glxinfo; fi
+- if [[ ${TRAVIS_OS_NAME} == "osx" ]]; then gem install xcpretty --no-rdoc --no-ri --no-document --quiet; fi
 
 install:
 - ulimit -c

Makefile

@@ -28,6 +28,9 @@ styles/styles:
 SMCalloutView:
 	git submodule update --init platform/ios/vendor/SMCalloutView
 
+KIF:
+	git submodule update --init test/ios/KIF
+
 #### Library builds ############################################################
 
 .PRECIOUS: Makefile/mbgl
@@ -124,6 +127,10 @@ ios: Xcode/ios
 isim: Xcode/ios
 	xcodebuild -sdk iphonesimulator ARCHS="x86_64 i386" -project ./build/ios/ios/app/mapboxgl-app.xcodeproj -configuration Debug -target iosapp -jobs $(JOBS)
 
+itest: Xcode/ios KIF
+	./scripts/package_ios.sh
+	./scripts/test_ios.sh
+
 # Legacy name
 iproj: ios-proj
 

scripts/package_ios.sh

@@ -33,7 +33,7 @@ export BUILDTYPE=${BUILDTYPE:-Release}
 export HOST=ios
 make Xcode/mbgl
 
-step "Building iOS targets..."
+step "Building iOS device targets..."
 xcodebuild -sdk iphoneos${IOS_SDK_VERSION} \
     ARCHS="arm64 armv7 armv7s" \
     -project ./build/ios/mbgl.xcodeproj \

scripts/test_ios.sh

@@ -0,0 +1,11 @@
+#!/usr/bin/env bash
+
+set -e
+set -o pipefail
+set -u
+
+xcodebuild test \
+    -project ./test/ios/ios-tests.xcodeproj \
+    -scheme 'Mapbox GL Tests' \
+    -destination 'platform=iOS Simulator,name=iPhone 5s,OS=latest' \
+    -destination-timeout 1

scripts/travis_script.sh

@@ -34,18 +34,23 @@ elif [[ ${TRAVIS_OS_NAME} == "linux" ]]; then
         mapbox_time_finish
     fi
 
-elif [[ ${TRAVIS_OS_NAME} == "osx" ]]; then
+elif [[ ${TRAVIS_OS_NAME} == "osx" && ${TEST_OS} == "osx" ]]; then
     #
     # build OS X
     #
-    mapbox_time "build_osx_project" \
+    mapbox_time "build_osx" \
     make xosx -j$JOBS
 
+elif [[ ${TRAVIS_OS_NAME} == "osx" && ${TEST_OS} == "ios" ]]; then
     #
-    # build iOS
+    # test iOS
     #
-    mapbox_time "build_ios_project_device_release" \
-    make ios -j$JOBS
-    mapbox_time "build_ios_project_simulator_debug" \
-    make isim -j$JOBS
+    mapbox_time "build_ios_static" \
+    ./scripts/package_ios.sh
+
+    mapbox_time "checkout_kif" \
+    git submodule update --init test/ios/KIF
+
+    mapbox_time "run_ios_tests" \
+    ./scripts/test_ios.sh | xcpretty -c
 fi

test/ios/KIFTestActor+MapboxGL.h

@@ -1,3 +1,4 @@
+#import <UIKit/UIKit.h>
 #import <KIF/KIF.h>
 
 @class MGLMapView;

test/ios/KIFTestActor+MapboxGL.m

@@ -1,7 +1,8 @@
 #import "KIFTestActor+MapboxGL.h"
+
+#import "MapboxGL.h"
+
 #import <KIF/UIApplication-KIFAdditions.h>
-#import <KIF/UIAccessibilityElement-KIFAdditions.h>
-#import "MGLMapView.h"
 
 @implementation KIFTestActor (MapboxGL)
 

test/ios/MGLTAppDelegate.h

@@ -2,6 +2,6 @@
 
 @interface MGLTAppDelegate : UIResponder <UIApplicationDelegate>
 
-@property (strong, nonatomic) UIWindow *window;
+@property (nonatomic) UIWindow *window;
 
 @end

test/ios/MGLTViewController.m

@@ -1,5 +1,5 @@
 #import "MGLTViewController.h"
-#import "MGLMapView.h"
+#import "MapboxGL.h"
 
 @implementation MGLTViewController
 

test/ios/MapViewTests.m

@@ -1,7 +1,10 @@
 #import "MapViewTests.h"
-#import <KIF/KIFTestStepValidation.h>
+
 #import "KIFTestActor+MapboxGL.h"
-#import "MGLMapView.h"
+
+#import "MapboxGL.h"
+
+#import <KIF/KIFTestStepValidation.h>
 
 @interface MapViewTests () <MGLMapViewDelegate>
 
@@ -144,12 +147,41 @@ - (void)testPanDisabled {
 
     [tester.mapView dragFromPoint:CGPointMake(10, 10) toPoint:CGPointMake(300, 300) steps:10];
 
-    __KIFAssertEqual(centerCoordinate.latitude,
-                     tester.mapView.centerCoordinate.latitude,
-                     @"disabling pan gesture should disallow vertical panning");
-    __KIFAssertEqual(centerCoordinate.longitude,
-                     tester.mapView.centerCoordinate.longitude,
-                     @"disabling pan gesture should disallow horizontal panning");
+    XCTAssertEqualWithAccuracy(centerCoordinate.latitude,
+                               tester.mapView.centerCoordinate.latitude,
+                               0.005,
+                               @"disabling pan gesture should disallow vertical panning");
+    XCTAssertEqualWithAccuracy(centerCoordinate.longitude,
+                               tester.mapView.centerCoordinate.longitude,
+                               0.005,
+                               @"disabling pan gesture should disallow horizontal panning");
+}
+
+- (void)testRotate {
+    CLLocationDirection startAngle = tester.mapView.direction;
+
+    XCTAssertFalse(startAngle == 45, @"start angle must not be destination angle");
+
+    [tester.mapView twoFingerRotateAtPoint:tester.mapView.center angle:45];
+
+    XCTAssertGreaterThanOrEqual(fabs(tester.mapView.direction - startAngle),
+                                20,
+                                @"rotating map should change angle");
+}
+
+- (void)testRotateDisabled {
+    tester.mapView.rotateEnabled = NO;
+
+    CLLocationDirection startAngle = tester.mapView.direction;
+
+    XCTAssertFalse(startAngle == 45, @"start angle must not be destination angle");
+
+    [tester.mapView twoFingerRotateAtPoint:tester.mapView.center angle:45];
+
+    XCTAssertEqualWithAccuracy(tester.mapView.direction,
+                               startAngle,
+                               0.005,
+                               @"disabling rotation show disallow rotation gestures");
 }
 
 - (void)testCenterSet {
@@ -233,7 +265,7 @@ - (void)testBottomLayoutGuide {
     logoBugFrame = [logoBug.superview convertRect:logoBug.frame toView:nil];
     toolbarFrame = [tester.window convertRect:toolbar.frame toView:nil];
     XCTAssertFalse(CGRectIntersectsRect(logoBugFrame, toolbarFrame),
-                   @"rotated device should not have logo buy under toolbar");
+                   @"rotated device should not have logo bug under toolbar");
 
     attributionButtonFrame = [attributionButton.superview convertRect:attributionButton.frame toView:nil];
     XCTAssertFalse(CGRectIntersectsRect(attributionButtonFrame, toolbarFrame),