jackyhu-db opened a new pull request, #3095:
URL: https://github.com/apache/arrow-adbc/pull/3095

   # PR Description
   
   ### Motivation 
   
   The tests under `csharp/test/Drivers/Databricks` include both unit test and 
E2E test, and E2E test depends on the connection setup with Databricks and 
cannot (currently) be run by the CI/voter, but unit test does not have this 
dependency and should be run by the CI to detect code regression early. So we 
should restructure the tests into sub-folder `Unit` and `E2E`. 
   
   ### Changes
   - Restructure the tests under `csharp/test/Drivers/Databricks` into `Unit` 
and `E2E` folder
   - Use namespace `Apache.Arrow.Adbc.Tests.Drivers.Databricks.Unit` as the 
root namespace for all the Databricks unit test
   - Update `ci/scripts/csharp_test.sh` to include Databricks unit test so that 
it can be run by the CI
   
   
   ### Testing
   
   - Run `ci/scripts/csharp_test.sh`
   - Run both E2E test and unit test under `csharp/test/Drivers/Databricks`


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to