From 0bedab23f3eb86878d894419614e1728c395a84e Mon Sep 17 00:00:00 2001
From: "Bryn M. Reeves" <bmr@redhat.com>
Date: Thu, 27 Mar 2014 21:06:24 +0000
Subject: Raise a TypeError if add_copy_specs() is called with a string

Since strings are iterable a plugin attempting to call
add_copy_specs("/something") results in a plugin calling
add_copy_spec("/"). Raise a TypeError if this happens.

Fixes Issue #141.

Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
---
 tests/plugin_tests.py | 3 +++
 1 file changed, 3 insertions(+)

(limited to 'tests/plugin_tests.py')

diff --git a/tests/plugin_tests.py b/tests/plugin_tests.py
index a4905cf9..c44c1629 100644
--- a/tests/plugin_tests.py
+++ b/tests/plugin_tests.py
@@ -243,6 +243,9 @@ class AddCopySpecTests(unittest.TestCase):
         self.mp.add_copy_specs(["tests/tail_test.txt"])
         self.assert_expect_paths()
 
+    def test_add_copy_spec_nostrings(self):
+        self.assertRaises(TypeError, self.mp.add_copy_specs,"stringsarebadmkay?")
+
     # add_copy_spec_limit()
 
     def test_single_file_over_limit(self):
-- 
cgit