[AURON #1757] Add cleanup logic to prevent flaky tests caused by leftover locations

[yew1eb]

Which issue does this PR close?

Closes #1757

Rationale for this change

What changes are included in this PR?

BaseAuronSQLSuite: reset suite workspace and set spark.sql.warehouse.dir

Are there any user-facing changes?

How was this patch tested?

[yew1eb]

@cxzl25 Could you help merge this PR?

[Copilot]

Pull request overview

This pull request addresses flaky tests caused by leftover managed table locations from interrupted test runs. The changes add workspace cleanup logic to BaseAuronSQLSuite to ensure a clean environment before tests execute, preventing LOCATION_ALREADY_EXISTS errors when tests are rerun.

Changes:

• Added workspace directory management with cleanup logic in BaseAuronSQLSuite's beforeAll() method
• Configured spark.sql.warehouse.dir to use a dedicated warehouse directory per suite
• Modified visibility modifiers in BuildInfoInSparkUISuite from protected to public for beforeAll/afterAll methods

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

File	Description
spark-extension-shims-spark/src/test/scala/org/apache/auron/BaseAuronSQLSuite.scala	Added workspace initialization and cleanup infrastructure with beforeAll/afterAll lifecycle methods, plus warehouse directory configuration
spark-extension-shims-spark/src/test/scala/org/apache/spark/sql/execution/BuildInfoInSparkUISuite.scala	Changed visibility modifiers from protected to public for beforeAll/afterAll methods to match parent trait requirements

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.