summaryrefslogtreecommitdiff
path: root/tests/unittests
diff options
context:
space:
mode:
authorDaniel Watkins <oddbloke@ubuntu.com>2020-06-17 12:59:23 -0400
committerGitHub <noreply@github.com>2020-06-17 10:59:23 -0600
commit3d13a9dd469f354d0ddc1e36be37d743b5a57c73 (patch)
treeeb6fbcf7f66c5c67e7703858f93369e8934b9c34 /tests/unittests
parente01e3ed5444c8093de47229e20abec440530d549 (diff)
downloadvyos-cloud-init-3d13a9dd469f354d0ddc1e36be37d743b5a57c73.tar.gz
vyos-cloud-init-3d13a9dd469f354d0ddc1e36be37d743b5a57c73.zip
util: rename write_file's copy_mode parameter to preserve_mode (#439)
When updating the docstring to include it, I realised that the current name is somewhat misleading; this makes it a little easier to understand, I think.
Diffstat (limited to 'tests/unittests')
-rw-r--r--tests/unittests/test_util.py12
1 files changed, 6 insertions, 6 deletions
diff --git a/tests/unittests/test_util.py b/tests/unittests/test_util.py
index 737dda8b..0821341f 100644
--- a/tests/unittests/test_util.py
+++ b/tests/unittests/test_util.py
@@ -110,29 +110,29 @@ class TestWriteFile(helpers.TestCase):
file_stat = os.stat(path)
self.assertEqual(0o666, stat.S_IMODE(file_stat.st_mode))
- def test_copy_mode_no_existing(self):
- """Verify that file is created with mode 0o644 if copy_mode
+ def test_preserve_mode_no_existing(self):
+ """Verify that file is created with mode 0o644 if preserve_mode
is true and there is no prior existing file."""
path = os.path.join(self.tmp, "NewFile.txt")
contents = "Hey there"
- util.write_file(path, contents, copy_mode=True)
+ util.write_file(path, contents, preserve_mode=True)
self.assertTrue(os.path.exists(path))
self.assertTrue(os.path.isfile(path))
file_stat = os.stat(path)
self.assertEqual(0o644, stat.S_IMODE(file_stat.st_mode))
- def test_copy_mode_with_existing(self):
+ def test_preserve_mode_with_existing(self):
"""Verify that file is created using mode of existing file
- if copy_mode is true."""
+ if preserve_mode is true."""
path = os.path.join(self.tmp, "NewFile.txt")
contents = "Hey there"
open(path, 'w').close()
os.chmod(path, 0o666)
- util.write_file(path, contents, copy_mode=True)
+ util.write_file(path, contents, preserve_mode=True)
self.assertTrue(os.path.exists(path))
self.assertTrue(os.path.isfile(path))