Sinon: v7.2.0 릴리스 후 예기치 않게 실패하는 테스트

일반적으로 훌륭한 버그 보고서에 감사드립니다. 하지만 한 가지 큰 결함이 있습니다. 검증이나 버그 추적을 위해 귀하의 예제를 실행할 수 없습니다! 자체적으로 실행할 수 없으며 재현 방법이 없으면이를 고치기가 어렵습니다.

최소한의 번식 후보를 만들 수 있습니까? var myObject = { aMethod: function(){}, aProp: 42 }; // etc 와 같은 단순화처럼 전체 원본 예제를 복제 할 필요가 없습니다.

이것이 고장났다는 것을 알았다면 물론 메이저 버전을 올렸을 것이므로 회귀 테스트가 좋을 것입니다!

추신 : sinon-chai 도 관련된 것 같습니다. 그것은 나열 할 가치가 있습니다.

@ fatso83 복제를 위해 분리되고 실행 가능한 예제를 얻는 방법을 보겠습니다.

업데이트가 있습니까?

이 문제는 최근 활동이 없었기 때문에 자동으로 오래된 것으로 표시되었습니다. 더 이상 활동이 발생하지 않으면 폐쇄됩니다. 귀하의 기여에 감사드립니다.

@mantoni 프로젝트에서 이런 일이 발생했다고보고하지 않았습니까?

업데이트가 없어서 죄송합니다. 더 이상 문제가 포함 된 프로젝트와 연결되어 있지 않습니다. 다른 사람이 같은 문제를 경험할 수 있다면 재현의 예를 제공하면 좋을 것입니다.

@ fatso83 같은 문제가 발생한 기억이 없습니다. 몇 가지 프로젝트에서 문제없이 최신 Sinon으로 업그레이드했습니다.

재현 할 수 없음으로 닫기

이 문제를 죽음에서 되살려 서 죄송하지만 같은 문제에 봉착했습니다.

_ 아래 코드는 설명을 위해 단순화되었습니다 ._

내 응용 프로그램에는이 수업이 있습니다.

class MyUser {
  // ... some user properties  
}

나중에 이와 같은 Koa 컨트롤러 기능에서 사용합니다.

const mqClient = require('./mqClient');

// GET /userDetails/<id> endpoint
async function getUserDetailsController(req, res) {
  const user = new MyUser();

  // populate user with properties etc.
  user.id = '1';
  user.name = 'Hans Gruber';
  user.tasks = [{ id: '42', title: 'execute Nakatomi tower heist' }];

  // ... get more details ...

  // Notify other users about this user
  mqClient.notifyOthers(user);

  res.json({ ok: true, user });
}

그리고 내 getUserDetailsController.test.js 다음과 같이합니다.

const mqClient = require('./mqClient');

const createExpectedUser = overrides => {
  id: '1',
  name: 'Hans Gruber',
  tasks: [{ id: '42', title: 'execute Nakatomi tower heist' }],
  ...overrides
};

it('does not fail', async () => {
  sinon.spy(mqClient, 'notifyOthers');

  const expectedUser = createExpectedUser();

  await presetRequest // helper object to run the controller code
    .get('/userDetails/1')
    .expect(200);

  // This works with sinon 7.1.1.
  // This fails with sinon 7.2.0 and onwards.
  sinon.assert.calledWithMatch(mqClient.notifyOthers, expectedUser);
});

나는 이것을 sinon 7.1.1 ( samsam 의 "2.1.3"을 사용함)으로 테스트했고, 작동합니다.
버전 7.2.0이 실패합니다 ( samsam 3.3.3 사용).
최신 버전 9.2.3도 실패합니다 ( samsam 5.3.0 사용).

저는 Node 14.5.3에 있지만 이것은 Node 12.20.0에서도 발생합니다.

컨트롤러의 user 와 expectedUser 를 통해 테스트에서 JSON.stringify 수동으로 비교했는데 100 % 동일합니다. 구문 상 유일한 차이점은 컨트롤러는 클래스를 사용하는 반면 테스트는 개체 리터럴을 사용한다는 것입니다.

왜 이런 일이 발생하는지 좋은 사람이 있습니까? 아니면이 문제를 해결하는 방법에 대한 좋은 아이디어가 있습니까?

감사합니다.
다니엘

이를 확인할 수있는 재현 가능한 사례가없는 한,이 문제를 조사하는 데 무료 시간을 보내는 것을 정당화 할 수 없습니다. 두렵습니다. 나는 찾을 것이없고 내가 옳다는 것을 증명할 것도 없을 것입니다.

그래도 버그를 고치는 것을 좋아하므로 _ 내가 실행할 수있는 _ 무언가를 만드는 데 시간을 할애 할 수 있다면 그렇게하세요! RunKit은이를위한 훌륭한 서비스입니다 : https://runkit.com/fatso83/sinon-issue-reproducible-bug-template

안녕하세요 @ fatso83 ,

시간을내어 조사해 주셔서 감사합니다. 나는 당신이 요청 한대로 버그를 보여주는 런킷 데모를 만들었습니다. 다른 require('sinon') 문을 주석 / 주석 해제하고 코드를 실행하여 버그를 확인하십시오.

https://runkit.com/danielkg/sinon-issue-reproducible-bug-template

어떤 이유로 든 runkit이 작동하지 않는 경우를 대비하여 여기에 코드를 게시합니다.

// Employs 'mini-mocha' to emulate running in the Mocha test runner (mochajs.org)
require("@fatso83/mini-mocha").install();
// const sinon = require('[email protected]'); // WORKS!
const sinon = require('[email protected]'); // FAILS!
// const sinon = require('sinon');       // FAILS!
const { assert } = require('@sinonjs/referee');

const SAMPLE_USER = {
    id: 1,
    name: 'Kid',
    age: 22,
    weight: 83.5,
    notes: [{ txt: 'abc' }, { txt: 'def' }, { txt: 'ghi' }],
};

class MyUser {
    constructor(id, name, age, weight, notes = []) {
        this.id = id;
        this.name = name;
        this.age = age;
        this.weight = weight;
        this.notes = notes;
    }
}

const createFakeUser = overrides => ({
    id: '007',
    name: 'Bond',
    ...overrides,
});

class Talkie {
    talk(user, topic) {
        console.log('talked to', user.name, 'about', topic);
    }
}

const action = (userId, talkie) => {
    const { name, age, weight, notes } = SAMPLE_USER;
    const user = new MyUser(userId, name, age, weight, notes);

    talkie.talk(user, 'apples');

    return user;
}


describe('stubbing', () => {
    const talkie = new Talkie();

    afterEach(() => {
        sinon.restore();
    });

    beforeEach(() => {
        sinon.spy(talkie, 'talk');
    });

    it('is the same user', () => {
        const userA = action(1, talkie);
        const userB = createFakeUser(SAMPLE_USER);

        assert.equals(JSON.stringify(userA), JSON.stringify(userB));
    });

    it('should set the return value', () => {
        const userA = action(1, talkie);
        const userB = createFakeUser(SAMPLE_USER);

        sinon.assert.calledWith(talkie.talk, userB, sinon.match.string);
    });
});

시간이있을 때 다시 한 번보세요. 감사합니다.

감사합니다.
다니엘