Merge lp:~joetalbott/utah/fix_duplicate_docstrings into lp:utah

[Joe Talbott]

This branch fixes the docstrings that were duplicated in some testcases.

[Javier Collado (javier.collado)]

When looking at a failure in nosetests output, if the method contains a
docstring, it prints the first line as an id of the test case:

======================================================================
ERROR: Test that a partially run state file with all jobs done
----------------------------------------------------------------------

Hence, if the first line is in the docstring is still the same one, the problem
isn't fixed. To be more specific, the test cases involved are:

test_load_state_partial
test_load_state_partial_run_all
test_load_state_partial_done_all_failed

review: Needs Fixing

[lp:~joetalbott/utah/fix_duplicate_docstrings]

783. By Joe Talbott on 2012-12-11

test_state_agent - Shorten docstrings and make them unique.

[Joe Talbott (joetalbott)]

I didn't realize it only uses the first line. This is actually one thing I don't like about nosetests. I've shortened the docstrings back to a single unique line.

review: Needs Resubmitting

[Javier Collado (javier.collado)]

Thanks for the update.

review: Approve

=== modified file 'utah/client/tests/test_state_agent.py'
--- utah/client/tests/test_state_agent.py	2012-12-11 09:47:51 +0000
+++ utah/client/tests/test_state_agent.py	2012-12-11 16:56:20 +0000
@@ -153,7 +153,7 @@
 
     def test_load_state_partial_run_all(self):
         """
-        Test that a partially run state file can be resumed.
+        Test partially run state file - run all jobs.
         """
         fp = open(self.state_file, 'w')
         fp.write(self.partial_state_file_content_run_all)
@@ -198,7 +198,7 @@
 
     def test_load_state_partial_done_all_failed(self):
         """
-        Test that a partially run state file can be resumed.
+        Test partially run state file - all jobs failed.
         """
         fp = open(self.state_file, 'w')
         fp.write(self.partial_state_file_content_done_all_failed)