Mercurial Mercurial > mercurial / changeset
summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | raw | help
Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
test-url: refactor with shorter lines
authorMartin Geisler <mg@aragost.com>
Fri, 01 Oct 2010 16:39:55 +0200
changeset 12606 5c8353692123
parent 12605 aa4a6e3ed0c9
child 12607 1393a81b3bdc
test-url: refactor with shorter lines
tests/test-url.py file | annotate | diff | comparison | revisions 
--- a/tests/test-url.py	Fri Oct 01 02:13:31 2010 +0200
+++ b/tests/test-url.py	Fri Oct 01 16:39:55 2010 +0200
@@ -4,38 +4,45 @@
     if a != b:
         print (a, b)
 
+def cert(cn):
+    return dict(subject=((('commonName', cn),),))
+
 from mercurial.url import _verifycert
 
 # Test non-wildcard certificates        
-check(_verifycert({'subject': ((('commonName', 'example.com'),),)}, 'example.com'),
-    None)
-check(_verifycert({'subject': ((('commonName', 'example.com'),),)}, 'www.example.com'),
-    'certificate is for example.com')
-check(_verifycert({'subject': ((('commonName', 'www.example.com'),),)}, 'example.com'),
-    'certificate is for www.example.com')
+check(_verifycert(cert('example.com'), 'example.com'),
+      None)
+check(_verifycert(cert('example.com'), 'www.example.com'),
+      'certificate is for example.com')
+check(_verifycert(cert('www.example.com'), 'example.com'),
+      'certificate is for www.example.com')
 
 # Test wildcard certificates
-check(_verifycert({'subject': ((('commonName', '*.example.com'),),)}, 'www.example.com'),
-    None)
-check(_verifycert({'subject': ((('commonName', '*.example.com'),),)}, 'example.com'),
-    'certificate is for *.example.com')
-check(_verifycert({'subject': ((('commonName', '*.example.com'),),)}, 'w.w.example.com'),
-    'certificate is for *.example.com')
+check(_verifycert(cert('*.example.com'), 'www.example.com'),
+      None)
+check(_verifycert(cert('*.example.com'), 'example.com'),
+      'certificate is for *.example.com')
+check(_verifycert(cert('*.example.com'), 'w.w.example.com'),
+      'certificate is for *.example.com')
 
 # Avoid some pitfalls
-check(_verifycert({'subject': ((('commonName', '*.foo'),),)}, 'foo'),
-    'certificate is for *.foo')
-check(_verifycert({'subject': ((('commonName', '*o'),),)}, 'foo'),
-    'certificate is for *o')
+check(_verifycert(cert('*.foo'), 'foo'),
+      'certificate is for *.foo')
+check(_verifycert(cert('*o'), 'foo'),
+      'certificate is for *o')
 
 import time
 lastyear = time.gmtime().tm_year - 1
 nextyear = time.gmtime().tm_year + 1
-check(_verifycert({'notAfter': 'May  9 00:00:00 %s GMT' % lastyear}, 'example.com'),
-    'certificate expired May  9 00:00:00 %s GMT' % lastyear)
-check(_verifycert({'notBefore': 'May  9 00:00:00 %s GMT' % nextyear}, 'example.com'),
-    'certificate not valid before May  9 00:00:00 %s GMT' % nextyear)
-check(_verifycert({'notAfter': 'Sep 29 15:29:48 %s GMT' % nextyear, 'subject': ()}, 'example.com'),
-    'no commonName found in certificate')
+check(_verifycert({'notAfter': 'May  9 00:00:00 %s GMT' % lastyear},
+                  'example.com'),
+      'certificate expired May  9 00:00:00 %s GMT' % lastyear)
+check(_verifycert({'notBefore': 'May  9 00:00:00 %s GMT' % nextyear},
+                  'example.com'),
+      'certificate not valid before May  9 00:00:00 %s GMT' % nextyear)
+check(_verifycert({'notAfter': 'Sep 29 15:29:48 %s GMT' % nextyear,
+                   'subject': ()},
+                  'example.com'),
+      'no commonName found in certificate')
 check(_verifycert(None, 'example.com'),
-    'no certificate received')
+      'no certificate received')
mercurial