Changeset 42

Timestamp:

2008年08月25日 18时22分44秒 (4 years ago)

Author:

jiangx

Message:

See #9: add model for svn plugins management.

Location:

trunk/pysvnmanager

Files:

• hooks (added)
• hooks/__init__.py (added)
• hooks/init (added)
• hooks/init/__init__.py (added)
• hooks/plugins (added)
• hooks/plugins/CaseInsensitive.py (added)
• hooks/plugins/EolStyleCheck.py (added)
• hooks/plugins/__init__.py (added)
• model/hooks.py (added)
• model/repos.py (modified) (4 diffs)
• tests/test_repos.py (modified) (4 diffs)

trunk/pysvnmanager/model/repos.py

@@ -2 +2 @@
 # -*- coding: utf-8 -*-
 
-"""Subversion authz config file management.
+"""Subversion repos management.
 
-Basic classes used for Subversion authz management.
+Basic classes used for Subversion repository add/remove.
 """
 
@@ -14 +14 @@
 log = logging.getLogger(__name__)
 
+from pylons import config
+config_path = config["here"] + '/config'
+if config_path not in sys.path:
+    sys.path.insert(0, config_path)
+from localconfig import LocalConfig as cfg
+
+sys.path.insert(0,os.path.dirname(os.path.dirname(os.path.dirname(os.path.abspath(__file__)))))
+from pysvnmanager import hooks
+
 # i18n works only as pysvnmanager (a pylons app) model.
-from pylons import config
 if config.get('package') and not config.has_key('unittest'):
     from pylons.i18n import _
@@ -60 +68 @@
     
     def hooks_init(self, repos_name):
-        version = self.svnversion()
-        repos_path = "%(root)s/%(entry)s" % { "root": self.repos_root, "entry": repos_name}
+        import distutils.version as dv
+        version = '.'.join(self.svnversion())
+        matched = 'default'
+        for ver in sorted(hooks.init.svn_hooks_init_dict.keys(),
+                          cmp = lambda x,y: cmp(dv.LooseVersion(x), dv.LooseVersion(y)),
+                          reverse=True):
+            if dv.LooseVersion(version) >= dv.LooseVersion(ver):
+                matched = ver
+                break
+        
+        #log.debug('Hooks version matched: %s' % matched)
+        #log.debug('svn_hooks_init_base: %s' % hooks.init.svn_hooks_init_base)
+        
+        # copy hook-scripts from matched hooks templates directory.
+        src = hooks.init.svn_hooks_init_base + '/' + hooks.init.svn_hooks_init_dict[matched]
+        dest = "%(root)s/%(entry)s/hooks" % { "root": self.repos_root, "entry": repos_name}
+        
+        log.debug('Now copy hooks from \n\t%s to \n\t%s' % (src, dest))
+        import shutil
+        shutil.rmtree(dest)
+        shutil.copytree(src, dest, symlinks=True)
+        
     
     def svnversion(self):
@@ -94 +122 @@
             
     def delete(self, repos_name):
-        if self.is_blank_svn_repos(repos_name):
-            repos_path = "%(root)s/%(entry)s" % { "root": self.repos_root, "entry": repos_name}
-            from svn import repos as _repos
-            _repos.delete(repos_path)
-        else:
-            raise Exception, _("Repos %s is not a blank repository.") % repos_name
+        repos_path = "%(root)s/%(entry)s" % { "root": self.repos_root, "entry": repos_name}
+        if os.path.exists(repos_path):
+            if self.is_blank_svn_repos(repos_name):
+                from svn import repos as _repos
+                _repos.delete(repos_path)
+            else:
+                raise Exception, _("Repos %s is not a blank repository.") % repos_name
         
 

trunk/pysvnmanager/tests/test_repos.py

@@ -9 +9 @@
 from pysvnmanager import model
 from pysvnmanager.model import repos
+from pysvnmanager.model import hooks
+from pysvnmanager.hooks import plugins
+
 import StringIO
 from pprint import pprint
@@ -15 +18 @@
     
     def __init__(self, *args):
-        repos_root = os.path.dirname(os.path.dirname(os.path.dirname(os.path.abspath(__file__)))) + '/svnroot'
-        self.repos = repos.Repos(repos_root)
+        self.repos_root = os.path.dirname(os.path.dirname(os.path.dirname(os.path.abspath(__file__)))) + '/svnroot'
+        self.repos = repos.Repos(self.repos_root)
         super(TestRepos, self).__init__(*args)
 
@@ -27 +30 @@
         pass
     
+    def testReposCreate(self):
+        self.assertRaises(Exception, self.repos.create, 'repos1')
+        self.repos.delete('repos3')
+        self.repos.create('repos3')
+        self.assert_(sorted(self.repos.repos_list) == ['repos1', 'repos2', 'repos3'], self.repos.repos_list)
+
+    def testReposDelete(self):
+        self.assertRaises(Exception, self.repos.delete, 'repos1')
+    
     def testReposRoot(self):
         repos.Repos('/tmp')
@@ -32 +44 @@
 
     def testReposlist(self):
-        self.assert_(sorted(self.repos.repos_list) == ['repos1', 'repos2'], u','.join(self.repos.repos_list).encode('utf-8'))
+        self.assert_(sorted(self.repos.repos_list) == ['repos1', 'repos2', 'repos3'], u','.join(self.repos.repos_list).encode('utf-8'))
 
-    def testReposCreate(self):
-        self.assertRaises(Exception, self.repos.create, 'repos1')
-        self.repos.create('repos3')
-        self.assert_(sorted(self.repos.repos_list) == ['repos1', 'repos2', 'repos3'], self.repos.repos_list)
-        self.repos.delete('repos3')
-
-    def testReposDelete(self):
-        self.assertRaises(Exception, self.repos.delete, 'repos1')
-    
     def testSvnVersion(self):
         svnversion = self.repos.svnversion()
         self.assert_(svnversion[0]!='', svnversion[0])
         self.assert_(svnversion[1]!='', svnversion[1])
+
+class TestReposPlugin(TestController):
     
+    def __init__(self, *args):
+        self.repos_root = os.path.dirname(os.path.dirname(os.path.dirname(os.path.abspath(__file__)))) + '/svnroot'
+        super(TestReposPlugin, self).__init__(*args)
+
+    def setUp(self):
+        #print '+'*40, 'setup'
+        pass
+
+    def tearDown(self):
+        #print '+'*40, 'teardown'
+        pass
+
+    def testPluginList(self):
+        self.assert_(plugins.modules==['CaseInsensitive', 'EolStyleCheck'], plugins.modules)
+      
+    def testPluginImport(self):
+        self.assertRaises(Exception,  plugins.getHandler("CaseInsensitive"), "")
+        module_ci = plugins.getHandler("CaseInsensitive")(self.repos_root + '/repos1')
+        self.assert_(module_ci.name=="check case insensitive", module_ci.name)
+        self.assert_(module_ci.description!="", module_ci.description)
+      
+    def testPluginSetting(self):
+        m = plugins.getHandler("CaseInsensitive")(self.repos_root + '/repos1')
+        self.assert_(m.is_set()==False)
+        m.set_plugin()
+        self.assert_(m.is_set()==True)
+        m.delete_plugin()
+        self.assert_(m.is_set()==False)
+      
+    def testHooks(self):
+        # self.repos_root is not a repository.
+        self.assertRaises(AssertionError, hooks.Hooks, self.repos_root)
+        
+        myhooks = hooks.Hooks(self.repos_root + '/repos1')
+        self.assert_(myhooks.pluginnames==['CaseInsensitive', 'EolStyleCheck'], myhooks.pluginnames)
+        self.assert_(myhooks.unapplied_plugins==['CaseInsensitive', 'EolStyleCheck'], myhooks.unapplied_plugins)
+        
+        m = myhooks.plugins['CaseInsensitive']
+        self.assert_(m.name=="check case insensitive", m.name)
+        self.assert_(m.description!="", m.description)
+    
+    def testHooksSetting(self):
+        myhooks = hooks.Hooks(self.repos_root + '/repos1')
+
+        m = myhooks.plugins['CaseInsensitive']
+        self.assert_(m.is_set()==False)
+        self.assert_(myhooks.applied_plugins==[], myhooks.applied_plugins)
+        self.assert_(myhooks.unapplied_plugins==['CaseInsensitive', 'EolStyleCheck'], myhooks.unapplied_plugins)
+
+        m.set_plugin()
+        self.assert_(m.is_set()==True)
+        self.assert_(myhooks.applied_plugins==['CaseInsensitive'], myhooks.applied_plugins)
+        self.assert_(myhooks.unapplied_plugins==['EolStyleCheck'], myhooks.unapplied_plugins)
+
+        m.delete_plugin()
+        self.assert_(m.is_set()==False)
+        self.assert_(myhooks.applied_plugins==[], myhooks.applied_plugins)
+        self.assert_(myhooks.unapplied_plugins==['CaseInsensitive', 'EolStyleCheck'], myhooks.unapplied_plugins)
+
+
 if __name__ == '__main__': 
     import unittest