@testream/dotnet-reporter 0.4.7 → 0.5.2

package/README.md

@@ -3,12 +3,11 @@
 Run .NET tests and send the results from your codebase to Jira via Testream.
 
 - **Docs:** https://testream.github.io/docs/reporters/dotnet
-- **Docs repository:** https://github.com/testream/docs
 
 ## Quick Start
 
 ```bash
-npx @testream/dotnet-reporter -p PROJ -k $TESTREAM_API_KEY
+npx @testream/dotnet-reporter -k $TESTREAM_API_KEY
 ```
 
 This command runs your tests and uploads the results so they show up in Jira.

package/dist/index.d.ts

@@ -1,5 +1,4 @@
 export { parseTrxFile, parseTrxFiles } from './trx-parser';
-export { uploadToApi, detectGitContext, defaultLogger } from './uploader';
-export type { Logger, UploadToApiOptions } from './uploader';
+export { uploadToApi, detectGitContext } from './uploader';
 export type { DotnetReporterConfig } from './types';
-export type { CTRFReport, CTRFTest, CTRFSummary, CTRFResults, UploadResult, } from '@jira-test-manager/shared-types';
+export type { CTRFReport, CTRFTest, CTRFSummary, CTRFResults, UploadResult, UploadTestRunOptions, } from '@jira-test-manager/shared-types';

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"","sourceRoot":"","sources":["file:///home/runner/work/jira-test-manager/jira-test-manager/packages/dotnet-reporter/src/index.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,YAAY,EAAE,aAAa,EAAE,MAAM,cAAc,CAAC;AAC3D,OAAO,EAAE,WAAW,EAAE,gBAAgB,EAAE,aAAa,EAAE,MAAM,YAAY,CAAC;AAC1E,YAAY,EAAE,MAAM,EAAE,kBAAkB,EAAE,MAAM,YAAY,CAAC;AAC7D,YAAY,EAAE,oBAAoB,EAAE,MAAM,SAAS,CAAC;AACpD,YAAY,EACV,UAAU,EACV,QAAQ,EACR,WAAW,EACX,WAAW,EACX,YAAY,GACb,MAAM,iCAAiC,CAAC"}
+{"version":3,"file":"","sourceRoot":"","sources":["file:///home/runner/work/jira-test-manager/jira-test-manager/packages/dotnet-reporter/src/index.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,YAAY,EAAE,aAAa,EAAE,MAAM,cAAc,CAAC;AAC3D,OAAO,EAAE,WAAW,EAAE,gBAAgB,EAAE,MAAM,YAAY,CAAC;AAC3D,YAAY,EAAE,oBAAoB,EAAE,MAAM,SAAS,CAAC;AACpD,YAAY,EACV,UAAU,EACV,QAAQ,EACR,WAAW,EACX,WAAW,EACX,YAAY,EACZ,oBAAoB,GACrB,MAAM,iCAAiC,CAAC"}

package/dist/index.js

@@ -2275,15 +2275,194 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.detectCIContext = void 0;
+exports.uploadArtifacts = exports.uploadTestRun = exports.detectCIContext = void 0;
 // CTRF types
 __exportStar(__nccwpck_require__(827), exports);
 var ci_detection_1 = __nccwpck_require__(406);
 Object.defineProperty(exports, "detectCIContext", ({ enumerable: true, get: function () { return ci_detection_1.detectCIContext; } }));
+var upload_1 = __nccwpck_require__(969);
+Object.defineProperty(exports, "uploadTestRun", ({ enumerable: true, get: function () { return upload_1.uploadTestRun; } }));
+Object.defineProperty(exports, "uploadArtifacts", ({ enumerable: true, get: function () { return upload_1.uploadArtifacts; } }));
 //# sourceMappingURL=index.js.map
 
 /***/ }),
 
