assert should take an optional message

[johnniwinther]

Currently the assert statement cannot provide any runtime information about what went wrong. Adding an addition message argument to the assert statement could provide such runtime information. For instance

assert(foo.isBar(), '$foo is not Bar');

could provide runtime information about the particalur foo instance that failed and not only, as now, that some foo instance failed. Such additional information would be valuable in debugging failed assertions.

[gramster]

There are alternatives to assert. assert should really just be used for language tests (Gilad, correct me if I'm wrong). For everything else there is a comprehensive set of matchers in the unit test library, which are soon going to be factored out into their own library. These matchers generate good messages automatically, and can be supplemented with user-supplied messages.

[gbracha]

I agree with the submitter that assert would be much more useful with an optional message parameter. I'm not sure I understand comment 1. I assume one rights code with asserts that is not in unit tests and does not have access to that library.

---

Added Accepted label.

[DartBot]

This comment was originally written by @davidB

---

but "expect" can not be ignored like assert via in non checked mode (VM or dart2js).
assertion can usefull to detect an throw error in dev/staging and simply do nothing in production live.

other example :

var nextState = states.get(next);
assert(nextState != null, "state '${next}' is not defined"); // For test/dev runtime only
if (nextState == null) {
  // nothing to change.
}

[DartBot]

This comment was originally written by nbilyk...@gmail.com

---

This post is old, but I'd like to +1 it if it hasn't been prioritized. assert seems like an awesome way to have extra dev/testing validation, but without a message the developer has to find that line number (which the dart IDE doesn't make clickable) and navigate manually to understand exactly what failed.

[gbracha]

Issue #17042 has been merged into this issue.

[DartBot]

This comment was originally written by zn...@google.com

---

Is there any update on this?

[sethladd]

@lrhn what do you think about this request?

(A customer asked to be able to add a message to assert.)

[lrhn]

It's a language change, not a library change, so if anything, it should be a DEP proposal.

I think there's a reason nothing has been done on this in almost three years - it's not that important.
I'm personally split on whether it's even a good idea.

The simple work-around is to use assert with a function:

assert(() => test || throw "message");

[sethladd]

Thanks for the feedback. If we don't plan to do this, let's close the issue.

[gramster]

You could possibly make the argument that Dart is mature enough now to not need things like language-level assert, but that's probably a non-starter.

On Jul 7, 2015, at 9:08 AM, Seth Ladd notifications@github.com wrote:

Thanks for the feedback. If we don't plan to do this, let's close the issue.

—
Reply to this email directly or view it on GitHub.