search

avwx-rest / avwx-engine

Aviation Weather parsing engine. METAR & TAF
MIT License
90 stars 31 forks source link

Fix AUBOM test #25

Closed AirbusDriver closed 4 years ago

AirbusDriver commented 4 years ago

YWOL doesn't seem to provide any TAF info any longer. Works with Brisbane, YBBN.

devdupont commented 4 years ago

The AUBOM server in particular is hit and miss when it comes to the CircleCI tests. I'm very tempted to mock all data source calls.

AirbusDriver commented 4 years ago

I think a good move would be mocking the service, or maybe checking out and integrating https://github.com/kevin1024/vcrpy . I haven’t used it, but it could be a drop in for caching API calls until we can refactor the services.

Also, I think using pytest’s marking features could help separate live API calls for validating changes to API calls from the integrated tests that just need to verify the post processing is working. Maybe a set of “live_test” marks that can be separated from the mocked tests can be called just as sanity checks.