| Index: build/android/pylib/utils/md5sum_test.py
|
| diff --git a/build/android/pylib/utils/md5sum_test.py b/build/android/pylib/utils/md5sum_test.py
|
| index 6c689fbd2bb6bfb2959511f8acc0e3f7c3ba4fe9..de9cd3578705a40d5d0efb5c3420efc9572e22e7 100755
|
| --- a/build/android/pylib/utils/md5sum_test.py
|
| +++ b/build/android/pylib/utils/md5sum_test.py
|
| @@ -39,8 +39,9 @@ class Md5SumTest(unittest.TestCase):
|
| with mock.patch('pylib.cmd_helper.GetCmdOutput', new=mock_get_cmd_output):
|
| out = md5sum.CalculateHostMd5Sums(test_path)
|
| self.assertEquals(1, len(out))
|
| - self.assertEquals('0123456789abcdeffedcba9876543210', out[0].hash)
|
| - self.assertEquals('/test/host/file.dat', out[0].path)
|
| + self.assertTrue('/test/host/file.dat' in out)
|
| + self.assertEquals('0123456789abcdeffedcba9876543210',
|
| + out['/test/host/file.dat'])
|
| mock_get_cmd_output.assert_called_once_with(
|
| [HOST_MD5_EXECUTABLE, '/test/host/file.dat'])
|
|
|
| @@ -52,10 +53,12 @@ class Md5SumTest(unittest.TestCase):
|
| with mock.patch('pylib.cmd_helper.GetCmdOutput', new=mock_get_cmd_output):
|
| out = md5sum.CalculateHostMd5Sums(test_paths)
|
| self.assertEquals(2, len(out))
|
| - self.assertEquals('0123456789abcdeffedcba9876543210', out[0].hash)
|
| - self.assertEquals('/test/host/file0.dat', out[0].path)
|
| - self.assertEquals('123456789abcdef00fedcba987654321', out[1].hash)
|
| - self.assertEquals('/test/host/file1.dat', out[1].path)
|
| + self.assertTrue('/test/host/file0.dat' in out)
|
| + self.assertEquals('0123456789abcdeffedcba9876543210',
|
| + out['/test/host/file0.dat'])
|
| + self.assertTrue('/test/host/file1.dat' in out)
|
| + self.assertEquals('123456789abcdef00fedcba987654321',
|
| + out['/test/host/file1.dat'])
|
| mock_get_cmd_output.assert_called_once_with(
|
| [HOST_MD5_EXECUTABLE, '/test/host/file0.dat',
|
| '/test/host/file1.dat'])
|
| @@ -68,10 +71,12 @@ class Md5SumTest(unittest.TestCase):
|
| with mock.patch('pylib.cmd_helper.GetCmdOutput', new=mock_get_cmd_output):
|
| out = md5sum.CalculateHostMd5Sums(test_paths)
|
| self.assertEquals(2, len(out))
|
| - self.assertEquals('0123456789abcdeffedcba9876543210', out[0].hash)
|
| - self.assertEquals('/test/host/file0.dat', out[0].path)
|
| - self.assertEquals('123456789abcdef00fedcba987654321', out[1].hash)
|
| - self.assertEquals('/test/host/file1.dat', out[1].path)
|
| + self.assertTrue('/test/host/file0.dat' in out)
|
| + self.assertEquals('0123456789abcdeffedcba9876543210',
|
| + out['/test/host/file0.dat'])
|
| + self.assertTrue('/test/host/file1.dat' in out)
|
| + self.assertEquals('123456789abcdef00fedcba987654321',
|
| + out['/test/host/file1.dat'])
|
| mock_get_cmd_output.assert_called_once_with(
|
| [HOST_MD5_EXECUTABLE, '/test/host/file0.dat', '/test/host/file1.dat'])
|
|
|
| @@ -99,8 +104,9 @@ class Md5SumTest(unittest.TestCase):
|
| new=mock_device_temp_file)):
|
| out = md5sum.CalculateDeviceMd5Sums(test_path, device)
|
| self.assertEquals(1, len(out))
|
| - self.assertEquals('0123456789abcdeffedcba9876543210', out[0].hash)
|
| - self.assertEquals('/storage/emulated/legacy/test/file.dat', out[0].path)
|
| + self.assertTrue('/storage/emulated/legacy/test/file.dat' in out)
|
| + self.assertEquals('0123456789abcdeffedcba9876543210',
|
| + out['/storage/emulated/legacy/test/file.dat'])
|
| device.adb.Push.assert_called_once_with(
|
| '/tmp/test/script/file.sh', '/data/local/tmp/test/script/file.sh')
|
| device.RunShellCommand.assert_called_once_with(
|
| @@ -132,10 +138,12 @@ class Md5SumTest(unittest.TestCase):
|
| new=mock_device_temp_file)):
|
| out = md5sum.CalculateDeviceMd5Sums(test_path, device)
|
| self.assertEquals(2, len(out))
|
| - self.assertEquals('0123456789abcdeffedcba9876543210', out[0].hash)
|
| - self.assertEquals('/storage/emulated/legacy/test/file0.dat', out[0].path)
|
| - self.assertEquals('123456789abcdef00fedcba987654321', out[1].hash)
|
| - self.assertEquals('/storage/emulated/legacy/test/file1.dat', out[1].path)
|
| + self.assertTrue('/storage/emulated/legacy/test/file0.dat' in out)
|
| + self.assertEquals('0123456789abcdeffedcba9876543210',
|
| + out['/storage/emulated/legacy/test/file0.dat'])
|
| + self.assertTrue('/storage/emulated/legacy/test/file1.dat' in out)
|
| + self.assertEquals('123456789abcdef00fedcba987654321',
|
| + out['/storage/emulated/legacy/test/file1.dat'])
|
| device.adb.Push.assert_called_once_with(
|
| '/tmp/test/script/file.sh', '/data/local/tmp/test/script/file.sh')
|
| device.RunShellCommand.assert_called_once_with(
|
| @@ -168,10 +176,47 @@ class Md5SumTest(unittest.TestCase):
|
| new=mock_device_temp_file)):
|
| out = md5sum.CalculateDeviceMd5Sums(test_path, device)
|
| self.assertEquals(2, len(out))
|
| - self.assertEquals('0123456789abcdeffedcba9876543210', out[0].hash)
|
| - self.assertEquals('/storage/emulated/legacy/test/file0.dat', out[0].path)
|
| - self.assertEquals('123456789abcdef00fedcba987654321', out[1].hash)
|
| - self.assertEquals('/storage/emulated/legacy/test/file1.dat', out[1].path)
|
| + self.assertTrue('/storage/emulated/legacy/test/file0.dat' in out)
|
| + self.assertEquals('0123456789abcdeffedcba9876543210',
|
| + out['/storage/emulated/legacy/test/file0.dat'])
|
| + self.assertTrue('/storage/emulated/legacy/test/file1.dat' in out)
|
| + self.assertEquals('123456789abcdef00fedcba987654321',
|
| + out['/storage/emulated/legacy/test/file1.dat'])
|
| + device.adb.Push.assert_called_once_with(
|
| + '/tmp/test/script/file.sh', '/data/local/tmp/test/script/file.sh')
|
| + device.RunShellCommand.assert_called_once_with(
|
| + ['sh', '/data/local/tmp/test/script/file.sh'])
|
| +
|
| + def testCalculateDeviceMd5Sums_singlePath_linkerWarning(self):
|
| + # See crbug/479966
|
| + test_path = '/storage/emulated/legacy/test/file.dat'
|
| +
|
| + device = mock.NonCallableMock()
|
| + device.adb = mock.NonCallableMock()
|
| + device.adb.Push = mock.Mock()
|
| + device_md5sum_output = [
|
| + 'WARNING: linker: /data/local/tmp/md5sum/md5sum_bin: '
|
| + 'unused DT entry: type 0x1d arg 0x15db',
|
| + '0123456789abcdeffedcba9876543210 '
|
| + '/storage/emulated/legacy/test/file.dat',
|
| + ]
|
| + device.RunShellCommand = mock.Mock(return_value=device_md5sum_output)
|
| +
|
| + mock_temp_file = mock.mock_open()
|
| + mock_temp_file.return_value.name = '/tmp/test/script/file.sh'
|
| +
|
| + mock_device_temp_file = mock.mock_open()
|
| + mock_device_temp_file.return_value.name = (
|
| + '/data/local/tmp/test/script/file.sh')
|
| +
|
| + with mock.patch('tempfile.NamedTemporaryFile', new=mock_temp_file), (
|
| + mock.patch('pylib.utils.device_temp_file.DeviceTempFile',
|
| + new=mock_device_temp_file)):
|
| + out = md5sum.CalculateDeviceMd5Sums(test_path, device)
|
| + self.assertEquals(1, len(out))
|
| + self.assertTrue('/storage/emulated/legacy/test/file.dat' in out)
|
| + self.assertEquals('0123456789abcdeffedcba9876543210',
|
| + out['/storage/emulated/legacy/test/file.dat'])
|
| device.adb.Push.assert_called_once_with(
|
| '/tmp/test/script/file.sh', '/data/local/tmp/test/script/file.sh')
|
| device.RunShellCommand.assert_called_once_with(
|
|
|