Fixed content-type of GET /_plugins/_observability/_local/stats.

Added missing observability object tests. Signed-off-by: dblock <[email protected]>
opensearch-project · Dec 9, 2024 · f07fea9 · f07fea9
1 parent 87847db
commit f07fea9
Show file tree

Hide file tree

Showing 4 changed files with 29 additions and 28 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -54,6 +54,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 - Added missing `repository` query parameter to `/_cat/snapshots` ([#700](https://github.com/opensearch-project/opensearch-api-specification/pull/700))
 - Fixed `hits` in `rank_eval` allowing numbers ([#704](https://github.com/opensearch-project/opensearch-api-specification/pull/704))
 - Fixed query DSL schemas ([#706](https://github.com/opensearch-project/opensearch-api-specification/pull/706))
+- Fixed content-type of `GET /_plugins/_observability/_local/stats` ([#711](https://github.com/opensearch-project/opensearch-api-specification/pull/711))
 
 ### Changed
 - Changed `tasks._common:TaskInfo` and `tasks._common:TaskGroup` to be composed of a `tasks._common:TaskInfoBase` ([#683](https://github.com/opensearch-project/opensearch-api-specification/pull/683))

diff --git a/spec/namespaces/observability.yaml b/spec/namespaces/observability.yaml
@@ -8,8 +8,8 @@ paths:
     get:
       operationId: observability.get_localstats.0
       x-operation-group: observability.get_localstats
-      x-version-added: '1.1'
-      description: Retrieves Local Stats of all observability objects.
+      x-version-added: '2.5'
+      description: Retrieves local stats of all observability objects.
       responses:
         '200':
           $ref: '#/components/responses/observability.get_localstats@200'
@@ -177,7 +177,7 @@ components:
     observability.get_localstats@200:
       description: Retrieves
       content:
-        application/json:
+        text/plain:
           schema:
             type: string
   parameters:

diff --git a/tests/default/observability/local/stats.yaml b/tests/default/observability/local/stats.yaml
@@ -0,0 +1,12 @@
+$schema: ../../../../json_schemas/test_story.schema.yaml
+
+description: Test backend stats.
+version: '>= 2.5'
+
+chapters:
+  - synopsis: Get observability stats.
+    path: /_plugins/_observability/_local/stats
+    method: GET
+    response:
+      status: 200
+      content_type: text/plain
diff --git a/.../default/observability/observability.yaml → tests/default/observability/object.yaml b/.../default/observability/observability.yaml → tests/default/observability/object.yaml
@@ -2,13 +2,9 @@ $schema: ../../../json_schemas/test_story.schema.yaml
 
 description: Test various operations of the OpenSearch Observability Object API.
 
-prologues:
-  - path: /_plugins/_observability/object/{object_id}
-    method: DELETE
-    parameters:
-      object_id: test_object
-    status: [200, 404]
-  - path: /_plugins/_observability/object
+chapters:
+  - synopsis: Create an observability object.
+    path: /_plugins/_observability/object
     method: POST
     request:
       payload:
@@ -56,12 +52,11 @@ prologues:
             tokens:
               - name: field1
                 type: text
-    status: [200]
   - path: /_refresh
     method: POST
-    status: [200]
-chapters:
-  - synopsis: Retrieve specific Observability object after creation.
+    warnings:
+      multiple-paths-detected: false
+  - synopsis: Retrieve an observability object.
     path: /_plugins/_observability/object/{object_id}
     id: observatory_object
     method: GET
@@ -74,7 +69,7 @@ chapters:
         totalHits: 1
         totalHitRelation: eq
         observabilityObjectList: []
-  - synopsis: Update specific Observability object.
+  - synopsis: Update an observability object.
     path: /_plugins/_observability/object/{object_id}
     method: PUT
     parameters:
@@ -128,19 +123,7 @@ chapters:
       status: 200
       payload:
         objectId: test_object
-  - synopsis: Retrieve specific Observability object after update.
-    path: /_plugins/_observability/object/{object_id}
-    method: GET
-    parameters:
-      object_id: test_object
-    response:
-      status: 200
-      payload:
-        startIndex: 0
-        totalHits: 1
-        totalHitRelation: eq
-        observabilityObjectList: []
-  - synopsis: Retrieve list of Observability objects.
+  - synopsis: Retrieve all observability objects.
     path: /_plugins/_observability/object
     method: GET
     response:
@@ -150,6 +133,11 @@ chapters:
         totalHits: 1
         totalHitRelation: eq
         observabilityObjectList: []
+  - synopsis: Delete an observability object.
+    path: /_plugins/_observability/object/{object_id}
+    method: DELETE
+    parameters:
+      object_id: test_object
 epilogues:
   - path: /_plugins/_observability/object/{object_id}
     method: DELETE