docstring in test_denied_DELETE_of_versioned_object confuses test output
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Object Storage (swift) |
Fix Released
|
Undecided
|
Brian Reitz |
Bug Description
Docstring confuses nose and produces atypical test output:
% nosetests -v test_server.
Verify that a request with read access to a versions container … ok
this looks really odd when mixed with other tests - and becomes somewhat harder to trace down to a specific file and test:
<snip>
test_copy_
test_cross_
test_custom_
Verify that a request with read access to a versions container ... ok
test_error_limiting (test.unit.
test_error_
test_expirer_
test_iter_nodes (test.unit.
<snip>
Because of a docstring introduced in the new unit test we get the abnormal test output:
+ def test_denied_
+ """
+ Verify that a request with read access to a versions container
+ is unable to cause any write operations on the versioned container.
+ """
+ methods = set()
+ authorize_
+
While it is great to have the test description it might be better to make it a pure comment and not a docstring to
get standard typical test output that the other tests produce:
% nosetests -v test_server.
test_denied_
Changed in swift: | |
assignee: | nobody → Brian Reitz (brian-reitz) |
Fixed in https:/ /github. com/openstack/ swift/commit/ 310ac18 (Swift 2.4.0)