Điều gì xảy ra với các thử nghiệm của phương thức khi phương thức đó trở thành riêng tư sau khi thiết kế lại trong TDD?

Giả sử tôi bắt đầu phát triển một trò chơi nhập vai với các nhân vật tấn công các nhân vật khác và những thứ đó.

Áp dụng TDD, tôi thực hiện một số trường hợp thử nghiệm để kiểm tra logic bên trong Character.receiveAttack(Int)phương thức. Một cái gì đó như thế này:

@Test
fun healthIsReducedWhenCharacterIsAttacked() {
    val c = Character(100) //arg is the health
    c.receiveAttack(50) //arg is the suffered attack damage
    assertThat(c.health, is(50));
}

Nói rằng tôi có 10 phương pháp thử nghiệm receiveAttackphương pháp. Bây giờ, tôi thêm một phương thức Character.attack(Character)(gọi receiveAttackphương thức đó) và sau một số chu kỳ TDD kiểm tra nó, tôi đưa ra quyết định: Character.receiveAttack(Int)nên private.

Điều gì xảy ra với 10 trường hợp thử nghiệm trước đó? Tôi có nên xóa chúng? Tôi có nên giữ phương pháp public(tôi không nghĩ vậy)?

Câu hỏi này không phải là về cách kiểm tra các phương thức riêng tư mà là cách xử lý chúng sau khi thiết kế lại khi áp dụng TDD

Trong TDD, các bài kiểm tra đóng vai trò là tài liệu thực thi của thiết kế của bạn. Thiết kế của bạn đã thay đổi, vì vậy rõ ràng, tài liệu của bạn cũng phải!

Lưu ý rằng, trong TDD, cách duy nhất mà attackphương thức có thể xuất hiện, là kết quả của việc vượt qua bài kiểm tra thất bại. Có nghĩa là, attackđang được thử nghiệm bởi một số thử nghiệm khác. Điều đó có nghĩa là gián tiếp receiveAttack được bao phủ bởi attackcác thử nghiệm. Lý tưởng nhất, bất kỳ thay đổi nào receiveAttacksẽ phá vỡ ít nhất một trong attackcác thử nghiệm.

Và nếu không, thì có chức năng trong receiveAttackđó không còn cần thiết và không còn tồn tại!

Vì vậy, vì receiveAttackđã được kiểm tra thông qua attack, nên bạn có giữ bài kiểm tra của mình hay không. Nếu khung thử nghiệm của bạn giúp dễ dàng thử nghiệm các phương thức riêng tư và nếu bạn quyết định thử nghiệm các phương thức riêng tư, thì bạn có thể giữ chúng. Nhưng bạn cũng có thể xóa chúng mà không mất bảo hiểm kiểm tra và sự tự tin.

Nếu phương thức đủ phức tạp để cần thử nghiệm, thì nó sẽ được công khai trong một số lớp. Vì vậy, bạn tái cấu trúc từ:

public class X {
  private int complexity(...) {
    ...
  }
  public void somethingElse() {
    int c = complexity(...);
  }
}

đến:

public class Complexity {
  public int calculate(...) {
    ...
  }
}

public class X {
  private Complexity complexity;
  public X(Complexity complexity) { // dependency injection happiness
    this.complexity = complexity;
  }

  public void something() {
    int c = complexity.calculate(...);
  }
}

Di chuyển bài kiểm tra hiện tại về X.complexity sang ComplexityTest. Sau đó nhắn tin cho X.s Something bằng cách chế giễu Độ phức tạp.

Theo kinh nghiệm của tôi, tái cấu trúc đối với các lớp nhỏ hơn và các phương thức ngắn hơn mang lại lợi ích rất lớn. Chúng dễ hiểu hơn, dễ kiểm tra hơn và cuối cùng được sử dụng lại nhiều hơn mong đợi.

Nói rằng tôi có 10 phương thức kiểm tra phương thức receiveAttack. Bây giờ, tôi thêm một phương thức Character.attack (Character) (gọi phương thức receiveAttack) và sau một số chu kỳ TDD kiểm tra nó, tôi đưa ra quyết định: Character.receiveAttack (Int) sẽ ở chế độ riêng tư.

Một điều cần lưu ý ở đây là quyết định bạn đưa ra là xóa phương thức khỏi API . Các biện pháp tương thích ngược sẽ đề nghị

1. Nếu bạn không cần xóa nó, hãy để nó trong API
2. Nếu bạn không cần phải loại bỏ nó chưa , sau đó đánh dấu nó như là bị phản đối và nếu tài liệu có thể khi kết thúc cuộc sống sẽ xảy ra
3. Nếu bạn cần phải loại bỏ nó, thì bạn có một thay đổi lớn về phiên bản

Các thử nghiệm được xóa / hoặc thay thế khi API của bạn không còn hỗ trợ phương thức. Tại thời điểm đó, phương thức riêng là một chi tiết triển khai mà bạn sẽ có thể cấu trúc lại.

Tại thời điểm đó, bạn quay lại câu hỏi tiêu chuẩn về việc liệu bộ thử nghiệm của bạn có nên truy cập trực tiếp vào các triển khai hay không, thay vì tương tác hoàn toàn thông qua API công khai. Một phương thức riêng tư là thứ mà chúng ta sẽ có thể thay thế mà không có bộ thử nghiệm cản trở . Vì vậy, tôi hy vọng cặp vợ chồng thử nghiệm sẽ biến mất - hoặc đã nghỉ hưu hoặc chuyển sang thực hiện thành một thành phần có thể kiểm tra riêng.