fix: checkSparkMaybeThrows should compare Spark and Comet results in success case

[andygrove]

Which issue does this PR close?

Closes #2657

Rationale for this change

What changes are included in this PR?

• Delegate to checkSparkAnswer in success case
• Rename method for consistency with other checkSparkAnswer methods

How are these changes tested?

Existing tests

[andygrove]

This is the fix. All other changes are for renaming the method.

[codecov-commenter]

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 58.64%. Comparing base (f09f8af) to head (8c4d13b).
⚠️ Report is 686 commits behind head on main.

Files with missing lines	Patch %	Lines
...scala/org/apache/comet/expressions/CometCast.scala	50.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #2728      +/-   ##
============================================
+ Coverage     56.12%   58.64%   +2.51%     
- Complexity      976     1477     +501     
============================================
  Files           119      163      +44     
  Lines         11743    13990    +2247     
  Branches       2251     2378     +127     
============================================
+ Hits           6591     8204    +1613     
- Misses         4012     4578     +566     
- Partials       1140     1208      +68     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[parthchandra]

lgtm

[andygrove]

I am looking into this failure:

2025-11-07T16:55:12.8149030Z - cast ArrayType to StringType *** FAILED *** (23 seconds, 552 milliseconds)
2025-11-07T16:55:12.8293960Z   Results do not match for query: