Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added DeckGL Test #4688

Closed
wants to merge 1 commit into from
Closed

Added DeckGL Test #4688

wants to merge 1 commit into from

Conversation

hughhhh
Copy link
Member

@hughhhh hughhhh commented Mar 25, 2018

Here are a few test I've added for DeckGL. Wanted to get some early feedback before doing another set.

@mistercrunch

@hughhhh hughhhh mentioned this pull request Mar 25, 2018
@codecov-io
Copy link

Codecov Report

Merging #4688 into master will increase coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4688      +/-   ##
==========================================
+ Coverage    71.4%   71.48%   +0.08%     
==========================================
  Files         190      190              
  Lines       14935    14935              
  Branches     1102     1102              
==========================================
+ Hits        10664    10676      +12     
+ Misses       4268     4256      -12     
  Partials        3        3
Impacted Files Coverage Δ
superset/viz.py 80.46% <0%> (+0.84%) ⬆️

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 9abc5c7...de05959. Read the comment docs.


def test_get_metrics(self):
form_data = {'size': 'large'}
datasource = {'type': 'table'}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think datasource expects a derivative of connectors.base.models.BaseDatasource, I'm surprised this works.

class BaseDeckGLVizTestCase(unittest.TestCase):

def test_get_metrics(self):
form_data = {'size': 'large'}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typically a form_data dict will be much larger and have as many keys a specified in that particular viz type. We should talk about a way to test viz.py with examples that represent reality, may by sharing code with the examples, along with assertions to complete/test the examples.

@hughhhh hughhhh closed this Apr 10, 2018
@hughhhh hughhhh mentioned this pull request Apr 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants