Promise改写的异步回调函数为何测试超时?

苏三
  • 512

在koa2使用了其他模块,是异步的方法,把他用promise包装了一下,结果出了问题

import Koa from "koa";
import Router from "koa-router";
import request from "request";
import supertest from "supertest";

const app = new Koa();
const router = new Router();
const client = supertest.agent(app.listen(3000));

router
    .get("/send", async (ctx, next)=> {
        let params = {...}

        let aa = await Prequest(params);
        console.log(aa);//不打印
        ctx.status = 200;
    })

app
    .use(router.routes())
    .use(router.allowedMethods());

//测试错误   超时了   
describe("test", function () {
    this.timeout(10000);
    it("send", function (done) {
        client
            .get('/send')
            .expect(200,done)
    });
})
function Prequest(params) {
    return new Promise(function (resolve, reject) {
        request(params, function (error, response, body){
            if (error || !body || body.code != 1) {
                reject(error);
            }
            resolve(body);//并没有resolve出去
        })
    });
}
评论
阅读 2.7k
2 个回答

koa没用过,一直觉得这种古怪的利用generator做中间件的手段太过“引人入胜”,实在不敢冒用。(可能我太low了)

首先,(由于不懂koa),不清楚如下设置status的方法是否正确:

ctx.status = 200;

其次,测试可以这么改改试试:

describe("test", function () {
    this.timeout(10000);
    it("send", function (done) {
        client
            .get('/send')
            .expect(200)
            .end(function(err, res) {
                if (err) return done(err);
                done();
            });
    });
})

补充:

图片描述

我看koa-router 7.x配合koa 2.x才支持async/await语法,你确定版本都用对了吧?

对照官方的例子看看吧supports promises for async/await

最后,真心没用过这怪玩意,你自己多找找文档吧

再次补充:

的确,如@wusisu 所说,你的request写的有点问题,应该改改:

function Prequest(params) {
    return new Promise(function (resolve, reject) {
        request(params, function (error, response, body){
            if (error || !body || body.code != 1) {
                return reject(error);//这里一定要return出去
            }
            resolve(body);
        })
    });
}

我是koa@2的老司机了。
不过问题不在koa上,而是那个Promise没有返回。

我也同样没有看懂request这个函数的使用,建议在request的回调中,打印一下看看。

宣传栏