[ckan-changes] [ckan/ckan] 82a328: [#1386] Fix bug when creating multiple sysadmins

Tue Apr 15 19:00:13 UTC 2014

  Branch: refs/heads/1386-improve-factories
  Home:   https://github.com/ckan/ckan
  Commit: 82a32806d46fb08f5394afa816683ea2a3db4758
      https://github.com/ckan/ckan/commit/82a32806d46fb08f5394afa816683ea2a3db4758
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2014-04-11 (Fri, 11 Apr 2014)

  Changed paths:
    M ckan/new_tests/factories.py

  Log Message:
  -----------
  [#1386] Fix bug when creating multiple sysadmins

  Commit: cacb9340fa25fb8a231450fe79ee0e6b2aa70e04
      https://github.com/ckan/ckan/commit/cacb9340fa25fb8a231450fe79ee0e6b2aa70e04
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2014-04-11 (Fri, 11 Apr 2014)

  Changed paths:
    M ckan/new_tests/factories.py
    A ckan/new_tests/test_factories.py

  Log Message:
  -----------
  [#1386] Add basic tests for the factories

These tests are very simple: try to create two objects using each factory, but
they already helped me find some bugs (fixed in this commit).

This also guarantees that we can safely call each factory with no parameters,
and they're not reusing unique variables (like was the case for the Sysadmin).

Compare: https://github.com/ckan/ckan/compare/f9961a56e441...cacb9340fa25