Patch 8.2.0969
Problem: Assert_equal() output for dicts is hard to figure out.
Solution: Only show the different items.
Files: src/testing.c, src/testdir/test_assert.vim
*** ../vim-8.2.0968/src/testing.c 2020-06-04 16:52:37.090058926 +0200
--- src/testing.c 2020-06-13 15:04:40.624679149 +0200
***************
*** 131,142 ****
garray_T *gap,
typval_T *opt_msg_tv,
char_u *exp_str,
! typval_T *exp_tv,
! typval_T *got_tv,
assert_type_T atype)
{
char_u numbuf[NUMBUFLEN];
char_u *tofree;
if (opt_msg_tv->v_type != VAR_UNKNOWN)
{
--- 131,146 ----
garray_T *gap,
typval_T *opt_msg_tv,
char_u *exp_str,
! typval_T *exp_tv_arg,
! typval_T *got_tv_arg,
assert_type_T atype)
{
char_u numbuf[NUMBUFLEN];
char_u *tofree;
+ typval_T *exp_tv = exp_tv_arg;
+ typval_T *got_tv = got_tv_arg;
+ int did_copy = FALSE;
+ int omitted = 0;
if (opt_msg_tv->v_type != VAR_UNKNOWN)
{
***************
*** 153,158 ****
--- 157,218 ----
ga_concat(gap, (char_u *)"Expected ");
if (exp_str == NULL)
{
+ // When comparing dictionaries, drop the items that are equal, so that
+ // it's a lot easier to see what differs.
+ if (atype != ASSERT_NOTEQUAL
+ && exp_tv->v_type == VAR_DICT && got_tv->v_type == VAR_DICT
+ && exp_tv->vval.v_dict != NULL && got_tv->vval.v_dict != NULL)
+ {
+ dict_T *exp_d = exp_tv->vval.v_dict;
+ dict_T *got_d = got_tv->vval.v_dict;
+ hashitem_T *hi;
+ dictitem_T *item2;
+ int todo;
+
+ did_copy = TRUE;
+ exp_tv->vval.v_dict = dict_alloc();
+ got_tv->vval.v_dict = dict_alloc();
+ if (exp_tv->vval.v_dict == NULL || got_tv->vval.v_dict == NULL)
+ return;
+
+ todo = (int)exp_d->dv_hashtab.ht_used;
+ for (hi = exp_d->dv_hashtab.ht_array; todo > 0; ++hi)
+ {
+ if (!HASHITEM_EMPTY(hi))
+ {
+ item2 = dict_find(got_d, hi->hi_key, -1);
+ if (item2 == NULL || !tv_equal(&HI2DI(hi)->di_tv,
+ &item2->di_tv, FALSE, FALSE))
+ {
+ // item of exp_d not present in got_d or values differ.
+ dict_add_tv(exp_tv->vval.v_dict,
+ (char *)hi->hi_key, &HI2DI(hi)->di_tv);
+ if (item2 != NULL)
+ dict_add_tv(got_tv->vval.v_dict,
+ (char *)hi->hi_key, &item2->di_tv);
+ }
+ else
+ ++omitted;
+ --todo;
+ }
+ }
+
+ // Add items only present in got_d.
+ todo = (int)got_d->dv_hashtab.ht_used;
+ for (hi = got_d->dv_hashtab.ht_array; todo > 0; ++hi)
+ {
+ if (!HASHITEM_EMPTY(hi))
+ {
+ item2 = dict_find(exp_d, hi->hi_key, -1);
+ if (item2 == NULL)
+ // item of got_d not present in exp_d
+ dict_add_tv(got_tv->vval.v_dict,
+ (char *)hi->hi_key, &HI2DI(hi)->di_tv);
+ --todo;
+ }
+ }
+ }
+
ga_concat_shorten_esc(gap, tv2string(exp_tv, &tofree, numbuf, 0));
vim_free(tofree);
}
***************
*** 168,173 ****
--- 228,248 ----
ga_concat(gap, (char_u *)" but got ");
ga_concat_shorten_esc(gap, tv2string(got_tv, &tofree, numbuf, 0));
vim_free(tofree);
+
+ if (omitted != 0)
+ {
+ char buf[100];
+
+ vim_snprintf(buf, 100, " - %d equal item%s omitted",
+ omitted, omitted == 1 ? "" : "s");
+ ga_concat(gap, (char_u *)buf);
+ }
+ }
+
+ if (did_copy)
+ {
+ clear_tv(exp_tv);
+ clear_tv(got_tv);
}
}
*** ../vim-8.2.0968/src/testdir/test_assert.vim 2020-06-04 16:52:37.090058926
+0200
--- src/testdir/test_assert.vim 2020-06-13 15:08:18.935798951 +0200
***************
*** 50,55 ****
--- 50,75 ----
call remove(v:errors, 0)
endfunc
+ func Test_assert_equal_dict()
+ call assert_equal(0, assert_equal(#{one: 1, two: 2}, #{two: 2, one: 1}))
+
+ call assert_equal(1, assert_equal(#{one: 1, two: 2}, #{two: 2, one: 3}))
+ call assert_match("Expected {'one': 1} but got {'one': 3} - 1 equal item
omitted", v:errors[0])
+ call remove(v:errors, 0)
+
+ call assert_equal(1, assert_equal(#{one: 1, two: 2}, #{two: 22, one: 11}))
+ call assert_match("Expected {'one': 1, 'two': 2} but got {'one': 11, 'two':
22}", v:errors[0])
+ call remove(v:errors, 0)
+
+ call assert_equal(1, assert_equal(#{}, #{two: 2, one: 1}))
+ call assert_match("Expected {} but got {'one': 1, 'two': 2}", v:errors[0])
+ call remove(v:errors, 0)
+
+ call assert_equal(1, assert_equal(#{two: 2, one: 1}, #{}))
+ call assert_match("Expected {'one': 1, 'two': 2} but got {}", v:errors[0])
+ call remove(v:errors, 0)
+ endfunc
+
func Test_assert_equalfile()
call assert_equal(1, assert_equalfile('abcabc', 'xyzxyz'))
call assert_match("E485: Can't read file abcabc", v:errors[0])
*** ../vim-8.2.0968/src/version.c 2020-06-13 13:20:21.092620451 +0200
--- src/version.c 2020-06-13 14:51:27.343914908 +0200
***************
*** 756,757 ****
--- 756,759 ----
{ /* Add new patch number below this line */
+ /**/
+ 969,
/**/
--
ARTHUR: Well, I can't just call you `Man'.
DENNIS: Well, you could say `Dennis'.
ARTHUR: Well, I didn't know you were called `Dennis.'
DENNIS: Well, you didn't bother to find out, did you?
The Quest for the Holy Grail (Monty Python)
/// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ an exciting new programming language -- http://www.Zimbu.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
---
You received this message because you are subscribed to the Google Groups
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/vim_dev/202006131314.05DDEicU580855%40masaka.moolenaar.net.
