@testomatio/mcp 1.0.2 → 1.0.3

package/index.js

@@ -45,13 +45,13 @@ class TestomatioMCPServer {
     }
 
     const data = await response.json();
-    
+
     if (!data.jwt) {
       throw new Error('Authentication failed: No JWT token received in response');
     }
 
     this.jwtToken = data.jwt;
-    
+
     return this.jwtToken;
   }
 
@@ -433,9 +433,9 @@ class TestomatioMCPServer {
   async makeRequest(path, params = {}) {
     // Ensure we have a valid JWT token
     const jwt = await this.authenticate();
-    
+
     const url = new URL(`${this.config.baseUrl}/api/${this.config.projectId}${path}`);
-    
+
     // Add query parameters with proper array handling
     Object.entries(params).forEach(([key, value]) => {
       if (value !== undefined && value !== null) {
@@ -486,7 +486,7 @@ class TestomatioMCPServer {
         this.jwtToken = null;
         return this.makePostRequest(path, data);
       }
-      throw new Error(`HTTP ${response.status}: ${response.statusText}`);
+      throw new Error(`HTTP ${response.status}: ${response.statusText}; ${await response.text()}`);
     }
 
     return await response.json();
@@ -580,10 +580,10 @@ class TestomatioMCPServer {
   formatModel(model, tagName, fields) {
     const attributes = model.attributes || {};
     const lines = [`<${tagName}>`];
-    
+
     // Always include ID from root level
     lines.push(`  <id>${model.id || ''}</id>`);
-    
+
     // Process specified fields
     fields.forEach(field => {
       let value;
@@ -599,7 +599,7 @@ class TestomatioMCPServer {
       }
 
       const formattedValue = this.formatValue(value, field);
-      
+
       if (field === 'test' && typeof value === 'object') {
         // Special case for nested test objects
         lines.push(`  <test>${formattedValue}\n  </test>`);
@@ -607,7 +607,7 @@ class TestomatioMCPServer {
         lines.push(`  <${xmlFieldName}>${formattedValue}</${xmlFieldName}>`);
       }
     });
-    
+
     lines.push(`</${tagName}>`);
     return lines.join('\n');
   }
@@ -615,13 +615,13 @@ class TestomatioMCPServer {
   async getTests(filters = {}) {
     const params = this.buildSearchParams(filters);
     const data = await this.makeRequest('/tests', params);
-    const formattedTests = data.data.map(test => 
+    const formattedTests = data.data.map(test =>
       this.formatModel(test, 'test', [
-        'title', 'description', 'code', 'priority', 
+        'title', 'description', 'code', 'priority',
         'state', 'suite-id', 'tags', 'file'
       ])
     ).join('\n\n');
-    
+
     return {
       content: [
         {
@@ -634,7 +634,7 @@ class TestomatioMCPServer {
 
   buildSearchParams(filters = {}) {
     const params = {};
-    
+
     // Handle basic filters
     Object.entries(filters).forEach(([key, value]) => {
       if (value !== undefined && value !== null) {
@@ -680,16 +680,16 @@ class TestomatioMCPServer {
   async searchTests(filters = {}) {
     const params = this.buildSearchParams(filters);
     const data = await this.makeRequest('/tests', params);
-    
-    const formattedTests = data.data.map(test => 
+
+    const formattedTests = data.data.map(test =>
       this.formatModel(test, 'test', [
-        'title', 'description', 'code', 'priority', 
+        'title', 'description', 'code', 'priority',
         'state', 'suite-id', 'tags', 'file'
       ])
     ).join('\n\n');
-    
+
     const searchDescription = this.buildSearchDescription(filters);
-    
+
     return {
       content: [
         {
@@ -704,15 +704,15 @@ class TestomatioMCPServer {
     // Add filter=true for suites search to include tests
     const params = this.buildSearchParams({ ...filters, filter: true });
     const data = await this.makeRequest('/suites', params);
-    
-    const formattedSuites = data.data.map(suite => 
+
+    const formattedSuites = data.data.map(suite =>
       this.formatModel(suite, 'suite', [
         'title', 'description', 'test-count', 'is-root', 'file-type'
       ])
     ).join('\n\n');
-    
+
     const searchDescription = this.buildSearchDescription(filters);
-    
+
     return {
       content: [
         {
@@ -725,7 +725,7 @@ class TestomatioMCPServer {
 
   buildSearchDescription(filters) {
     const descriptions = [];
-    
+
     if (filters.query) {
       if (filters.query.startsWith('@')) {
         descriptions.push(`tagged with "${filters.query}"`);
@@ -735,41 +735,41 @@ class TestomatioMCPServer {
         descriptions.push(`containing "${filters.query}"`);
       }
     }
-    
+
     if (filters.tql) {
       descriptions.push(`matching TQL: "${filters.tql}"`);
     }
-    
+
     if (filters.labels && filters.labels.length > 0) {
       descriptions.push(`with labels: ${filters.labels.join(', ')}`);
     }
-    
+
     if (filters.state) {
       descriptions.push(`state: ${filters.state}`);
     }
-    
+
     if (filters.priority) {
       descriptions.push(`priority: ${filters.priority}`);
     }
-    
+
     if (filters.filter && typeof filters.filter === 'object') {
       const filterDesc = Object.entries(filters.filter)
         .map(([key, value]) => `${key}: ${value}`)
         .join(', ');
       descriptions.push(`filtered by: ${filterDesc}`);
     }
-    
+
     return descriptions.length > 0 ? ` (${descriptions.join(', ')})` : '';
   }
 
   async getRootSuites() {
     const data = await this.makeRequest('/suites');
-    const formattedSuites = data.data.map(suite => 
+    const formattedSuites = data.data.map(suite =>
       this.formatModel(suite, 'suite', [
         'title', 'description', 'test-count', 'is-root', 'file-type'
       ])
     ).join('\n\n');
-    
+
     return {
       content: [
         {
@@ -785,7 +785,7 @@ class TestomatioMCPServer {
     const formattedSuite = this.formatModel(data.data, 'suite', [
       'title', 'description', 'test-count', 'is-root', 'file-type'
     ]);
-    
+
     // Format child suites and tests if they exist
     let childContent = '';
     if (data.data.relationships?.children?.data) {
@@ -797,18 +797,18 @@ class TestomatioMCPServer {
         childContent += `\n\nChild Suites:\n${childSuites}`;
       }
     }
-    
+
     if (data.data.relationships?.tests?.data) {
       const tests = data.data.relationships.tests.data
         .map(test => this.formatModel(test, 'test', [
-          'title', 'description', 'code', 'priority', 
+          'title', 'description', 'code', 'priority',
           'state', 'suite-id', 'tags', 'file'
         ])).join('\n\n');
       if (tests) {
         childContent += `\n\nTests:\n${tests}`;
       }
     }
-    
+
     return {
       content: [
         {
@@ -821,13 +821,13 @@ class TestomatioMCPServer {
 
   async getRuns() {
     const data = await this.makeRequest('/runs');
-    const formattedRuns = data.data.map(run => 
+    const formattedRuns = data.data.map(run =>
       this.formatModel(run, 'run', [
         'status', 'title', 'tests-count', 'automated', 'duration',
         'passed', 'failed', 'skipped', 'created-at', 'finished-at'
       ])
     ).join('\n\n');
-    
+
     return {
       content: [
         {
@@ -845,7 +845,7 @@ class TestomatioMCPServer {
       'status', 'title', 'tests-count', 'automated', 'duration',
       'passed', 'failed', 'skipped', 'created-at', 'finished-at'
     ]);
-    
+
     return {
       content: [
         {
@@ -861,14 +861,14 @@ class TestomatioMCPServer {
     if (dateRange) {
       params.finished_at_date_range = dateRange;
     }
-    
+
     const data = await this.makeRequest('/testruns', params);
-    const formattedTestruns = data.data.map(testrun => 
+    const formattedTestruns = data.data.map(testrun =>
       this.formatModel(testrun, 'testrun', [
         'status', 'run-time', 'message', 'run-id', 'test'
       ])
     ).join('\n\n');
-    
+
     return {
       content: [
         {
@@ -881,12 +881,12 @@ class TestomatioMCPServer {
 
   async getPlans(filters = {}) {
     const data = await this.makeRequest('/plans', filters);
-    const formattedPlans = data.data.map(plan => 
+    const formattedPlans = data.data.map(plan =>
       this.formatModel(plan, 'plan', [
         'title', 'test-count', 'kind', 'created-at', 'tests-ids', 'labels'
       ])
     ).join('\n\n');
-    
+
     return {
       content: [
         {
@@ -902,7 +902,7 @@ class TestomatioMCPServer {
     const formattedPlan = this.formatModel(data.data, 'plan', [
       'title', 'test-count', 'kind', 'created-at', 'tests-ids', 'labels'
     ]);
-    
+
     return {
       content: [
         {
@@ -914,19 +914,33 @@ class TestomatioMCPServer {
   }
 
   async createTest(args) {
-    const { test_id, labels_ids, ...attributes } = args;
+    const { suite_id, labels_ids, ...attributes } = args;
     const requestData = {
-      type: 'test',
-      attributes: Object.fromEntries(Object.entries(attributes).map(([k, v]) => [k.replace(/_/g, '-'), v])),
+      data: {
+        type: 'tests',
+        attributes: Object.fromEntries(Object.entries(attributes).map(([k, v]) => [k.replace(/_/g, '-'), v]))
+      }
     };
+    
+    if (suite_id) {
+      requestData.data.relationships = {
+        suite: {
+          data: {
+            type: 'suites',
+            id: suite_id
+          }
+        }
+      };
+    }
+    
     if (labels_ids) requestData.labels_ids = labels_ids;
 
     const data = await this.makePostRequest('/tests', requestData);
     const formattedTest = this.formatModel(data.data, 'test', [
-      'title', 'description', 'code', 'priority', 
+      'title', 'description', 'code', 'priority',
       'state', 'suite-id', 'tags', 'file'
     ]);
-    
+
     return {
       content: [
         {
@@ -938,19 +952,33 @@ class TestomatioMCPServer {
   }
 
   async updateTest(args) {
-    const { test_id, labels_ids, ...attributes } = args;
+    const { test_id, suite_id, labels_ids, ...attributes } = args;
     const requestData = {
-      type: 'test',
-      attributes: Object.fromEntries(Object.entries(attributes).map(([k, v]) => [k.replace(/_/g, '-'), v])),
+      data: {
+        type: 'tests',
+        attributes: Object.fromEntries(Object.entries(attributes).map(([k, v]) => [k.replace(/_/g, '-'), v]))
+      }
     };
+    
+    if (suite_id) {
+      requestData.data.relationships = {
+        suite: {
+          data: {
+            type: 'suites',
+            id: suite_id
+          }
+        }
+      };
+    }
+    
     if (labels_ids) requestData.labels_ids = labels_ids;
 
-    const data = await this.makePutRequest(`/tests/${args.test_id}`, requestData);
+    const data = await this.makePutRequest(`/tests/${test_id}`, requestData);
     const formattedTest = this.formatModel(data.data, 'test', [
-      'title', 'description', 'code', 'priority', 
+      'title', 'description', 'code', 'priority',
       'state', 'suite-id', 'tags', 'file'
     ]);
-    
+
     return {
       content: [
         {
@@ -989,7 +1017,7 @@ function parseArgs() {
     .parse();
 
   const options = program.opts();
-  
+
   const token = options.token || process.env.TESTOMATIO_API_TOKEN;
   const projectId = options.project;
   const baseUrl = options.baseUrl || process.env.TESTOMATIO_BASE_URL || 'https://app.testomat.io';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testomatio/mcp",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "Model Context Protocol server for Testomatio API",
   "main": "index.js",
   "bin": {