You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Sep 3, 2024. It is now read-only.
At the moment the tests run against whatever postgres version is current on travis, but the code contains special cases to support Postgres < 9.4 and >= 9.4 so the build config should declare environments which allow us run the tests on multiple postgres versions to properly test that support. added in 0c784fa
Quite a lot of the untested code is making calls to external APIs so there is some work do on mocking out the interactions with other services (and perhaps some refactoring to make that a bit easier to do)
The text was updated successfully, but these errors were encountered:
I've started adding some tests for this package but our coverage level is still low. There's a few problems to tackle:
In order to write tests for https://github.com/DemocracyClub/dc_signup_form/blob/master/dc_signup_form/views.py we need to create a stub django project for the tests to run in. With the minimal stub app defined in https://github.com/DemocracyClub/dc_signup_form/blob/master/run_tests.py it falls over as soon as you try to render a template because it is expecting stuff like static files,done in 07d5b1fbase.html
template,dc_base_theme
, context processors + so on to exist, and they don't.At the moment the tests run against whatever postgres version is current on travis, but the code contains special cases to support Postgresadded in 0c784fa< 9.4
and>= 9.4
so the build config should declare environments which allow us run the tests on multiple postgres versions to properly test that support.Quite a lot of the untested code is making calls to external APIs so there is some work do on mocking out the interactions with other services (and perhaps some refactoring to make that a bit easier to do)
The text was updated successfully, but these errors were encountered: