chore: todo items remove class extension constructor overrides

packages/analytics/__tests__/errors/AnalyticsError.test.ts

@@ -0,0 +1,12 @@
+import { AnalyticsError } from '../../src/errors/AnalyticsError';
+
+describe('AnalyticsError', () => {
+	it('works with instanceof operator', () => {
+		const error = new AnalyticsError({
+			name: 'TestError',
+			message: 'message',
+		});
+
+		expect(error instanceof AnalyticsError).toBe(true);
+	});
+});

packages/analytics/src/errors/AnalyticsError.ts

@@ -1,21 +1,9 @@
 // Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
 // SPDX-License-Identifier: Apache-2.0
 
-import {
-	AmplifyError,
-	AmplifyErrorParams,
-} from '@aws-amplify/core/internals/utils';
+import { AmplifyError } from '@aws-amplify/core/internals/utils';
 
 /**
  * @internal
  */
-export class AnalyticsError extends AmplifyError {
-	constructor(params: AmplifyErrorParams) {
-		super(params);
-
-		// Hack for making the custom error class work when transpiled to es5
-		// TODO: Delete the following 2 lines after we change the build target to >= es2015
-		this.constructor = AnalyticsError;
-		Object.setPrototypeOf(this, AnalyticsError.prototype);
-	}
-}
+export class AnalyticsError extends AmplifyError {}

packages/api-graphql/__tests__/utils/errors/GraphQLApiError.test.ts

@@ -0,0 +1,12 @@
+import { GraphQLApiError } from '../../../src/utils/errors';
+
+describe('GraphQLApiError', () => {
+	it('works with instanceof operator', () => {
+		const error = new GraphQLApiError({
+			name: 'TestError',
+			message: 'message',
+		});
+
+		expect(error instanceof GraphQLApiError).toBe(true);
+	});
+});

packages/api-graphql/src/utils/errors/GraphQLApiError.ts

@@ -1,21 +1,9 @@
 // Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
 // SPDX-License-Identifier: Apache-2.0
 
-import {
-	AmplifyError,
-	AmplifyErrorParams,
-} from '@aws-amplify/core/internals/utils';
+import { AmplifyError } from '@aws-amplify/core/internals/utils';
 
 /**
  * @internal
  */
-export class GraphQLApiError extends AmplifyError {
-	constructor(params: AmplifyErrorParams) {
-		super(params);
-
-		// Hack for making the custom error class work when transpiled to es5
-		// TODO: Delete the following 2 lines after we change the build target to >= es2015
-		this.constructor = GraphQLApiError;
-		Object.setPrototypeOf(this, GraphQLApiError.prototype);
-	}
-}
+export class GraphQLApiError extends AmplifyError {}

packages/api-rest/__tests__/errors/CanceledError.test.ts

@@ -0,0 +1,12 @@
+import { CanceledError } from '../../src/errors';
+
+describe('CanceledError', () => {
+	it('works with instanceof operator', () => {
+		const error = new CanceledError({
+			name: 'TestError',
+			message: 'message',
+		});
+
+		expect(error instanceof CanceledError).toBe(true);
+	});
+});

packages/api-rest/__tests__/errors/RestApiError.test.ts

@@ -0,0 +1,12 @@
+import { RestApiError } from '../../src/errors';
+
+describe('RestApiError', () => {
+	it('works with instanceof operator', () => {
+		const error = new RestApiError({
+			name: 'TestError',
+			message: 'message',
+		});
+
+		expect(error instanceof RestApiError).toBe(true);
+	});
+});

packages/api-rest/src/errors/CanceledError.ts

@@ -17,10 +17,6 @@ export class CanceledError extends RestApiError {
 			message: 'Request is canceled by user',
 			...params,
 		});
-
-		// TODO: Delete the following 2 lines after we change the build target to >= es2015
-		this.constructor = CanceledError;
-		Object.setPrototypeOf(this, CanceledError.prototype);
 	}
 }
 

packages/api-rest/src/errors/RestApiError.ts

@@ -1,14 +1,6 @@
 // Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
 // SPDX-License-Identifier: Apache-2.0
 
-import { ApiError, ApiErrorParams } from '@aws-amplify/core/internals/utils';
+import { ApiError } from '@aws-amplify/core/internals/utils';
 
-export class RestApiError extends ApiError {
-	constructor(params: ApiErrorParams) {
-		super(params);
-
-		// TODO: Delete the following 2 lines after we change the build target to >= es2015
-		this.constructor = RestApiError;
-		Object.setPrototypeOf(this, RestApiError.prototype);
-	}
-}
+export class RestApiError extends ApiError {}

packages/auth/__tests__/Error.test.ts

