datopian / datapackage-views-js / 57 / 1

73%

master: 73%

push

travis-ci

[data-explorer][m]: refactored to work with data explorer concept. (#19)

* [data-explorer][m]: refactored to work with data explorer concept.

* update props of the App container
* update logic behind App container
* update fixtures to do dev with cosmos
* update unit tests

* [package][m]: Package App for distribution

* Add babel resources to package.json
* Add `yarn build:package` command
* Add updated resources at /dist

* [Chart][m]: added Chart component for rendering graphs from simple spec.

- includes fixtures for testing via cosmos

* [package][m]: Add dist files

* [tidy up][s]: refactored App.js to remove redundant code.

* [fixtures][xs]: fix loading fixture.

* [fixtures][xs]: fix failed fixture.

* [fixtures][xs]: fix map fixture.

* [Map][s]: fix popup on markers so it uses 'feature.properties.name' to display on click.

* [tests][xs]: mock Chart component as Plotly lib errors when running tests.

This means we aren't testing Chart compnent.

Error details:

  ● Test suite failed to run

    TypeError: Cannot read property 'document' of undefined

      1 | import React from 'react';
    > 2 | import Plotly from 'plotly.js-basic-dist';
        | ^
      3 | import createPlotlyComponent from "react-plotly.js/factory";
      4 |
      5 |

      at document (node_modules/plotly.js-basic-dist/plotly-basic.js:667:26)
      at Object.7 (node_modules/plotly.js-basic-dist/plotly-basic.js:660:2)
      at call (node_modules/plotly.js-basic-dist/plotly-basic.js:7:631)
      at o (node_modules/plotly.js-basic-dist/plotly-basic.js:7:682)
      at Object._dereq_ (node_modules/plotly.js-basic-dist/plotly-basic.js:36595:10)
      at call (node_modules/plotly.js-basic-dist/plotly-basic.js:7:631)
      at o (node_modules/plotly.js-basic-dist/plotly-basic.js:7:682)
      at Object._dereq_ (node_modules/plotly.js-basic-dist/plotly-basic.js:10:11)
      at call (node_modules/plotly.js-basic-dist/plotl... (continued)

59 of 102 branches covered (57.84%)

Branch coverage included in aggregate %.

99 of 150 relevant lines covered (66.0%)

2.14 hits per line