ckan / ckan / 7180

87%

Pull #2570

travis-ci

Do not use the write_url for datastore_search

This commit remove the use of the write_url in the datastore_search
action, as the permissions are already checked against the
auth.datastore_search function (which delegates to resource_show).
Looks like the non-removal of write_url in the action method was an
accidental oversight.

Tests (including the privacy tests for TestDatastoreSearch.test_search_private_dataset)
continue to pass.

Allows datastore_search_sql on private datasets

Moves the auth check for datastore_search_sql to the db module
where a list of the resource tables involved in the query is
available.  This should allow users to query across resources
that they have access to whether they are public or private.

PEP8 cleanup and skipping tests with 8.4

As Postgres 8.4 is now legacy, we will skip tests that rely on features that no longer work.

38 of 38 new or added lines in 9 files covered. (100.0%)

38191 of 44023 relevant lines covered (86.75%)

0.87 hits per line