[Feature request] Add test coverage for all of the parsing logic.

mrhappyasthma / IsThisStockGood

A tool for evaluating companies using the Rule #1 investing principles.

http://www.isthisstockgood.com

22 stars 10 forks source link

[Feature request] Add test coverage for all of the parsing logic. #31

Closed mrhappyasthma closed 3 months ago

mrhappyasthma commented 3 years ago

Add more canned responses to the TestData/ subfolder
Add python tests to verify testing logic

We could consider even trying to write end-to-end tests that actually fetch data from a production URL, too. But then the asserts need to be fuzzy since we won't be able to have an expected output that's fixed.

mrhappyasthma commented 3 months ago

Folding this in to https://github.com/mrhappyasthma/IsThisStockGood/issues/59. I think functionality tests are more important than parsing at this stage.