Skip to content

Commit 8cf16dc

Browse files
committed
fix some test comments
1 parent 8111d32 commit 8cf16dc

File tree

4 files changed

+3
-11
lines changed

4 files changed

+3
-11
lines changed

dev-packages/node-integration-tests/suites/tracing/nestjs-errors-no-express/scenario.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
// eslint-disable-next-line @typescript-eslint/ban-ts-comment
2-
// @ts-nocheck These are only tests
31
/* eslint-disable @typescript-eslint/naming-convention */
42
/* eslint-disable @typescript-eslint/explicit-member-accessibility */
53
import { loggingTransport, sendPortToRunner } from '@sentry-internal/node-integration-tests';
@@ -35,7 +33,7 @@ class AppController {
3533
constructor(private readonly appService: AppService) {}
3634

3735
@Get('test-exception/:id')
38-
async testException(@Param('id') id: string): void {
36+
async testException(@Param('id') id: string): Promise<void> {
3937
Sentry.captureException(new Error(`error with id ${id}`));
4038
}
4139
}

dev-packages/node-integration-tests/suites/tracing/nestjs-errors/scenario.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
// eslint-disable-next-line @typescript-eslint/ban-ts-comment
2-
// @ts-nocheck These are only tests
31
/* eslint-disable @typescript-eslint/naming-convention */
42
/* eslint-disable @typescript-eslint/explicit-member-accessibility */
53
import { loggingTransport, sendPortToRunner } from '@sentry-internal/node-integration-tests';
@@ -33,7 +31,7 @@ class AppController {
3331
constructor(private readonly appService: AppService) {}
3432

3533
@Get('test-exception/:id')
36-
async testException(@Param('id') id: string): void {
34+
async testException(@Param('id') id: string): Promise<void> {
3735
Sentry.captureException(new Error(`error with id ${id}`));
3836
}
3937
}

dev-packages/node-integration-tests/suites/tracing/nestjs-no-express/scenario.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
// eslint-disable-next-line @typescript-eslint/ban-ts-comment
2-
// @ts-nocheck These are only tests
31
/* eslint-disable @typescript-eslint/naming-convention */
42
/* eslint-disable @typescript-eslint/explicit-member-accessibility */
53
import { loggingTransport, sendPortToRunner } from '@sentry-internal/node-integration-tests';
@@ -35,7 +33,7 @@ class AppController {
3533
constructor(private readonly appService: AppService) {}
3634

3735
@Get('test-exception/:id')
38-
async testException(@Param('id') id: string): void {
36+
async testException(@Param('id') id: string): Promise<void> {
3937
Sentry.captureException(new Error(`error with id ${id}`));
4038
}
4139
}

dev-packages/node-integration-tests/suites/tracing/nestjs/scenario.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
// eslint-disable-next-line @typescript-eslint/ban-ts-comment
2-
// @ts-nocheck These are only tests
31
/* eslint-disable @typescript-eslint/naming-convention */
42
/* eslint-disable @typescript-eslint/explicit-member-accessibility */
53
import { loggingTransport, sendPortToRunner } from '@sentry-internal/node-integration-tests';

0 commit comments

Comments
 (0)