Revert "Fix Credentials Manager Test (auth0#325)"

This reverts commit 5b9d6f3.

.circleci/config.yml

@@ -2,7 +2,7 @@ version: 2
 jobs:
   build-iOS-Swift-5.1:
     macos:
-      xcode: "11.2.0"
+      xcode: "11.0.0"
     shell: /bin/bash --login -eo pipefail
     environment:
           LC_ALL: en_US.UTF-8
@@ -11,6 +11,9 @@ jobs:
           HOMEBREW_TEMP: ~/homebrew-temp
           HOMEBREW_NO_AUTO_UPDATE: 1
     steps:
+      - run:
+          name: Set Ruby Version
+          command:  echo "ruby-2.5" > ~/.ruby-version 
       - checkout
       - run: |
           brew install swiftlint
@@ -26,18 +29,12 @@ jobs:
             SCHEME: Auth0.iOS
             DEVICE: iPhone 8
             FASTLANE_EXPLICIT_OPEN_SIMULATOR: 2
-      - run: |
-          slather
-          mkdir /tmp/artifacts
-          cp ./fastlane/test_output/* /tmp/artifacts/
-          cp ./cobertura.xml /tmp/artifacts/
-          bash <(curl -s https://codecov.io/bash) -J Auth0 -f ./cobertura.xml
       - save_cache:
           key: dependency-cache
           paths:
             - Carthage/Build
-      - store_artifacts:
-          path: /tmp/artifacts
+      - store_test_results:
+          path: fastlane/test_output
   build-iOS-Swift-5.0:
     macos:
       xcode: "10.3.0"
@@ -63,10 +60,14 @@ jobs:
             SCHEME: Auth0.iOS
             DEVICE: iPhone 8
             FASTLANE_EXPLICIT_OPEN_SIMULATOR: 2
+      - run: |
+          bash <(curl -s https://codecov.io/bash) -J 'Auth0'
       - save_cache:
           key: dependency-cache
           paths:
             - Carthage/Build
+      - store_test_results:
+          path: fastlane/test_output
   build-iOS-Swift-4.2:
     macos:
       xcode: "10.3.0"
@@ -97,6 +98,8 @@ jobs:
           key: dependency-cache
           paths:
             - Carthage/Build
+      - store_test_results:
+          path: fastlane/test_output
   build-iOS-Swift-4.0:
     macos:
       xcode: "10.3.0"

Auth0.xcodeproj/project.pbxproj

@@ -2122,7 +2122,6 @@
 				ENABLE_TESTABILITY = YES;
 				GCC_C_LANGUAGE_STANDARD = gnu99;
 				GCC_DYNAMIC_NO_PIC = NO;
-				GCC_GENERATE_TEST_COVERAGE_FILES = NO;
 				GCC_NO_COMMON_BLOCKS = YES;
 				GCC_OPTIMIZATION_LEVEL = 0;
 				GCC_PREPROCESSOR_DEFINITIONS = (
@@ -2182,7 +2181,6 @@
 				ENABLE_NS_ASSERTIONS = NO;
 				ENABLE_STRICT_OBJC_MSGSEND = YES;
 				GCC_C_LANGUAGE_STANDARD = gnu99;
-				GCC_GENERATE_TEST_COVERAGE_FILES = NO;
 				GCC_NO_COMMON_BLOCKS = YES;
 				GCC_WARN_64_TO_32_BIT_CONVERSION = YES;
 				GCC_WARN_ABOUT_RETURN_TYPE = YES_ERROR;

Auth0/CredentialsManager.swift

@@ -195,17 +195,20 @@ public struct CredentialsManager {
 
     func hasExpired(_ credentials: Credentials) -> Bool {
 
+        var hasATExpired = true
+        var hasIDTExpired = true
+
         if let expiresIn = credentials.expiresIn {
-            if expiresIn < Date() { return true }
+            if expiresIn > Date() { hasATExpired = false }
         }
 
         if let token = credentials.idToken,
             let tokenDecoded = decode(jwt: token),
             let exp = tokenDecoded["exp"] as? Double {
-            if Date(timeIntervalSince1970: exp) < Date() { return true }
+            if Date(timeIntervalSince1970: exp) > Date() { hasIDTExpired = false }
         }
 
-        return false
+        return hasATExpired || hasIDTExpired
     }
 }
 

Auth0Tests/CredentialsManagerSpec.swift

@@ -301,7 +301,7 @@ class CredentialsManagerSpec: QuickSpec {
                 expect(credentialsManager.hasValid()).to(beTrue())
             }
 
-            it("should not be valid when at valid and id token expired") {
+            fit("should not be valid when at valid and id token expired") {
                 let credentials = Credentials(accessToken: AccessToken, tokenType: TokenType, idToken: ExpiredToken, refreshToken: nil, expiresIn: Date(timeIntervalSinceNow: ExpiresIn))
                 expect(credentialsManager.store(credentials: credentials)).to(beTrue())
                 expect(credentialsManager.hasValid()).to(beFalse())

Gemfile

@@ -2,7 +2,6 @@ source 'https://rubygems.org'
 
 gem 'fastlane'
 gem 'semantic',  '~> 1.5'
-gem 'slather'
 
 plugins_path = File.join(File.dirname(__FILE__), 'fastlane', 'Pluginfile')
 eval_gemfile(plugins_path) if File.exist?(plugins_path)

fastlane/Fastfile

@@ -35,7 +35,7 @@ platform :ios do
       device: device,
       clean: true,
       skip_build: true,
-      code_coverage: true
+      output_types: 'junit'
       )
   end
 

fastlane/Scanfile

@@ -2,3 +2,5 @@ scheme "Auth0.iOS"
 devices ["iPhone 8"]
 
 clean true
+
+output_types "html"