Task #530
closedPlatform unit tests require qpidtoollibs to be installed
0%
Description
Description of problem:
The platform unit tests will not pass if qpidtoollibs is not installed. This is unfriendly for community developers who wish to use rabbitmq. It would be better if the tests ran only if qpidtoollibs is installed.
Version-Release number of selected component (if applicable):
2.4.0-1
How reproducible:
Every time.
Steps to Reproduce:
1. Ensure that python-qpid-qmf is not installed.
2. Run the platform test suite.
Actual results:
======================================================================
ERROR: test_migrate (unit.server.db.migrations.test_migration_0009.TestMigration)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/usr/lib/python2.6/site-packages/mock.py", line 1216, in patched
arg = patching.__enter__()
File "/usr/lib/python2.6/site-packages/mock.py", line 1275, in enter
original, local = self.get_original()
File "/usr/lib/python2.6/site-packages/mock.py", line 1263, in get_original
"%s does not have the attribute %r" % (target, name)
AttributeError: <module 'pulp.server.db.migrations.0009_qpid_queues' from '/home/rbarlow/devel/pulp/server/pulp/server/db/migrations/0009_qpid_queues.py'> does not have the attribute 'BrokerAgent'
Expected results:
The Qpid specific tests should be skipped.
+ This bug was cloned from Bugzilla Bug #1142325 +
Updated by bmbouter about 8 years ago
The tests could also be written to not require any 3rd party libraries.
+ This comment was cloned from Bugzilla #1142325 comment 1 +
Updated by jcline@redhat.com about 8 years ago
https://github.com/pulp/pulp/pull/1418
+ This comment was cloned from Bugzilla #1142325 comment 2 +
Updated by cduryee about 8 years ago
fixed in pulp 2.6.0-0.2.beta
+ This comment was cloned from Bugzilla #1142325 comment 3 +
Updated by igulina@redhat.com about 8 years ago
uname -r
3.17.1-302.fc21.x86_64
rpm -qa python-qpid-qmf
$
./run-tests.py
Running flake8
Running Unit Tests
.....
....
Ran 3236 tests in 214.981s
OK
+ This comment was cloned from Bugzilla #1142325 comment 4 +
Updated by dkliban@redhat.com over 7 years ago
- Status changed from 6 to CLOSED - CURRENTRELEASE