<p>sinon.restore () &amp;&amp; sandbox.restore () 가짜를 복원하지 않음</p>

현재로서는 _ 버그처럼 보이지만 _ 솔직히 저는 이것에 대해 완전히 확신하지 못합니다. 가짜에 대한 요점은 어떤 의미에서는 불변이어야한다는 점을 기억하지만, 상태가 있기 때문에 (즉, 호출되었는지 알고 있음) 불변성은 스터 빙 동작으로 만 제한되어야합니다.

@mroderick 이 대답하기에 적합한 사람이 될 수 있습니다 ...

이것이 올바른 동작이라면 최소한 문서를 업데이트해야합니다.

@ fatso83 감사합니다! .restore 문서는 가짜를 언급했습니다.

여기에서 .

이 버그는 나 자신이 조금만 있었으므로 수정하면 좋을 것입니다.

며칠 동안 다른 사람이 PR을하지 않으면 문제를 해결할 시간을 찾을 수 있는지 확인할 수 있습니다.

나는 이것을 지금 조사했고 위의 테스트와 같이 문서는 약간 오해의 소지가 있습니다. 실제로는 (거의) 말하는 것처럼 보이지만 먼저 몇 가지 용어를 정의하겠습니다.

복원 -샌드 박스를 복원한다는 것은 모든 스텁 기능을 원래 상태로 복원하는 것을 의미합니다. 즉, 스텁이 원본을 덮어 쓰면 각 개체에서 제거됩니다.
재설정 -재설정은 생성 된 각 페이크의 상태가 지워지지 만 개체에는 여전히 존재 함을 의미합니다.

위 테스트의 문제점은

1. 샌드 박스에서 모든 가짜를 제거합니다.
2. 그런 다음 샌드 박스 (현재 비어 있음)에서 가짜 컬렉션을 진행하고 가짜 목록 (빈)을 재설정하려고합니다.

말이 안돼 :-)

나는 테스트했고 reset 및 restore 호출이 제대로 작동합니다. 거의 ... reset 는 calledOnce 의 상태를 정리하지만 lastArgument 은 (는) 정리되지 않은 것 같습니다. 세부 사항을 잃어 버렸는지 좀 더 확인합니다.

재현 할 수없는 것으로 종료하지만 fake.lastArg 버그에 대한 새로운 문제가 열립니다.

확인하려면 : https://runkit.com/fatso83/sinon-issue-2065

이 버그가 나에게도 발생하고 있음을 확인할 수 있습니다. 구현이 그대로 유지되어야하는 테스트 시나리오가 있지만 함수는 이전 테스트의 스텁 결과를 전달합니다. 즉, sandbox.restore() 이 (가) 할 예정입니다.

7.5.0 버전에서 이런 일이 발생합니다.

@ oscarr-reyes https://github.com/sinonjs/sinon/commit/54df7f7000a9db2dd05319daa49518f3cd5a5dd7이 문제를 해결하는지 확인할 수 있습니까? 아직 새 릴리스로 만들지 않은 것 같습니다.

@ oscarr-reyes 재현 가능한 예를 만들 수 있습니까? 우리는 아직이 버그를 보지 못했습니다. 위의 스레드와 제 설명을 살펴보면 우리가 발견 한 유일한 버그는 하나의 플래그가 지워지지 않은 것입니다. 다른 모든 것이 작동합니다. 따라서 실제로 증명을 보지 않는 한 버그는 없습니다 (lastArg 플래그를 제외하고).

나는 같은 문제에 직면했다.

내 "validateCaptcha"미들웨어를 샌드 박싱하고 복원 할 때 스텁을 계속 발행합니다.

/src/middlewares/captcha.js

const request = require("got");

const validateCaptcha = async (req, res, next) => {
    // if (process.env.NODE_ENV !== "production") return next();

    const captcha = req.body["g-recaptcha-response"] || req.body.g_recaptcha_response;

    // g-recaptcha-response is the key that browser will generate upon form submit.
    // if its blank or null means user has not selected the captcha, so return the error.
    if (captcha === undefined || captcha === "" || captcha === null)
        return res
            .status(400)
            .json({ message: "Please, verify that you are not a robot" });

    // Put your secret key here.
    const secretKey = process.env.GOOGLE_CAPTCHA_API_SECRET_KEY;

    // req.connection.remoteAddress will provide IP address of connected user.
    const verificationUrl = `https://www.google.com/recaptcha/api/siteverify?secret=${secretKey}&response=${captcha}`;

    try {

        // Hitting GET request to the URL, Google will respond with success or error scenario.
        const { body } = await request.post(verificationUrl, { responseType: "json" });

        // Success will be true or false depending upon captcha validation.
        if (body.success !== undefined && !body.success)
            return res
                .status(400)
                .json({ message: "Captcha validation error" });
    } catch (error) {
        return res
            .status(503)
            .json({ message: "Couldn't validate captcha" });
    }

    next();
};

