[ADD] ListOrders TDD - ViewUI (visible in ViewController)

- Create a TableViewSpy to spy the UITableView and:
+ Test if tableView fetch proper orders via reloadData().
- Moreover, test all UITableViewDataSource and UITableViewDelegate
function:
+ Test if tableView display proper number of sections.
+ Test if tableView display proper number of rows in one section.
+ Test if tableView configure proper cells.
+

Author: peihsendoyle

CleanStore/CleanStore/Scenes/ListOrders/ListOrdersViewController.swift

@@ -19,62 +19,94 @@ protocol ListOrdersViewControllerInput
 
 protocol ListOrdersViewControllerOutput
 {
-//  func doSomething(request: ListOrdersRequest)
+    //  func doSomething(request: ListOrdersRequest)
 
     func fetchOrders(request: ListOrders_FetchOrders_Request)
 }
 
-class ListOrdersViewController: UITableViewController, ListOrdersViewControllerInput
-{
-  var output: ListOrdersViewControllerOutput!
-  var router: ListOrdersRouter!
-  
-  // MARK: Object lifecycle
-  
-  override func awakeFromNib()
-  {
-    super.awakeFromNib()
-    ListOrdersConfigurator.sharedInstance.configure(self)
-  }
-  
-  // MARK: View lifecycle
-  
-  override func viewDidLoad()
-  {
-    super.viewDidLoad()
-//    doSomethingOnLoad()
-    
-    fetchOrdersOnLoad()
-  }
-  
-  // MARK: Event handling
-  
-//  func doSomethingOnLoad()
-//  {
-//    // NOTE: Ask the Interactor to do some work
-//    
-//    let request = ListOrdersRequest()
-////    output.doSomething(request)
-//  }
+class ListOrdersViewController: UITableViewController, ListOrdersViewControllerInput {
+    
+    var output: ListOrdersViewControllerOutput!
+    var router: ListOrdersRouter!
+    
+    var displayedOrders : [ListOrders_FetchOrders_ViewModel.DisplayedOrder] = []
+    
+    // MARK: Object lifecycle
+    
+    override func awakeFromNib()
+    {
+        super.awakeFromNib()
+        ListOrdersConfigurator.sharedInstance.configure(self)
+    }
+    
+    // MARK: View lifecycle
+    
+    override func viewDidLoad()
+    {
+        super.viewDidLoad()
+        //    doSomethingOnLoad()
+        
+        fetchOrdersOnLoad()
+    }
+    
+    // MARK: Event handling
+    
+    //  func doSomethingOnLoad()
+    //  {
+    //    // NOTE: Ask the Interactor to do some work
+    //
+    //    let request = ListOrdersRequest()
+    ////    output.doSomething(request)
+    //  }
 
     func fetchOrdersOnLoad() {
 
         let request = ListOrders_FetchOrders_Request()
 
         output.fetchOrders(request)
     }
-  
-  // MARK: Display logic
-  
-  func displaySomething(viewModel: ListOrdersViewModel)
-  {
-    // NOTE: Display the result from the Presenter
 
-    // nameTextField.text = viewModel.name
-  }
+    // MARK: Display logic
+    
+    func displaySomething(viewModel: ListOrdersViewModel)
+    {
+        // NOTE: Display the result from the Presenter
+        
+        // nameTextField.text = viewModel.name
+    }
 
     func displayFetchedOrders(viewModel: ListOrders_FetchOrders_ViewModel) {
 
+        displayedOrders = viewModel.displayedOrders
+        
+        tableView.reloadData()
+    }
+    
+    override func numberOfSectionsInTableView(tableView: UITableView) -> Int {
+        
+        return 1
+    }
+    
+    override func tableView(tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
+        
+        return displayedOrders.count
+    }
+    
+    override func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
+        
+        let displayedOrder = displayedOrders[indexPath.row]
+        
+        var cell = tableView.dequeueReusableCellWithIdentifier("OrderTableViewCell")
+        
+        if cell == nil {
+            
+            cell = UITableViewCell(style: .Value1, reuseIdentifier: "OrderTableViewCell")
+        }
+        
+        cell?.textLabel?.text = displayedOrder.date
+        
+        cell?.detailTextLabel?.text = displayedOrder.total
 
+        return cell!
     }
 }

