Versuchen zu verstehen, Jasmine toHaveBeenCalled() matcher

Ich bin neu jasmine hier ist mein src - Datei, in die ich erstellen Auth Klasse 

function Auth() {
}

Auth.prototype.isEmpty = function(str) {
    return (!str || 0 === str.length);
}

Auth.prototype.Login = function (username , password) {
    if (this.isEmpty(username) || this.isEmpty(password)) {
        return "Username or Password cann't be blank ";
    }
    else {
        return "Logged In !";
    }
}

nun will ich testen, jasmine toHaveBeenCalled() matcher . Hier ist, was ich Schreibe 

it("should be able to Login", function () {
    spyOn(authobj);
    expect(authobj.Login('abc', 'abc')).toHaveBeenCalled();
});

aber Sie sagt, dass undefined() method does not exist

InformationsquelleAutor Ancient | 2013-05-28
  1. 17

    EDIT: Schau basecode Antwort für einen besseren Ansatz

    Von den docs, sollten Sie es verwenden, wie die folgenden:

    spyOn(foo, 'setBar');

it("tracks that the spy was called", function() {
  expect(foo.setBar).toHaveBeenCalled();
});

    So sollten Sie schreiben:

    it("should be able to Login", function () {
  spyOn(authobj, 'isEmpty');  
  authobj.Login('abc', 'abc');  
  expect(authobj.isEmpty).toHaveBeenCalled();
});
    • nach dem Versuch, was Sie sagte, es gibt mir diese Error: Expected a spy, but got undefined. in http://localhost:57494/Scripts/Jasmine/jasmine.js (line 1361)
    • Oops my bad, ich kann nicht versuchen, dieses Recht jetzt, aber siehe meine aktualisierte Antwort.
    • Ich will wissen, dass, Wenn ich rufe Login Funktion , dann tut es nennen isEmpty Funktion oder nicht ? .
    • Dann sollte der Spion auf isEmpty einen Anruf Login ich update meine Antwort.
    • Downvoted diese Antwort, weil Sie Befürworter der Spionage auf Einzelheiten der Implementierung. Siehe die Antwort von @basecode für einen besseren Ansatz.
    • vielen Dank für diesen Hinweis, werde ich prüfen basecode Antwort, und siehe, wo ich mich verbessern.

    InformationsquelleAutor Alberto Zaccagni
  2. 24

    Blick auf deinen use-case, kann ich nicht empfehlen, mit toHaveBeenCalled hier. toHaveBeenCalled ist in Fällen nützlich, wo Sie wollen testen callbacks (async) oder in Kombination mit mocks.

    Betrachten alles, was geschieht innerhalb Auth.prototype.Login als Implementierungsdetail, das nicht sichtbar ist "outerworld". Sie sollten nicht test-Implementierung-details. Was löst zwei Fragen.

    Warum sollte ich nicht test-Implementierung-details?

    Macht es refactoring schwer. Lassen Sie uns sagen, das Sie ersetzen möchten Auth.prototype.isEmpty für einige Gründe, die von underscore.isEmpty. Einige Tage später werden Sie entscheiden, Sie zu ersetzen underscore vollständig durch lodash. Dies würde Sie zwingen, sich zu ändern, Ihre test drei mal. Betrachten Sie alles, was verhindert, dass Sie von refactoring leicht als "no-go".

    Was soll ich das testen statt?

    Öffentliche API. Alles, was sichtbar ist, um die "outerworld". Das ist in deinem Fall "Eingeloggt !" und "Benutzername oder Passwort kann nicht leer sein ".

    Die Ergebnisse in 3 tests:

    describe('Login', function() {

 it('returns "success" string when username and password are not empty', function() {
   expect(new Auth().Login('non-empty', 'non-empty')).toBe('Logged In !');
 });

 it('returns "failure" string when username is empty', function() {
   expect(new Auth().Login('', 'non-empty')).toBe('Username or Password cannot be blank');
 });

 it('returns "failure" string when password is empty', function() {
   expect(new Auth().Login('non-empty', '')).toBe('Username or Password cannot be blank');
 });

});
    • Dies ist eine ausgezeichnete Antwort. Vielen Dank für die Zeit nehmen, zu erklären, den richtigen Weg zu testen.
    InformationsquelleAutor basecode
  3. 0

    Seine einfach zu bedienen, im Grunde:-

    spyOn(<name_of_the_object>, '<name_of_the_method>')

expect(<name_of_the_object>.<name_of_the_method>).toHaveBeenCalled();
    InformationsquelleAutor prashantsahni
Schreibe einen Kommentar