[ckan-changes] [ckan/ckan] 82a328: [#1386] Fix bug when creating multiple sysadmins
GitHub
noreply at github.com
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
More information about the ckan-changes
mailing list