测试异步Ajax调用

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完成时从User.GetUsername检索数据，因此它将返回undefined 。 再加上你对done()调用正在完成测试，然后你就会发出任何expect 。

您可以让User.GetUsername进行回调，就像您正在为Git.VerifyGitUser做的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，而不是让它们接受回调。