+/***/ 969:
+/***/ ((__unused_webpack_module, exports, __nccwpck_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.uploadTestRun = uploadTestRun;
+exports.uploadArtifacts = uploadArtifacts;
+const DEFAULT_API_URL = 'https://test-manager-backend.fly.dev';
+/**
+ * Upload test run to Testream backend API
+ */
+async function uploadTestRun(options) {
+    const { report, apiKey, apiUrl = DEFAULT_API_URL } = options;
+    // Ensure reportId exists
+    if (!report.reportId) {
+        report.reportId = crypto.randomUUID();
+    }
+    // Build type-safe IngestRequest payload
+    const ingestPayload = {
+        report,
+        reportId: report.reportId,
+        commitSha: options.commitSha,
+        branch: options.branch,
+        repositoryUrl: options.repositoryUrl,
+        buildName: options.buildName,
+        buildNumber: options.buildNumber,
+        buildUrl: options.buildUrl,
+        testEnvironment: options.testEnvironment,
+        appName: options.appName,
+        appVersion: options.appVersion,
+        testType: options.testType,
+    };
+    // Professional logging
+    console.log('Uploading test results...');
+    let response;
+    try {
+        response = await fetch(`${apiUrl}/api/v1/ingest`, {
+            method: 'POST',
+            headers: {
+                'X-API-KEY': apiKey,
+                'Content-Type': 'application/json',
+            },
+            body: JSON.stringify(ingestPayload),
+        });
+    }
+    catch (error) {
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        return {
+            success: false,
+            reportId: report.reportId,
+            error: `Connection failed: ${errorMessage}`,
+        };
+    }
+    // Handle 409 (report already exists) - idempotent
+    if (response.status === 409) {
+        console.warn('Report already exists (workflow may have been re-run)');
+        return {
+            success: true,
+            reportId: report.reportId,
+            summary: {
+                passed: report.results.summary.passed,
+                failed: report.results.summary.failed,
+                skipped: report.results.summary.skipped,
+                total: report.results.summary.tests,
+            },
+            alreadyExists: true,
+        };
+    }
+    // Handle other errors
+    if (!response.ok) {
+        const errorText = await response.text();
+        return {
+            success: false,
+            reportId: report.reportId,
+            error: `Upload failed (HTTP ${response.status}): ${errorText}`,
+        };
+    }
+    // Success
+    const result = (await response.json());
+    console.log(`✓ Test results uploaded successfully`);
+    console.log(`  Report ID: ${result.reportId}`);
+    console.log(`  Test Run ID: ${result.testRunId}`);
+    console.log(`  Tests: ${result.summary.passed}/${result.summary.total} passed`);
+    return {
+        success: true,
+        reportId: result.reportId,
+        testRunId: result.testRunId,
+        summary: {
+            passed: result.summary.passed,
+            failed: result.summary.failed,
+            skipped: result.summary.skipped,
+            total: result.summary.total,
+        },
+        testResults: result.testResults,
+    };
+}
+/**
+ * Upload test artifacts (screenshots, videos, etc.) to Testream backend API
+ */
+async function uploadArtifacts(options) {
+    const { reportId, apiKey, testResults, apiUrl = DEFAULT_API_URL } = options;
+    let uploadedCount = 0;
+    for (const testResult of testResults) {
+        for (const attachment of testResult.attachments) {
+            try {
+                const success = await uploadSingleArtifact({
+                    testResultId: testResult.testResultId,
+                    attachment,
+                    reportId,
+                    apiKey,
+                    apiUrl,
+                });
+                if (success)
+                    uploadedCount++;
+            }
+            catch (error) {
+                const errorMessage = error instanceof Error ? error.message : String(error);
+                console.error(`Failed to upload artifact ${attachment.name}: ${errorMessage}`);
+            }
+        }
+    }
+    if (uploadedCount > 0) {
+        console.log(`✓ Uploaded ${uploadedCount} artifact(s)`);
+    }
+    return uploadedCount;
+}
+async function uploadSingleArtifact(options) {
+    const { testResultId, attachment, reportId, apiKey, apiUrl } = options;
+    // Dynamically import fs for Node.js environments
+    const fs = await Promise.resolve(/* import() */).then(__nccwpck_require__.t.bind(__nccwpck_require__, 943, 23));
+    const path = await Promise.resolve(/* import() */).then(__nccwpck_require__.t.bind(__nccwpck_require__, 928, 23));
+    const filePath = path.resolve(attachment.path);
+    // Check if file exists
+    try {
+        await fs.access(filePath);
+    }
+    catch {
+        console.warn(`Artifact not found: ${path.basename(filePath)}`);
+        return false;
+    }
+    // Read file
+    let fileBuffer;
+    try {
+        fileBuffer = await fs.readFile(filePath);
+    }
+    catch (error) {
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        throw new Error(`Failed to read file: ${errorMessage}`);
+    }
+    // Create native FormData with Blob
+    const blob = new Blob([fileBuffer], { type: attachment.contentType });
+    const formData = new FormData();
+    formData.append('testResultId', testResultId);
+    formData.append('ctrfAttachmentName', attachment.name);
+    formData.append('file', blob, path.basename(filePath));
+    // Upload to API
+    const uploadUrl = `${apiUrl}/api/v1/artifacts/${reportId}`;
+    const response = await fetch(uploadUrl, {
+        method: 'POST',
+        headers: {
+            'X-API-KEY': apiKey,
+            // Don't set Content-Type - fetch sets it with boundary
+        },
+        body: formData,
+    });
+    if (!response.ok) {
+        const errorText = await response.text();
+        throw new Error(`HTTP ${response.status}: ${errorText}`);
+    }
+    return true;
+}
+//# sourceMappingURL=upload.js.map
+
+/***/ }),
+
 /***/ 328:
 /***/ (function(__unused_webpack_module, exports, __nccwpck_require__) {
 
@@ -2335,10 +2514,9 @@ function printUsage() {
 @testream/dotnet-reporter - Run .NET tests and upload results to Testream
 
 Usage:
-  npx @testream/dotnet-reporter --project-key <key> --api-key <key> [options]
+  npx @testream/dotnet-reporter --api-key <key> [options]
 
 Required arguments:
-  --project-key, -p     Project key
   --api-key, -k         API key for authentication
 
 Optional arguments:
@@ -2371,25 +2549,25 @@ Other:
 
 Examples:
   # Run tests and upload (simplest usage)
-  npx @testream/dotnet-reporter -p MYPROJ -k your-api-key
+  npx @testream/dotnet-reporter -k your-api-key
 
   # Run tests for a specific project
-  npx @testream/dotnet-reporter -p MYPROJ -k your-api-key --project ./MyTests
+  npx @testream/dotnet-reporter -k your-api-key --project ./MyTests
 
   # With environment metadata
-  npx @testream/dotnet-reporter -p MYPROJ -k your-api-key \\
+  npx @testream/dotnet-reporter -k your-api-key \\
     --test-environment staging --app-name MyApp --app-version 1.0.0
 
   # Pass additional arguments to dotnet test
-  npx @testream/dotnet-reporter -p MYPROJ -k your-api-key -- --filter "Category=Unit"
+  npx @testream/dotnet-reporter -k your-api-key -- --filter "Category=Unit"
 
   # Use existing TRX file (skip running tests)
-  npx @testream/dotnet-reporter -p MYPROJ -k your-api-key --trx-path TestResults/*.trx
+  npx @testream/dotnet-reporter -k your-api-key --trx-path TestResults/*.trx
 
 How it works:
   1. Runs 'dotnet test' with TRX logging enabled
   2. Parses the TRX output and converts to CTRF format
-  3. Uploads to Testream
+  3. Uploads to Testream (project key inferred from API key)
 
 No manual steps required - just run and upload!
 `);
@@ -2419,11 +2597,6 @@ function parseArgs(args) {
                 printUsage();
                 process.exit(0);
                 break;
-            case '--project-key':
-            case '-p':
-                options.projectKey = nextArg;
-                i++;
-                break;
             case '--api-key':
             case '-k':
                 options.apiKey = nextArg;
@@ -2493,13 +2666,8 @@ function parseArgs(args) {
         options.dotnetArgs = dotnetArgs;
     }
     // Validate required options
-    const missing = [];
-    if (!options.projectKey)
-        missing.push('--project-key');
-    if (!options.apiKey && options.uploadEnabled)
-        missing.push('--api-key');
-    if (missing.length > 0) {
-        console.error(`Error: Missing required arguments: ${missing.join(', ')}`);
+    if (!options.apiKey && options.uploadEnabled) {
+        console.error('Error: Missing required argument: --api-key');
         console.error('Run with --help for usage information.');
         return null;
     }
@@ -2521,7 +2689,7 @@ async function findTrxFiles(pattern) {
  * Run dotnet test and return the path to generated TRX file
  */
 async function runDotnetTest(projectPath, additionalArgs = []) {
-    const logger = uploader_1.defaultLogger;
+    const logger = console;
     // Create temp directory for TRX output
     const tempDir = await fs.mkdtemp(path.join(os.tmpdir(), 'dotnet-reporter-'));
     const trxFileName = `results-${Date.now()}.trx`;
@@ -2579,7 +2747,7 @@ async function main() {
     if (!options) {
         process.exit(1);
     }
-    const logger = uploader_1.defaultLogger;
+    const logger = console;
     try {
         let trxPath;
         // Either use provided TRX path or run dotnet test
@@ -2631,7 +2799,6 @@ async function main() {
         // Upload to API
         const result = await (0, uploader_1.uploadToApi)({
             report,
-            projectKey: options.projectKey,
             apiKey: options.apiKey,
             // Git context
             branch,
@@ -2646,14 +2813,13 @@ async function main() {
             appName: options.appName,
             appVersion: options.appVersion,
             testType: options.testType,
-            logger,
         });
         if (!result.success) {
             if (options.failOnUploadError) {
                 process.exit(1);
             }
             else {
-                logger.warning('Upload failed but continuing (use --fail-on-error to fail on upload errors)');
+                logger.warn('Upload failed but continuing (use --fail-on-error to fail on upload errors)');
             }
         }
     }
@@ -2981,173 +3147,27 @@ async function parseTrxFiles(trxPaths) {
 "use strict";
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.detectGitContext = exports.defaultLogger = void 0;
+exports.detectGitContext = void 0;
 exports.uploadToApi = uploadToApi;
 const shared_types_1 = __nccwpck_require__(322);
-const crypto_1 = __nccwpck_require__(982);
-/**
- * Hardcoded API URL for Testream backend
- */
-const API_URL = 'https://test-manager-backend.fly.dev';
-/**
- * Default console logger
- */
-exports.defaultLogger = {
-    info: (msg) => console.log(msg),
-    warning: (msg) => console.warn(msg),
-    error: (msg) => console.error(msg),
-};
 /**
  * Upload CTRF report to Testream API
  */
 async function uploadToApi(options) {
-    const { report, projectKey, apiKey, commitSha = '', branch = '', repositoryUrl = '', buildName, buildNumber, buildUrl, testEnvironment, appName, appVersion, testType, logger = exports.defaultLogger, } = options;
-    // Ensure reportId exists
-    if (!report.reportId) {
-        report.reportId = (0, crypto_1.randomUUID)();
-        logger.info(`Generated reportId: ${report.reportId}`);
-    }
-    try {
-        logger.info('='.repeat(60));
-        logger.info('Testream - .NET Test Results Upload');
-        logger.info('='.repeat(60));
-        logger.info(`Project Key: ${projectKey}`);
-        logger.info(`Test Tool: ${report.results.tool.name}`);
-        logger.info('');
-        logger.info(`Test Summary:`);
-        logger.info(`  Total: ${report.results.summary.tests}`);
-        logger.info(`  Passed: ${report.results.summary.passed}`);
-        logger.info(`  Failed: ${report.results.summary.failed}`);
-        logger.info(`  Skipped: ${report.results.summary.skipped}`);
-        logger.info('');
-        // Log git context if provided
-        if (branch || commitSha || repositoryUrl) {
-            logger.info(`Git Context:`);
-            if (branch)
-                logger.info(`  Branch: ${branch}`);
-            if (commitSha)
-                logger.info(`  Commit: ${commitSha.substring(0, 7)}`);
-            if (repositoryUrl)
-                logger.info(`  Repository: ${repositoryUrl}`);
-            logger.info('');
-        }
-        // Log build info if provided
-        if (buildName || buildNumber || buildUrl) {
-            logger.info(`Build Info:`);
-            if (buildName)
-                logger.info(`  Name: ${buildName}`);
-            if (buildNumber)
-                logger.info(`  Number: ${buildNumber}`);
-            if (buildUrl)
-                logger.info(`  URL: ${buildUrl}`);
-            logger.info('');
-        }
-        // Log environment info if provided
-        if (testEnvironment || appName || appVersion || testType) {
-            logger.info(`Environment:`);
-            if (testEnvironment)
-                logger.info(`  Environment: ${testEnvironment}`);
-            if (appName)
-                logger.info(`  App: ${appName}`);
-            if (appVersion)
-                logger.info(`  Version: ${appVersion}`);
-            if (testType)
-                logger.info(`  Test Type: ${testType}`);
-            logger.info('');
-        }
-        // Prepare payload with all metadata
-        const ingestPayload = {
-            report,
-            reportId: report.reportId,
-            projectKey,
-            // Git context
-            commitSha,
-            branch,
-            repositoryUrl,
-            // Build metadata
-            buildName,
-            buildNumber,
-            buildUrl,
-            // Environment metadata
-            testEnvironment,
-            appName,
-            appVersion,
-            testType,
-        };
-        // Upload to API
-        logger.info(`Uploading test results...`);
-        logger.info(`   Report ID: ${report.reportId}`);
-        let response;
-        try {
-            response = await fetch(`${API_URL}/api/v1/ingest`, {
-                method: 'POST',
-                headers: {
-                    'X-API-KEY': apiKey,
-                    'Content-Type': 'application/json',
-                },
-                body: JSON.stringify(ingestPayload),
-            });
-        }
-        catch (error) {
-            const errorMessage = error instanceof Error ? error.message : String(error);
-            throw new Error(`Failed to connect to API: ${errorMessage}`);
-        }
-        // Log response status for debugging
-        logger.info(`   Response status: ${response.status}`);
-        // Handle errors
-        if (!response.ok) {
-            const responseText = await response.text();
-            let errorData = null;
-            try {
-                errorData = JSON.parse(responseText);
-            }
-            catch {
-                // Not JSON
-            }
-            // Handle 409 (already exists)
-            if (response.status === 409) {
-                logger.warning(`Report already exists: ${errorData?.reportId || 'unknown'}`);
-                logger.info('This is expected if the workflow was re-run');
-                return {
-                    success: true,
-                    reportId: errorData?.reportId || report.reportId,
-                    summary: {
-                        passed: report.results.summary.passed,
-                        failed: report.results.summary.failed,
-                        skipped: report.results.summary.skipped,
-                        total: report.results.summary.tests,
-                    },
-                };
-            }
-            // Other errors
-            const errorMessage = errorData?.error
-                ? `${errorData.error}${errorData.details ? `: ${errorData.details}` : ''}`
-                : responseText || `HTTP ${response.status}`;
-            throw new Error(`API ingest failed (${response.status}): ${errorMessage}`);
-        }
-        const result = (await response.json());
-        logger.info('');
-        logger.info('='.repeat(60));
-        logger.info('Upload completed successfully');
-        logger.info(`  Report ID: ${result.reportId}`);
-        logger.info(`  Test Run ID: ${result.testRunId}`);
-        logger.info('='.repeat(60));
-        return {
-            success: true,
-            reportId: result.reportId,
-            testRunId: result.testRunId,
-            summary: result.summary,
-        };
-    }
-    catch (error) {
-        const errorMessage = error instanceof Error ? error.message : String(error);
-        logger.error(`Upload failed: ${errorMessage}`);
-        return {
-            success: false,
-            reportId: '',
-            error: errorMessage,
-        };
-    }
+    return (0, shared_types_1.uploadTestRun)({
+        report: options.report,
+        apiKey: options.apiKey,
+        commitSha: options.commitSha,
+        branch: options.branch,
+        repositoryUrl: options.repositoryUrl,
+        buildName: options.buildName,
+        buildNumber: options.buildNumber,
+        buildUrl: options.buildUrl,
+        testEnvironment: options.testEnvironment,
+        appName: options.appName,
+        appVersion: options.appVersion,
+        testType: options.testType,
+    });
 }
 // Keep old function name for backwards compatibility
 exports.detectGitContext = shared_types_1.detectCIContext;
@@ -3163,14 +3183,6 @@ module.exports = require("child_process");
 
 /***/ }),
 
-/***/ 982:
-/***/ ((module) => {
-
-"use strict";
-module.exports = require("crypto");
-
-/***/ }),
-
 /***/ 896:
 /***/ ((module) => {
 
@@ -11526,6 +11538,64 @@ exports.PathScurry = process.platform === 'win32' ? PathScurryWin32
 /******/ 	}
 /******/ 	
 /************************************************************************/
+/******/ 	/* webpack/runtime/create fake namespace object */
+/******/ 	(() => {
+/******/ 		var getProto = Object.getPrototypeOf ? (obj) => (Object.getPrototypeOf(obj)) : (obj) => (obj.__proto__);
+/******/ 		var leafPrototypes;
+/******/ 		// create a fake namespace object
+/******/ 		// mode & 1: value is a module id, require it
+/******/ 		// mode & 2: merge all properties of value into the ns
+/******/ 		// mode & 4: return value when already ns object
+/******/ 		// mode & 16: return value when it's Promise-like
+/******/ 		// mode & 8|1: behave like require
+/******/ 		__nccwpck_require__.t = function(value, mode) {
+/******/ 			if(mode & 1) value = this(value);
+/******/ 			if(mode & 8) return value;
+/******/ 			if(typeof value === 'object' && value) {
+/******/ 				if((mode & 4) && value.__esModule) return value;
+/******/ 				if((mode & 16) && typeof value.then === 'function') return value;
+/******/ 			}
+/******/ 			var ns = Object.create(null);
+/******/ 			__nccwpck_require__.r(ns);
+/******/ 			var def = {};
+/******/ 			leafPrototypes = leafPrototypes || [null, getProto({}), getProto([]), getProto(getProto)];
+/******/ 			for(var current = mode & 2 && value; typeof current == 'object' && !~leafPrototypes.indexOf(current); current = getProto(current)) {
+/******/ 				Object.getOwnPropertyNames(current).forEach((key) => (def[key] = () => (value[key])));
+/******/ 			}
+/******/ 			def['default'] = () => (value);
+/******/ 			__nccwpck_require__.d(ns, def);
+/******/ 			return ns;
+/******/ 		};
+/******/ 	})();
+/******/ 	
+/******/ 	/* webpack/runtime/define property getters */
+/******/ 	(() => {
+/******/ 		// define getter functions for harmony exports
+/******/ 		__nccwpck_require__.d = (exports, definition) => {
+/******/ 			for(var key in definition) {
+/******/ 				if(__nccwpck_require__.o(definition, key) && !__nccwpck_require__.o(exports, key)) {
+/******/ 					Object.defineProperty(exports, key, { enumerable: true, get: definition[key] });
+/******/ 				}
+/******/ 			}
+/******/ 		};
+/******/ 	})();
+/******/ 	
+/******/ 	/* webpack/runtime/hasOwnProperty shorthand */
+/******/ 	(() => {
+/******/ 		__nccwpck_require__.o = (obj, prop) => (Object.prototype.hasOwnProperty.call(obj, prop))
+/******/ 	})();
+/******/ 	
+/******/ 	/* webpack/runtime/make namespace object */
+/******/ 	(() => {
+/******/ 		// define __esModule on exports
+/******/ 		__nccwpck_require__.r = (exports) => {
+/******/ 			if(typeof Symbol !== 'undefined' && Symbol.toStringTag) {
+/******/ 				Object.defineProperty(exports, Symbol.toStringTag, { value: 'Module' });
+/******/ 			}
+/******/ 			Object.defineProperty(exports, '__esModule', { value: true });
+/******/ 		};
+/******/ 	})();
+/******/ 	
 /******/ 	/* webpack/runtime/compat */
 /******/ 	
 /******/ 	if (typeof __nccwpck_require__ !== 'undefined') __nccwpck_require__.ab = __dirname + "/";