[ckan-changes] [okfn/ckan] 645e2d: [#299] Clear up changes to Pylons' config between ...

GitHub noreply at github.com
Tue Feb 5 18:53:42 UTC 2013


  Branch: refs/heads/299-bug-ckanext-tests-failures
  Home:   https://github.com/okfn/ckan
  Commit: 645e2d736bcbf41588e1e4b4f517fa8138ea6078
      https://github.com/okfn/ckan/commit/645e2d736bcbf41588e1e4b4f517fa8138ea6078
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/config/middleware.py
    M ckan/tests/functional/api/test_email_notifications.py
    M ckan/tests/functional/test_storage.py
    M ckan/tests/functional/test_upload.py
    M ckan/tests/test_plugins.py
    M ckanext/jsonpreview/tests/test_preview.py
    M ckanext/pdfpreview/tests/test_preview.py
    M ckanext/reclinepreview/tests/test_preview.py
    M ckanext/resourceproxy/tests/test_proxy.py
    M ckanext/stats/tests/__init__.py
    M doc/writing-extensions.rst

  Log Message:
  -----------
  [#299] Clear up changes to Pylons' config between tests

We also, instead of always loading test.ini in these tests, we use whatever Pylons loaded.


  Commit: 16849724065ba4461965d71f86826861487130ad
      https://github.com/okfn/ckan/commit/16849724065ba4461965d71f86826861487130ad
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/new_authz.py

  Log Message:
  -----------
  [#299] Reload AuthFunctions if couldn't find the action we're looking for

The problem I found was when you're testing a bunch of things
sequentially. Then, the first time that method is called, it
loads all known AuthFunctions and puts into a cache. Then it
only checks that cache.

But what happens if some code, after this has happened, adds
a new AuthFunction? It never gets added to the cache.

With this change, if we have the requested function on cache,
we use it. If not, we refresh the cache and see if something
changed.


  Commit: f87ebf41251cb5e050f0adcb557497898505cced
      https://github.com/okfn/ckan/commit/f87ebf41251cb5e050f0adcb557497898505cced
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckanext/jsonpreview/tests/test_preview.py
    M ckanext/pdfpreview/tests/test_preview.py
    M ckanext/reclinepreview/tests/test_preview.py
    M ckanext/stats/tests/test_stats_lib.py

  Log Message:
  -----------
  [#299] Clear test data after each test is run

This might not be the best way to do it. It's too repetitive, but it's
better than leaving trash behind.

We might investigate running the tests inside a transaction, and rolling
back when we're done (is it done already?).


  Commit: 7496756a0b0a68a6e238c64c1be600fae6b0dd79
      https://github.com/okfn/ckan/commit/7496756a0b0a68a6e238c64c1be600fae6b0dd79
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  [#299] Run ckan and ckanext tests on Travis


  Commit: efc74721bf5f5f0cfab381562cf403c1f79da4b5
      https://github.com/okfn/ckan/commit/efc74721bf5f5f0cfab381562cf403c1f79da4b5
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckanext/jsonpreview/tests/test_preview.py
    M ckanext/pdfpreview/tests/test_preview.py
    M ckanext/reclinepreview/tests/test_preview.py

  Log Message:
  -----------
  [#299] Fix ckanext.{json,pdf,recline}preview tests

The problem is that we were looking for each *_preview.js, when the included
files are already minified (*_preview.min.js).


  Commit: b65391680b861f2df0a80eda94e999ade348f4da
      https://github.com/okfn/ckan/commit/b65391680b861f2df0a80eda94e999ade348f4da
  Author: kindly <kindly at gmail.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/plugins/core.py
    M ckan/tests/ckantestplugin/ckantestplugin.egg-info/entry_points.txt
    M ckan/tests/ckantestplugin/ckantestplugin/__init__.py
    M ckan/tests/ckantestplugin/setup.py
    M ckan/tests/test_plugins.py

  Log Message:
  -----------
  #299 clear action cache when plugin loads


  Commit: c5f55511e50f03dac7880014799a2eb61e008c5f
      https://github.com/okfn/ckan/commit/c5f55511e50f03dac7880014799a2eb61e008c5f
  Author: kindly <kindly at gmail.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckanext/datastore/tests/test_create.py
    M ckanext/datastore/tests/test_delete.py
    M ckanext/datastore/tests/test_search.py
    M ckanext/datastore/tests/test_upsert.py

  Log Message:
  -----------
  #299 unload and load datastore plugin at correct times


  Commit: 03d67a0eedcf138e77a22e364a0ee19da25720dd
      https://github.com/okfn/ckan/commit/03d67a0eedcf138e77a22e364a0ee19da25720dd
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M .travis.yml

  Log Message:
  -----------
  Revert "Run DataStore tests in Travis-CI"

This reverts commit 76cc9d3b8641489dc8e6674cb0e46ade02ce0b25.

Conflicts:

	.travis.yml


  Commit: b874c3b36adce3bf6ddc2051609d0422bf2af2f2
      https://github.com/okfn/ckan/commit/b874c3b36adce3bf6ddc2051609d0422bf2af2f2
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/logic/action/get.py

  Log Message:
  -----------
  [#299] Don't call WHERE statements if there're no terms

This removes the following warning:

/home/vagrant/pyenv/local/lib/python2.7/site-packages/sqlalchemy/sql/expression.py:1925:
SAWarning: The IN-predicate on "term_translation.term" was invoked with an empty sequence.
This results in a contradiction, which nonetheless can be expensive to evaluate.
Consider alternative strategies for improved performance.


  Commit: b47237fb17f1b648f707639487c9149b34064e71
      https://github.com/okfn/ckan/commit/b47237fb17f1b648f707639487c9149b34064e71
  Author: kindly <kindly at gmail.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/controllers/revision.py
    M ckan/lib/activity_streams_session_extension.py
    M ckan/lib/dictization/model_dictize.py
    M ckan/logic/action/get.py
    M ckan/model/__init__.py
    M ckan/tests/functional/api/model/test_package.py
    M ckan/tests/functional/api/model/test_tag.py
    M ckan/tests/logic/test_tag.py

  Log Message:
  -----------
  309 no private datasets in activity streams, revisions, group listing and tag listings


  Commit: 3177bb657eec359b05a41d884338e320ac8012ff
      https://github.com/okfn/ckan/commit/3177bb657eec359b05a41d884338e320ac8012ff
  Author: kindly <kindly at gmail.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/lib/dictization/model_dictize.py
    M ckan/tests/functional/test_group.py
    M ckan/tests/logic/test_action.py

  Log Message:
  -----------
  make group_list_dictize reasonably fast by going through search index


  Commit: f8908fb4691a1fc3810d28f0e11aaeedbd86b30a
      https://github.com/okfn/ckan/commit/f8908fb4691a1fc3810d28f0e11aaeedbd86b30a
  Author: kindly <kindly at gmail.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/controllers/home.py
    M ckan/lib/dictization/model_dictize.py
    M ckan/logic/action/get.py
    M ckan/templates/snippets/organization.html
    M ckan/tests/lib/test_dictization.py

  Log Message:
  -----------
  [#309] add package count using search index


  Commit: 46c2a86d23981f6d761077aa7618d5449c1e6b64
      https://github.com/okfn/ckan/commit/46c2a86d23981f6d761077aa7618d5449c1e6b64
  Author: Dominik Moritz <domoritz at gmail.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/model/meta.py

  Log Message:
  -----------
  [#345] Also recognize driver `postgres` in `engine_is_pg`


  Commit: 0932cf9613aa9290bf3bcf26700da87b6b2608b7
      https://github.com/okfn/ckan/commit/0932cf9613aa9290bf3bcf26700da87b6b2608b7
  Author: Sean Hammond <seanhammond at lavabit.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/lib/activity_streams_session_extension.py

  Log Message:
  -----------
  [#309] Tidy up a couple of comments

No point in putting "TODO" comments in the source code no one will
remember to go back and do them and the comment may become outdated/misleading.


  Commit: 3912384c2c4fb16d893d6ca9b49486fddb429d56
      https://github.com/okfn/ckan/commit/3912384c2c4fb16d893d6ca9b49486fddb429d56
  Author: tobes <toby.junk at gmail.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/lib/base.py

  Log Message:
  -----------
  [#285] Remove code in base controller that is not used in the ckan codebase


  Commit: 810d90c09d5c31f4d7b6eee79e34ccc85751c041
      https://github.com/okfn/ckan/commit/810d90c09d5c31f4d7b6eee79e34ccc85751c041
  Author: tobes <toby.junk at gmail.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/lib/base.py

  Log Message:
  -----------
  [#285] Re add function mis-used in api controller


  Commit: 34c4457c2927c32f2a37a65c39ce7838c29d90a4
      https://github.com/okfn/ckan/commit/34c4457c2927c32f2a37a65c39ce7838c29d90a4
  Author: tobes <toby.junk at gmail.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/lib/base.py

  Log Message:
  -----------
  [#285] _make_unicode() still has user


  Commit: ee7514eda1db12ed25f225cd193a8f2d69d037e7
      https://github.com/okfn/ckan/commit/ee7514eda1db12ed25f225cd193a8f2d69d037e7
  Author: tobes <toby.junk at gmail.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/lib/base.py

  Log Message:
  -----------
  [#285] Move function into only function using it


  Commit: 9586a88916880bb5ac27778f5a29a527d5947416
      https://github.com/okfn/ckan/commit/9586a88916880bb5ac27778f5a29a527d5947416
  Author: tobes <toby.junk at gmail.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/controllers/api.py
    M ckan/lib/base.py

  Log Message:
  -----------
  [#285] Move function into API controller as only user


  Commit: 708c5d6f2b3530231237a86bb81f44f0c2b008df
      https://github.com/okfn/ckan/commit/708c5d6f2b3530231237a86bb81f44f0c2b008df
  Author: tobes <toby.junk at gmail.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/controllers/api.py

  Log Message:
  -----------
  [#285] Fix missing imports


  Commit: 0225f91e26b99b0aee8e5d1c983fe568e086322f
      https://github.com/okfn/ckan/commit/0225f91e26b99b0aee8e5d1c983fe568e086322f
  Author: tobes <toby.junk at gmail.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/controllers/api.py

  Log Message:
  -----------
  [#285] Use json defined in h


  Commit: dea3b3ae04d459280fdc4eea60dba42792530003
      https://github.com/okfn/ckan/commit/dea3b3ae04d459280fdc4eea60dba42792530003
  Author: John Glover <j at johnglover.net>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/controllers/api.py
    M ckan/lib/base.py

  Log Message:
  -----------
  [#285] PEP8


  Commit: b777c9776be934eec111299ee3a98384dd8d33fa
      https://github.com/okfn/ckan/commit/b777c9776be934eec111299ee3a98384dd8d33fa
  Author: Dominik Moritz <domoritz at gmail.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckanext/datastore/bin/datastore_setup.py

  Log Message:
  -----------
  [#290] Do not request password in datastore permissions script


  Commit: e82823ef5c35ba670cf290cab2be6246d13c1096
      https://github.com/okfn/ckan/commit/e82823ef5c35ba670cf290cab2be6246d13c1096
  Author: Dominik Moritz <domoritz at gmail.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckanext/pdfpreview/tests/test_preview.py

  Log Message:
  -----------
  [#290] Fix test class name


  Commit: 7dbf278765ba96ffc472f03fea3bb26a2a6f377e
      https://github.com/okfn/ckan/commit/7dbf278765ba96ffc472f03fea3bb26a2a6f377e
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckanext/resourceproxy/tests/file_server.py

  Log Message:
  -----------
  [#299] Ignore SimpleHTTPServer's log messages


  Commit: 4a780f4c68403a8ee382c2d9a2512104c5563445
      https://github.com/okfn/ckan/commit/4a780f4c68403a8ee382c2d9a2512104c5563445
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Changed paths:
    M ckan/lib/dictization/model_dictize.py

  Log Message:
  -----------
  [#299] Run before_show on ITag before running it on IPackage


  Commit: a1943a2591d1233a7f4a1fd857d465a73d6b6d8e
      https://github.com/okfn/ckan/commit/a1943a2591d1233a7f4a1fd857d465a73d6b6d8e
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2013-02-05 (Tue, 05 Feb 2013)

  Log Message:
  -----------
  Merge branch 'master' into 299-bug-ckanext-tests-failures

Conflicts:
	ckan/lib/dictization/model_dictize.py


Compare: https://github.com/okfn/ckan/compare/b82456aca926...a1943a2591d1


More information about the ckan-changes mailing list