Question

我正在尝试使用函数GetUsername来调用实际的实现，并返回用户名，如果它被找回到变量结果中。我正在使用Jasmine的完成功能，但测试不正确。即使预期值和实际值不相同，它也会继续通过。任何帮助或建议都会很棒！提前谢谢。

以下是我的目标：

var Git = {
    VerifyGitUser: function (username, callback) {
        $.ajax({
            url: 'https://api.github.com/users/' + username
        })
        .done(function (data) {
            callback.call(this, data);
        })
        .fail(function (data) {
            callback.call(this, data);
        })
    }
}

var User = {
    GetUsername: function (username) {
        Git.VerifyGitUser(username, function (data) {
            if (data.login) {
                return data.login;
            } else {
                return null;
            }
        });
    }
}

这是我的测试：

describe('User', function () {
    it('should return username', function (done) {
        spyOn(Git, 'VerifyGitUser');

        spyOn(User, 'GetUsername').and.callThrough();

        var result = User.GetUsername('test');
        done();

        expect(Git.VerifyGitUser).toHaveBeenCalledWith('test');
        expect(User.GetUsername).toHaveBeenCalled();
        expect(result).toEqual('test');
    })
});

Answer 1

目前无法在User.GetUsername完成后检索数据，因此它只会返回undefined。另外，在使用done()声明任何内容之前，您对expect的调用正在完成测试。

您可以User.GetUsername接听回电，就像您为Git.VerifyGitUser所做的那样：

var User = {
    GetUsername: function (username, callback) {
        Git.VerifyGitUser(username, function (data) {
            if (data.login) {
                callback(data.login);
            } else {
                callback(null);
            }
        });
    }
}

现在您知道User.GetUserName何时完成。因此，在测试中，您可以传递User.GetUserName一个可以调用done()的回调：

describe('User', function () {
    it('should return username', function (done) {
        spyOn(Git, 'VerifyGitUser');

        spyOn(User, 'GetUsername').and.callThrough();

        User.GetUsername('test', function(result) {
            expect(Git.VerifyGitUser).toHaveBeenCalledWith('test');
            expect(User.GetUsername).toHaveBeenCalled();
            expect(result).toEqual('test');

            done();
        });
    })
});

其他想法：

您是否需要在此测试期间调用实际的API？您可以使用Jasmine间谍查看从VerifyGitUser返回的模拟API数据。
在VerifyGitUser我不明白为什么你需要使用callback.call(this, data)强制上下文。似乎callback(data)就足够了。
您可能希望查看从异步函数返回的promise，而不是让它们接受回调。

测试异步Ajax调用

1 个答案: