Phpunit: 데이터 공급자는 setupBeforeClass 전에 실행됩니다.

정적 리소스가 setupBeforeClass 에서 초기화될 수 있도록 몇 가지 순서를 적용하는 것이 데이터 공급자가 정적이거나 동일한 테스트 클래스에 있을 필요가 없다는 점을 고려하면 정말 의미가 있는지 확신할 수 없습니다.

어댑터를 지연 초기화하지 않는 이유는 무엇입니까? 그러면 순서는 전혀 중요하지 않습니다.

데이터 공급자를 실행하지 않으면 테스트 메서드가 실행되는 테스트 수를 알 수 없습니다. 완벽한 세상에서 우리는 B/C를 깨고 데이터 제공자가 Countable 를 확장하는 인터페이스를 구현하는 객체가 되도록 요구할 것입니다. 그렇게 하면 이러한 문제를 분리하고 훨씬 깨끗한 상황을 만들 수 있습니다.

이것은 @whatthejeff 무엇을 우리는 일을 끝낸다. 그러나 공급자에서 반환될 때 매개 변수를 배열로 래핑해야 하기 때문에 생성자에서 초기화하는 것이 조금 더 깔끔할 수 있습니다.

@sebastianbergmann ok 그들이 왜 전에 호출되었는지 이해합니다. 문서에서 언급하는 것은 어떻습니까? (당신이 괜찮다면 변경되는 PR을 열 수 있습니다). 나는 객체를 갖는 것이 많은 도움이 될 것인지 확신하지 못합니다. 제 전 사람을 보면 한 공급자에서 다른 공급자로의 종속성이 있습니다.

답장을 보내주셔서 감사합니다. 문제를 종료합니다.

@vicb , 이에 대한 문서 관련 PR을 기꺼이 병합하겠습니다.

장면에 약간 늦을 수 있지만 초기화해야 하는 종속성이 있을 때 수행하는 일은 @dataProvider 주석을 버리고 대신 yield 사용하는 것입니다.

@srosato 나는 바보입니다 (나는 yield를 사용하지 않았습니다). 이 작업을 수행하는 방법에 대한 간단한 예를 들어 주시겠습니까? 여기에 있는 예가 적절하지 않은 경우 https://gist.github.com을 통해.

@dataProvider 에 설정 코드가 필요하고 그다지 크지 않은 경우 테스트에서 문자열로 지정하고 eval 할 수 있습니다. 예를 들어:

public function myProvider() {
    return [
        'new Klass("param 1", "param 2")',
        'new Klass("param 1", "param 2")',
    ];
}

/**
 * <strong i="8">@dataProvider</strong> myProvider
 */
public testMyFunction($instance_str) {
    $klass = eval("return {$instance_str};");
    # continue testing $klass ...
}

내 두 센트:

정적 속성(불가능하다고 알고 있음)에 의존하는 데이터 공급자 대신에 setUp() 메서드에서 테스트 클래스의 클래스 멤버로 필요한 데이터/객체를 설정하고 있습니다.
tearDown() 에서 null로 설정하거나 복잡한 객체가 있을 때 clear() 메서드를 구현합니다.

테스트 클래스 전체의 종속성이 너무 많지 않은 한 이 접근 방식이 편안합니다.
그러나 그러한 종속성이 하나 또는 두 개 이상 있는 경우 전체 디자인을 지나치게 생각해야 할 수도 있습니다.

특히, DB 연결을 setUpBeforeClass() 설정된 정적 속성으로 사용하고 $queryBuilder 의 클래스 멤버로 연결을 주입하는 setUp() (대신 데이터 공급자에서 반환).

해결책은 새로운 개인 메소드를 정의하고 그 안에 static 만큼 많은 변수를 정의하는 것입니다. 그런 다음 그 중 하나가 설정되어 있는지 확인하고, 설정되어 있지 않으면 모두 설정하고 반환(또는 양보)합니다. 이렇게 하면 _변수는 한 번만 선언됩니다_. 제공자가 10개이든 뭐든 상관없습니다. 게다가 setUpBeforeClass() 또는 setUp() 있습니다.

예에서 그것을 보십시오:

private static function getData()
{
    static $data, $anotherData;

    if (!isset($data)) {
        $data = new TestClass();
        $anotherData = [];
    }

    return [
        $data,
        // Or: clone $data
        $anotherData,
    ];
}

public static function setUpBeforeClass()
{
    list(self::$sampleJson, self::$sampleData) = self::getData(); 
}

public function sampleProvider()
{
    $data = self::getData();

    return [
        $data
    ];
}

@MAChitgarha 어제 귀하의 의견 덕분에 이 티켓에 대해 알게 되었습니다. :)

저는 #3736의 데이터 공급자 로직을 리팩토링하여 몇 가지 일반적인 문제를 해결하기 위해 노력하고 있습니다.

• 모든 데이터 공급자가 로드되기 때문에 실행 시작 시 활동이 더 이상 급증하지 않습니다.
• 데이터 제공자는 고정 setUpBeforeClass 및 setUp 이후에 실행할 수 있습니다.
• 비정적 데이터 제공자가 가능해짐
• 발전기는 훨씬 더 효율적일 것입니다

@epdenouden 반갑습니다 ! ;) 목록의 두 번째 항목은 좋은 항목입니다. 그것을 기다리고 있습니다.

비정적 데이터 제공자가 가능해짐

현재 가능하지 않습니까? 저는 항상 정적 데이터 대신 비정적 데이터 공급자를 테스트에 사용하며 문제 없이 예상대로 작동합니다(PHPUnit 7.5.6 사용). 내가 잘못?

