~jcsackett/charmworld/bac-tag-constraints

« back to all changes in this revision

Viewing changes to charmworld/tests/test_search.py

  • Committer: Tarmac
  • Author(s): Brad Crittenden
  • Date: 2014-04-02 18:25:54 UTC
  • mfrom: (497.1.1 add-maintainers)
  • Revision ID: tarmac-20140402182554-h4fo6e0qnroitzni
Add a 'maintainer's field.

It was decided on the mailing list that the 'maintainer' field should be
renamed to 'maintainers' to make it clear that there can be more than one.
Note the old 'maintainer' field was capable of being a single entity or a list
just fine.

So for backward compatability, 'maintainer' is preserved, 'maintainers' is
added and the actual value used is the union of the two, with duplicates
removed.  

On the charm page the heading has been changed to 'Maintainer(s)" and an
unordered list is used to present the names.

https://codereview.appspot.com/83620043/

R=rharding.

Approved by Juju Gui Bot.

Show diffs side-by-side

added added

removed removed

Lines of Context:
360
360
        charm['provides']['website']['interface'] = 'gopher'
361
361
        charm['requires']['database']['interface'] = 'reiserfs'
362
362
        charm['_id'] = 'asdf'
363
 
        # Best-effort ensure there's no overlap between search terms
 
363
        charm['maintainer'] = 'John Doe <jdoe@example.com>'
 
364
        charm['maintainers'] = 'Jane Doe <jane@example.com>'
 
365
        # Best-effort ensure there's no overlap between search terms.
364
366
        items = charm.items()
365
367
        for num, (key, value) in enumerate(items):
366
368
            if not isinstance(value, basestring):