Merge lp:~sil2100/cupstream2distro/quick_fix_for_typo into lp:cupstream2distro

[Łukasz Zemczak]

Fix a typo introduced by robru's changes that breaks the spreadsheet working - the '.' at the end of Packages built breaks many formulas and is unnecessary

Fix a typo introduced by robru's changes that breaks the spreadsheet working - the '.' at the end of Packages built breaks many formulas and is unnecessary

[Timo Jyrinki (timo-jyrinki)]

Yes, please.

review: Approve

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:754
http://jenkins.qa.ubuntu.com/job/cu2d-choo-choo-ci/159/
Executed test runs:

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/cu2d-choo-choo-ci/159/rebuild

review: Approve (continuous-integration)

=== modified file 'cupstream2distro/silomanager.py'
--- cupstream2distro/silomanager.py	2014-09-24 19:31:48 +0000
+++ cupstream2distro/silomanager.py	2014-09-25 11:39:04 +0000
@@ -128,7 +128,7 @@
         return self.config_step == SILO_PUBLISHED
 
     def set_build_successful(self):
-        self.set_config_status(SILO_STATE_BUILD_SUCCESSFUL, 'Packages built.')
+        self.set_config_status(SILO_STATE_BUILD_SUCCESSFUL, 'Packages built')
 
     def set_cleaning(self, message):
         self.set_config_status(SILO_STATE_CLEANING, message)
 
=== modified file 'tests/unit/test_silomanager.py'
--- tests/unit/test_silomanager.py	2014-09-24 19:31:48 +0000
+++ tests/unit/test_silomanager.py	2014-09-25 11:39:04 +0000
@@ -110,7 +110,7 @@
     def test_silostate_status_setters(self):
         setMock = self.state.set_config_status = Mock()
         self.state.set_build_successful()
-        setMock.assert_called_once_with(3, 'Packages built.')
+        setMock.assert_called_once_with(3, 'Packages built')
         self.state.set_cleaning('Foo')
         setMock.assert_called_with(11, 'Foo')
         self.state.set_merging()