[ckan-changes] [okfn/ckan] b70536: [#1117] Remove setup db stuff from some new test c...

GitHub noreply at github.com
Thu Aug 1 09:38:21 UTC 2013


  Branch: refs/heads/1117-start-new-test-suite
  Home:   https://github.com/okfn/ckan
  Commit: b705361263ce686bebbb107dbc8bbecde5f7abf1
      https://github.com/okfn/ckan/commit/b705361263ce686bebbb107dbc8bbecde5f7abf1
  Author: Sean Hammond <seanhammond at lavabit.com>
  Date:   2013-08-01 (Thu, 01 Aug 2013)

  Changed paths:
    M ckan/new_tests/helpers.py
    M ckan/new_tests/logic/action/test_update.py
    M ckan/new_tests/logic/auth/test_update.py
    M ckan/new_tests/logic/test_validators.py

  Log Message:
  -----------
  [#1117] Remove setup db stuff from some new test classes

Only test classes that use the db (which should be few of them) need to
do things like cleaning and initing the db. This makes the tests much
faster.


  Commit: a0a2f699b156f9a5bec0e545ba5b3e51e93aabcd
      https://github.com/okfn/ckan/commit/a0a2f699b156f9a5bec0e545ba5b3e51e93aabcd
  Author: Sean Hammond <seanhammond at lavabit.com>
  Date:   2013-08-01 (Thu, 01 Aug 2013)

  Changed paths:
    M ckan/new_tests/lib/navl/test_validators.py
    M ckan/new_tests/logic/action/test_update.py
    M ckan/new_tests/logic/test_validators.py

  Log Message:
  -----------
  [#1117] Always use `as context` when doing `with nose.tools.assert_raises`

It looks like `with nose.tools.assert_raises:` without the `as context`
doesn't work in Python 2.6. In commit 135c59b3de2d518643d94ea I thought
it was the other way round. Hopefully *this* fixes the tests with Python
2.6.


Compare: https://github.com/okfn/ckan/compare/135c59b3de2d...a0a2f699b156


More information about the ckan-changes mailing list