Skip to content
  • David Chelimsky's avatar
    dcce0113
    Raise Assertion instead of RoutingError for routing assertion failures. · dcce0113
    David Chelimsky authored
    Before this change, assert_recognizes, assert_generates, and
    assert_routing raised ActionController::RoutingError when they failed to
    recognize the route.
    
    This commit changes them to raise Assertion instead. This aligns with
    convention for logical failures, and supports reporting tools that care
    about the difference between logical failures and errors e.g. the
    summary at the end of a test run.
    
    - Fixes #5899
    dcce0113
    Raise Assertion instead of RoutingError for routing assertion failures.
    David Chelimsky authored
    Before this change, assert_recognizes, assert_generates, and
    assert_routing raised ActionController::RoutingError when they failed to
    recognize the route.
    
    This commit changes them to raise Assertion instead. This aligns with
    convention for logical failures, and supports reporting tools that care
    about the difference between logical failures and errors e.g. the
    summary at the end of a test run.
    
    - Fixes #5899
Loading