@lowerdeck/error 1.0.10 → 1.2.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/package.json +1 -1
- package/src/error.test.ts +20 -0
- package/src/error.ts +2 -2
package/package.json
CHANGED
package/src/error.test.ts
CHANGED
|
@@ -71,4 +71,24 @@ describe('ServiceError', () => {
|
|
|
71
71
|
let response = apiError.toResponse();
|
|
72
72
|
expect(response).toEqual({ ...errorRecord.data, object: 'error', ok: false });
|
|
73
73
|
});
|
|
74
|
+
|
|
75
|
+
test('should not mutate response payload in fromResponse', () => {
|
|
76
|
+
let payload = {
|
|
77
|
+
code: 'bad_request',
|
|
78
|
+
message: 'Bad Request',
|
|
79
|
+
status: 400,
|
|
80
|
+
ok: false,
|
|
81
|
+
object: 'error'
|
|
82
|
+
};
|
|
83
|
+
|
|
84
|
+
ServiceError.fromResponse(payload);
|
|
85
|
+
|
|
86
|
+
expect(payload).toEqual({
|
|
87
|
+
code: 'bad_request',
|
|
88
|
+
message: 'Bad Request',
|
|
89
|
+
status: 400,
|
|
90
|
+
ok: false,
|
|
91
|
+
object: 'error'
|
|
92
|
+
});
|
|
93
|
+
});
|
|
74
94
|
});
|
package/src/error.ts
CHANGED
|
@@ -49,7 +49,7 @@ export class ServiceError<InnerError extends ErrorRecord<any, any>> extends Erro
|
|
|
49
49
|
}
|
|
50
50
|
|
|
51
51
|
constructor(private readonly error: InnerError) {
|
|
52
|
-
super(error.data.message);
|
|
52
|
+
super(`[@lowerdeck/error]: ${error.data.message} (${JSON.stringify(error.data)})`);
|
|
53
53
|
}
|
|
54
54
|
|
|
55
55
|
setParent(parent: Error) {
|
|
@@ -66,7 +66,7 @@ export class ServiceError<InnerError extends ErrorRecord<any, any>> extends Erro
|
|
|
66
66
|
}
|
|
67
67
|
|
|
68
68
|
static fromResponse(raw: ErrorData<any, any>) {
|
|
69
|
-
let data = raw;
|
|
69
|
+
let data = { ...raw };
|
|
70
70
|
delete data.ok;
|
|
71
71
|
delete data.object;
|
|
72
72
|
|