test_pack_extra: Add additional cases.
authorMartin Pool <mbp@samba.org>
Mon, 4 Nov 2002 22:01:25 +0000 (22:01 +0000)
committerMartin Pool <mbp@samba.org>
Mon, 4 Nov 2002 22:01:25 +0000 (22:01 +0000)
test_pack_failures: Remove cases for which old code is too lax.
(This used to be commit 631945a03a3bf4982177bc22ebccf096b2bb5b6c)

source3/python/examples/tdbpack/test_tdbpack.py

index c17f08097372040f5e3b1e39f04e9c0cd6e986b0..fccf876c205bb0622aa6ac69812e55de75309e89 100755 (executable)
@@ -125,6 +125,8 @@ class PackTests(unittest.TestCase):
         """Leftover values when packing"""
         cases = [
             ('d', [10, 20], [10]),
         """Leftover values when packing"""
         cases = [
             ('d', [10, 20], [10]),
+            ('d', [10, 'hello'], [10]),
+            ('ff', ['hello', 'world', 'sailor'], ['hello', 'world']),
             ]
         for unpacker in both_unpackers:
             for packer in both_packers:
             ]
         for unpacker in both_unpackers:
             for packer in both_packers:
@@ -160,23 +162,26 @@ class PackTests(unittest.TestCase):
 #                 ('w', [0x60A15EC5L]),
                  ('w', [None]),
                  ('d', []),
 #                 ('w', [0x60A15EC5L]),
                  ('w', [None]),
                  ('d', []),
-                 ('d', [0L]),
                  ('p', []),
                  ('f', [2]),
                  ('P', [None]),
                  ('P', ()),
                  ('f', [hex]),
                  ('fw', ['hello']),
                  ('p', []),
                  ('f', [2]),
                  ('P', [None]),
                  ('P', ()),
                  ('f', [hex]),
                  ('fw', ['hello']),
-                 ('f', [u'hello']),
+                 ('f', [u'hello']),
                  ('B', [2]),
                  (None, [2, 3, 4]),
                  (ord('f'), [20]),
                  ('B', [2]),
                  (None, [2, 3, 4]),
                  (ord('f'), [20]),
-                 (['w', 'w'], [2, 2]),
-                 ('Q', [2]),
-                 ('fQ', ['2', 3]),
-                 ('fQ', ['2']),
+                 # old code doesn't distinguish string from seq-of-char
+#                 (['w', 'w'], [2, 2]),
+                 # old code just ignores invalid characters
+#                 ('Q', [2]),
+#                 ('fQ', ['2', 3]),
+#                 ('fQ', ['2']),
                  (2, [2]),
                  (2, [2]),
-                 ({}, {})]
+                 # old code doesn't typecheck format
+#                 ({}, {})
+                 ]
         for packer in both_packers:
             for format, values in cases:
                 try:
         for packer in both_packers:
             for format, values in cases:
                 try: