update validation logic

Author: morford-brex

packages/cubejs-snowflake-driver/package.json

@@ -25,6 +25,7 @@
     "lint:fix": "eslint --fix src/* --ext .ts"
   },
   "dependencies": {
+    "@aws-sdk/client-s3": "^3.726.0",
     "@cubejs-backend/base-driver": "1.3.75",
     "@cubejs-backend/shared": "1.3.75",
     "date-fns-timezone": "^0.1.4",

packages/cubejs-snowflake-driver/src/SnowflakeDriver.ts

@@ -23,8 +23,8 @@ import {
 import { formatToTimeZone } from 'date-fns-timezone';
 import fs from 'fs/promises';
 import crypto from 'crypto';
-import { HydrationMap, HydrationStream } from './HydrationStream';
 import { S3ClientConfig } from '@aws-sdk/client-s3';
+import { HydrationMap, HydrationStream } from './HydrationStream';
 
 const SUPPORTED_BUCKET_TYPES = ['s3', 'gcs', 'azure'];
 
@@ -397,10 +397,7 @@ export class SnowflakeDriver extends BaseDriver implements DriverInterface {
   ): string[] {
     if (exportBucket.bucketType === 's3') {
       const s3Config = exportBucket as SnowflakeDriverExportAWS;
-      const hasCredentials = s3Config.keyId && s3Config.secretKey;
-      const hasIntegration = s3Config.integrationName;
-      
-      if (!hasCredentials && !hasIntegration) {
+      if (s3Config.integrationName) {
         return emptyKeys.filter(key => key !== 'keyId' && key !== 'secretKey');
       }
     }

packages/cubejs-testing-drivers/fixtures/snowflake.json

@@ -27,7 +27,8 @@
         "environment": {
           "CUBEJS_DB_EXPORT_BUCKET_TYPE": "s3",
           "CUBEJS_DB_EXPORT_BUCKET": "snowflake-drivers-tests-preaggs",
-          "CUBEJS_DB_EXPORT_BUCKET_AWS_REGION": "us-west-1"
+          "CUBEJS_DB_EXPORT_BUCKET_AWS_REGION": "us-west-1",
+          "CUBEJS_DB_EXPORT_INTEGRATION": "aws_int"
         }
       }
     },