From 102e5ca36df1c9ca77c284561a57f5efd7c01680 Mon Sep 17 00:00:00 2001
From: Martin Walsh <martin.walsh@gmail.com>
Date: Wed, 13 Sep 2017 16:22:34 +0100
Subject: [PATCH] Added Tests

---
 .../Presenters/DatabasePresenterSpec.swift    | 44 +++++++++++++------
 1 file changed, 30 insertions(+), 14 deletions(-)

diff --git a/LockTests/Presenters/DatabasePresenterSpec.swift b/LockTests/Presenters/DatabasePresenterSpec.swift
index 0d73bc790..38c5b3b0d 100644
--- a/LockTests/Presenters/DatabasePresenterSpec.swift
+++ b/LockTests/Presenters/DatabasePresenterSpec.swift
@@ -221,6 +221,10 @@ class DatabasePresenterSpec: QuickSpec {
                 expect(view.passwordManagerButton).to(beNil())
             }
 
+            it("should have show password button") {
+                expect(view.showPasswordButton).toNot(beNil())
+            }
+
             context("with password manager available") {
 
                 beforeEach {
@@ -237,6 +241,10 @@ class DatabasePresenterSpec: QuickSpec {
                     view = presenter.view as! DatabaseOnlyView
                     expect(view.passwordManagerButton).to(beNil())
                 }
+
+                it("should not have show password button") {
+                    expect(view.showPasswordButton).to(beNil())
+                }
             }
 
             describe("user input") {
@@ -293,6 +301,12 @@ class DatabasePresenterSpec: QuickSpec {
                     expect(input.valid) == false
                 }
 
+                it("should toggle show password") {
+                    expect(view.passwordField?.textField?.isSecureTextEntry).to(beTrue())
+                    view.showPasswordButton?.onPress(view.showPasswordButton!)
+                    expect(view.passwordField?.textField?.isSecureTextEntry).to(beFalse())
+                }
+
             }
 
             // MARK:- Log In
@@ -445,6 +459,10 @@ class DatabasePresenterSpec: QuickSpec {
                 expect(view.primaryButton?.title) == "SIGN UP"
             }
 
+            it("should have show password button") {
+                expect(view.showPasswordButton).toNot(beNil())
+            }
+
             describe("user input") {
 
                 it("should clear global message") {
@@ -504,6 +522,10 @@ class DatabasePresenterSpec: QuickSpec {
                     expect(view.passwordManagerButton).to(beNil())
                 }
 
+                it("should not show password manager when disabled") {
+                    expect(view.passwordManagerButton).to(beNil())
+                }
+
                 context("with password manager available") {
 
                     beforeEach {
@@ -517,23 +539,17 @@ class DatabasePresenterSpec: QuickSpec {
                         expect(view.passwordManagerButton).toNot(beNil())
                     }
 
-                    context("disable password manager") {
-
-                        beforeEach {
-                            presenter.passwordManager = passwordManager
-                            presenter.passwordManager.enabled = false
-                            view = presenter.view as! DatabaseOnlyView
-                            view.switcher?.selected = .signup
-                            view.switcher?.onSelectionChange(view.switcher!)
-                        }
-
-                        it("should not show password manager when disabled") {
-                            expect(view.passwordManagerButton).to(beNil())
-                        }
-
+                    it("should not have show password button") {
+                        expect(view.showPasswordButton).to(beNil())
                     }
                 }
 
+                it("should toggle show password") {
+                    expect(view.passwordField?.textField?.isSecureTextEntry).to(beTrue())
+                    view.showPasswordButton?.onPress(view.showPasswordButton!)
+                    expect(view.passwordField?.textField?.isSecureTextEntry).to(beFalse())
+                }
+
             }
 
             describe("sign up action") {