[ckan-changes] [ckan/ckan] 00b192: [#2035] Further fixes to auth tests

GitHub noreply at github.com
Mon Nov 10 17:15:02 UTC 2014


  Branch: refs/heads/2035-remove-auth-permissions-cache
  Home:   https://github.com/ckan/ckan
  Commit: 00b192d489f1f680e8bb8372a23b3e3caf7f689c
      https://github.com/ckan/ckan/commit/00b192d489f1f680e8bb8372a23b3e3caf7f689c
  Author: amercader <amercadero at gmail.com>
  Date:   2014-11-10 (Mon, 10 Nov 2014)

  Changed paths:
    M ckan/tests/logic/test_auth.py

  Log Message:
  -----------
  [#2035] Further fixes to auth tests

The old auth tests are terrible in that they depend on each other to
run. Also the auth cache thing masked some bad assumptions.

The whole file should be rewritten as new tests.


  Commit: 7d30f7cc0b8c2879dfa1e1481269f6a55958287b
      https://github.com/ckan/ckan/commit/7d30f7cc0b8c2879dfa1e1481269f6a55958287b
  Author: amercader <amercadero at gmail.com>
  Date:   2014-11-10 (Mon, 10 Nov 2014)

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

  Log Message:
  -----------
  [#2035] Don't use http://foo.bar as a test url

Because `.bar` is being recognized as a gTLD, it points to
`127.0.53.53`, which if you have a web server running will return
localhost (and a 200 status).

More details:

https://stackoverflow.com/questions/25662590/domain-foo-bar-points-127-0-53-53-why


Compare: https://github.com/ckan/ckan/compare/3665e3851d12...7d30f7cc0b8c


More information about the ckan-changes mailing list