diff options
author | Mike Bayer <mike_mp@zzzcomputing.com> | 2010-04-06 18:28:40 -0400 |
---|---|---|
committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2010-04-06 18:28:40 -0400 |
commit | afba8ab659f2c9ee7084358b6db5f61920b1691c (patch) | |
tree | 00e01f0ec976137acd63d2d2b8461da73bfc7bae /lib/sqlalchemy/test | |
parent | 376b8276f0a7acb1f95313ca604e273d7c981f57 (diff) | |
download | sqlalchemy-afba8ab659f2c9ee7084358b6db5f61920b1691c.tar.gz |
- added missing coverage for self-referential many-to-many flushes
- some other areas where per-state deps are called and an empty result returned
are still lacking coverage.
Diffstat (limited to 'lib/sqlalchemy/test')
-rw-r--r-- | lib/sqlalchemy/test/assertsql.py | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/lib/sqlalchemy/test/assertsql.py b/lib/sqlalchemy/test/assertsql.py index 1417c2e43..81a6191a1 100644 --- a/lib/sqlalchemy/test/assertsql.py +++ b/lib/sqlalchemy/test/assertsql.py @@ -156,12 +156,15 @@ class CompiledSQL(SQLMatchRule): if not isinstance(params, list): params = [params] - # do a positive compare only - for param, received in zip(params, _received_parameters): - for k, v in param.iteritems(): - if k not in received or received[k] != v: - equivalent = False - break + while params: + param = params.pop(0) + if param not in _received_parameters: + equivalent = False + break + else: + _received_parameters.remove(param) + if _received_parameters: + equivalent = False else: params = {} |