测试承诺导致未定义的值

时间:2016-08-22 02:59:27

标签: javascript promise ecmascript-6 mocha

我在测试代码时收到此错误:

1) Sourcerer Testing:  getStatusCode :
     Error: Expected undefined to equal 200

我不确定为什么我在测试中获得undefined但是当我运行代码时,我得到200。可能是因为没有正确处理promises

Test code:

import expect from 'expect';
import rp from 'request-promise';
import Sourcerer from './sourcerer';


describe("Sourcerer Testing: ", () => {
    let sourcerer = new Sourcerer(null);
    const testCases = {
        "https://www.google.com": 200,
        // "www.google.com":
    };

    describe("getStatusCode", () => {
        it("", () => {
            for (let testCase in testCases) {
                sourcerer.setSourcererUrl(testCase);
                expect(sourcerer.url).toEqual(testCase);
                expect(sourcerer.getStatusCode()).toEqual(testCases[testCase]);
            }
        });
    });
});

代码:

import rp from 'request-promise';

export default class Sourcerer {
    constructor(url) {
        this.options = {
            method: 'GET',
            url,
            resolveWithFullResponse: true
        };
        this.payload = {};
    }

    setSourcererUrl(url) {
        this.url = url;
    }

    getSourcererUrl() {
        return this.url;
    }

    analyzeSourcePage() {
        rp(this.options).then((res) => {
            console.log(res);
        }).catch((err) => {
            console.log("ERROR");
            throw(err);
        });

    }

    getStatusCode() {
        rp(this.options).then((res) => {
            console.log(res.statusCode);
            return res.statusCode;
        }).catch((err) => {
            console.log("STATUS CODE ERROR");
            return 0;
        });
    }
}

2 个答案:

答案 0 :(得分:1)

免责声明:我不会在单个it()中运行for循环,因为我想知道哪个迭代失败了。认为有办法实现这一点,但这是另一个故事。此外,这在很大程度上取决于你的测试跑步者,但这里有一些我觉得有用的经验法则。

但是对于你所问的问题,在承诺得到解决之前,测试不应该进行评估。有时(例如在mocha中),这意味着从it()内部函数返回promise。有时,这意味着获得一个完成功能,并在准备好进行测试评估时调用它。如果您提供有关测试框架的更多信息,我可能会提供帮助(其他人肯定会)

答案 1 :(得分:1)

getStatusCode不会返回任何内容。它应该回复一个承诺:

getStatusCode() {
    return rp(this.options)...
}

在这种情况下,规范将失败,因为它期望promise对象等于200。

它更复杂,因为规范是异步的,并且在规范完成之前应该等待几个promise。它应该像

    it("", () => {
        let promises = [];
        for (let testCase in testCases) {
            sourcerer.setSourcererUrl(testCase);
            let statusCodePromise = sourcerer.getStatusCode()
            .then((statusCode) => {
                expect(sourcerer.url).toEqual(testCase);
                expect(statusCode).toEqual(testCases[testCase]);
            })
            .catch((err) => {
                throw err;
            });
            promises.push(statusCodePromise);
        }

        return promises;
    });

co为流量控制Promise.all提供了一个很棒的替代方法:

    it("", co.wrap(function* () {
        for (let testCase in testCases) {
            sourcerer.setSourcererUrl(testCase);
            expect(sourcerer.url).toEqual(testCase);
            let statusCode = yield sourcerer.getStatusCode();
            expect(statusCode).toEqual(testCases[testCase]);
        }
    });