Codecov Report

:exclamation: No coverage uploaded for pull request base (master@40c9e85). Click here to learn what that means. The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #12   +/-   ##
=========================================
  Coverage          ?   89.30%           
=========================================
  Files             ?       13           
  Lines             ?      346           
  Branches          ?        0           
=========================================
  Hits              ?      309           
  Misses            ?       37           
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 40c9e85...e06fc15. Read the comment docs.

nepito commented 3 years ago

Woow @nepito ya está quedando muchísimo más limpio el repo! Muchas gracias por atender mis comentarios. Todo quedó muy bien, te mereces unas jejeje.

jajajajaj Hoy me quedé pasando el scroll mientras veía lo limpio :raised_hands: que está ahora.

xwalls commented 3 years ago

Del issue #4 chequé y ya está todo. Solo una duda en el punto:

Prueba para import_tabular_data_resource: Estas son las pruebas de test_constructor.py?

Si, si ya quedó. Si, no, falta ese punto.

Muchas gracias por tus dudas fernando, en esta ocasión respecificamos el nombre del archivo test_constructor.py a test_datapackage.py para que quedara un poco más claro.

Además nos dimos cuenta que, las pruebas de la función import_tabular_data_resource() es de forma indirecta. Las otras dos funciones también las probamos igual: _build_metadata() y add_variable_metadata().

IslasGECI / wrangler

🔖 GECI-73: Lanzamiento versión 0.2.2 #12

Codecov Report