TDD Red-Green-Refactor 및 사적인 방법을 테스트하는 경우 / 방법


91

내가 이해하는 한, 대부분의 사람들은 개인 메소드를 직접 테스트하지 말고 공개 메소드가 호출하는 방식을 통해 테스트해야한다는 데 동의하는 것 같습니다. 나는 그들의 요점을 볼 수 있지만 "TDD의 3 가지 법칙"을 따르고 "적색-녹색-리 팩터"주기를 사용할 때 이것에 약간의 문제가 있습니다. 예를 들어 설명하는 것이 가장 좋습니다.

지금은 탭으로 구분 된 데이터가 포함 된 파일을 읽고 숫자가 아닌 데이터가 포함 된 모든 열을 필터링 할 수있는 프로그램이 필요합니다. 아마 이미이 작업을 수행 할 수있는 몇 가지 간단한 도구가있을 것 같지만 TDD로 연습 할 수있는 멋지고 깨끗한 프로젝트가 될 수 있다고 생각했기 때문에 처음부터 직접 구현하기로 결정했습니다.

따라서 먼저 "빨간 모자를 착용하십시오", 즉 실패한 테스트가 필요합니다. 숫자가 아닌 모든 필드를 한 줄에서 찾는 방법이 필요하다고 생각했습니다. 그래서 간단한 테스트를 작성합니다. 물론 즉시 컴파일되지 않으므로 함수 자체를 작성하기 시작하고 몇 번의주기 (빨간색 / 녹색)가 끝나면 작동하는 함수와 완전한 테스트가 있습니다.

다음으로 한 번에 한 줄씩 파일을 읽고 각 행에서 "findNonNumericFields"함수를 호출하여 결국 제거해야하는 모든 열을 수집하는 "gatherNonNumericColumns"함수를 계속 사용합니다. 적록주기가 몇 번되었고 작동 기능과 완전한 테스트를 다시 마쳤습니다.

이제 리팩토링해야한다고 생각합니다. "findNonNumericFields"메소드는 "gatherNonNumericColumns"를 구현할 때 필요하다고 생각했기 때문에 설계되었으므로 "findNonNumericFields"를 비공개로 설정하는 것이 합리적이라고 생각됩니다. 그러나 더 이상 테스트하는 방법에 액세스 할 수 없으므로 첫 번째 테스트가 중단됩니다.

그래서 나는 개인적인 방법과 그것을 테스트하는 일련의 테스트로 끝납니다. 많은 사람들이 개인적인 방법을 테스트해서는 안된다고 조언하기 때문에 여기 모퉁이에 그려진 것처럼 느껴집니다. 하지만 정확히 어디서 실패 했습니까?

나는 더 높은 수준에서 시작할 수 있었으며 결국 내 공개 메소드가 될 것 (즉, findAndFilterOutAllNonNumericalColumns)을 테스트하는 테스트를 작성했지만 TDD의 전체 지점에 다소 반대되는 느낌을 얻었습니다 (최소한 밥 삼촌에 따르면) : 테스트 작성과 프로덕션 코드 간을 끊임없이 전환해야하며 어느 시점에서든 모든 테스트가 마지막 1 분 안에 작동했습니다. 공개 메소드에 대한 테스트를 작성하여 시작하면 개인 메소드의 모든 세부 정보가 작동하여 테스트를 공개하기 전에 몇 분 (또는 매우 복잡한 경우에는 며칠 또는 몇 일)이 걸릴 수 있기 때문에 메소드가 전달됩니다.

그래서 뭐 할까? TDD (빨간색-녹색-리 팩터 사이클이있는)는 단순히 개인용 메소드와 호환되지 않습니까? 아니면 내 디자인에 결함이 있습니까?



2
이 두 가지 기능은 서로 다른 단위가 될만큼 충분히 다릅니다.이 경우 개인 메서드는 아마도 자신의 클래스에 있어야합니다. 또는 같은 단위 인 경우 테스트를 작성하는 이유를 알 수 없습니다 장치 내부의 행동 . 두 번째 단락과 관련하여 충돌이 보이지 않습니다. 하나의 테스트 사례를 통과하기 위해 전체 복잡한 개인 메소드를 작성해야하는 이유는 무엇입니까? 공개 메소드를 통해 점차적으로 구동하거나 인라인으로 시작한 다음 추출하십시오.
Ben Aaronson

