Capomastro uses transaction.commit_unless_managed which is a no-op and will be removed in django 1.8
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Capomastro |
Fix Released
|
High
|
Unassigned |
Bug Description
When running tox tests for capomastro on django 1.7 I get:
Creating test database for alias 'default'...
.../capomastro/
transaction.
the tests pass, but when I let the requirements.txt file install the latest matching django version (1.8a1 at the moment), the removal has taken effect and I get a test failure:
=======
ERROR: test_run_
-------
Traceback (most recent call last):
File "/capomastro/
update=False)
File "/capomastro/
output = self.handle(*args, **options)
File "/capomastro/
transaction
AttributeError: 'module' object has no attribute 'commit_
Removing the call to transaction.
Related branches
- Sheila Miguez (community): Approve
-
Diff: 21 lines (+1/-2)2 files modifiedcredentials/management/commands/import_sshkeypair.py (+0/-2)
projects/forms.py (+1/-0)
- James Jesudason (community): Approve
- Sheila Miguez (community): Approve
-
Diff: 527 lines (+478/-3)3 files modifiedarchives/models.py (+2/-2)
archives/tests/test_transports.py (+267/-1)
archives/transports.py (+209/-0)
Changed in capomastro: | |
status: | New → Triaged |
importance: | Undecided → High |
Changed in capomastro: | |
milestone: | none → 2015-04 |
Changed in capomastro: | |
status: | Triaged → Fix Released |