Cucumber-js: 단계 정의에 대한 인수 확인을 완화하는 옵션

같은 요청을 했습니다. #445를 참조하세요. :)

@riaan53 , 예, 나는 그것을 보았고 불행히도 아마도 올바른 이유 때문에 요청이 거부되었습니다. 나는 그 추론에 동의하지 않지만, 더 큰 프레임워크 구축의 일부로 이 기능은 사용자가 그것을 남용하지 않는 한 중요하다고 생각합니다. 따라서 이것을 엄격하게 선택적인 구성으로 만드는 것이 좋습니다.

당신이 말하는 이 일반 처리 논리는 무엇입니까?

확신하는. 한 가지 사용 사례에서는 모든 개발자가 단계 정의에서 표현식 참조를 사용할 수 있기를 바랍니다. 예: '${admin} 사용자가 시스템에 로그인할 때'.

이제 이 경우 ${admin}은 JSON 파일에서 해결될 수 있으며 해결에 대한 책임은 단계 정의를 구현하는 동안 개발자에게 있습니다. 그런 종류의 속성 확인을 실제로 보면 개발자가 전혀 인식하지 못하는 일반 코드로 수행할 수 있습니다.

이를 허용하기 위해 Cucumber가 주입한 원시 인수를 수락하고 해결한 다음 해결된 값을 실제 단계 구현에 주입하는 개발자 단계 구현 주위에 일반 함수 래퍼를 쉽게 만들 수 있습니다.

현재 Cucumber의 유효성 검사를 통해 함수에 올바른 수의 매개 변수가 있는지 확인하는 것과 같은 일반 함수를 작성할 수 없습니다. 제 경우에는 일반 래퍼 함수가 명명된 인수를 허용하지 않고 '인수' 개체입니다.

내가 이해할 수 있기를 바랍니다. 현재 프로젝트에는 단계 구현에 대한 일반 래퍼 기능이 필요한 다른 사용 사례도 있습니다.

단계 인수 변환을 구현하면 문제가 해결됩니다. https://github.com/cucumber/cucumber/wiki/Step-Argument-Transforms?

아 예, 잠재적으로 참조 해상도 문제를 해결할 수 있습니다. 그러나 다른 이유로 단계 구현 주위에 일반 래퍼 함수를 ​​추가해야 합니다.

예를 들어 한 가지 이유는 프레임워크로 Cucumber와 함께 Protractor를 사용하고 Protractor에서 WebDriver 제어 흐름에 사용자 지정 약속을 등록해야 다음 단계로 진행하기 전에 이러한 사용자 지정 약속이 해결될 때까지 부지런히 기다려야 한다는 것입니다.

Cucumber는 Promise가 해결될 때까지 기다리지만(단계 구현에서 반환된 경우) 분명히 WebDriver를 인식하지 않으며 반환된 Promise를 WebDriver에 등록하기 위해 각 단계 구현에 추가 코드를 추가해야 하는 경우가 가장 많습니다.

이것은 매개변수 검사가 Cucumber에 의해 완화되어야 하는 일반 함수 래퍼를 통해 해결하기 매우 쉬운 문제입니다.

나는 지금이 문제에 두 번 부딪쳤다.

가장 눈에 띄는 것은 Then 단계마다 retry 도우미를 구현하는 것입니다.

cucumber.Then = function(match, callback) {
  const retryingCallback = (...args) => retry(async () => await callback(...args));
  cucumber.Then(match, retryingCallback);
};

나는 이 도우미를 사용하여 궁극적으로 일관된 백엔드에서 타이밍 문제를 처리합니다. 기본적으로 y초가 지나거나 콜백이 지나갈 때까지 x초마다 콜백을 실행합니다.

안타깝게도 이로 인해

function has 0 arguments, should have 1 (if synchronous or returning a promise) or 2 (if accepting a callback)

내가 지금 사용하고 있는 대안은 모든 Then 단계에서 도우미를 호출하는 것이므로 많은 코드 중복이 발생합니다.

this.Then(/^I see that "([^"]*)" does not have a destination$/, async clientName => {
  return retry(async () => {
    const client = await homeView.clientByName(clientName);
    expect(client.destinationName).to.not.exist;
  });
});

또 다른 경우는 로그인을 위한 도우미 기능을 원하는 더 간단한 경우입니다.

function loggedIn(username, func) {
  return (...args) => {
    await accounts.login(username);
    return func(...args)
  };
}

