Fix the protocol for NavigationRow

Author: bcylin

QuickTableViewController-iOSTests/Row/NavigationRowSpec.swift

@@ -43,11 +43,14 @@ internal final class NavigationRowSpec: QuickSpec {
         expect(row.subtitle) == subtitle
         expect(row.icon) == icon
         expect(row.cellReuseIdentifier) == "UITableViewCell.subtitle"
-        expect(row.action).notTo(beNil())
 
         row.action?(row)
         expect(invoked) == true
       }
+
+      it("should conform to the protocol") {
+        expect(row).to(beAKindOf(NavigationRowCompatible.self))
+      }
     }
 
     describe("cellReuseIdentifier") {

QuickTableViewController-iOSTests/Row/OptionRowSpec.swift

@@ -42,7 +42,9 @@ internal final class OptionRowSpec: QuickSpec {
         expect(row.title) == "title"
         expect(row.subtitle).to(beNil())
         expect(row.isSelected) == true
-        expect(row.action).notTo(beNil())
+
+        row.action?(row)
+        expect(invoked) == true
 
         // RowStyle
         expect(row.cellReuseIdentifier) == "UITableViewCell"
@@ -51,9 +53,10 @@ internal final class OptionRowSpec: QuickSpec {
         expect(row.accessoryType) == UITableViewCellAccessoryType.checkmark
         expect(row.isSelectable) == true
         expect(row.customize).to(beNil())
+      }
 
-        row.action?(row)
-        expect(invoked) == true
+      it("should conform to the protocol") {
+        expect(row).to(beAKindOf(OptionRowCompatible.self))
       }
     }
 
@@ -103,6 +106,7 @@ internal final class OptionRowSpec: QuickSpec {
 
         it("should invoke the action closure") {
           row.isSelected = true
+          expect(row.accessoryType) == UITableViewCellAccessoryType.checkmark
           expect(invoked).toEventually(beTrue())
         }
       }
@@ -113,6 +117,7 @@ internal final class OptionRowSpec: QuickSpec {
 
         it("should not invoke the action closure") {
           row.isSelected = false
+          expect(row.accessoryType) == UITableViewCellAccessoryType.none
           expect(invoked).toEventually(beFalse())
         }
       }

QuickTableViewController-iOSTests/Row/SwitchRowSpec.swift

@@ -39,11 +39,14 @@ internal final class SwitchRowSpec: QuickSpec {
         expect(row.title) == "title"
         expect(row.switchValue) == true
         expect(row.cellReuseIdentifier) == "SwitchCell"
-        expect(row.action).notTo(beNil())
 
         row.action?(row)
         expect(invoked) == true
       }
+
+      it("should conform to the protocol") {
+        expect(row).to(beAKindOf(SwitchRowCompatible.self))
+      }
     }
 
     describe("equatable") {

QuickTableViewController-iOSTests/Row/TapActionRowSpec.swift

@@ -38,11 +38,14 @@ internal final class TapActionRowSpec: QuickSpec {
       it("should initialize with given parameters") {
         expect(row.title) == "title"
         expect(row.cellReuseIdentifier) == "TapActionCell"
-        expect(row.action).notTo(beNil())
 
         row.action?(row)
         expect(invoked) == true
       }
+
+      it("should conform to the protocol") {
+        expect(row).to(beAKindOf(TapActionRowCompatible.self))
+      }
     }
 
     describe("equatable") {

Source/Protocol/RowCompatible.swift

@@ -27,7 +27,7 @@
 import Foundation
 
 /// This protocol defines the compatible interface of a `NavigationRow` regardless of its associated cell type.
-public protocol NavigationRowCompatible: class, Row, RowStyle {}
+public protocol NavigationRowCompatible: Row, RowStyle {}
 
 
 /// This protocol defines the compatible interface of a `TapActionRow` regardless of its associated cell type.

Source/Rows/NavigationRow.swift

@@ -27,7 +27,7 @@
 import UIKit
 
 /// A class that represents a row that triggers certain navigation when selected.
-open class NavigationRow<T: UITableViewCell>: Row, RowStyle, Equatable {
+open class NavigationRow<T: UITableViewCell>: NavigationRowCompatible, Equatable {
 
   // MARK: - Initializer