gregdardis / stock-market-search

Search a stock symbol to receive a graph and financial metrics
MIT License
5 stars 0 forks source link

Test data item #162

Closed gregdardis closed 6 years ago

gregdardis commented 6 years ago

Took out ternary for rendering value. Value should be required, I made it not required because it was breaking Column tests. Column tests are now using a dumb component instead of the real DataItem component (that was really dumb of me) so the ternary isn't required.

coveralls commented 6 years ago

Pull Request Test Coverage Report for Build 172


Totals Coverage Status
Change from base Build 208: 1.0%
Covered Lines: 330
Relevant Lines: 495

💛 - Coveralls