26
사람들이 프로그래밍 책과 블로그에서 관용구와 진부한 내용을 프로그래밍 방법에 대한 실제 지침으로 삼는 이유는 무엇입니까?
AK_

7
나는 이런 이유로 정확하게 TDD를 좋아하지 않는다. 만약 당신이 새로운 영역에 있다면 아키텍처가 어떻게 작동해야하고 어떤 것들이 어떻게 작동하는지 알아내는 동안 많은 추가 작업을해야 할 것이다. 반면에, 이미 경험이 많은 지역에 있다면 intellisense가 컴파일 할 수없는 코드를 작성하는 이유를 이해하지 못하기 때문에 테스트를 먼저 작성하는 것이 좋습니다. 저는 디자인에 대해 생각하고 쓰고 나서 단위 테스트를하는 것에 대해 더 큰 팬입니다.
Jeroen Vannevel

1
"대부분의 사람들은 개인 메소드를 직접 테스트해서는 안된다는 데 동의하는 것 같습니다."아닙니다. 그렇지 않은 경우 메소드를 직접 테스트하십시오. private그렇게하는 것이 타당한 것처럼 숨 깁니다 .
osa

답변:


44

단위

문제가 시작된 곳을 정확하게 지적 할 수 있다고 생각합니다.

숫자가 아닌 모든 필드를 한 줄에서 찾는 방법이 필요하다고 생각했습니다.

"즉 gatherNonNumericColumns, 동일한 장치에 대해 별도의 테스트 가능한 장치 일까요?" 라고 스스로에게 물어야합니다.

대답이 " 예, 분리 "인 경우, 행동 과정은 간단합니다. 해당 방법은 적절한 클래스에서 공개되어야하므로 단위로 테스트 할 수 있습니다. 당신의 사고 방식은 "한 가지 방법으로 드라이브를 테스트해야하고 다른 방법으로 드라이브를 테스트해야합니다"와 같은 것입니다.

당신이 말한 것으로부터, 당신은 그 대답이 " 아니요, 같은 것 " 이라고 생각했습니다 . 이 시점에서 더 이상 계획을 완전히 작성하고 테스트 findNonNumericFields 한 다음 쓰지 않아야합니다 gatherNonNumericColumns. 대신 간단히 작성해야합니다 gatherNonNumericColumns. 지금 findNonNumericFields은 다음 빨간색 테스트 사례를 선택하고 리팩토링을 수행 할 때 염두에 두어야 할 대상이 될 것입니다. 이번에는 "한 가지 방법으로 드라이브를 테스트해야하는데, 그렇게하는 동안 완성 된 구현에는이 다른 방법이 포함될 것"이라는 점을 명심해야합니다.


짧은주기 유지

위의 작업을 수행하면 두 번째 단락에서 설명하는 문제가 발생 하지 않아야합니다 .

공개 메소드에 대한 테스트를 작성하여 시작하면 개인 메소드의 모든 세부 정보가 작동하여 테스트를 공개하기 전에 몇 분 (또는 매우 복잡한 경우에는 며칠 또는 몇 일)이 걸릴 수 있기 때문에 메소드가 전달됩니다.

이 기술은 어느 시점에서도 전체 findNonNumericFields를 처음부터 구현할 때만 녹색으로 바뀌는 빨간색 테스트를 작성하도록 요구하지 않습니다 . 아마도 findNonNumericFields테스트중인 공개 메소드에서 일부 코드가 인라인으로 시작될 가능성이 훨씬 높습니다. 이 코드는 여러주기에 걸쳐 빌드되어 리팩토링 중에 추출됩니다.


로드맵

이 특정 예제에 대한 대략적인 로드맵을 제공하기 위해 사용한 정확한 테스트 사례를 모르지만 gatherNonNumericColumns공개 방법 으로 작성했다고 말합니다 . 그런 다음 테스트 사례는 findNonNumericFields각각 한 행만있는 테이블을 사용하여 작성한 사례와 동일 할 것 입니다. 해당 한 행 시나리오가 완전히 구현되고 메소드를 추출하도록 테스트를 작성하려는 경우 반복을 추가해야하는 2 행 케이스를 작성합니다.


