search

tableau / TabPy

Execute Python code on the fly and display results in Tableau visualizations:
https://tableau.github.io/TabPy/
MIT License
1.56k stars 598 forks source link

add test for mult entries #246

Closed johng42 closed 5 years ago

johng42 commented 5 years ago

there was already a test for this condition but since it was missing a \n character it caused a different failure - too many arguments on a single line of the pwd file change that test name to be more accurate and add a test for duplicate usernames this is expected to hit 100% of app\util.py

pep8speaks commented 5 years ago

Hello @johng42! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:

Comment last updated at 2019-04-04 19:02:20 UTC