expectHEAD è documentato ma non implementato?

Question

Nel nostro progetto angularjs interno, uno dei servizi ha la chiamata $http.head() che sto tentando di testare.

Per il test, sto usando Fake HTTP backend fornito da angular-mocks. Ecco il codice rilevante:

it('handle status code 200', inject(function ($httpBackend, ConnectionService) { 
    spyOn(Math, 'random').andReturn(0.1234); 

    httpBackend = $httpBackend; 
    httpBackend.expectHEAD('ping?rand=1234').respond(200); 
    ConnectionService.sendRequest(); 
    httpBackend.flush(); 

    expect(ConnectionService.stats.packetsReceived).toEqual(5); 

    httpBackend.verifyNoOutstandingExpectation(); 
    httpBackend.verifyNoOutstandingRequest(); 
})); 

esecuzione i risultati del test nel seguente errore:

PhantomJS 1.9.7 (Mac OS X) connection service tests sendRequest function handle status code 200 
FAILED TypeError: 'undefined' is not a function (evaluating 'httpBackend.expectHEAD('ping?rand=1234')') 
at /path/to/app/app-connection-service_test.js:66 
at d (/path/to/app/bower_components/angular/angular.min.js:35) 
at workFn (/path/to/app/bower_components/angular-mocks/angular-mocks.js:2159) 

Dopo aver scavato un po ', ho trovato il problema github rilevanti:

• fix($httpBackend): Add missing expectHEAD() and expectOPTIONS() methods

che, per quanto ho capito, significa che Non c'è davvero il metodo expectHEAD() in angock-mock: è documentato, ma, in realtà, non fa ancora parte di una versione angolare stabile.

Qual è il modo migliore per procedere?

Nota che devo stare con angolare < = 1.2, poiché questa applicazione deve essere eseguita su IE8 (Angular 1.3 è dropping support for IE8).

---

Una delle soluzioni che sto pensando al momento è quello di sostituire head() con get(). In questo caso, posso testarlo con il metodo esistente expectGET(). Ma non sono sicuro degli svantaggi.

Answer 1

Contrariamente al forking e alla modifica di una libreria, sarebbe molto meglio utilizzare il metodo sottostante.

$httpBackend.expect('HEAD', 'ping?rand=1234').respond(200); 

L'altro metodo che non è possibile utilizzare è solo un metodo di scelta rapida. È molto meglio e più facile usare questo metodo che cambiare la libreria.

Answer 2

Si può solo aggiungere nel cambiare se stessi: https://github.com/revolunet/angular.js/commit/b2955dd52725241dd9519baa12fe8ca74659004b

Si tratta di un cambiamento di una riga in angolari-mocks.js on line 1523:

1523: - angular.forEach(['GET', 'DELETE', 'JSONP'], function(method) { 
1523: + angular.forEach(['GET', 'DELETE', 'JSONP', 'HEAD'], function(method) {  

Visto che si deve stare con un non- versione corrente di Angular comunque, non dovrebbe causare troppi problemi in futuro.

Modifica: rivedere la risposta di JoelJeske, fa un buon punto: si potrebbe facilmente farlo senza forking. L'unico cambiamento che vorrei fare per la sua soluzione, è invece di chiamare il metodo direttamente

$httpBackend.expect('HEAD', 'ping?rand=1234').respond(200); 

avrei invece creare il metodo di scelta rapida direttamente:

$httpBackend.expectHEAD = function(url, headers) { 
    return $httpBackend.expect('HEAD', url, undefined, headers); 
}; 

E poi si può fare:

$httpBackend.expectHEAD('ping?rand=1234').respond(200);