현재 가능하지 않습니까? 저는 항상 정적 데이터 대신 비정적 데이터 공급자를 테스트에 사용하며 문제 없이 예상대로 작동합니다(PHPUnit 7.5.6 사용). 내가 잘못?

아니, 당신이 맞습니다! 제 표현이 정확하지 않았습니다. 알려주셔서 감사합니다. 현재 읽고 있는 데이터 공급자 처리 코드를 자세히 살펴보십시오.

private static function getDataFromDataProviderAnnotation($allTheParameters): ?iterable
    // code for locating the data provider
    // [...]
                if ($dataProviderMethod->isStatic()) {
                    $object = null;
                } else {
                    $object = $dataProviderClass->newInstance();
                }

    // code for preparing returned data rows
    // [...]
}

여기에 더러운 작은 비밀이 있습니다.

• 비정적 데이터 공급자를 사용할 수 있으며
• 그들은 실제 instanceof 에서 호출되지만...
• 픽스쳐와 테스트에 사용되는 _같은 인스턴스가 아닙니다_

데이터 제공자 리팩토링과 관련하여 생각하고 확인할 수 있는 정보를 제공해 주셨습니다! 개체 인스턴스가 동일한 유형이나 복제본이 아니라 예상되는 개체인지 확인하기 위해 추가 테스트를 추가해야 합니다. ☕️ 그리고 🍰 당신이 암스테르담에 있을 때 나에게.

@MAChitgarha 인스턴스가 실제로 다릅니다. https://github.com/epdenouden/phpunit/blob/4718e0bb470170b2da46bf0e05b7275110a2a29d/tests/_files/DataProviderInstanceTest.php#L14 -L27

@epdenouden 그러나 나는 어떤 실패도 얻지 않습니다! PHPUnit 7.5.13에서 어설션은 오류를 생성하지 않습니다. 이 오류는 어떻게 발생합니까? 이전 버전과의 비호환성을 의미합니까?

이것은 8.2를 기반으로 작업 중인 지연 로딩 데이터 공급자 분기에 있습니다. 다음에 7.5.x를 확인하겠습니다. 미리 알려주셔서 감사합니다.

이것이 작동한다면 비정적 공급자 메서드를 구현하도록 명시적으로 요청했던 다른 이전 문제로 돌아가서 정확한 사용 사례를 다시 살펴봐야 합니다.

어쨌든: 귀하의 의견은 이미 매우 유용한 테스트에 영감을 주었습니다. :)

그리고 @MAChitgarha 예, BC 휴식이 될 것입니다.

@epdenouden 기다리겠습니다. 어떤 이유로 내 프로젝트에서 PHPUnit 7.5.13을 사용하고 있습니다. 하지만 곧 8.2.*로 업데이트할 예정입니다. 그리고 좋은 소식은 BC-break가 수정될 수 있다는 것입니다. PHPUnit 자체의 구조를 몰라서 무엇을 변경했는지는 모르겠지만 수정은 가능하지 않을까 싶습니다. 그러나 그것은 모두 당신에게 달려 있습니다! :)

그리고 좋은 소식은 BC-break가 수정될 수 있다는 것입니다. PHPUnit 자체의 구조를 몰라서 무엇을 변경했는지는 모르겠지만 수정은 가능하지 않을까 싶습니다. 그러나 그것은 모두 당신에게 달려 있습니다! :)

아주 좋은 이유 없이 이전 버전과의 호환성 중단을 도입하는 것은 @sebastianbergmann 이 허용하는 것이

예, 테스트의 최종 사용자/개발자로서 보고 싶은 것이 무엇인지 알려주십시오. 이러한 프로젝트에는 자원 봉사자 개발자가 많지 않다는 점을 염두에 두십시오.

재미있는 사실: 위의 테스트는 실제로 버전 7.x와 8.x에서 모두 작동하며 JIT 프로토타입에서는 실패합니다.

내가 작업하고 있는 코드는 여전히 원래 코드 흐름과 훨씬 비슷합니다. 그래서 그것은 버그의 결과일 수도 있고, 여전히 더 리팩토링해야 하는 일부 논리일 수도 있고, PHP의 객체 및 리플렉션에 대해 낮은 수준의 것, 조회해야 하는 것 또는 내 부분에 대한 순진한 테스트일 수도 있습니다. 🔬

@epdenouden JIT? PHP용 JIT로 테스트 중입니까, 아니면 제가 모르는 것이 있습니까? 죄송합니다. 약간 혼란스럽습니다. PHP7.x 및 PHP8.x 또는 PHPUnit7.x 및 PHPUnit8.x에 대해 이야기하고 있습니까?!

@epdenouden JIT? PHP용 JIT로 테스트 중입니까, 아니면 제가 모르는 것이 있습니까? 죄송합니다. 약간 혼란스럽습니다. PHP7.x 및 PHP8.x 또는 PHPUnit7.x 및 PHPUnit8.x에 대해 이야기하고 있습니까?!

@MAChitgarha PHP8이 Just-In-Time _compiler_를 얻고 있으므로 혼란을 이해합니다. 나는 동일한 아이디어를 기반으로 하는 새로운 데이터 제공자 구현 이라는 용어를 사용하고 있었습니다. Just-in-time _loader_(일명 지연 로딩). 필요할 때 필요한 것만 초기화하십시오.

이것은 PHP 언어 자체의 까다로운 기능을 사용하지 않기 때문에 곧 출시될 PHP 버전 8 JIT에서 잘 작동합니다. 내 모든 작업은 구성 구문 분석, 테스트 로드 및 실행 등과 같은 작업을 다시 수행하는 PHPUnit 내부 깊숙이 있습니다.

내가 아무 것도 하지 않는 것처럼 보이면 제대로 하고 있는 것입니다. 낮아진 GCP 또는 AWS 인보이스를 제외하고. 💸