[ckan-changes] [ckan/ckan] 543c6e: Add migration test for revision being missing

Fri Mar 1 14:53:23 UTC 2019

  Branch: refs/heads/3484_revision_ui_removal2
  Home:   https://github.com/ckan/ckan
  Commit: 543c6e35b50766794404382e874b9c3fb95f6215
      https://github.com/ckan/ckan/commit/543c6e35b50766794404382e874b9c3fb95f6215
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2019-03-01 (Fri, 01 Mar 2019)

  Changed paths:
    M ckan/migration/migrate_package_activity.py
    M ckan/tests/migration/test_migrate_package_activity.py
    M ckan/tests/migration/test_revision_legacy_code.py

  Log Message:
  -----------
  Add migration test for revision being missing

Also PEP8

  Commit: 1a238968aa764873f0cbadf32bc380e87e3ecc46
      https://github.com/ckan/ckan/commit/1a238968aa764873f0cbadf32bc380e87e3ecc46
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2019-03-01 (Fri, 01 Mar 2019)

  Changed paths:
    M ckan/migration/migrate_package_activity.py

  Log Message:
  -----------
  Migration cleaner with context, not that there has been any trouble.

  Commit: 1836e812ce02dae82cf85312d9f7493db601e393
      https://github.com/ckan/ckan/commit/1836e812ce02dae82cf85312d9f7493db601e393
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2019-03-01 (Fri, 01 Mar 2019)

  Changed paths:
    M ckan/migration/migrate_package_activity.py

  Log Message:
  -----------
  Do migration of revisions in chronological order - makes more sense

  Commit: 404a0139bbcd3bd16b96e4910072d50a00124289
      https://github.com/ckan/ckan/commit/404a0139bbcd3bd16b96e4910072d50a00124289
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2019-03-01 (Fri, 01 Mar 2019)

  Changed paths:
    M ckan/migration/revision_legacy_code.py
    M ckan/tests/migration/test_migrate_package_activity.py

  Log Message:
  -----------
  Fix package_show of old revision

Fix in package_dictize_with_revisions of 'metadata_modified' and 'organization' -
it was showing the current package's version, not the older revision.

Test fix - del activity.data['package'] was unreliably committed.

Compare: https://github.com/ckan/ckan/compare/52f230ed8c95...404a0139bbcd