Skip to content

Conversation

blueyed
Copy link
Owner

@blueyed blueyed commented Nov 6, 2019

Follow-up to 946434c (pytest-dev#5924).

Before this patch the test would look like this:

{'env': {'sub...s wrapped'}}}} == {'env': {'sub...}}}, 'new': 1}
Omitting 1 identical items, use -vv to show
Right contains 1 more item:
{'new': 1}
Full diff:
  {
   'env': {'sub': {'long_a': 'aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa',
-                  'sub1': {'long_a': 'substring '
+                  'sub1': {'long_a': 'substring that gets wrapped'}}},
?                                                +++++++++++++++++ ++++
+  'new': 1,
-                                     'that '
-                                     'gets '
-                                     'wrapped'}}},
  }

Ref: pytest-dev#6137

@blueyed blueyed force-pushed the fix-_compare_eq_iterable branch from 8d1e71e to 81577d5 Compare November 6, 2019 10:36
Follow-up to 946434c (pytest-dev#5924).

Before this patch the test would look like this:

    {'env': {'sub...s wrapped'}}}} == {'env': {'sub...}}}, 'new': 1}
    Omitting 1 identical items, use -vv to show
    Right contains 1 more item:
    {'new': 1}
    Full diff:
      {
       'env': {'sub': {'long_a': 'aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa',
    -                  'sub1': {'long_a': 'substring '
    +                  'sub1': {'long_a': 'substring that gets wrapped'}}},
    ?                                                +++++++++++++++++ ++++
    +  'new': 1,
    -                                     'that '
    -                                     'gets '
    -                                     'wrapped'}}},
      }
@blueyed blueyed force-pushed the fix-_compare_eq_iterable branch from 81577d5 to fb5bca9 Compare November 6, 2019 10:58
@blueyed blueyed merged commit 5771368 into my-master Nov 6, 2019
@blueyed blueyed deleted the fix-_compare_eq_iterable branch November 6, 2019 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant