Made operator property on errors optional.

[loucyx]

If I just do inside a test:

throw Error("Mesage");

I get this:

Message  (undefined)

Followed by the stack trace, instead of just getting Message. This is because the format function is expecting the error to have a string property operator to put after the message of the error.

This MR makes it so if that property is missing, we just skip it.

[codecov-commenter]

Codecov Report

Merging #41 into master will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master      #41   +/-   ##
=======================================
  Coverage   90.06%   90.06%           
=======================================
  Files           3        3           
  Lines         302      302           
=======================================
  Hits          272      272           
  Misses         30       30           

Impacted Files	Coverage Δ
src/index.js	78.09% <0.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2bc0026...e21d258. Read the comment docs.

[lukeed]

Hey~! How do you feel about (unknown) instead?

[loucyx]

@lukeed I still feel is "adding stuff" to my error message, but is better than (undefined) for sure!

[lukeed]

Ok. Can you please "allow edits from maintainers" so that I can push a commit?

[loucyx]

For some reason I can find the option to allow edits from maintainer (and I think is because of this). I'll try adding you as maintainer to the fork I did.

[lukeed]

No problem. Can you please insert this for the entire format function?

function format(name, err, suite = '') {
	let { details, operator='' } = err;
	let idx = err.stack && err.stack.indexOf('\n');
	if (err.name.startsWith('AssertionError') && !operator.includes('not')) details = compare(err.actual, err.expected); // TODO?
	let str = '  ' + FAILURE + (suite ? kleur.red(SUITE(` ${suite} `)) : '') + ' ' + QUOTE + kleur.red().bold(name) + QUOTE;
	str += '\n    ' + err.message + (operator ? kleur.italic().dim(`  (${operator})`) : '') + '\n';
	if (details) str += GUTTER + details.split('\n').join(GUTTER);
	if (!!~idx) str += stack(err.stack, idx);
	return str + '\n';
}

[loucyx]

Sure thing!

[lukeed]

Thanks!