Fix supportPage optional

auth0 · Apr 6, 2017 · 7b8b70c · 7b8b70c
1 parent 00334d5
commit 7b8b70c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/Lock/OptionBuildable.swift b/Lock/OptionBuildable.swift
@@ -140,13 +140,13 @@ public extension OptionBuildable {
     }
 
         /// Support Page URL. By default is not set.
-    var supportPage: String {
+    var supportPage: String? {
         get {
-            guard let url = self.supportURL else { return "" }
+            guard let url = self.supportURL else { return nil }
             return url.absoluteString
         }
         set {
-            guard let url = URL(string: newValue) else { return } // FIXME: log error
+            guard let value = newValue, let url = URL(string: value) else { return } // FIXME: log error
             self.supportURL = url
         }
     }

diff --git a/LockTests/OptionsSpec.swift b/LockTests/OptionsSpec.swift
@@ -61,8 +61,8 @@ class OptionsSpec: QuickSpec {
                 expect(options.supportURL).to(beNil())
             }
 
-            it("should have Auth0 support as empty String") {
-                expect(options.supportPage) == ""
+            it("should return Auth0 supportPage as nil") {
+                expect(options.supportPage).to(beNil())
             }
 
             it("should have openid as scope") {