module.exports = {
    validateCaptcha,
};

/test/api/user.spec.js

require("../setup.spec");
const supertest = require("supertest");
const captcha = require("../../src/middlewares/captcha");
const sinon = require("sinon");
let app;
let agent;
let sandbox;

const login = (anAgent, userCredentials) =>
    anAgent
        .post("/api/users/signIn")
        .send(userCredentials);

const logout = (anAgent) =>
    anAgent
        .post("/api/users/signOut");

describe("User endpoints", function () {
    this.beforeAll(async () => {
        sandbox = sinon.createSandbox();
        sandbox
            .stub(captcha, "validateCaptcha")
            .callsFake(async (req, res, next) => {
                console.log("Validate Captcha FAKE");
                return next();
            });
        // sandbox.restore();
    });

    this.beforeEach(async () => {
        app = require("../../src/app");
        agent = supertest.agent(app);
        const sequelize = require("../../src/models");
        await sequelize.sync({ force: true });

        const Customer = require("../../src/models/customer");
        const User = require("../../src/models/user");
        await Customer.create(
            {
                firstName: "Test",
                lastName: "Customer",
                user: {
                    email: "[email protected]",
                    password: "boo",
                },
            },
            {
                include: [User],
            }
        );
    });

    this.afterEach(async () => {
        sandbox.restore();
    });

    it("should create a new user", function (done) {
        agent
            .post("/api/users/signUp")
            .send({
                firstName: "firstName",
                lastName: "lastName",
                email: "[email protected]",
                password: "aPassword",
            })
            .expect(201)
            .end((err, res) => {
                err ? done(err) : done();
            });
    });

    it("should sign in", function (done) {
        login(agent, { email: "[email protected]", password: "boo" })
            .expect(200)
            .end((err, res) => {
                const { message, user } = res.body;
                message.should.be.equal("valid");
                user.should.have.property("email").equal("[email protected]");
                user.should.have.property("customer").have.property("firstName").equal("Test");
                user.should.have.property("customer").have.property("lastName").equal("Customer");

                agent
                    .get("/api/users/me")
                    .expect(200)
                    .end((err, res) => {
                        const user = res.body;
                        user.should.have.property("email").equal("[email protected]");
                        user.should.have.property("customer").have.property("firstName").equal("Test");
                        user.should.have.property("customer").have.property("lastName").equal("Customer");
                        err ? done(err) : done();
                    });
            });
    });

    it("should sign out", async function () {
        await login(agent, { email: "[email protected]", password: "boo" });
        await logout(agent)
            .expect(302)
            .expect("Location", "/");

        return agent
            .get("/api/users/me")
            .expect(401);
    });

    it("should return unauthorized", function (done) {
        agent
            .get("/api/users/me")
            .expect(401)
            .end((err, res) => {
                err ? done(err) : done();
            });
    });
});

작동하지 않는지 확인하기 위해 내가하는 일은 다음과 같습니다.
1) 첫 번째 테스트를 위해 모의
2) 다음 항목에 대해 복원하십시오.

beforeAll 안에 "sandbox.restore ();"라는 주석이 있습니다. 내가 시도했고 거기에서 작동했습니다 ...하지만 쓸모가 없습니다

나는 또한 내 app.js에 매번 요구하고 있습니다.

5 분 전에 다음과 같이 수정했습니다.

require("../setup.spec");
const decache = require("decache");
const supertest = require("supertest");
const sequelize = require("../../src/models");

const sinon = require("sinon");
const sandbox = sinon.createSandbox();
let agent;

const login = (anAgent, userCredentials) =>
    anAgent
        .post("/api/users/signIn")
        .send(userCredentials);

const logout = (anAgent) =>
    anAgent
        .post("/api/users/signOut");

describe("User endpoints", function () {
    this.beforeAll(async () => {

    });

    this.beforeEach(async () => {
        decache("../../src/app");
        decache("../../src/middlewares/captcha");

        const captcha = require("../../src/middlewares/captcha");

        sandbox
            .stub(captcha, "validateCaptcha")
            .callsFake(async (req, res, next) => {
                console.log("Validate Captcha FAKE");
                return next();
            });

        const app = require("../../src/app");
        agent = supertest.agent(app);
        await sequelize.sync({ force: true });


        const Customer = require("../../src/models/customer");
        const User = require("../../src/models/user");
        await Customer.create(
            {
                firstName: "Test",
                lastName: "Customer",
                user: {
                    email: "[email protected]",
                    password: "boo",
                },
            },
            {
                include: [User],
            }
        );
    });

    this.afterEach(async () => {
        sandbox.restore();
    });

    it("should create a new user", function (done) {
        agent
            .post("/api/users/signUp")
            .send({
                firstName: "firstName",
                lastName: "lastName",
                email: "[email protected]",
                password: "aPassword",
            })
            .expect(201)
            .end((err, res) => {
                err ? done(err) : done();
            });
    });

    it("should sign in", function (done) {
        login(agent, { email: "[email protected]", password: "boo" })
            .expect(200)
            .end((err, res) => {
                const { message, user } = res.body;
                message.should.be.equal("valid");
                user.should.have.property("email").equal("[email protected]");
                user.should.have.property("customer").have.property("firstName").equal("Test");
                user.should.have.property("customer").have.property("lastName").equal("Customer");

                agent
                    .get("/api/users/me")
                    .expect(200)
                    .end((err, res) => {
                        const user = res.body;
                        user.should.have.property("email").equal("[email protected]");
                        user.should.have.property("customer").have.property("firstName").equal("Test");
                        user.should.have.property("customer").have.property("lastName").equal("Customer");
                        err ? done(err) : done();
                    });
            });
    });

    it("should sign out", async function () {
        await login(agent, { email: "[email protected]", password: "boo" });
        await logout(agent)
            .expect(302)
            .expect("Location", "/");

        return agent
            .get("/api/users/me")
            .expect(401);
    });

    it("should return unauthorized", function (done) {
        agent
            .get("/api/users/me")
            .expect(401)
            .end((err, res) => {
                err ? done(err) : done();
            });
    });
});

이 문제도보고 있습니다. 왜 이것이 닫혀 있습니까?

복원 동작은 스텁을 설정하는 데 사용되는 특정 구문에 민감 할 수 있습니다. Afaik 두 가지 모두이 작업을 수행하는 유효한 방법이어야하지만 restore ()는 다릅니다. 하나에서는 작동하고 다른 하나에서는 예기치 않은 결과가 발생합니다.

const sinon = require('sinon')

// restore does not work as expected

let db = {
  query: () => {
    return 'my query 1'
  }
}
const dbStub = sinon.stub().resolves('somejson')
db.query = dbStub
db.query() // somejson
sinon.restore()
db.query() // resolves nothing

// restore works as expected

db = {
  query: () => {
    return 'my query 1'
  }
}
sinon.stub(db, 'query').resolves('somejson')
db.query() // somejson
sinon.restore()
db.query() //my query 1

@scottpatrickwright db.query 직접 할당 할 때 sinon 는 복원해야 할 항목이나 할당 전에 값이 무엇인지 알 수 없습니다.

@scottpatrickwright 스레드에서 분명히 알 수 있듯이 버그가 없기 때문에 닫힙니다. 귀하의 경우와 같이 사용자 오류였습니다. 설명되고 의도 한대로 작동합니다. :-)

Sinon 샌드 박스는 알고있는 정보를 정리할 수 있습니다. 즉, Sinon은 상호 작용하는 대상에 대한 지식이 있어야합니다. 위에서 설명한 스텁은 독립형으로 만들어집니다. Sinon은 사용자가 할당 한 개체를 인식하지 못합니다. 두 번째 예에서 변경됩니다. sinon.replace(obj, fieldname, fake) 을 (를) 수행하여 동일한 결과를 얻을 수도 있습니다. JavaScript 일뿐입니다 🙂

충분히 공평합니다-설명 감사합니다.

2021 년 2 월 19 일 금요일 오전 7:02 Carl-Erik Kopseng [email protected]
썼다 :

@scottpatrickwright https://github.com/scottpatrickwright 폐쇄되었습니다
스레드에서 명백한 버그가 없기 때문입니다. 그냥
귀하의 경우와 같이 사용자 오류 : 설명 및 의도대로 작동합니다 :-)

Sinon 샌드 박스는 알고있는 정보를 정리할 수 있습니다. 이는 Sinon이
상호 작용하는 대상에 대한 지식을 갖습니다. 스텁 당신
위의 설명은 독립형으로 만들어집니다. Sinon은
할당 한 개체. 두 번째 예에서 변경됩니다. 당신은 할 수 있습니다
또한 sinon.replace (obj, fieldname, fake)를 수행하여 동일한 결과를 얻습니다. 그냥
자바 스크립트 🙂

—
당신이 언급 되었기 때문에 이것을 받고 있습니다.
이 이메일에 직접 답장하고 GitHub에서 확인하세요.
https://github.com/sinonjs/sinon/issues/2065#issuecomment-782032440 또는
구독 취소
https://github.com/notifications/unsubscribe-auth/AAKKRTFVMGEYVIB5GSIDXKTS7ZHMRANCNFSM4IGBASQQ
.