`OptimizerConfig::skip_failing_rules` should be false by default in sql tests

[andygrove]

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
We currently default to OptimizerConfig::skip_failing_rules to true in tests, and this means that we fail to catch regressions.

Describe the solution you'd like
I would like to default to true intests so that regressions are caught much sooner.

Describe alternatives you've considered
None

Additional context
None

[andygrove]

These tests fail when we stop ignoring optimizer errors:

failures:
    sql::group_by::csv_query_having_without_group_by
    sql::joins::equijoin_implicit_syntax_with_filter
    sql::joins::equijoin_left_and_condition_from_left
    sql::joins::equijoin_right_and_condition_from_left
    sql::joins::equijoin_right_and_condition_from_right
    sql::joins::reduce_left_join_1
    sql::joins::reduce_left_join_2
    sql::joins::reduce_left_join_3
    sql::joins::self_join_non_equijoin
    sql::select::csv_select_nested
    sql::select::csv_select_nested_without_aliases
    sql::select::unprojected_filter
    sql::timestamp::cast_string_to_time