Skip to content

Commit 9bf466f

Browse files
general-kroll-4-lifegeneral-kroll-4-life
committed
- Added robot test Busted Auth Throws Error Then Set Statement Update Auth Scenario Working.
1 parent 03f16a1 commit 9bf466f

File tree

2 files changed

+49
-2
lines changed

2 files changed

+49
-2
lines changed

‎test/robot/functional/stackql_mocked_from_cmd_line.robot‎

Lines changed: 41 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7196,3 +7196,44 @@ Set Statement Update Auth Scenario Working
71967196
... ${EMPTY}
71977197
... stdout=${CURDIR}/tmp/Set-Statement-Update-Auth-Scenario-Working-Working.tmp
71987198
... stderr=${CURDIR}/tmp/Set-Statement-Update-Auth-Scenario-Working-Working-stderr.tmp
7199+
7200+
Busted Auth Throws Error Then Set Statement Update Auth Scenario Working
7201+
${inputStr} = Catenate
7202+
... select name, id, network, split_part(network, '/', 8) as network_region from google.compute.firewalls where project \= 'testing-project' order by id desc;
7203+
... set session "$.auth.google.credentialsfilepath"='${AUTH_GOOGLE_SA_KEY_PATH}';
7204+
... select name, id, network, split_part(network, '/', 8) as network_region from google.compute.firewalls where project \= 'testing-project' order by id desc;
7205+
${outputStr} = Catenate SEPARATOR=\n
7206+
... |------------------------|---------------|----------------------------------------------------------------------------------------|----------------|
7207+
... |${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}name${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}|${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}id${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}|${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}network${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}|${SPACE}network_region${SPACE}|
7208+
... |------------------------|---------------|----------------------------------------------------------------------------------------|----------------|
7209+
... |${SPACE}selected-allow-rdesk${SPACE}${SPACE}${SPACE}|${SPACE}8888888888888${SPACE}|${SPACE}https://www.googleapis.com/compute/v1/projects/testing-project/global/networks/default${SPACE}|${SPACE}global${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}|
7210+
... |------------------------|---------------|----------------------------------------------------------------------------------------|----------------|
7211+
... |${SPACE}default-allow-ssh${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}|${SPACE}${SPACE}777777777777${SPACE}|${SPACE}https://www.googleapis.com/compute/v1/projects/testing-project/global/networks/default${SPACE}|${SPACE}global${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}|
7212+
... |------------------------|---------------|----------------------------------------------------------------------------------------|----------------|
7213+
... |${SPACE}default-allow-rdp${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}|${SPACE}${SPACE}${SPACE}${SPACE}6666666666${SPACE}|${SPACE}https://www.googleapis.com/compute/v1/projects/testing-project/global/networks/default${SPACE}|${SPACE}global${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}|
7214+
... |------------------------|---------------|----------------------------------------------------------------------------------------|----------------|
7215+
... |${SPACE}default-allow-internal${SPACE}|${SPACE}5555555555555${SPACE}|${SPACE}https://www.googleapis.com/compute/v1/projects/testing-project/global/networks/default${SPACE}|${SPACE}global${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}|
7216+
... |------------------------|---------------|----------------------------------------------------------------------------------------|----------------|
7217+
... |${SPACE}default-allow-icmp${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}|${SPACE}4444444444444${SPACE}|${SPACE}https://www.googleapis.com/compute/v1/projects/testing-project/global/networks/default${SPACE}|${SPACE}global${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}|
7218+
... |------------------------|---------------|----------------------------------------------------------------------------------------|----------------|
7219+
... |${SPACE}default-allow-https${SPACE}${SPACE}${SPACE}${SPACE}|${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}33333333${SPACE}|${SPACE}https://www.googleapis.com/compute/v1/projects/testing-project/global/networks/default${SPACE}|${SPACE}global${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}|
7220+
... |------------------------|---------------|----------------------------------------------------------------------------------------|----------------|
7221+
... |${SPACE}default-allow-http${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}|${SPACE}${SPACE}${SPACE}22222222222${SPACE}|${SPACE}https://www.googleapis.com/compute/v1/projects/testing-project/global/networks/default${SPACE}|${SPACE}global${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}|
7222+
... |------------------------|---------------|----------------------------------------------------------------------------------------|----------------|
7223+
... |${SPACE}allow-spark-ui${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}|${SPACE}${SPACE}111111111111${SPACE}|${SPACE}https://www.googleapis.com/compute/v1/projects/testing-project/global/networks/default${SPACE}|${SPACE}global${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}${SPACE}|
7224+
... |------------------------|---------------|----------------------------------------------------------------------------------------|----------------|
7225+
${outputErrStr} = Catenate SEPARATOR=\n
7226+
... service account credentials error: open ${NON_EXISTENT_AUTH_GOOGLE_SA_KEY_PATH}: no such file or directory
7227+
Should Stackql Exec Inline Equal Both Streams
7228+
... ${STACKQL_EXE}
7229+
... ${OKTA_SECRET_STR}
7230+
... ${GITHUB_SECRET_STR}
7231+
... ${K8S_SECRET_STR}
7232+
... ${REGISTRY_NO_VERIFY_CFG_STR}
7233+
... ${AUTH_CFG_DEFECTIVE_STR}
7234+
... ${SQL_BACKEND_CFG_STR_CANONICAL}
7235+
... ${inputStr}
7236+
... ${outputStr}
7237+
... ${outputErrStr}
7238+
... stdout=${CURDIR}/tmp/Busted-Auth-Throws-Error-Then-Set-Statement-Update-Auth-Scenario-Working-Working.tmp
7239+
... stderr=${CURDIR}/tmp/Busted-Auth-Throws-Error-Then-Set-Statement-Update-Auth-Scenario-Working-Working-stderr.tmp

