[ckan-changes] [ckan/ckan] b68a83: Modernize package model tests

Adrià Mercader noreply at github.com
Tue May 14 12:07:17 UTC 2019


  Branch: refs/heads/master
  Home:   https://github.com/ckan/ckan
  Commit: b68a836a416127bb119722d0a75b296d865fd32b
      https://github.com/ckan/ckan/commit/b68a836a416127bb119722d0a75b296d865fd32b
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    M ckan/model/tag.py
    R ckan/tests/legacy/models/test_extras.py
    M ckan/tests/legacy/models/test_package.py
    A ckan/tests/model/test_package.py
    A ckan/tests/model/test_package_extra.py
    A ckan/tests/model/test_tags.py

  Log Message:
  -----------
  Modernize package model tests

* The rest of ckan/tests/legacy/models/test_package.py will go with the
revisioning.
* tag.by_name() is deprecated because it doesnt seem used anywhere and
is broken - missing a join in the query, so the Tag.name filter does
nothing.
* tag.search_by_name() is deprecated because it isn't used in core ckan.
I think some sites will have a tag search, but they should really have
the code in their own repo, so we're giving them warning.


  Commit: 4bac5eac82c253dbaaa74e169c909403d94e5c77
      https://github.com/ckan/ckan/commit/4bac5eac82c253dbaaa74e169c909403d94e5c77
  Author: Adrià Mercader <amercadero at gmail.com>
  Date:   2019-05-14 (Tue, 14 May 2019)

  Changed paths:
    M ckan/model/tag.py
    R ckan/tests/legacy/models/test_extras.py
    M ckan/tests/legacy/models/test_package.py
    A ckan/tests/model/test_package.py
    A ckan/tests/model/test_package_extra.py
    A ckan/tests/model/test_tags.py

  Log Message:
  -----------
  Merge pull request #4776 from ckan/modernize-model-tests

Modernize package model tests


Compare: https://github.com/ckan/ckan/compare/e8d430c0a5ee...4bac5eac82c2



More information about the ckan-changes mailing list