CleanStore/CleanStoreTests/Scenes/ListOrders/ListOrdersViewControllerTests.swift

@@ -65,6 +65,19 @@ class ListOrdersViewControllerTests: XCTestCase
         }
     }
 
+    class TableViewSpy: UITableView {
+        
+        // MARK: Method call expectations
+        
+        var reloadDataCalled = false
+        
+        // Spied methods
+        
+        override func reloadData() {
+            
+            reloadDataCalled = true
+        }
+    }
     // MARK: Tests
 
     func testShouldFetchOrdersWhenViewIsLoaded() {
@@ -84,12 +97,81 @@ class ListOrdersViewControllerTests: XCTestCase
         XCTAssert(listOrdersViewControllerOutputSpy.fetchOrdersCalled, "Should fetch orders when view is loaded")
     }
 
-    func testSomething()
-    {
+    func testShouldDisplayFetchedOrders() {
+        
+        // Given
+        
+        let tableViewSpy = TableViewSpy()
+        
+        sut.tableView = tableViewSpy
+        
+        // When
+        
+        let displayedOrders = [ListOrders_FetchOrders_ViewModel.DisplayedOrder(id: "doyle", date: "03/03/16", email: "peihsen.doyle@gmail.com", name: "Nguyen Hieu Hiep", total: "$1.23")]
+        
+        let viewModel = ListOrders_FetchOrders_ViewModel(displayedOrders: displayedOrders)
+        
+        sut.displayFetchedOrders(viewModel)
+        
+        // Then
+        
+        XCTAssert(tableViewSpy.reloadDataCalled, "Displaying fetched orders should reload the table view")
+    }
+    
+    func testNumberOfSectionsInTableViewShouldAlwaysBeOne() {
+        
         // Given
 
+        let tableView = sut.tableView
+        
         // When
 
+        let numberOfSections = sut.numberOfSectionsInTableView(tableView)
+        
         // Then
+        
+        XCTAssertEqual(numberOfSections, 1, "Should be the same")
+    }
+    
+    func testNumberOfRowsInAnySectionShouldEqaulNumberOfOrdersToDisplay() {
+        
+        // Given
+        
+        let tableView = sut.tableView
+        
+        // When
+        
+        let testDisplayedOrders = [ListOrders_FetchOrders_ViewModel.DisplayedOrder(id: "doyle", date: "03/03/16", email: "peihsen.doyle@gmail.com", name: "Nguyen Hieu Hiep", total: "$1.23")]
+        
+        sut.displayedOrders = testDisplayedOrders
+        
+        // Then
+        
+        let numberOfRows = tableView.numberOfRowsInSection(0)
+        
+        XCTAssertEqual(numberOfRows, testDisplayedOrders.count, "Number of test displayed orders should be the same as numberOfRows")
+    }
+    
+    func testShouldConfigureTableViewCellToDisplayOrder() {
+        
+        // Given
+        
+        let tableView = sut.tableView
+        
+        // When
+        
+        let testDisplayedOrders = [ListOrders_FetchOrders_ViewModel.DisplayedOrder(id: "doyle", date: "03/03/16", email: "peihsen.doyle@gmail.com", name: "Nguyen Hieu Hiep", total: "$1.23")]
+        
+        sut.displayedOrders = testDisplayedOrders
+        
+        let indexPath = NSIndexPath(forRow: 0, inSection: 0)
+        
+        let cell = sut.tableView(tableView, cellForRowAtIndexPath: indexPath)
+        
+        // Then
+        
+        XCTAssertEqual(cell.textLabel?.text, "03/03/16", "Should be the same")
+        
+        XCTAssertEqual(cell.detailTextLabel?.text, "$1.23", "Should be the same")
     }
 }