diff options
author | John Perkins <john.d.perkins@gmail.com> | 2016-08-05 10:26:51 -0600 |
---|---|---|
committer | John Perkins <john.d.perkins@gmail.com> | 2016-08-05 11:32:35 -0600 |
commit | 1cc68e8cd7d7dc29ff34ae1aca9c43a94ef539f9 (patch) | |
tree | b575893c6ba024b67dbc0f343ce86876d1adafd8 /test/ext/declarative/test_basic.py | |
parent | f2fa9d000b44a54b0fd3ae6114eb5d53ef20c3b8 (diff) | |
download | sqlalchemy-pr/298.tar.gz |
Clarify mappers.configure_mappers error messagepr/298
Sometimes the message generated by configure_mappers lacks a reference
to the failing mapper, and only the mapper._configure_failed is shown. This
makes debugging problems extremely difficult. Example:
sqlalchemy.exc.InvalidRequestError: One or more mappers failed to
initialize - can't proceed with initialization of other mappers. Original
exception was: Class 'neutron.objects.router.Router' is not mapped
In the above failure, the actual object having a problem is FloatingIP, which
has a knock-on effect on the Router object when it fails to map. A more helpful
error message might look like this example:
sqlalchemy.exc.InvalidRequestError: One (Mapper|FloatingIp|floatingip) or
more mappers failed to initialize - can't proceed with initialization of
other mappers. Original exception was: Class
'neutron.objects.router.Router' is not mapped
This patch adds the failing mapper name to the beginning of the message after
'One', as shown in the second example.
Change-Id: I9f23bfa90b26dde9229ab7ec812eec9ceae48153
Diffstat (limited to 'test/ext/declarative/test_basic.py')
-rw-r--r-- | test/ext/declarative/test_basic.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/test/ext/declarative/test_basic.py b/test/ext/declarative/test_basic.py index ae1a85f8b..2fc9dcafd 100644 --- a/test/ext/declarative/test_basic.py +++ b/test/ext/declarative/test_basic.py @@ -772,9 +772,9 @@ class DeclarativeTest(DeclarativeTestBase): for i in range(3): assert_raises_message( sa.exc.InvalidRequestError, - "^One or more mappers failed to initialize - " - "can't proceed with initialization of other " - "mappers. Original exception was: When initializing.*", + "^One (Mapper|User|users) or more mappers failed to initialize" + " - can't proceed with initialization of other mappers. " + "Original exception was: When initializing.*", configure_mappers) def test_custom_base(self): |