2
나는 이것이 바로 대답이라고 생각합니다. OOP 환경에서 TDD를 채택하면서 나는 종종 내 자신의 상향식 본능을 극복하는 데 어려움을 겪었다. 예, 함수는 작아야하지만 리팩토링이 끝난 후입니다. 전에는 거대한 모놀리스가 될 수 있습니다. +1
João Mendes

2
@ JoãoMendes 글쎄, 리팩토링하기 전에, 특히 매우 짧은 RGR주기에서 거대한 모놀리스 상태에 도달해야할지 모르겠다. 그러나 테스트 가능한 장치 에서 상향식으로 작업하면 OP가 설명하는 문제가 발생할 수 있습니다.
Ben Aaronson

1
좋아, 나는 그것이 어디에서 잘못되었는지 이해한다고 생각한다. 모두에게 감사합니다 (이 답변으로 표시되었지만 대부분의 다른 답변도 동일하게 도움이됩니다)
Henrik Berg

66

많은 사람들이 단위 테스트가 방법 기반이라고 생각합니다. 그렇지 않습니다. 가장 작은 단위를 기반으로해야합니다. 대부분의 경우 이것은 클래스가 전체 실체로서 테스트해야한다는 것을 의미합니다. 개별적인 방법이 아닙니다.

이제 클래스에서 메소드를 호출 할 것입니다. 그러나 테스트는 현재 보유하고있는 블랙 박스 객체에 적용되는 것으로 생각해야합니다. 따라서 클래스가 제공하는 모든 논리 연산을 볼 수 있습니다. 이것들은 테스트해야 할 것들입니다. 클래스가 너무 커서 논리 연산이 너무 복잡하면 디자인 문제가 먼저 해결되어야합니다.

수천 개의 메서드가있는 클래스는 테스트 가능한 것처럼 보일 수 있지만 각 메서드를 개별적으로 테스트하는 경우 실제로 클래스를 테스트하지는 않습니다. 메소드를 호출하기 전에 데이터를 보내기 전에 연결을 설정해야하는 네트워크 클래스와 같은 일부 클래스는 특정 상태에 있어야합니다. 데이터 전송 방법은 전체 클래스와 독립적으로 고려 될 수 없습니다.

따라서 개인 메소드는 테스트와 관련이 없음을 알아야합니다. 클래스의 공용 인터페이스를 호출하여 개인용 메소드를 사용할 수없는 경우 해당 개인용 메소드는 쓸모가 없으며 어쨌든 사용되지 않습니다.

많은 사람들이 테스트를 실행하기 쉬운 것처럼 보이기 때문에 개인 메소드를 테스트 가능한 단위로 바꾸려고 시도하지만 테스트 세분성이 너무 멀리 걸립니다. 마틴 파울러는 말한다

나는 단위라는 클래스의 개념으로 시작하지만, 종종 밀접하게 관련된 많은 클래스를 하나의 단위로 취급합니다.

이는 객체 지향 시스템, 객체가 단위로 설계되는 데 많은 의미가 있습니다. 개별 메소드를 테스트하려면 C와 같은 절차 적 시스템 또는 정적 함수로 구성된 클래스를 작성해야합니다.


14
나 에게이 답변은 OP의 질문에서 TDD 접근법을 완전히 무시하는 것처럼 보입니다. 이것은 "비공개 메소드를 테스트하지 않음"만트라의 반복 일 뿐이지 만 실제로 메소드 기반 인 TDD 메소드 기반이 아닌 단위 테스트 방식에서 어떻게 작동 하는지 설명하지 않습니다 .
Doc Brown

6
@DocBrown 아니오, 그것은 당신의 단위를 "과도하게 세분화하지 말라"고 말하며 자신을 위해 인생을 어렵게 만든다고 전적으로 대답합니다. TDD는 메소드 기반 이 아니며 , 단위가 의미가있는 모든 단위 기반입니다. C 라이브러리가 있다면 각 장치가 기능합니다. 수업이 있으면 단위는 객체입니다. Fowler가 말했듯이, 때때로 단위는 밀접하게 관련된 여러 클래스입니다. 많은 사람들이 단위 테스트를 방법 별 방법이라고 생각합니다. 일부 바보 도구는 방법에 따라 스텁을 생성하기 때문입니다.
gbjbaanb