@@ -0,0 +1,10 @@
+import { NoUserPoolError } from '../src/Errors';
+import { AuthErrorTypes } from '../src/types/Auth';
+
+describe('NoUserPoolError', () => {
+	it('works with instanceof operator', () => {
+		const error = new NoUserPoolError(AuthErrorTypes.NoConfig);
+
+		expect(error instanceof NoUserPoolError).toBe(true);
+	});
+});

packages/auth/__tests__/errors/AuthError.test.ts

@@ -0,0 +1,12 @@
+import { AuthError } from '../../src/errors/AuthError';
+
+describe('AuthError', () => {
+	it('works with instanceof operator', () => {
+		const error = new AuthError({
+			name: 'TestError',
+			message: 'message',
+		});
+
+		expect(error instanceof AuthError).toBe(true);
+	});
+});

packages/auth/src/Errors.ts

@@ -16,11 +16,6 @@ export class AuthError extends Error {
 		const { message, log } = authErrorMessages[type];
 		super(message);
 
-		// Hack for making the custom error class work when transpiled to es5
-		// TODO: Delete the following 2 lines after we change the build target to >= es2015
-		this.constructor = AuthError;
-		Object.setPrototypeOf(this, AuthError.prototype);
-
 		this.name = 'AuthError';
 		this.log = log || message;
 
@@ -32,11 +27,6 @@ export class NoUserPoolError extends AuthError {
 	constructor(type: AuthErrorTypes) {
 		super(type);
 
-		// Hack for making the custom error class work when transpiled to es5
-		// TODO: Delete the following 2 lines after we change the build target to >= es2015
-		this.constructor = NoUserPoolError;
-		Object.setPrototypeOf(this, NoUserPoolError.prototype);
-
 		this.name = 'NoUserPoolError';
 	}
 }

packages/auth/src/errors/AuthError.ts

@@ -1,18 +1,6 @@
 // Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
 // SPDX-License-Identifier: Apache-2.0
 
-import {
-	AmplifyError,
-	AmplifyErrorParams,
-} from '@aws-amplify/core/internals/utils';
+import { AmplifyError } from '@aws-amplify/core/internals/utils';
 
-export class AuthError extends AmplifyError {
-	constructor(params: AmplifyErrorParams) {
-		super(params);
-
-		// Hack for making the custom error class work when transpiled to es5
-		// TODO: Delete the following 2 lines after we change the build target to >= es2015
-		this.constructor = AuthError;
-		Object.setPrototypeOf(this, AuthError.prototype);
-	}
-}
+export class AuthError extends AmplifyError {}

packages/core/__tests__/errors/APIError.test.ts

@@ -0,0 +1,12 @@
+import { ApiError } from '../../src/errors/APIError';
+
+describe('ApiError', () => {
+	it('works with instanceof operator', () => {
+		const error = new ApiError({
+			name: 'TestError',
+			message: 'message',
+		});
+
+		expect(error instanceof ApiError).toBe(true);
+	});
+});

packages/core/__tests__/errors/AmplifyError.test.ts

@@ -0,0 +1,12 @@
+import { AmplifyError } from '../../src/errors/AmplifyError';
+
+describe('AmplifyError', () => {
+	it('works with instanceof operator', () => {
+		const error = new AmplifyError({
+			name: 'TestError',
+			message: 'message',
+		});
+
+		expect(error instanceof AmplifyError).toBe(true);
+	});
+});

packages/core/src/errors/APIError.ts

@@ -42,10 +42,6 @@ export class ApiError extends AmplifyError {
 	constructor(params: ApiErrorParams) {
 		super(params);
 
-		// TODO: Delete the following 2 lines after we change the build target to >= es2015
-		this.constructor = ApiError;
-		Object.setPrototypeOf(this, ApiError.prototype);
-
 		if (params.response) {
 			this._response = params.response;
 		}

packages/core/src/errors/AmplifyError.ts

@@ -25,10 +25,5 @@ export class AmplifyError extends Error {
 		this.name = name;
 		this.underlyingError = underlyingError;
 		this.recoverySuggestion = recoverySuggestion;
-
-		// Hack for making the custom error class work when transpiled to es5
-		// TODO: Delete the following 2 lines after we change the build target to >= es2015
-		this.constructor = AmplifyError;
-		Object.setPrototypeOf(this, AmplifyError.prototype);
 	}
 }

packages/interactions/__tests__/errors/InteractionsError.test.ts

@@ -0,0 +1,12 @@
+import { InteractionsError } from '../../src/errors/InteractionsError';
+
+describe('InteractionsError', () => {
+	it('works with instanceof operator', () => {
+		const error = new InteractionsError({
+			name: 'TestError',
+			message: 'message',
+		});
+
+		expect(error instanceof InteractionsError).toBe(true);
+	});
+});

packages/interactions/src/errors/InteractionsError.ts

@@ -1,18 +1,6 @@
 // Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
 // SPDX-License-Identifier: Apache-2.0
 
-import {
-	AmplifyError,
-	AmplifyErrorParams,
-} from '@aws-amplify/core/internals/utils';
+import { AmplifyError } from '@aws-amplify/core/internals/utils';
 
-export class InteractionsError extends AmplifyError {
-	constructor(params: AmplifyErrorParams) {
-		super(params);
-
-		// Hack for making the custom error class work when transpiled to es5
-		// TODO: Delete the following 2 lines after we change the build target to >= es2015
-		this.constructor = InteractionsError;
-		Object.setPrototypeOf(this, InteractionsError.prototype);
-	}
-}
+export class InteractionsError extends AmplifyError {}

packages/notifications/__tests__/inAppMessaging/errors/InAppMessagingError.test.ts

@@ -0,0 +1,12 @@
+import { InAppMessagingError } from '../../../src/inAppMessaging/errors/InAppMessagingError';
+
+describe('InAppMessagingError', () => {
+	it('works with instanceof operator', () => {
+		const error = new InAppMessagingError({
+			name: 'TestError',
+			message: 'message',
+		});
+
+		expect(error instanceof InAppMessagingError).toBe(true);
+	});
+});

packages/notifications/__tests__/pushNotifications/errors/PushNotificationError.test.ts

@@ -0,0 +1,12 @@
+import { PushNotificationError } from '../../../src/pushNotifications/errors/PushNotificationError';
+
+describe('PushNotificationError', () => {
+	it('works with instanceof operator', () => {
+		const error = new PushNotificationError({
+			name: 'TestError',
+			message: 'message',
+		});
+
+		expect(error instanceof PushNotificationError).toBe(true);
+	});
+});

packages/notifications/src/inAppMessaging/errors/InAppMessagingError.ts

@@ -1,21 +1,9 @@
 // Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
 // SPDX-License-Identifier: Apache-2.0
 
-import {
-	AmplifyError,
-	AmplifyErrorParams,
-} from '@aws-amplify/core/internals/utils';
+import { AmplifyError } from '@aws-amplify/core/internals/utils';
 
 /**
  * @internal
  */
-export class InAppMessagingError extends AmplifyError {
-	constructor(params: AmplifyErrorParams) {
-		super(params);
-
-		// Hack for making the custom error class work when transpiled to es5
-		// TODO: Delete the following 2 lines after we change the build target to >= es2015
-		this.constructor = InAppMessagingError;
-		Object.setPrototypeOf(this, InAppMessagingError.prototype);
-	}
-}
+export class InAppMessagingError extends AmplifyError {}

packages/notifications/src/pushNotifications/errors/PushNotificationError.ts

@@ -1,18 +1,6 @@
 // Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
 // SPDX-License-Identifier: Apache-2.0
 
-import {
-	AmplifyError,
-	AmplifyErrorParams,
-} from '@aws-amplify/core/internals/utils';
+import { AmplifyError } from '@aws-amplify/core/internals/utils';
 
-export class PushNotificationError extends AmplifyError {
-	constructor(params: AmplifyErrorParams) {
-		super(params);
-
-		// Hack for making the custom error class work when transpiled to es5
-		// TODO: Delete the following 2 lines after we change the build target to >= es2015
-		this.constructor = PushNotificationError;
-		Object.setPrototypeOf(this, PushNotificationError.prototype);
-	}
-}
+export class PushNotificationError extends AmplifyError {}

packages/predictions/__tests__/errors/PredictionsError.test.ts

@@ -0,0 +1,12 @@
+import { PredictionsError } from '../../src/errors/PredictionsError';
+
+describe('PredictionsError', () => {
+	it('works with instanceof operator', () => {
+		const error = new PredictionsError({
+			name: 'TestError',
+			message: 'message',
+		});
+
+		expect(error instanceof PredictionsError).toBe(true);
+	});
+});

packages/predictions/src/errors/PredictionsError.ts

@@ -1,16 +1,5 @@
 // Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
 // SPDX-License-Identifier: Apache-2.0
-import {
-	AmplifyError,
-	AmplifyErrorParams,
-} from '@aws-amplify/core/internals/utils';
+import { AmplifyError } from '@aws-amplify/core/internals/utils';
 
-export class PredictionsError extends AmplifyError {
-	constructor(params: AmplifyErrorParams) {
-		super(params);
-
-		// TODO: Delete the following 2 lines after we change the build target to >= es2015
-		this.constructor = PredictionsError;
-		Object.setPrototypeOf(this, PredictionsError.prototype);
-	}
-}
+export class PredictionsError extends AmplifyError {}

packages/storage/__tests__/errors/CanceledError.test.ts

@@ -0,0 +1,12 @@
+import { CanceledError } from '../../src/errors/CanceledError';
+
+describe('CanceledError', () => {
+	it('works with instanceof operator', () => {
+		const error = new CanceledError({
+			name: 'TestError',
+			message: 'message',
+		});
+
+		expect(error instanceof CanceledError).toBe(true);
+	});
+});