Skip to content

Commit 6e4460d

Browse files
committed
test(sql): scope yaml profile test to demo cases
Remove sql.explain.json and trim sql.profile.yml to two demo cases: happy path and "ignored for csv format". The csv case demonstrates the silently-ignored behavior using the same `sql:` action, removing the need for the sql.explain.json spec. The remaining sql.json spec is required: every test case dispatches through the `sql:` action. Refs: opensearch-project#5317 Signed-off-by: Chen Dai <daichen@amazon.com>
1 parent 43c0a08 commit 6e4460d

2 files changed

Lines changed: 0 additions & 66 deletions

File tree

integ-test/src/yamlRestTest/resources/rest-api-spec/api/sql.explain.json

Lines changed: 0 additions & 22 deletions
This file was deleted.

integ-test/src/yamlRestTest/resources/rest-api-spec/test/api/sql.profile.yml

Lines changed: 0 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -59,21 +59,6 @@ teardown:
5959
- gt: {profile.plan.time_ms: 0.0}
6060
- match: {profile.plan.rows: 2}
6161

62-
---
63-
"Profile ignored for explain api":
64-
- skip:
65-
features:
66-
- headers
67-
- allowed_warnings
68-
- do:
69-
headers:
70-
Content-Type: 'application/json'
71-
sql.explain:
72-
body:
73-
query: 'SELECT message FROM sql_profile'
74-
profile: true
75-
- match: {profile: null}
76-
7762
---
7863
"Profile ignored for csv format":
7964
- skip:
@@ -89,32 +74,3 @@ teardown:
8974
profile: true
9075
format: 'csv'
9176
- match: {profile: null}
92-
93-
---
94-
"Profile absent when not requested":
95-
- skip:
96-
features:
97-
- headers
98-
- allowed_warnings
99-
- do:
100-
headers:
101-
Content-Type: 'application/json'
102-
sql:
103-
body:
104-
query: 'SELECT message FROM sql_profile'
105-
- match: {profile: null}
106-
107-
---
108-
"Profile ignored for explain query":
109-
- skip:
110-
features:
111-
- headers
112-
- allowed_warnings
113-
- do:
114-
headers:
115-
Content-Type: 'application/json'
116-
sql:
117-
body:
118-
query: 'EXPLAIN SELECT message FROM sql_profile'
119-
profile: true
120-
- match: {profile: null}

0 commit comments

Comments
 (0)