3
@gbjbaanb : OP가 자신이 작성하려고하는 클래스의 공용 인터페이스없이 순수한 TDD를 먼저 사용하여 OP가 자신의 "비 숫자 필드를 한 줄에 모아서"구현할 수있는 테스트를 제안하려고합니다.
Doc Brown

8
@DocBrown에 동의해야합니다. 질문자의 문제는 사적인 방법을 테스트하지 않고 달성 할 수있는 것보다 더 많은 테스트 세분성을 원한다는 것이 아닙니다. 그는 엄격한 TDD 접근 방식을 따르려고 시도했지만 계획을 세우지 않은 채 갑자기 벽에 부딪쳐서 개인 방법이 무엇인지에 대한 테스트가 갑자기 있음을 알게되었습니다. 이 답변은 도움이되지 않습니다. 이 질문이 아닌 어떤 질문에 대한 좋은 대답입니다.
Ben Aaronson

7
@Matthew : 실수는 처음부터 기능을 썼다는 것입니다. 이상적으로 그는 공용 메소드를 스파게티 코드로 작성한 다음 리 팩터주기에서 개인 함수로 리팩토링해야합니다. 리 팩터주기에서 개인으로 표시하지 마십시오.
slebetman

51

데이터 수집 방법이 테스트를 수행하기에 충분히 복잡 하고 솔루션에 대한 일부 루프 포인트가 아닌 자체 방법이 될 수 있도록 기본 목표 충분히 분리되어 있다는 사실 : 이러한 방법은 개인 이 아닌 다른 클래스의 구성원으로 만듭니다. 수집 / 필터링 / 탭 기능을 제공합니다.

그런 다음 헬퍼 클래스의 멍청한 데이터 감지 측면 (예 : "문자와 숫자 구별")에 대한 테스트를 작성하고 다른 위치에서 기본 목표 (예 : "판매량 확보")를 테스트합니다. 일반적인 비즈니스 로직 테스트에서 기본 필터링 테스트를 반복 할 필요가 없습니다.

일반적으로 한 작업을 수행하는 클래스에 기본 목적과는 별개로 다른 작업을 수행하기위한 광범위한 코드가 포함 된 경우 해당 코드는 다른 클래스에 거주하며 공용 메서드를 통해 호출되어야합니다. 실수로 해당 코드를 포함 하는 클래스의 비공개 코너에 숨겨져서는 안됩니다 . 이것은 테스트 성과 이해성을 동시에 향상시킵니다.


그래 나도 너와 같은 생각이야. 그러나 나는 "충분히 복잡한"부분과 "충분히 충분한"부분이라는 첫 진술에 문제가있다. "복잡한 정도"에 관해 : 나는 빠른 빨강-녹색주기를 시도하고 있는데, 이는 테스트로 전환하기 전에 최대 1 분 정도 또는 한 번만 코딩 할 수 있음을 의미합니다. 그것은 내 테스트가 실제로 매우 세밀하게 될 것임을 의미합니다. 나는 그것이 TDD의 장점 중 하나라고 생각했지만 어쩌면 그것을 과장하여 단점이되었습니다.
Henrik Berg

"충분히 분리 된"에 관하여 : 나는 기능이 작고 그보다 작아야한다는 것을 (unclebob으로부터 다시) 배웠다. 기본적으로 3-4 줄 기능을 만들려고합니다. 따라서 모든 기능은 규모에 관계없이 고유 한 방법으로 분리됩니다.
Henrik Berg

어쨌든, 나는 데이터 통신 측면 (예 : findNonNumericFields)이 실제로 사적이어야한다고 생각합니다. 그리고 그것을 다른 수업으로 분리하면 어쨌든 공개해야하므로 그 요점을 잘 알지 못합니다.
Henrik Berg

6
@HenrikBerg는 왜 왜 객체를 가지고 있는지를 생각합니다. 그것들은 기능을 그룹화하는 편리한 방법은 아니지만 복잡한 시스템을보다 쉽게 ​​작업 할 수있는 독립적 인 유닛입니다. 따라서 클래스를 테스트하는 것을 생각해야합니다.
gbjbaanb

@ gbjbaanb 나는 그들이 하나이고 동일하다고 주장합니다.
RubberDuck

29

개인적으로 테스트를 작성할 때 구현 마인드에 갔다고 생각합니다. 당신은 가정 특정 방법이 필요합니다. 하지만 수업이해야 할 일을하기 위해 정말로 필요합니까? 누군가가 와서 내부적으로 리팩터링하면 수업이 실패합니까? 수업 을 사용 하는 경우 (그리고 내 의견으로는 테스터의 사고 방식이어야 함) 숫자를 확인하는 명시적인 방법이 있다면 신경 쓰지 않아도됩니다.

클래스의 공용 인터페이스를 테스트해야합니다. 개인 구현은 사유로 인해 개인용입니다. 필요하지 않고 변경할 수 있기 때문에 공용 인터페이스의 일부가 아닙니다. 구현 세부 사항입니다.

공용 인터페이스에 대해 테스트를 작성하면 실제로 발생한 문제는 발생하지 않습니다. 개인 메소드를 다루는 공용 인터페이스에 대한 테스트 케이스를 작성할 수 있거나 그렇지 않습니다. 이 경우, 개인 메소드에 대해 열심히 생각하고 어쨌든 도달 할 수없는 경우 모두 제거 할 수 있습니다.


1
"구현 세부 사항"은 "변수간에 정수를 교환하기 위해 XOR 또는 임시 변수를 사용 했습니까"와 같은 것입니다. 보호 / 개인 방법에는 다른 것과 같은 계약이 있습니다. 특정 제약 조건 하에서 입력을 받고 작업하며 출력을 생성합니다. 계약을 맺은 모든 것은 궁극적으로 라이브러리를 소비하는 사람이 아니라 유지 관리하고 수정 한 사용자를 위해 테스트해야합니다. 그것은 "공공"아니다 그냥 있기 때문에 포함되지 않은 것을 의미하지 않는다 API.
Knetic

11

수업이 내부적으로 할 것으로 기대하는 것에 따라 TDD를 수행하지 않습니다.

테스트 사례는 클래스 / 기능 / 프로그램이 외부 세계에 어떤 역할을하는지에 근거해야합니다. 귀하의 예에서, 사용자는 독자 클래스를find all the non-numerical fields in a line?

대답이 "아니오"이면 처음에 글을 쓰는 것은 나쁜 시험입니다. 클래스 / 인터페이스 레벨 에서 기능 에 대한 테스트를 작성하려고 합니다. "클래스 메소드가이를 구현하기 위해 구현해야하는 것"레벨이 아니라 테스트입니다.

TDD의 흐름은 다음과 같습니다.

  • 빨간색 (외부 세계에서 클래스 / 객체 / 기능 / 등이하는 일)
  • 녹색 (이 외부 세계 기능이 작동하도록 최소 코드 작성)
  • 리팩터링 (이 작업을 수행하는 더 좋은 코드는 무엇입니까)

"나중에 X를 개인 메소드로 필요로하기 때문에 구현하고 먼저 테스트 해 보도록하겠습니다." 이 작업을 수행하면 "빨간색"단계를 잘못 수행 한 것입니다. 여기에 문제가있는 것 같습니다.

개인 메소드가되는 메소드에 대한 테스트를 자주 작성하는 경우 몇 가지 작업 중 하나를 수행하는 것입니다.

  • 테스트를 작성하기에 충분한 인터페이스 / 공용 레벨 사용 사례를 제대로 이해하지 못함
  • 디자인을 극적으로 변경하고 여러 테스트를 리팩토링했습니다 (기능이 최신 테스트에서 테스트되는지 여부에 따라 좋은 결과 일 수 있음)

9

일반적으로 테스트에 대한 일반적인 오해가 있습니다.

테스트를 처음 접하는 대부분의 사람들은 다음과 같이 생각합니다.

  • 함수 F에 대한 테스트 작성
  • F를 구현
  • 함수 G에 대한 테스트 작성
  • F를 호출하여 G를 구현하십시오.
  • 함수 H에 대한 테스트 작성
  • G 호출을 사용하여 H 구현

등등.

여기서 문제는 실제로 함수 H에 대한 단위 테스트가 없다는 것입니다. H를 테스트해야하는 테스트는 실제로 H, G 및 F를 동시에 테스트하는 것입니다.

