Improve output for the assertion to contain

[sunesimonsen]

This is out current output:

expect([0,1,2,3,4,5,6,7,8,9], 'to contain', [0,1,3,4,5,6,7,8,9])
UnexpectedError:
expected [ 0, 1, 2, 3, 4, 5, 6, 7, 8, 9 ] to contain [ 0, 1, 3, 4, 5, 6, 7, 8, 9 ]

It would be better if we did the following:

expect([0,1,2,3,4,5,6,7,8,9], 'to contain', [0,1,3,4,5,6,7,8,9])
UnexpectedError:
expected [ 0, 1, 2, 3, 4, 5, 6, 7, 8, 9 ] to contain [ 0, 1, 3, 4, 5, 6, 7, 8, 9 ]
  Did not contain: 2

[papandreou]

to contain takes varargs, not an array of items that the subject array must contain. The first example should probably have looked like this:

> expect([0, 1, 3, 4, 5, 6, 7, 8, 9], 'to contain', 0, 1, 2, 3, 4, 5, 6, 7, 8, 9)
UnexpectedError: 
expected [ 0, 1, 3, 4, 5, 6, 7, 8, 9 ] to contain 0, 1, 2, 3, 4, 5, 6, 7, 8, 9

I think this feature req is a great idea. It should be relatively easy, so it might even work as a good "first PR".

[sunesimonsen]

@papandreou you are right that we shouldn't output the arguments as an array either.

[HyunSangHan]

@sunesimonsen @papandreou
I tried to make PR for this, but it already seems make output when I ran it like below:

> expect([0, 1, 3, 4, 5, 6, 7, 8, 9], 'to contain', 0, 1, 2, 3, 4, 5, 6, 7, 8, 9)
UnexpectedError: 
expected [ 0, 1, 3, 4, 5, 6, 7, 8, 9 ] to contain 0, 1, 2, 3, 4, 5, 6, 7, 8, 9

(same as papandreou said)

Is it already resolved? I am wondering if this issue is valid until now. :-)