mozilla / active-data-recipes

A repository of various activedata queries and recipes
Mozilla Public License 2.0
8 stars 24 forks source link

Add recipe to compare usage across branches #102

Closed ahal closed 5 years ago

TrangNguyenBC commented 5 years ago

Hi @ahal I think the "branch" should be changed to "branches" in test_context.py and the order of import also need to be updated because you enabled flake8-import-order :-)

ahal commented 5 years ago

Yes, thank you! Looks like I also broke your web app test. You got it landed just in time:)

ahal commented 5 years ago

I fixed test_context.py and flake8, but I'm still failing test_web_api.py. I think it's failing when the branches context is being modified within a recipe's run function. I'll have to do some more digging to figure out how to fix it.