이 문제를 해결하려면 테스트 가능한 장치가 서로에 의존하지 말고 인터페이스 에 의존해야한다는 것을 알아야합니다 . 귀하의 경우, 단위가 간단한 기능인 경우 인터페이스는 단지 호출 서명입니다. 당신은 따라서이 함께 사용할 수 있음을, 같은 방법으로 G를 구현해야합니다 모든 기능 F.과 동일한 서명을 가진

이것이 정확히 수행되는 방법은 프로그래밍 언어에 따라 다릅니다. 많은 언어에서 함수 (또는 함수에 대한 포인터)를 다른 함수의 인수로 전달할 수 있습니다. 이를 통해 각 기능을 개별적으로 테스트 할 수 있습니다.


3
나는 이것을 더 많이 투표 할 수 있기를 바랍니다. 솔루션을 올바르게 설계하지 않았다고 요약하면됩니다.
Justin Ohms

C와 같은 언어에서는 이것이 의미가 있지만, 단위가 일반적으로 클래스 (공용 및 개인 메소드 포함)이어야하는 OO 언어의 경우 모든 개인 메소드가 분리되어있는 것이 아니라 클래스를 테스트해야합니다. 수업 분리하기 각 클래스에서 메소드를 분리합니다.
gbjbaanb 2016 년

8

Test Driven Development 중에 작성하는 테스트는 클래스가 퍼블릭 API를 올바르게 구현하는 동시에 퍼블릭 API를 테스트하고 사용하기 쉬운 지 확인해야합니다.

반드시 프라이빗 메소드를 사용하여 해당 API를 구현할 수 있지만 TDD를 통해 테스트를 작성할 필요는 없습니다. 퍼블릭 API가 올바르게 작동하기 때문에 기능이 테스트됩니다.

이제 개인 메소드가 독립형 테스트를 수행 할 정도로 복잡하다고 가정하지만 원래 클래스의 공개 API의 일부로 의미가 없습니다. 아마도 이것은 아마도 다른 클래스 (실제 클래스가 자체 구현에서 활용하는 클래스)의 공용 메소드 여야한다는 것을 의미합니다.

공개 API 만 테스트하면 향후 구현 세부 사항을 훨씬 쉽게 수정할 수 있습니다. 도움이되지 않는 테스트는 방금 발견 한 우아한 리팩토링을 지원하기 위해 다시 작성해야 할 때만 귀찮게합니다.


4

정답은 공개 방법으로 시작한 결론이라고 생각합니다. 해당 메소드를 호출하는 테스트를 작성하여 시작합니다. 실패하면 그 이름으로 아무것도하지 않는 메소드를 만듭니다. 그렇다면 반환 값을 확인하는 테스트가 적합 할 수 있습니다.

(함수의 기능에 대해서는 완전히 명확하지 않습니다. 숫자가 아닌 값이 제거 된 파일 내용이 포함 된 문자열을 반환합니까?)

메서드가 문자열을 반환하면 해당 반환 값을 확인합니다. 그래서 당신은 그것을 계속 구축합니다.

개인 메서드에서 발생하는 모든 작업은 프로세스 중 어느 시점에서 공개 메서드에 있어야한다고 생각하며 리팩토링 단계의 일부로 개인 메서드로만 이동했습니다. 리팩토링은 내가 아는 한 실패한 테스트를 요구하지 않습니다. 기능을 추가 할 때 실패한 테스트 만 필요합니다. 리 팩터 후에 테스트를 실행하면 모두 통과 할 수 있습니다.


3

내가 여기 모퉁이에 그려진 것처럼 느껴집니다. 하지만 정확히 어디서 실패 했습니까?

오래된 격언이 있습니다.

계획을 세우지 않으면 실패 할 계획입니다.

사람들은 TDD를 할 때 앉아서 테스트를 작성하면 디자인이 마술처럼 일어날 것이라고 생각하는 것 같습니다. 사실이 아닙니다. 높은 수준의 계획이 필요합니다. 인터페이스 (공용 API)를 처음 디자인 할 때 TDD에서 최상의 결과를 얻는다는 것을 알았습니다. 개인적으로 interface클래스를 먼저 정의 하는 실제 를 만듭니다 .

gasp 테스트를 작성하기 전에 "코드"를 작성했습니다! 음 ... 아니. 나는하지 않았다. 나는 따라야 할 계약 , 디자인 을 썼습니다 . 그래프 용지에 UML 다이어그램을 적어두면 비슷한 결과를 얻을 수 있다고 생각합니다. 요점은 계획이 있어야한다는 것입니다. TDD는 코드에서 윌리 닐리 해킹을 할 수있는 라이센스가 아닙니다.

"Test First"가 잘못된 것 같습니다. 먼저 디자인 한 다음 테스트하십시오.

물론 코드에서 더 많은 클래스를 추출하는 것에 대해 다른 사람들이 제공 한 조언을 따르십시오. 수업 내부를 테스트해야한다고 생각되면 내부를 쉽게 테스트 할 수있는 단위로 추출하여 주입하십시오.


2

테스트도 리팩토링 할 수 있습니다. 메소드를 개인용으로 만들면 공개 API가 줄어들 기 때문에 "손실 된 기능"(일명 복잡성 감소)에 대한 해당 테스트를 버리는 것이 좋습니다.

다른 사람들은 개인 메소드가 다른 API 테스트의 일부로 호출되거나 도달 할 수 없으므로 삭제할 수 있다고 말했습니다. 실제로 실행 경로 에 대해 생각하면 상황이 더 세분화됩니다 .

예를 들어, 나눗셈을 수행하는 퍼블릭 메서드가 있으면 경로를 테스트하여 0으로 나눕니다. 우리는 방법은 비공개 경우에, 우리는 선택의 여지를 얻을 : 하나 우리는 0으로 나누기 경로를 고려할 수 있습니다, 또는 우리가 다른 방법으로 불리는 방법을 고려하여 해당 경로를 제거 할 수 있습니다.

이런 식으로 일부 테스트 (예 : 0으로 나누기)를 버리고 나머지 공개 API 측면에서 다른 테스트를 리팩터링 할 수 있습니다. 물론 이상적인 세계에서 기존 테스트는 남아있는 모든 경로를 처리하지만 현실은 항상 타협입니다.)


1
다른 방법은 개인 방법이 빨간색 주기로 작성되어서는 안된다는 점에서 정확하지만 인간은 실수를합니다. 그리고 당신이 실수의 길을 충분히 갔을 때 이것은 적절한 해결책입니다.
slebetman

2

개인 메소드가 다른 클래스의 공용 메소드가 될 수있는 경우가 있습니다.

예를 들어 스레드로부터 안전하지 않은 개인 메서드가 있고 클래스를 임시 상태로 둘 수 있습니다. 이 방법들은 첫 번째 수업에 의해 개인적으로 개최되는 별도의 수업으로 옮길 수 있습니다. 따라서 클래스가 큐인 경우 공용 메소드가있는 InternalQueue 클래스가있을 수 있으며 Queue 클래스는 InternalQueue 인스턴스를 비공개로 보유합니다. 이를 통해 내부 대기열을 테스트 할 수 있으며 InternalQueue의 개별 작업이 무엇인지 명확하게 알 수 있습니다.

(List 클래스가 없다고 생각할 때와 List 함수를 사용하는 클래스에서 List 함수를 전용 메소드로 구현하려고 시도한 경우 가장 분명합니다.)


2
"개인 메서드가 다른 클래스의 공용 메서드가 될 수있는 경우가 있습니다." 나는 그것을 충분히 강조 할 수 없습니다. 때로는 사적인 방법은 단순히 자신의 정체성을 외치는 다른 클래스 일뿐입니다.

0

왜 당신의 언어가 완전히 공개적이고 완전히 사적인 두 가지 수준의 프라이버시만을 가지고 있는지 궁금합니다.

비공개 메소드를 패키지 액세스 가능 또는 이와 유사한 방식으로 배열 할 수 있습니까? 그런 다음 테스트를 동일한 패키지에 넣고 공용 인터페이스의 일부가 아닌 내부 작업을 테스트하십시오. 빌드 바이너리는 빌드 바이너리를 빌드 할 때 테스트를 제외합니다.

물론 때로는 정의 클래스 외에는 액세스 할 수없는 진정한 개인 메소드가 필요합니다. 그러한 모든 방법이 매우 작기를 바랍니다. 일반적으로 메소드를 작게 유지하면 (예 : 20 줄 미만) 테스트, 유지 관리 및 코드 이해가 쉬워집니다.


