Open jordanw-bq opened 6 years ago
We should adopt a strategy for testing this class to ensure getting/setting configuration options are sufficiently tested.
Proposed strategy: Similar to Presto, use Airlift's configuration module for verifying configuration behaviour (example: https://github.com/prestodb/presto/blob/master/presto-hive/src/test/java/com/facebook/presto/hive/TestHiveClientConfig.java)
We should adopt a strategy for testing this class to ensure getting/setting configuration options are sufficiently tested.
Proposed strategy: Similar to Presto, use Airlift's configuration module for verifying configuration behaviour (example: https://github.com/prestodb/presto/blob/master/presto-hive/src/test/java/com/facebook/presto/hive/TestHiveClientConfig.java)