fpb, deploy.sh script should have execution permissions

Otherwise plugin doesn't work out of the box.

Change-Id: If87d2ccbcc5b94fdf64293d876d9f1a3ab5d8e4e
Closes-bug: #1392736
This commit is contained in:
Evgeniy L 2014-11-14 16:49:11 +04:00
parent 29f5362265
commit b5798c95af
3 changed files with 14 additions and 1 deletions

View File

@ -3,6 +3,8 @@
## 1.0.1 (SET DATE)
- Show instruction for CentOS if not all requirements are installed
- Fixed bug, deploy.sh doesn't have execution permission
https://bugs.launchpad.net/fuel/+bug/1392736
## 1.0.0 (2014-11-13)

View File

@ -216,10 +216,12 @@ class TestUtils(BaseTestCase):
open_mock.assert_called_once_with(path)
yaml_mock.load.assert_called_once_with(file_mock)
@mock.patch('fuel_plugin_builder.utils.copy_file_permissions')
@mock.patch('fuel_plugin_builder.utils.render_to_file')
@mock.patch('fuel_plugin_builder.utils.remove')
@mock.patch('fuel_plugin_builder.utils.os.walk')
def test_render_files_in_dir(self, walk_mock, remove_mock, render_mock):
def test_render_files_in_dir(
self, walk_mock, remove_mock, render_mock, copy_permissions_mock):
dir_path = '/tmp/some_plugin'
walk_mock.return_value = [
[dir_path, '', ['file1.txt.mako', 'file2.txt']],
@ -227,6 +229,7 @@ class TestUtils(BaseTestCase):
params = {'param1': 'value1', 'param2': 'value2'}
utils.render_files_in_dir(dir_path, params)
self.assertEqual(
[mock.call('/tmp/some_plugin/file1.txt.mako',
'/tmp/some_plugin/file1.txt',
@ -240,3 +243,10 @@ class TestUtils(BaseTestCase):
[mock.call('/tmp/some_plugin/file1.txt.mako'),
mock.call('/tmp/some_plugin/file4.mako')],
remove_mock.call_args_list)
self.assertEqual(
[mock.call('/tmp/some_plugin/file1.txt.mako',
'/tmp/some_plugin/file1.txt'),
mock.call('/tmp/some_plugin/file4.mako',
'/tmp/some_plugin/file4')],
copy_permissions_mock.call_args_list)

View File

@ -152,6 +152,7 @@ def render_files_in_dir(dir_path, params):
src_path = os.path.join(root, file_path)
dst_path = os.path.join(root, name)
render_to_file(src_path, dst_path, params)
copy_file_permissions(src_path, dst_path)
remove(src_path)