3
테스트를 실행하기 위해 메소드 액세스 수정자를 변경하는 것은 꼬리가 개를 흔들는 상황입니다. 유닛의 내부 부품을 테스트하면 나중에 리팩터링하기가 더 어려워집니다. 반대로 공용 인터페이스 테스트는 장치의 "계약"으로 작동하기 때문에 훌륭합니다.
scriptin

메소드의 액세스 레벨을 변경할 필요가 없습니다. 내가 말하려는 것은 공개 계약을하지 않고도 테스트를 포함한 특정 코드를 쉽게 작성할 수있는 중간 액세스 수준이라는 것입니다. 물론 공용 인터페이스를 테스트해야하지만 일부 내부 작업을 개별적으로 테스트하는 것이 유리한 경우도 있습니다.
9000

0

때로는 더 세밀한 테스트 (노출 된 공개 API보다 더 세밀한)를 허용하도록 보호하기 위해 개인 메소드를 충돌했습니다. 이것은 규칙이 아닌 (희망적으로 매우 드문) 예외 여야하지만 특정 특정 경우에 유용 할 수 있습니다. 또한, 공개 API를 구축 할 때 전혀 고려하지 않을 것입니다. 드문 상황에서 내부 사용 소프트웨어에 사용할 수있는 "치트"가 더 많습니다.


0

나는 이것을 경험하고 당신의 고통을 느꼈습니다.

내 해결책은 다음과 같습니다.

모놀리스 빌드와 같은 테스트 처리를 중단하십시오.

몇 가지 기능을 사용하기 위해 5 가지 테스트를 작성 했을 때, 특히 다른 테스트의 일부가 될 때 이러한 테스트를 모두 유지할 필요는 없습니다 .

예를 들어 나는 종종 :

  • 저수준 시험 1
  • 그것을 충족시키는 코드
  • 저수준 시험 2
  • 그것을 충족시키는 코드
  • 저수준 시험 3
  • 그것을 충족시키는 코드
  • 저수준 시험 4
  • 그것을 충족시키는 코드
  • 저수준 시험 5
  • 그것을 충족시키는 코드

그래서 나는

  • 저수준 시험 1
  • 저수준 시험 2
  • 저수준 시험 3
  • 저수준 시험 4
  • 저수준 시험 5

지금 시험을 많이 가지고 그, 그것을 호출하는 높은 수준의 기능 (들)을 추가하지만, 난 수도 지금은 단지 수에 그 낮은 수준의 테스트를 줄일 수 :

  • 저수준 시험 1
  • 저수준 시험 5

악마는 세부 사항에 있으며 그것을 할 수있는 능력은 상황에 따라 다릅니다.


-2

태양이 지구를 중심으로 또는 지구를 중심으로 회전합니까? 아인슈타인에 따르면 그 대답은 '그렇다'또는 두 모델이 관점에 의해서만 다르기 때문에 캡슐화와 테스트 주도 개발은 우리가 생각하기 때문에 충돌에 불과합니다. 우리는 갈릴레오와 교황처럼 여기 앉아 서로를 모욕합니다. 바보, 당신은 개인적인 방법도 시험을해야한다는 것을 보지 않습니까; 이단자, 캡슐화를 중단하지 마십시오! 마찬가지로 우리가 생각하는 것보다 진실이 더 크다는 것을 인식 할 때 개인 인터페이스에 대한 테스트를 캡슐화하여 공용 인터페이스에 대한 테스트가 캡슐화를 깨뜨리지 않는 것과 같은 것을 시도 할 수 있습니다.

시도해보십시오 : 두 가지 방법을 추가하십시오. 하나는 입력이 없지만 단지 개인 테스트 수를 반환하고 다른 하나는 테스트 번호를 매개 변수로 사용하고 통과 / 실패를 반환합니다.


1
선택한 모욕은 갈릴레오와 교황에 의해 사용되었으며이 질문에 대한 답은 아닙니다.
초에 hildred
당사 사이트를 사용함과 동시에 당사의 쿠키 정책개인정보 보호정책을 읽고 이해하였음을 인정하는 것으로 간주합니다.
Licensed under cc by-sa 3.0 with attribution required.