swiftでの非同期処理のテストコードの書き方

Question

自作アプリのテストコードを書きたいと思っています。
下記のようなFireStoreのデータを編集し、画面遷移をするコードのテストを下記場合は、どのように書くのが通例でしょうか？
一つの関数につき一つのテストという認識でしたが、これは一つの関数にデータの編集と画面遷移という二つのロジックが書かれているため、画面遷移の方を別の関数にまとめて、それぞれをテストした方がいいのでしょうか？
また、画面遷移のテストコードの書き方はどのように記述すればいいのでしょうか？調べてみましたが、自分では理解することが困難でした。
テストのことを全く考えずにコードを書いてしまったので、テストしやすいコードの書き方という観点からも、こういう書き方にするべきというご意見がありましたら、ご指摘お願いいたします。

import UIKit
import FirebaseFirestore
import Firebase

class EditViewController: UIViewController,UITextFieldDelegate {

    let db = Firestore.firestore()
    var titleString = String()
    var moneyString = String()
    var descriptionString = String()
    var documentIdString = String()
    
    @IBOutlet weak var editButton: UIButton!
    @IBOutlet weak var titleTextField: UITextField!
    @IBOutlet weak var moneyTextField: UITextField!
    @IBOutlet weak var descriptionTextField: UITextField!
    override func viewDidLoad() {
        super.viewDidLoad()
        titleTextField.text = titleString
        moneyTextField.text = moneyString
        descriptionTextField.text = descriptionString
        moneyTextField.keyboardType = UIKeyboardType.numberPad
    }
    
    @IBAction func edit(_ sender: Any) {
        let title = titleTextField.text
        let money = moneyTextField.text!
        let description = descriptionTextField.text
        let data = ["title":title,"money":money,"description":description] as [String : Any]
        var ref: DocumentReference? = nil
        ref = db.collection("patiences").document("(documentIdString)")
        ref?.updateData(data as [AnyHashable : Any]){ err in
            if let err = err {
                print("Error updating document: (err)")
            } else {
                print("Document successfully updated")
            }
        }
        let indexVC = self.storyboard?.instantiateViewController(identifier: "index") as! IndexViewController
        indexVC.modalPresentationStyle = .fullScreen
        present(indexVC, animated: true, completion: nil)
    }
    
    func textFieldShouldReturn(_ textField: UITextField) -> Bool {
        if titleTextField.text == ""||moneyTextField.text == ""{
            editButton.isEnabled = false
        }else{
            editButton.isEnabled = true
        }
        titleTextField.resignFirstResponder()
        moneyTextField.resignFirstResponder()
        descriptionTextField.resignFirstResponder()
        return true
    }
    
    override func touchesBegan(_ touches: Set<UITouch>, with event: UIEvent?) {
        if titleTextField.text == ""||moneyTextField.text == ""{
            editButton.isEnabled = false
        }else{

            editButton.isEnabled = true
        }
        titleTextField.resignFirstResponder()
        moneyTextField.resignFirstResponder()
        descriptionTextField.resignFirstResponder()

    

Comments

[hayabusabusash]

テストコードというのはUnit TestのコードかUI Testのコードどちらでしょうか？

[oeiqgfodgfhps]

unit testのコードです！

[hayabusabusash]

返信ありがとうございます????‍♂️
あと確認ですが、上記のコードはどこかのViewControllerで実装されているコードでしょうか？

[oeiqgfodgfhps]

そうです！

[oeiqgfodgfhps]

ViewControllerの全てのコードを載せました

Answer 1

質問欄に記載していただいたコードは ViewController 上の処理みたいですが、
以下のような問題が出てくると思います。

• 画面の描画ではない処理が ViewController 上に実装されているのでテストが書きにくい
• edit() が実行されると複数の処理( Firestoreへデータの追加、画面遷移 )が走る

この二つを解決するために自分なら MVC のような設計パターンを適応させて、
画面の描画関係以外の処理は Model に実装させ、ViewController は Model を保持するような作りにします。
まずはこの辺りを参考に MVC を適応させて、画面からビジネスロジックを切り離すような構成にするのはどうでしょうか？
非同期処理自体のテストはこちらの記事が参考になると思います。

また、画面遷移のテストについてですが、
画面遷移のテストは UI Test の範囲になると思うので、Unit Test で無理にテストする必要はないんじゃないかなと思います。