Skip to content
This repository has been archived by the owner on Dec 12, 2024. It is now read-only.

fix: Fix migr8 crashing on windows due to Unix path delimiters #160

fix: Fix migr8 crashing on windows due to Unix path delimiters

fix: Fix migr8 crashing on windows due to Unix path delimiters #160

Triggered via pull request May 15, 2024 11:11
Status Failure
Total duration 1m 25s
Artifacts

ci.yml

on: pull_request
Build & Test (Linux)
41s
Build & Test (Linux)
Build & Test (Windows)
1m 11s
Build & Test (Windows)
Pack & Upload Artifacts
0s
Pack & Upload Artifacts
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 2 warnings
src/index.test.js > index > works with an empty app & default options: src/index.test.js#L7
Error: wait for prompt, but proccess is terminate with error ❯ TestRunner.stdin node_modules/.pnpm/[email protected]/node_modules/clet/lib/runner.js:433:25 ❯ TestRunner._runChain node_modules/.pnpm/[email protected]/node_modules/clet/lib/runner.js:163:7 ❯ node_modules/.pnpm/[email protected]/node_modules/clet/lib/runner.js:199:7 ❯ initCwd node_modules/.pnpm/[email protected]/node_modules/clet/lib/runner.js:479:11 ❯ cleanup node_modules/.pnpm/[email protected]/node_modules/clet/lib/runner.js:172:9 ❯ src/index.test.js:7:5
src/transformers/addMigrationComments/addMigrationComments.test.js > transformers | tracingConfig > works with startTransaction: src/transformers/addMigrationComments/addMigrationComments.test.js#L39
AssertionError: expected 'import * as Sentry from \'@sentry/bro…' to deeply equal 'import * as Sentry from \'@sentry/bro…' - Expected + Received - import * as Sentry from '@sentry/browser'; - import { startTransaction } from '@sentry/browser'; - - function doSomething() { - // TODO(sentry): Use `startInactiveSpan()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md - startTransaction(); - } - - function doSomethingElse() { - // TODO(sentry): Use `startInactiveSpan()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md - const transaction = Sentry.startTransaction(); - transaction.finish(); - - const obj = { - // TODO(sentry): Use `startInactiveSpan()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md - transaction: Sentry.startTransaction(), - }; - } + import * as Sentry from '@sentry/browser'; + import { startTransaction } from '@sentry/browser'; + + function doSomething() { + // TODO(sentry): Use `startInactiveSpan()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md + startTransaction(); + } + + function doSomethingElse() { + // TODO(sentry): Use `startInactiveSpan()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md + const transaction = Sentry.startTransaction(); + transaction.finish(); + + const obj = { + // TODO(sentry): Use `startInactiveSpan()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md + transaction: Sentry.startTransaction(), + }; + } ❯ src/transformers/addMigrationComments/addMigrationComments.test.js:39:20
src/transformers/addMigrationComments/addMigrationComments.test.js > transformers | tracingConfig > works with TS: src/transformers/addMigrationComments/addMigrationComments.test.js#L70
AssertionError: expected 'function indexMembersByProject(member…' to deeply equal 'function indexMembersByProject(member…' - Expected + Received - function indexMembersByProject(members: Member[]): IndexedMembersByProject { - return members.reduce<IndexedMembersByProject>((acc, member) => { - for (const project of member.projects) { - if (!acc.hasOwnProperty(project)) { - acc[project] = []; - } - if (member.user) { - acc[project].push(member.user); - } - } - return acc; - }, {}); - } + function indexMembersByProject(members: Member[]): IndexedMembersByProject { + return members.reduce<IndexedMembersByProject>((acc, member) => { + for (const project of member.projects) { + if (!acc.hasOwnProperty(project)) { + acc[project] = []; + } + if (member.user) { + acc[project].push(member.user); + } + } + return acc; + }, {}); + } ❯ src/transformers/addMigrationComments/addMigrationComments.test.js:70:20
src/transformers/addMigrationComments/addMigrationComments.test.js > transformers | tracingConfig > works with span.startChild: src/transformers/addMigrationComments/addMigrationComments.test.js#L96
AssertionError: expected '\r\nfunction doSomething(span) {\r\n …' to deeply equal '\nfunction doSomething(span) {\n // …' - Expected + Received - - function doSomething(span) { - // TODO(sentry): Use `startInactiveSpan()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md - span.startChild(); - } - - function doSomethingElse() { - const transaction = Sentry.getActiveSpan(); - // TODO(sentry): Use `startInactiveSpan()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md - transaction.startChild().end(); - transaction.finish(); - } + + function doSomething(span) { + // TODO(sentry): Use `startInactiveSpan()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md + span.startChild(); + } + + function doSomethingElse() { + const transaction = Sentry.getActiveSpan(); + // TODO(sentry): Use `startInactiveSpan()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md + transaction.startChild().end(); + transaction.finish(); + } ❯ src/transformers/addMigrationComments/addMigrationComments.test.js:96:20
src/transformers/addMigrationComments/addMigrationComments.test.js > transformers | tracingConfig > works with hub: src/transformers/addMigrationComments/addMigrationComments.test.js#L121
AssertionError: expected 'import * as Sentry from \'@sentry/bro…' to deeply equal 'import * as Sentry from \'@sentry/bro…' - Expected + Received - import * as Sentry from '@sentry/browser'; - import { makeMain, Hub } from '@sentry/browser'; - - // TODO(sentry): Use `new Scope()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-initializing.md - const hub = new Hub(); - // TODO(sentry): Use `setCurrentClient()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-initializing.md - makeMain(hub); - - function doSomethingElse() { - // TODO(sentry): Use `new Scope()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-initializing.md - const hub = new Sentry.Hub(); - // TODO(sentry): Use `setCurrentClient()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-initializing.md - Sentry.makeMain(hub); - } + import * as Sentry from '@sentry/browser'; + import { makeMain, Hub } from '@sentry/browser'; + + // TODO(sentry): Use `new Scope()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-initializing.md + const hub = new Hub(); + // TODO(sentry): Use `setCurrentClient()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-initializing.md + makeMain(hub); + + function doSomethingElse() { + // TODO(sentry): Use `new Scope()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-initializing.md + const hub = new Sentry.Hub(); + // TODO(sentry): Use `setCurrentClient()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-initializing.md + Sentry.makeMain(hub); + } ❯ src/transformers/addMigrationComments/addMigrationComments.test.js:121:20
src/transformers/addMigrationComments/addMigrationComments.test.js > transformers | tracingConfig > works with getActiveTransaction: src/transformers/addMigrationComments/addMigrationComments.test.js#L148
AssertionError: expected 'import * as Sentry from \'@sentry/bro…' to deeply equal 'import * as Sentry from \'@sentry/bro…' - Expected + Received - import * as Sentry from '@sentry/browser'; - import { getActiveTransaction } from '@sentry/browser'; - - function doSomething() { - // TODO(sentry): Use `getActiveSpan()` instead. If you use this only to start a child, use `startInactiveSpan({ onlyIfParent: true })` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md - getActiveTransaction(); - } - - function doSomethingElse() { - // TODO(sentry): Use `getActiveSpan()` instead. If you use this only to start a child, use `startInactiveSpan({ onlyIfParent: true })` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md - const transaction = Sentry.getActiveTransaction(); - transaction.finish(); - - const obj = { - // TODO(sentry): Use `getActiveSpan()` instead. If you use this only to start a child, use `startInactiveSpan({ onlyIfParent: true })` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md - transaction: Sentry.getActiveTransaction(), - }; - } + import * as Sentry from '@sentry/browser'; + import { getActiveTransaction } from '@sentry/browser'; + + function doSomething() { + // TODO(sentry): Use `getActiveSpan()` instead. If you use this only to start a child, use `startInactiveSpan({ onlyIfParent: true })` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md + getActiveTransaction(); + } + + function doSomethingElse() { + // TODO(sentry): Use `getActiveSpan()` instead. If you use this only to start a child, use `startInactiveSpan({ onlyIfParent: true })` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md + const transaction = Sentry.getActiveTransaction(); + transaction.finish(); + + const obj = { + // TODO(sentry): Use `getActiveSpan()` instead. If you use this only to start a child, use `startInactiveSpan({ onlyIfParent: true })` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md + transaction: Sentry.getActiveTransaction(), + }; + } ❯ src/transformers/addMigrationComments/addMigrationComments.test.js:148:20
src/transformers/addMigrationComments/addMigrationComments.test.js > transformers | tracingConfig > avoids adding duplicate comments: src/transformers/addMigrationComments/addMigrationComments.test.js#L179
AssertionError: expected 'import * as Sentry from \'@sentry/bro…' to deeply equal 'import * as Sentry from \'@sentry/bro…' - Expected + Received - import * as Sentry from '@sentry/browser'; - import { startTransaction } from '@sentry/browser'; - - function doSomething() { - // TODO(sentry): Use `startInactiveSpan()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md - startTransaction(); - } - - function doSomethingElse() { - // TODO(sentry): Use `startInactiveSpan()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md - const transaction = Sentry.startTransaction(); - transaction.finish(); - - const obj = { - // TODO(sentry): Use `startInactiveSpan()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md - transaction: Sentry.startTransaction(), - }; - } + import * as Sentry from '@sentry/browser'; + import { startTransaction } from '@sentry/browser'; + + function doSomething() { + // TODO(sentry): Use `startInactiveSpan()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md + startTransaction(); + } + + function doSomethingElse() { + // TODO(sentry): Use `startInactiveSpan()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md + const transaction = Sentry.startTransaction(); + transaction.finish(); + + const obj = { + // TODO(sentry): Use `startInactiveSpan()` instead - see https://github.com/getsentry/sentry-javascript/blob/develop/docs/v8-new-performance-apis.md + transaction: Sentry.startTransaction(), + }; + } ❯ src/transformers/addMigrationComments/addMigrationComments.test.js:179:20
src/transformers/configureScope/configureScope.test.js > transformers | configureScope > works with example files: src/transformers/configureScope/configureScope.test.js#L40
AssertionError: expected 'import { getCurrentScope } from \'@se…' to deeply equal 'import { getCurrentScope } from \'@se…' - Expected + Received - import { getCurrentScope } from '@sentry/browser'; - import * as Sentry from '@sentry/browser'; - - function orig() { - // do something - } - - function doSomething() { - getCurrentScope().setTag('aaa', 'aaa'); - Sentry.getCurrentScope().setTag('ccc', 'ccc'); - - { - const scope = getCurrentScope(); - scope.setTag('aaa', 'aaa'); - scope.setExtra('bbb', { bbb: 'bbb' }); - }; - - { - const scope = getCurrentScope(); - scope.setTag('aaa', 'aaa'); - scope.setExtra('bbb', { bbb: 'bbb' }); - }; - - configureScope(orig); - - { - const scope = Sentry.getCurrentScope(); - scope.setTag('ccc', 'ccc'); - scope.setExtra('ddd', { ddd: 'ddd' }); - }; - - getCurrentScope().addAttachment({ filename: 'scope.file', data: 'great content!' }); - Sentry.getCurrentScope().addAttachment({ filename: 'scope.file', data: 'great content!' }); - } + import { getCurrentScope } from '@sentry/browser'; + import * as Sentry from '@sentry/browser'; + + function orig() { + // do something + } + + function doSomething() { + getCurrentScope().setTag('aaa', 'aaa'); + Sentry.getCurrentScope().setTag('ccc', 'ccc'); + + { + const scope = getCurrentScope(); + scope.setTag('aaa', 'aaa'); + scope.setExtra('bbb', { bbb: 'bbb' }); + }; + + { + const scope = getCurrentScope(); + scope.setTag('aaa', 'aaa'); + scope.setExtra('bbb', { bbb: 'bbb' }); + }; + + configureScope(orig); + + { + const scope = Sentry.getCurrentScope(); + scope.setTag('ccc', 'ccc'); + scope.setExtra('ddd', { ddd: 'ddd' }); + }; + + getCurrentScope().addAttachment({ filename: 'scope.file', data: 'great content!' }); + Sentry.getCurrentScope().addAttachment({ filename: 'scope.file', data: 'great content!' }); + } ❯ src/transformers/configureScope/configureScope.test.js:40:25
src/transformers/convertEnums/convertEnums.test.js > transformers | utilExports > works with imports: src/transformers/convertEnums/convertEnums.test.js#L39
AssertionError: expected 'import { other } from \'@sentry/brows…' to deeply equal 'import { other } from \'@sentry/brows…' - Expected + Received - import { other } from '@sentry/browser'; - import * as Sentry from '@sentry/browser'; - - function doSomething() { - const a = "fatal"; - const b = "unknown_error"; - const c = "debug"; - const d = "failed_precondition"; - const x = other(); - } + import { other } from '@sentry/browser'; + import * as Sentry from '@sentry/browser'; + + function doSomething() { + const a = "fatal"; + const b = "unknown_error"; + const c = "debug"; + const d = "failed_precondition"; + const x = other(); + } ❯ src/transformers/convertEnums/convertEnums.test.js:39:20
src/transformers/convertEnums/convertEnums.test.js > transformers | utilExports > works with typescript: src/transformers/convertEnums/convertEnums.test.js#L62
AssertionError: expected 'import { other } from \'@sentry/brows…' to deeply equal 'import { other } from \'@sentry/brows…' - Expected + Received - import { other } from '@sentry/browser'; - import * as Sentry from '@sentry/browser'; - - function doSomething() { - const a = "fatal"; - const b = "unknown_error"; - const c = "debug"; - const d = "failed_precondition"; - const x = other(); - } - - enum SpanStatus { - Fatal = 'fatal', - UnknownError = 'unknown_error', - } + import { other } from '@sentry/browser'; + import * as Sentry from '@sentry/browser'; + + function doSomething() { + const a = "fatal"; + const b = "unknown_error"; + const c = "debug"; + const d = "failed_precondition"; + const x = other(); + } + + enum SpanStatus { + Fatal = 'fatal', + UnknownError = 'unknown_error', + } ❯ src/transformers/convertEnums/convertEnums.test.js:62:20
Build & Test (Linux)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, pnpm/action-setup@v2, actions/setup-node@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Build & Test (Windows)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, pnpm/action-setup@v2, actions/setup-node@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.