Skip to content

Commit ca4a168

Browse files
committed
test: Rename empty env vars test variable
Signed-off-by: Tomas Kislan <tomas@kislan.sk>
1 parent b10def7 commit ca4a168

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

src/platform/notebooks/deepnote/sqlIntegrationEnvironmentVariablesProvider.unit.test.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ import {
1414
} from './integrationTypes';
1515
import { mockedVSCodeNamespaces, resetVSCodeMocks } from '../../../test/vscode-mock';
1616

17-
const EXPECTED_EMPTY_ENV_VARS = {
17+
const EXPECTED_DATAFRAME_ONLY_ENV_VARS = {
1818
SQL_DEEPNOTE_DATAFRAME_SQL: '{"url":"deepnote+duckdb:///:memory:","params":{},"param_style":"qmark"}'
1919
};
2020

@@ -59,7 +59,7 @@ suite('SqlIntegrationEnvironmentVariablesProvider', () => {
5959
when(mockedVSCodeNamespaces.workspace.notebookDocuments).thenReturn([notebook]);
6060

6161
const envVars = await provider.getEnvironmentVariables(uri);
62-
assert.deepStrictEqual(envVars, EXPECTED_EMPTY_ENV_VARS);
62+
assert.deepStrictEqual(envVars, EXPECTED_DATAFRAME_ONLY_ENV_VARS);
6363
});
6464

6565
test('Returns empty object when SQL cells have no integration ID', async () => {
@@ -71,7 +71,7 @@ suite('SqlIntegrationEnvironmentVariablesProvider', () => {
7171
when(mockedVSCodeNamespaces.workspace.notebookDocuments).thenReturn([notebook]);
7272

7373
const envVars = await provider.getEnvironmentVariables(uri);
74-
assert.deepStrictEqual(envVars, EXPECTED_EMPTY_ENV_VARS);
74+
assert.deepStrictEqual(envVars, EXPECTED_DATAFRAME_ONLY_ENV_VARS);
7575
});
7676

7777
test('Returns environment variable for internal DuckDB integration', async () => {
@@ -256,7 +256,7 @@ suite('SqlIntegrationEnvironmentVariablesProvider', () => {
256256
const envVars = await provider.getEnvironmentVariables(uri);
257257

258258
// Should return only dataframe integration when integration config is missing
259-
assert.deepStrictEqual(envVars, EXPECTED_EMPTY_ENV_VARS);
259+
assert.deepStrictEqual(envVars, EXPECTED_DATAFRAME_ONLY_ENV_VARS);
260260
});
261261

262262
test('Properly encodes special characters in PostgreSQL credentials', async () => {
@@ -627,7 +627,7 @@ suite('SqlIntegrationEnvironmentVariablesProvider', () => {
627627

628628
// Should return only dataframe integration when unsupported auth method is encountered
629629
const envVars = await provider.getEnvironmentVariables(uri);
630-
assert.deepStrictEqual(envVars, EXPECTED_EMPTY_ENV_VARS);
630+
assert.deepStrictEqual(envVars, EXPECTED_DATAFRAME_ONLY_ENV_VARS);
631631
});
632632

633633
test('Skips unsupported Snowflake auth method (AZURE_AD)', async () => {
@@ -651,7 +651,7 @@ suite('SqlIntegrationEnvironmentVariablesProvider', () => {
651651
when(integrationStorage.getIntegrationConfig(integrationId)).thenResolve(config);
652652

653653
const envVars = await provider.getEnvironmentVariables(uri);
654-
assert.deepStrictEqual(envVars, EXPECTED_EMPTY_ENV_VARS);
654+
assert.deepStrictEqual(envVars, EXPECTED_DATAFRAME_ONLY_ENV_VARS);
655655
});
656656

657657
test('Skips unsupported Snowflake auth method (KEY_PAIR)', async () => {
@@ -675,7 +675,7 @@ suite('SqlIntegrationEnvironmentVariablesProvider', () => {
675675
when(integrationStorage.getIntegrationConfig(integrationId)).thenResolve(config);
676676

677677
const envVars = await provider.getEnvironmentVariables(uri);
678-
assert.deepStrictEqual(envVars, EXPECTED_EMPTY_ENV_VARS);
678+
assert.deepStrictEqual(envVars, EXPECTED_DATAFRAME_ONLY_ENV_VARS);
679679
});
680680
});
681681
});

0 commit comments

Comments
 (0)