From 631945a03a3bf4982177bc22ebccf096b2bb5b6c Mon Sep 17 00:00:00 2001
From: Martin Pool <mbp@samba.org>
Date: Mon, 4 Nov 2002 22:01:25 +0000
Subject: [PATCH] test_pack_extra: Add additional cases.

test_pack_failures: Remove cases for which old code is too lax.
---
 .../python/examples/tdbpack/test_tdbpack.py   | 19 ++++++++++++-------
 1 file changed, 12 insertions(+), 7 deletions(-)

diff --git a/source/python/examples/tdbpack/test_tdbpack.py b/source/python/examples/tdbpack/test_tdbpack.py
index c17f0809737..fccf876c205 100755
--- a/source/python/examples/tdbpack/test_tdbpack.py
+++ b/source/python/examples/tdbpack/test_tdbpack.py
@@ -125,6 +125,8 @@ class PackTests(unittest.TestCase):
         """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:
@@ -160,23 +162,26 @@ class PackTests(unittest.TestCase):
 #                 ('w', [0x60A15EC5L]),
                  ('w', [None]),
                  ('d', []),
-                 ('d', [0L]),
                  ('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]),
-                 (['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]),
-                 ({}, {})]
+                 # old code doesn't typecheck format
+#                 ({}, {})
+                 ]
         for packer in both_packers:
             for format, values in cases:
                 try: