commit 1307d1c003b01b4f67524c95feb07c3d91c7c428
author Bram Moolenaar <Bram@vim.org> Sun Oct 07 20:16:49 2018 +0200
committer Bram Moolenaar <Bram@vim.org> Sun Oct 07 20:16:49 2018 +0200
tree 8c365b3c02351b128d4aea2744f4411425a84b30
parent a05a0d325c7615439f4a42f00682b2ebad43c8d9

patch 8.1.0460: assert_fails() does not take a message argument

Problem:    assert_fails() does not take a message argument
Solution:   Add the argument.

src/eval.c

diff --git a/src/eval.c b/src/eval.c
index 4f457a4..8746d55 100644
--- a/src/eval.c
+++ b/src/eval.c
@@ -9041,6 +9041,8 @@
     char_u	*cmd = get_tv_string_chk(&argvars[0]);
     garray_T	ga;
     int		ret = 0;
+    char_u	numbuf[NUMBUFLEN];
+    char_u	*tofree;
 
     called_emsg = FALSE;
     suppress_errthrow = TRUE;
@@ -9050,7 +9052,14 @@
     {
 	prepare_assert_error(&ga);
 	ga_concat(&ga, (char_u *)"command did not fail: ");
-	ga_concat(&ga, cmd);
+	if (argvars[1].v_type != VAR_UNKNOWN
+					   && argvars[2].v_type != VAR_UNKNOWN)
+	{
+	    ga_concat(&ga, echo_string(&argvars[2], &tofree, numbuf, 0));
+	    vim_free(tofree);
+	}
+	else
+	    ga_concat(&ga, cmd);
 	assert_error(&ga);
 	ga_clear(&ga);
 	ret = 1;

src/evalfunc.c

diff --git a/src/evalfunc.c b/src/evalfunc.c
index c9f4c45..0d79996 100644
--- a/src/evalfunc.c
+++ b/src/evalfunc.c
@@ -512,7 +512,7 @@
     {"assert_equal",	2, 3, f_assert_equal},
     {"assert_equalfile", 2, 2, f_assert_equalfile},
     {"assert_exception", 1, 2, f_assert_exception},
-    {"assert_fails",	1, 2, f_assert_fails},
+    {"assert_fails",	1, 3, f_assert_fails},
     {"assert_false",	1, 2, f_assert_false},
     {"assert_inrange",	3, 4, f_assert_inrange},
     {"assert_match",	2, 3, f_assert_match},
@@ -1507,7 +1507,7 @@
 }
 
 /*
- * "assert_fails(cmd [, error])" function
+ * "assert_fails(cmd [, error[, msg]])" function
  */
     static void
 f_assert_fails(typval_T *argvars, typval_T *rettv)

src/testdir/test_assert.vim

diff --git a/src/testdir/test_assert.vim b/src/testdir/test_assert.vim
index 28db289..fb31478 100644
--- a/src/testdir/test_assert.vim
+++ b/src/testdir/test_assert.vim
@@ -152,6 +152,14 @@
   call assert_equal(1, assert_fails('xxx', {}))
   call assert_match("Expected {} but got 'E731:", v:errors[0])
   call remove(v:errors, 0)
+
+  call assert_equal(1, assert_fails('xxx', {}, 'stupid'))
+  call assert_match("stupid: Expected {} but got 'E731:", v:errors[0])
+  call remove(v:errors, 0)
+
+  call assert_equal(1, assert_fails('echo', '', 'echo command'))
+  call assert_match("command did not fail: echo command", v:errors[0])
+  call remove(v:errors, 0)
 endfunc
 
 func Test_assert_beeps()

src/version.c

diff --git a/src/version.c b/src/version.c
index c868496..7a2e94a 100644
--- a/src/version.c
+++ b/src/version.c
@@ -793,6 +793,8 @@
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    460,
+/**/
     459,
 /**/
     458,