this.Then(/^someone logged in as "([^"]*)" sees a destination named "([^"]*)"$/, loggedIn(assert.destinationExists));

또한 이것은 많은 코드 중복을 저장합니다.

마지막으로 모든 승인 테스트를 실행하지만 모든 Then 콜백 전에 서버를 다시 시작하는 제품군을 추가하고 싶을 때가 있을 것으로 예상합니다(서버 다시 시작으로 인해 문제가 발생하지 않도록 하기 위해). 다시 말하지만, 많은 중복.

PS 재시도 도우미:

const patience = 250;
const interval = 5;

function delay(time) {
  return new Promise(function (fulfill) {
    setTimeout(fulfill, time);
  });
}

async function attempt(start, func) {
  const attemptDate = new Date();
  try {
    return await func();
  } catch (errr) {
    const timeElapsed = attemptDate.getTime() - start.getTime();
    if (timeElapsed < patience) {
      await delay(interval);
      return await attempt(start, func);
    } else {
      throw errr;
    }
  }
}

export async function retry(func) {
  const start = new Date();
  return await attempt(start, func);
}

_편집하다_

내 방법을 해킹하려고했습니다.

function splat(func) {
  return (one, two, three, four, five, six, seven, eight, nine, ten) => {
    if (typeof ten !== 'undefined') {
      return func(one, two, three, four, five, six, seven, eight, nine, ten);
    } else if (typeof nine !== 'undefined') {
      return func(one, two, three, four, five, six, seven, eight, nine);
    } else if (typeof eight !== 'undefined') {
      return func(one, two, three, four, five, six, seven, eight);
    } else if (typeof seven !== 'undefined') {
      return func(one, two, three, four, five, six, seven);
    } else if (typeof six !== 'undefined') {
      return func(one, two, three, four, five, six);
    } else if (typeof five !== 'undefined') {
      return func(one, two, three, four, five);
    } else if (typeof four !== 'undefined') {
      return func(one, two, three, four);
    } else if (typeof three !== 'undefined') {
      return func(one, two, three);
    } else if (typeof two !== 'undefined') {
      return func(one, two);
    } else if (typeof one !== 'undefined') {
      return func(one);
    } else {
      return func();
    }
  };
}

cucumber.Then = function(match, callback) {
  const retryingCallback = splat((...args) => retry(async () => await callback(...args)));
  cucumber.Then(match, retryingCallback);
};

하지만

function has 10 arguments, should have 1 (if synchronous or returning a promise) or 2 (if accepting a callback)

나를 슬픈 팬더로 만들어

다음 은 길이를 유지하기 위해 함수를 래핑하는 예입니다. 이 작업을 수행하는 작은 노드 모듈만 있으면 좋을 것입니다.

제안 감사합니다! 단계 정의당 인수 수를 지정하면 작동합니다. 맞죠?

예를 들어

this.Then(/^someone logged in as "([^"]*)" sees a destination named "([^"]*)"$/, createProxy(loggedIn(assert.destinationExists), 2));

나에게 가장 시급한 문제는 여러 단계 정의에 대한 일반 미들웨어를 추가할 수 없다는 것입니다. createProxy 와 같은 것은 미들웨어 등록을 허용하는 개체로 만든 다음 모든 단일 단계 정의에서 인수 수를 알려주면 작동할 수 있습니다. (제 첫 번째 예를 자세히 살펴보면 retry 함수가 이를 래핑하기 때문에 createProxy 직접 사용할 수 없음을 알 수 있습니다. 그 반대가 되어야 하지만 createProxy 는 각 콜백에 대한 인수의 수를 알 수 없습니다)

그래도 오류를 끌 수 있는 것과 비교하면 여전히 어색합니다. :순결한:

proxyLength 를 전달하는 대신 래핑하고 function.length 사용하는 함수의 변형을 사용할 수 있다고 생각합니다.

좋은 제안, 감사합니다!

나는 다음과 같은 것을 얻었다.

cucumber.Then = function(match, callback) {
  cucumber.Then(match, retryProxy(callback));
};

function retryProxy(func) {
  const numberOfArgs = func.length;
  switch (numberOfArgs) {
    case 0: return () => retry(func);
    case 1: return (a) => retry(func, a);
    case 2: return (a, b) => retry(func, a, b);
    case 3: return (a, b, c) => retry(func, a, b, c);
    case 4: return (a, b, c, d) => retry(func, a, b, c, d);
    case 5: return (a, b, c, d, e) => retry(func, a, b, c, d, e);
  }
}

해결되지 않는 두 가지는 로그인 도우미의 경우와 기본 매개변수를 허용하는 것이지만 이 두 가지를 해결할 수 있습니다.

이제 단계 정의를 조정하지 않고 미들웨어를 추가할 수 있어 기쁩니다!

@thomasvanlankveld 알다시피, 특정 함수 길이를 제공하기 위해 함수를 래핑하는 이 라이브러리를 찾았습니다: https://github.com/blakeembrey/arity

@susil-rxr 원래 함수 길이를 유지하기 위해 일반 함수 래퍼에서와 같이 할 수 있습니까?

2.0.0-rc.1 이제 일반 함수 래퍼를 추가할 수 있습니다. 또한 원래 기능 길이를 유지하는 기능이 내장되어 있습니다.

이 스레드는 닫힌 후 최근 활동이 없었기 때문에 자동으로 잠겼습니다. 관련 버그에 대한 새 문제를 여십시오.