diff options
| author | Chris McDonough <chrism@plope.com> | 2011-12-02 14:59:11 -0500 |
|---|---|---|
| committer | Chris McDonough <chrism@plope.com> | 2011-12-02 14:59:11 -0500 |
| commit | 116ec77f7e2c06f06af94ced45b5b5ce71340455 (patch) | |
| tree | 470ca0511883f0e1855d8082c1ec83bf7a2f682c | |
| parent | 65fb3b371d63f59fa94ec2b5a431894eb78060be (diff) | |
| download | pyramid-116ec77f7e2c06f06af94ced45b5b5ce71340455.tar.gz pyramid-116ec77f7e2c06f06af94ced45b5b5ce71340455.tar.bz2 pyramid-116ec77f7e2c06f06af94ced45b5b5ce71340455.zip | |
prevent a scaffold rendering from being named site
| -rw-r--r-- | pyramid/scaffolds/__init__.py | 4 | ||||
| -rw-r--r-- | pyramid/tests/test_scaffolds/test_init.py | 5 |
2 files changed, 9 insertions, 0 deletions
diff --git a/pyramid/scaffolds/__init__.py b/pyramid/scaffolds/__init__.py index dc6926024..50c04017a 100644 --- a/pyramid/scaffolds/__init__.py +++ b/pyramid/scaffolds/__init__.py @@ -7,6 +7,10 @@ from pyramid.scaffolds.template import Template class PyramidTemplate(Template): def pre(self, command, output_dir, vars): + if vars['package'] == 'site': + raise ValueError('Sorry, you may not name your package "site". ' + 'The package name "site" has a special meaning in ' + 'Python. Please name it anything except "site".') vars['random_string'] = native_(binascii.hexlify(os.urandom(20))) package_logger = vars['package'] if package_logger == 'root': diff --git a/pyramid/tests/test_scaffolds/test_init.py b/pyramid/tests/test_scaffolds/test_init.py index 6b038914a..4988e66ff 100644 --- a/pyramid/tests/test_scaffolds/test_init.py +++ b/pyramid/tests/test_scaffolds/test_init.py @@ -11,6 +11,11 @@ class TestPyramidTemplate(unittest.TestCase): inst.pre('command', 'output dir', vars) self.assertTrue(vars['random_string']) self.assertEqual(vars['package_logger'], 'one') + + def test_pre_site(self): + inst = self._makeOne() + vars = {'package':'site'} + self.assertRaises(ValueError, inst.pre, 'command', 'output dir', vars) def test_pre_root(self): inst = self._makeOne() |