[ckan-changes] [ckan/ckan] 308758: [#3229] Fix various issues in tests

Mon Sep 5 13:55:14 UTC 2016

  Branch: refs/heads/3229-api-blueprint
  Home:   https://github.com/ckan/ckan
  Commit: 308758cefea41c1cab794ea787c95bd9c92e9d8a
      https://github.com/ckan/ckan/commit/308758cefea41c1cab794ea787c95bd9c92e9d8a
  Author: amercader <amercadero at gmail.com>
  Date:   2016-09-05 (Mon, 05 Sep 2016)

  Changed paths:
    M ckan/tests/legacy/logic/test_auth.py
    M ckanext/datastore/tests/test_search.py
    M ckanext/multilingual/plugin.py

  Log Message:
  -----------
  [#3229] Fix various issues in tests

The DataStore ones were encoding things badly.
On the multilingual plugin, the origin of the failure is unclear
but it was related to the PylonsTestCase in
the legacy tests messing with the pylons request object.

  Commit: 99911be091bf68ed258cac277056ab46deac7605
      https://github.com/ckan/ckan/commit/99911be091bf68ed258cac277056ab46deac7605
  Author: amercader <amercadero at gmail.com>
  Date:   2016-09-05 (Mon, 05 Sep 2016)

  Changed paths:
    M ckan/lib/activity_streams.py
    M ckan/lib/base.py

  Log Message:
  -----------
  [#3229] Temporary fixes to allow templates to be rendered in Flask

There is one action (`dashboard_activity_list_html`) that calls the
template rendering logic. This needs proper work to cleanup and be made
compatible with both Flask and Pylons, but given that this will be the
only instance of a template being rendered in Flask, it makes sense to
put this temporary changes in place until the whole template rendering
logic can be sorted out properly.

Compare: https://github.com/ckan/ckan/compare/2ef2c5720398...99911be091bf