Unit tests don't seem to be fallible

[gdennie]

The following tests in lib.rs module test does not appear to be fallible.

    #[test]
    fn deeply_nested_whitelisted() {
        clean(&"<b>".repeat(60_000));
    }

[notriddle]

In an older version of ammonia, that test case would have caused a stack overflow.

[gdennie]

The tests or their names are confusing requiring your clarification. Perhaps the name could be more indicative, such as the following:

#[test]
fn deeply_nested_whitelisted_elements_do_not_cause_stack_overflow() {
    clean(&"<b>".repeat(60_000));
}

or perhaps contain such test within a dedicated module...

mod confirm_these_do_not_cause_stack_overflow {
    #[test]
    fn deeply_nested_whitelisted() {
        clean(&"<b>".repeat(60_000));
    }
}

While pruning old tests might open the gate to regressions, organizing them for clarity and perhaps selective execution might be a good compromise.

[notriddle]

Very good point. Thanks!