¿Cómo tener una prueba atómica real en QUnit?

Question

decir que tengo dos clases llamada Book y Library de la siguiente manera:

var Book = function(title, author) 
{ 
    this.title = title; 
    this.author = author; 
}; 

y

var Library = function() 
{ 
    var dbName = 'test'; 

    this.getLibrary = function() { 
     return JSON.parse(window.localStorage.getItem(dbName)); 
    }; 

    this.save = function(library) { 
     window.localStorage.setItem(dbName, JSON.stringify(library)); 
    }; 
} 

Library.prototype.addBook = function(book) { 
    var library = this.getLibrary(); 
    library.push(book); 

    this.save(library); 

    return library; 
}; 

Library.prototype.removeBook = function(book) { 
    var library = this.getLibrary(); 

    // Find and delete the right book 

    this.save(library); 

    return library; 
}; 

Mi pregunta es: ¿Cómo puedo Unidad a probar la clase Library usando QUnit a disfrutar de una auténtica atómica y prueba independiente?

Escribí esta función de prueba pero no me ha satisfecho. No parece ser muy atómico e independiente ya que mezcla varias funciones que creo deberían probarse de forma independiente. Me gustaría saber si hay una manera mejor o ya la estoy probando bien.

test("test", function() { 
    var library = new Library(); 

    deepEqual(library.getLibrary(), []); 

    // Add book1 
    var book1 = new Book("A book", "An author"); 
    deepEqual(library.addBook(book1), [book1]); 
    deepEqual(library.getLibrary(), [book1]); 

    // Add book2  
    book2 = new Result("A new book", "Another author"); 
    deepEqual(library.addBook(book2), [book2, book1]); 
    deepEqual(library.getLibrary(), [book2, book1]]); 

    // Remove book1 
    deepEqual(library.removeResult(book1), [book2]); 
    deepEqual(library.getLibrary(), [book2]); 

    // Remove book2 
    deepEqual(library.removeResult(book2), []); 
    deepEqual(library.getLibrary(), []); 
}); 

Answer 1

Evite escribir pruebas "omnibus" que dependen del estado interno de la unidad que está probando. Hacer pruebas de grano fino a determinados comportamientos y no estatales:

test("addBooksReturnsLibrary", function(){ 
    var l = new Library(); 
    l.addBook(new Book(...)); 
    l.addBook(new Book(...)); 
    var addBooksRetval = l.addBook(new Book(...)); 
    deepEqual(addBooksRetval, l.getLibrary(); 
}); 

test("libraryContainsAddedBook"), function() { 
    var l = new Library(); 
    var b = new Book(...); 
    l.addBook(b); 
    notEqual(-1, library.indexOf(b)); 
}); 

// etc and so forth 

El uso intensivo de deepEqual parece que es una prueba de olor más a menudo que no. Si necesita, por ejemplo, que los libros estén en la biblioteca en un orden específico, diría que es mejor probar específicamente las restricciones de ordenamiento.