‎test/robot/lib/stackql_context.py‎

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -188,6 +188,8 @@ def get_registry_mocked(execution_env :str) -> RegistryCfg:
188188

189189
_AUTH_GOOGLE_SA_KEY_PATH = os.path.join(REPOSITORY_ROOT, 'test', 'assets', 'credentials', 'dummy', 'google', 'functional-test-dummy-sa-key.json')
190190

191+
_NON_EXISTENT_AUTH_GOOGLE_SA_KEY_PATH = get_unix_path(os.path.join(REPOSITORY_ROOT, 'test', 'assets', 'credentials', 'dummy', 'google', 'non-existent-dummy-sa-key.json'))
192+
191193
_AUTH_CFG={
192194
"google": {
193195
"credentialsfilepath": _AUTH_GOOGLE_SA_KEY_PATH,
@@ -228,7 +230,7 @@ def get_registry_mocked(execution_env :str) -> RegistryCfg:
228230
}
229231

230232
_AUTH_CFG_DEFECTIVE= copy.deepcopy(_AUTH_CFG)
231-
_AUTH_CFG_DEFECTIVE["google"]["credentialsfilepath"] = get_unix_path(os.path.join(REPOSITORY_ROOT, 'test', 'assets', 'credentials', 'dummy', 'google', 'defective-dummy-sa-key.json'))
233+
_AUTH_CFG_DEFECTIVE["google"]["credentialsfilepath"] = _NON_EXISTENT_AUTH_GOOGLE_SA_KEY_PATH
232234

233235
_AUTH_PLUS_EXTERNAL_POSTGRES = copy.deepcopy(_AUTH_CFG)
234236

@@ -241,6 +243,8 @@ def get_registry_mocked(execution_env :str) -> RegistryCfg:
241243

242244
_AUTH_GOOGLE_SA_KEY_PATH_DOCKER = os.path.join('/opt', 'stackql', 'credentials', 'dummy', 'google', 'docker-functional-test-dummy-sa-key.json')
243245

246+
_NON_EXISTENT_AUTH_GOOGLE_SA_KEY_PATH_DOCKER = get_unix_path(os.path.join('/opt', 'stackql', 'credentials', 'dummy', 'google', 'non-existent-dummy-sa-key.json'))
247+
244248
_AUTH_CFG_DOCKER={
245249
"google": {
246250
"credentialsfilepath": _AUTH_GOOGLE_SA_KEY_PATH_DOCKER,
@@ -281,7 +285,7 @@ def get_registry_mocked(execution_env :str) -> RegistryCfg:
281285
}
282286

283287
_AUTH_CFG_DEFECTIVE_DOCKER= copy.deepcopy(_AUTH_CFG_DOCKER)
284-
_AUTH_CFG_DEFECTIVE_DOCKER["google"]["credentialsfilepath"] = get_unix_path(os.path.join('/opt', 'stackql', 'credentials', 'dummy', 'google', 'docker-defective-dummy-sa-key.json'))
288+
_AUTH_CFG_DEFECTIVE_DOCKER["google"]["credentialsfilepath"] = _NON_EXISTENT_AUTH_GOOGLE_SA_KEY_PATH_DOCKER
285289

286290
_AUTH_PLUS_EXTERNAL_POSTGRES_DOCKER = copy.deepcopy(_AUTH_CFG_DOCKER)
287291

@@ -1097,6 +1101,7 @@ def get_variables(execution_env :str, sql_backend_str :str, use_stackql_preinsta
10971101
rv['AUTH_CFG_STR'] = AUTH_CFG_STR_DOCKER
10981102
rv['AUTH_CFG_DEFECTIVE_STR'] = AUTH_CFG_DEFECTIVE_STR_DOCKER
10991103
rv['AUTH_GOOGLE_SA_KEY_PATH'] = _AUTH_GOOGLE_SA_KEY_PATH_DOCKER
1104+
rv['NON_EXISTENT_AUTH_GOOGLE_SA_KEY_PATH'] = _NON_EXISTENT_AUTH_GOOGLE_SA_KEY_PATH_DOCKER
11001105
rv['AUTH_PLUS_EXTERNAL_POSTGRES'] = AUTH_PLUS_EXTERNAL_POSTGRES_DOCKER
11011106
rv['AUTH_CFG_STR_INTEGRATION'] = AUTH_CFG_INTEGRATION_STR_DOCKER
11021107
rv['GET_IAM_POLICY_AGG_ASC_INPUT_FILE'] = GET_IAM_POLICY_AGG_ASC_INPUT_FILE_DOCKER
@@ -1137,6 +1142,7 @@ def get_variables(execution_env :str, sql_backend_str :str, use_stackql_preinsta
11371142
rv['AUTH_CFG_STR'] = AUTH_CFG_STR
11381143
rv['AUTH_CFG_DEFECTIVE_STR'] = AUTH_CFG_DEFECTIVE_STR
11391144
rv['AUTH_GOOGLE_SA_KEY_PATH'] = _AUTH_GOOGLE_SA_KEY_PATH
1145+
rv['NON_EXISTENT_AUTH_GOOGLE_SA_KEY_PATH'] = _NON_EXISTENT_AUTH_GOOGLE_SA_KEY_PATH
11401146
rv['AUTH_PLUS_EXTERNAL_POSTGRES'] = AUTH_PLUS_EXTERNAL_POSTGRES
11411147
rv['AUTH_CFG_STR_INTEGRATION'] = AUTH_CFG_INTEGRATION_STR
11421148
rv['GET_IAM_POLICY_AGG_ASC_INPUT_FILE'] = GET_IAM_POLICY_AGG_ASC_INPUT_FILE

0 commit comments

Comments
 (0)