@testream/dotnet-reporter 1.0.0 → 1.0.1

package/README.md

@@ -23,4 +23,4 @@ Run .NET tests and send the results from your codebase to Jira via Testream.
 
 This installs the reporter, runs your .NET tests, and uploads results to Testream.
 
-For CLI options and CI examples, see https://testream.github.io/docs/reporters/dotnet.
+For CLI options and CI examples, see https://docs.testream.app/reporters/dotnet.

package/dist/index.js

@@ -2299,6 +2299,7 @@ exports.ensureReportId = ensureReportId;
 exports.uploadTestRun = uploadTestRun;
 exports.mapAttachmentsToTestResults = mapAttachmentsToTestResults;
 exports.uploadArtifacts = uploadArtifacts;
+const ci_detection_1 = __nccwpck_require__(406);
 const DEFAULT_API_URL = 'https://test-manager-backend.fly.dev';
 function ensureReportId(report) {
     if (typeof report.reportId === 'string' && report.reportId.trim().length > 0) {
@@ -2335,6 +2336,52 @@ function normalizeApiUrl(value) {
     }
     return trimmed.replace(/\/+$/, '');
 }
+function isReportAlreadyExistsConflict(error) {
+    return error.errorCode === 'report_already_exists';
+}
+function formatUploadError(error) {
+    const detail = error.detail || error.title || error.rawBody || 'Unknown error';
+    const codeLabel = error.errorCode ? `, code=${error.errorCode}` : '';
+    return `Upload failed (HTTP ${error.statusCode}${codeLabel}): ${detail}`;
+}
+async function parseHttpError(response) {
+    const rawBody = await response.text();
+    const contentType = response.headers.get('content-type') || '';
+    if (contentType.toLowerCase().includes('json') && rawBody.trim().length > 0) {
+        try {
+            const parsed = JSON.parse(rawBody);
+            return {
+                statusCode: response.status,
+                title: parsed.title,
+                detail: parsed.detail || parsed.details || parsed.error,
+                errorCode: parsed.errorCode,
+                rawBody,
+            };
+        }
+        catch {
+            // Fall through to raw body handling
+        }
+    }
+    return {
+        statusCode: response.status,
+        rawBody,
+    };
+}
+function resolveUploadContext(options) {
+    const ciContext = (0, ci_detection_1.detectCIContext)();
+    return {
+        commitSha: options.commitSha ?? ciContext.commitSha,
+        branch: options.branch ?? ciContext.branch,
+        repositoryUrl: options.repositoryUrl ?? ciContext.repositoryUrl,
+        buildName: options.buildName,
+        buildNumber: options.buildNumber ?? ciContext.buildNumber,
+        buildUrl: options.buildUrl ?? ciContext.buildUrl,
+        testEnvironment: options.testEnvironment,
+        appName: options.appName,
+        appVersion: options.appVersion,
+        testType: options.testType,
+    };
+}
 /**
  * Upload test run to Testream backend API
  */
@@ -2342,20 +2389,12 @@ async function uploadTestRun(options) {
     const { report, apiKey } = options;
     const apiUrl = resolveApiUrl(options.apiUrl);
     const reportId = ensureReportId(report);
+    const uploadContext = resolveUploadContext(options);
     // Build type-safe IngestRequest payload
     const ingestPayload = {
         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,
+        ...uploadContext,
     };
     // Professional logging
     console.log('Uploading test results...');
@@ -2372,34 +2411,41 @@ async function uploadTestRun(options) {
     }
     catch (error) {
         const errorMessage = error instanceof Error ? error.message : String(error);
+        const connectionError = `Connection failed: ${errorMessage}`;
+        console.error(connectionError);
         return {
             success: false,
             reportId,
-            error: `Connection failed: ${errorMessage}`,
+            error: connectionError,
         };
     }
-    // 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,
-            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();
+        const parsedError = await parseHttpError(response);
+        if (response.status === 409 && isReportAlreadyExistsConflict(parsedError)) {
+            console.warn('Report already exists (workflow may have been re-run)');
+            return {
+                success: true,
+                reportId,
+                summary: {
+                    passed: report.results.summary.passed,
+                    failed: report.results.summary.failed,
+                    skipped: report.results.summary.skipped,
+                    total: report.results.summary.tests,
+                },
+                alreadyExists: true,
+                statusCode: parsedError.statusCode,
+                errorCode: parsedError.errorCode,
+            };
+        }
+        const formattedError = formatUploadError(parsedError);
+        console.error(formattedError);
         return {
             success: false,
             reportId,
-            error: `Upload failed (HTTP ${response.status}): ${errorText}`,
+            error: formattedError,
+            statusCode: parsedError.statusCode,
+            errorCode: parsedError.errorCode,
+            errorDetail: parsedError.detail || parsedError.rawBody,
         };
     }
     // Success
@@ -2958,26 +3004,18 @@ async function main() {
             logger.info('='.repeat(60));
             return;
         }
-        // Detect CI context from environment
-        const ciContext = (0, uploader_1.detectGitContext)();
-        // Merge CLI options with auto-detected CI context (CLI takes precedence)
-        const branch = options.branch || ciContext.branch;
-        const commitSha = options.commitSha || ciContext.commitSha;
-        const repositoryUrl = options.repositoryUrl || ciContext.repositoryUrl;
-        const buildNumber = options.buildNumber || ciContext.buildNumber;
-        const buildUrl = options.buildUrl || ciContext.buildUrl;
         // Upload to API
         const result = await (0, uploader_1.uploadToApi)({
             report,
             apiKey: options.apiKey,
             // Git context
-            branch,
-            commitSha,
-            repositoryUrl,
+            branch: options.branch,
+            commitSha: options.commitSha,
+            repositoryUrl: options.repositoryUrl,
             // Build metadata
             buildName: options.buildName,
-            buildNumber,
-            buildUrl,
+            buildNumber: options.buildNumber,
+            buildUrl: options.buildUrl,
             // Environment metadata
             testEnvironment: options.testEnvironment,
             appName: options.appName,