These need to be re-thought, as they are now just confusing. For now,
things should just use the _assert_ primitive.
* assert.xml (_assert-nonzero_, _fail-on-empty_): Remove templates.
Keep things consistent. A deprecation warning will be displayed for
`_fail-on-empty_'.
* assert.xml (_assert-nonzero_): Rename from `_fail-on-empty'.
(_fail-on-empty_): Alias to `_assert-nonzero'. Display deprecation
warning.
This completely breaks BC by completely reversing existing behavior.
I'm not sure what I was thinking to begin with.
Assertions are supposed to state an _expected_ condition and fail if they
yield _false_; this is the opposite of what `_assert_' was doing before this
change!
To keep things simple, the template now requires a body and the extra
`@when@' and `@class@' params have been removed (but are still available on
`_fail-on-empty_').
* assert.xml (_assert_): Negate behavior.
[@when@, @class@]: Remove params.
Otherwise, they're not included in package eligibility classes.
@external will be removed at some point in the future anyway; it's no
longer needed.
* assert.xml (_assert_): Remove @external.
Mike Gerwitz
Mike Gerwitz