google/closure-compiler

Jasmine Externs are out of Date

Open

#3045 aperta il 6 ago 2018

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Java (1197 fork)batch import
Externshelp wanted

Metriche repository

Star
 (7176 star)
Metriche merge PR
 (Merge medio 12g 15h) (1 PR mergiata in 30 g)

Descrizione

I've found that many Closure externs are out of date for Jasmine. Specifically, I see:

  • Many custom matcher types are missing, including:
    • CustomMatcherFactories
    • MatchersUtil
    • CustomEqualityTester
    • CustomMatcher
  • Jasmine's pretty print utility is missing (jasmine.pp())
  • Most matcher functions (toBe(), toEqual(), etc.) accept an optional string to use as an error message if the expectation fails. This is also missing.
  • Likely others...

Without these types, custom matchers are basically impossible. Would be great to see these types updated.

See: https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/jasmine/index.d.ts

Guida contributor

Direzione di ricerca
Studia le dichiarazioni esterne esistenti per Jasmine nel repository closure compiler (probabilmente nella directory externs). Confronta con l'API dell'ultima versione di Jasmine, utilizzando come riferimento il file types/jasmine/index.d.ts di DefinitelyTyped. Identifica i tipi mancanti come CustomMatcherFactories, MatchersUtil, CustomEqualityTester, CustomMatcher, jasmine.pp() e il parametro stringa opzionale nei matcher. Quindi aggiungi o aggiorna di conseguenza il file di dichiarazioni esterne.
Tech stack
javascript
Dominio
testingdeveloper experience
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScript basicsClosure compiler externsJasmine testing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30