chore: fix content type

Signed-off-by: Tokesh <tokesh789@gmail.com>
opensearch-project · Tokesh · Jan 7, 2025 · Jan 7, 2025 · Jan 7, 2025 · Jan 7, 2025
commit 14292d4216677f287c9ccae5b6821245dd417022
diff --git a/tools/src/tester/PostmanManager.ts b/tools/src/tester/PostmanManager.ts
@@ -1,10 +1,10 @@
 import fs from 'fs';

 export class PostmanManager {
  private readonly collection: any;
  private readonly collectionPath: string;

  constructor(collectionPath: string = './postman_collection.json') {
    this.collectionPath = collectionPath;
    this.collection = {
      info: {
@@ -15,49 +15,33 @@
    };
  }

  addToCollection(
    url: string | undefined,
    method: string,
    path: string,
    headers: Record<string, any> | undefined,
    params: Record<string, any>,
     body: any,
     contentType: string
   ): void {
-    const bodyContent = body
-      ? (() => {
-          switch (contentType) {
-            case 'application/json':
-              return { mode: 'raw', raw: JSON.stringify(body) };
-            case 'text/plain':
-              return { mode: 'raw', raw: body.toString() };
-            case 'application/x-www-form-urlencoded':
-              return {
-                mode: 'urlencoded',
-                urlencoded: Object.entries(body).map(([key, value]) => ({ key, value: value.toString() })),
-              };
-            default:
-              throw new Error(`Unsupported content type: ${contentType}`);
-          }
-        })()
-      : undefined;
-
     const item = {
       name: path,
       request: {
        method,
        header: Object.entries(headers || {}).map(([key, value]) => ({ key, value })),
         url: {
           raw: `${url}${path}`,
           path: path.split('/').filter(Boolean),
-          query: Object.entries(params).map(([key, value]) => ({ key, value: value.toString() })),
+          query: Object.entries(params).map(([key, value]) => ({ key, value: value as string })),
         },
-        body: bodyContent,
+        body: body
+          ? { mode: contentType === 'application/json' ? 'raw' : 'formdata', raw: JSON.stringify(body) }
+          : undefined,
       },
     };
-  
+
     this.collection.item.push(item);
-  }  
+  }
 
   saveCollection(): void {
     fs.writeFileSync(this.collectionPath, JSON.stringify(this.collection, null, 2));

diff --git a/tools/src/OpenSearchHttpClient.ts b/tools/src/OpenSearchHttpClient.ts
    }
  }

  getUrl(): string | undefined {
    if(this._opts != null && !this._opts.url) return this._opts.url ;
    return DEFAULT_URL;
  }

diff --git a/tools/src/tester/ChapterReader.ts b/tools/src/tester/ChapterReader.ts
  private readonly logger: Logger
  private readonly postmanManager: PostmanManager;

  constructor (client: OpenSearchHttpClient, logger: Logger, collectionPath: string = './postman_collection.json') {
    this._client = client
    this.logger = logger
    this.postmanManager = new PostmanManager(collectionPath);
	}
	}

	getUrl(): string \| undefined {
Check failure on line 224 in tools/src/OpenSearchHttpClient.ts GitHub Actions / lint Class Method name `getUrl` must match one of the following formats: snake_case
	if(this._opts != null && !this._opts.url) return this._opts.url ;
Check failure on line 225 in tools/src/OpenSearchHttpClient.ts GitHub Actions / lint `Expected space(s) after "if"` Check failure on line 225 in tools/src/OpenSearchHttpClient.ts GitHub Actions / lint `Unexpected nullable string value in conditional. Please handle the nullish/empty cases explicitly`
	return DEFAULT_URL;
	}
	private readonly logger: Logger
	private readonly postmanManager: PostmanManager;

	constructor (client: OpenSearchHttpClient, logger: Logger, collectionPath: string = './postman_collection.json') {
Check failure on line 28 in tools/src/tester/ChapterReader.ts GitHub Actions / lint Parameter name `collectionPath` must match one of the following formats: snake_case, UPPER_CASE
	this._client = client
	this.logger = logger
	this.postmanManager = new PostmanManager(collectionPath);