ソースを参照

Revert "AMBARI-6064. Server unittests should remove temporary files (aonishuk)"

This reverts commit b33ec74ed9bcb10b290657f1eb27f7ba493ec2a6.
Andrew Onishuk 11 年 前
コミット
d2cc6b31cd

+ 0 - 7
ambari-server/src/test/python/TestAmbariServer.py

@@ -24,7 +24,6 @@ from mock.mock import patch
 from mock.mock import MagicMock
 from mock.mock import create_autospec
 import os, errno, tempfile
-import glob
 import signal
 import stat
 import datetime
@@ -3568,8 +3567,6 @@ MIIFHjCCAwYCCQDpHKOBI+Lt0zANBgkqhkiG9w0BAQUFADBRMQswCQYDVQQGEwJV
     self.assertEquals(result, 0)
 
     os.unlink(fn2)
-    os.unlink(ambari_server.AMBARI_PROPERTIES_RPMSAVE_FILE
-              + '.' + timestamp.strftime('%Y%m%d%H%M%S'))
 
     #if ambari.properties file is absent then "ambari-server upgrade" should
     # fail
@@ -3578,8 +3575,6 @@ MIIFHjCCAwYCCQDpHKOBI+Lt0zANBgkqhkiG9w0BAQUFADBRMQswCQYDVQQGEwJV
 
     result = ambari_server.update_ambari_properties()
     self.assertNotEquals(result, 0)
-    os.unlink(os.path.join(tempfile.gettempdir(),'password.dat'))
-    os.unlink(fn)
 
   @patch.object(ambari_server.Properties, '__init__')
   @patch.object(ambari_server, 'search_file')
@@ -3633,8 +3628,6 @@ MIIFHjCCAwYCCQDpHKOBI+Lt0zANBgkqhkiG9w0BAQUFADBRMQswCQYDVQQGEwJV
     self.assertEqual(value, "root")
 
     os.unlink(fn2)
-    prev_conf_file = glob.glob(ambari_server.AMBARI_PROPERTIES_RPMSAVE_FILE+'*')
-    os.unlink(prev_conf_file[0])
 
 
   @patch.object(ambari_server, 'verify_setup_allowed')

+ 0 - 4
ambari-server/src/test/python/TestOSCheck.py

@@ -21,7 +21,6 @@ limitations under the License.
 import platform
 import datetime
 import os
-import glob
 import errno
 import tempfile
 import sys
@@ -212,9 +211,6 @@ class TestOSCheck(TestCase):
     # Command should not fail if *.rpmsave file is missing
     result = ambari_server.update_ambari_properties()
     self.assertEquals(result, 0)
-    os.unlink(fn2)
-    prev_conf_file = glob.glob(ambari_server.AMBARI_PROPERTIES_RPMSAVE_FILE+'*')
-    os.unlink(prev_conf_file[0])
 
   @patch("platform.linux_distribution")
   def test_os_type_check(self, mock_linux_distribution):