Sinon: v7.2.0 रिलीज के बाद अप्रत्याशित रूप से विफल परीक्षण

आम तौर पर महान बग रिपोर्ट के लिए धन्यवाद। हालांकि, इसकी एक बड़ी कमी है: मैं सत्यापन या बग को ट्रैक करने के लिए आपका उदाहरण नहीं चला सकता! यह अपने आप चलने योग्य नहीं है, और पुनरुत्पादन के तरीके के बिना, इसे ठीक करना मुश्किल है।

क्या आप प्रजनन के लिए न्यूनतम उम्मीदवार बना सकते हैं? आपको var myObject = { aMethod: function(){}, aProp: 42 }; // etc जैसे सरलीकरण की तरह पूरे मूल उदाहरण को दोहराने की ज़रूरत नहीं है।

अगर हम जानते थे कि यह टूट रहा है, तो हम निश्चित रूप से प्रमुख संस्करण को ऊपर उठाएंगे, इसलिए एक रिग्रेशन टेस्ट बहुत अच्छा होगा!

PS लगता है जैसे sinon-chai भी शामिल है। यह सूचीबद्ध करने लायक है।

@ fatso83 मुझे देखने दो कि मैं प्रजनन के लिए एक अलग, चलने योग्य उदाहरण कैसे प्राप्त कर सकता हूं

कोई अपडेट्स?

इस समस्या को स्वचालित रूप से पुराने के रूप में चिह्नित कर दिया गया है क्योंकि इसमें हाल की गतिविधि नहीं है। आगे कोई गतिविधि नहीं होने पर इसे बंद कर दिया जाएगा। आपके योगदानों के लिए धन्यवाद।

@mantoni क्या आपने अपनी किसी परियोजना में ऐसा कुछ होने का अनुभव करने की रिपोर्ट नहीं की?

अपडेट की कमी के लिए क्षमा करें, मैं अब उस प्रोजेक्ट से संबद्ध नहीं हूं जिसमें समस्या थी। अगर किसी और को एक ही समस्या का सामना करना पड़ रहा है, तो यह बहुत अच्छा होगा अगर वे रेप्रो के लिए एक उदाहरण प्रदान कर सकें

@ fatso83 मुझे एक ही मुद्दे पर चलने की याद नहीं है। मैंने बिना किसी समस्या के कुछ परियोजनाओं में नवीनतम सिनॉन में अपग्रेड किया है।

पुन: पेश नहीं किया जा सकता के रूप में बंद करना

इस मुद्दे को मृतकों में से पुनर्जीवित करने के लिए क्षमा करें, लेकिन मैं उसी समस्या में भाग रहा हूं।

_नीचे दिए गए कोड को चित्रण उद्देश्यों के लिए सरल बनाया गया है।_

मेरे आवेदन में मेरे पास यह वर्ग है

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

और मैं इसे बाद में कोआ नियंत्रक फ़ंक्शन में इस तरह उपयोग करता हूं

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.13" का उपयोग करता है) के साथ परीक्षण किया, और यह काम करता है।
संस्करण 7.2.0 विफल रहता है ( samsam 3.3.3 का उपयोग करता है)।
नवीनतम संस्करण 9.2.3 भी विफल रहता है ( samsam 5.3.0 का उपयोग करता है)।

मैं नोड 14.5.3 पर हूं, लेकिन यह नोड 12.20.0 पर भी होता है।

मैं तुलना में user नियंत्रक में और expectedUser के माध्यम से परीक्षण में JSON.stringify मैन्युअल रूप से और 100 वे कर रहे हैं% के बराबर। वाक्य रचनात्मक रूप से एकमात्र अंतर यह है कि नियंत्रक कक्षा का उपयोग करता है, जबकि परीक्षण वस्तु का शाब्दिक उपयोग करता है।

क्या किसी के पास कोई अच्छा विचार है कि ऐसा क्यों होता है? या किसी के पास इस पर काम करने का अच्छा विचार है?

सधन्यवाद,
डैनियल

जब तक हमारे पास इसे सत्यापित करने के लिए कोई प्रतिलिपि प्रस्तुत करने योग्य मामला नहीं है, तब तक हम इस पर ध्यान देने के लिए अपने खाली घंटे खर्च करने को उचित नहीं ठहरा सकते, मुझे डर है। मेरे पास देखने के लिए कुछ नहीं होगा और यह सत्यापित करने के लिए कुछ भी नहीं होगा कि मैं सही था।

हालाँकि, मुझे बग ठीक करना पसंद है, इसलिए यदि आप कुछ ऐसा बनाने की कोशिश में कुछ समय बिता सकते हैं _मैं चला सकता हूँ, तो ऐसा करें! रनकिट ऐसा करने के लिए एक बेहतरीन सेवा है: https://runkit.com/fatso83/sinon-issue-reproducible-bug-template

हाय @ fatso83 ,

इसे देखने के लिए अपना समय निकालने के लिए धन्यवाद। जैसा आपने मुझसे पूछा था मैंने वैसा ही किया और एक रनकिट डेमो बनाया जो बग दिखाता है। बस अलग-अलग require('sinon') स्टेटमेंट पर कमेंट/अनकमेंट करें और बग देखने के लिए कोड रन करें।

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

मैं यहां कोड भी पोस्ट करता हूं, अगर किसी भी कारण से रनकिट काम नहीं करता है।

// 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);
    });
});

जब आपके पास समय हो तो कृपया एक और नज़र डालें। धन्यवाद।

सधन्यवाद,
डैनियल