[ckan-changes] [okfn/ckan] 866c73: Add some docstrings to model/activity.py

Mon Nov 26 13:55:06 UTC 2012

  Branch: refs/heads/refactor-dashboard-tests
  Home:   https://github.com/okfn/ckan
  Commit: 866c7376330062ad1d14723a1dc609e819d314bd
      https://github.com/okfn/ckan/commit/866c7376330062ad1d14723a1dc609e819d314bd
  Author: Sean Hammond <seanhammond at lavabit.com>
  Date:   2012-11-26 (Mon, 26 Nov 2012)

  Changed paths:
    M ckan/model/activity.py

  Log Message:
  -----------
  Add some docstrings to model/activity.py

  Commit: d1f122afb3a8701b2dbf8eee495e9993bf157ec1
      https://github.com/okfn/ckan/commit/d1f122afb3a8701b2dbf8eee495e9993bf157ec1
  Author: Sean Hammond <seanhammond at lavabit.com>
  Date:   2012-11-26 (Mon, 26 Nov 2012)

  Changed paths:
    M ckan/model/activity.py
    M ckan/tests/functional/api/test_dashboard.py

  Log Message:
  -----------
  Fix dashboard activity stream from followed groups

It was returning all activities from the entire site, fix to return
activities from followed groups only.

The tests didn't catch this error because they didn't test doing random
activities that should _not_ appear in the user's dashboard and
asserting that they don't.

Add a quick test that catches this.

This is probably a problem for the other activity streams tests in
test_activity.py as well.

Compare: https://github.com/okfn/ckan/compare/8225e7306c31...d1f122afb3a8