Skip to content

Commit ff3815d

Browse files
committed
optimize rtc error message and fix its tests
- read response body error first - add rpqId property - always try to delete rtc app after tests
1 parent f5c2e0a commit ff3815d

File tree

2 files changed

+11
-2
lines changed

2 files changed

+11
-2
lines changed

qiniu/rtc/app.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,8 @@ function get(credentials, options, fn) {
2323
if (res.statusCode != 200) {
2424
var result = {
2525
code: res.statusCode,
26-
message: res.statusMessage
26+
message: resultObject.error || res.statusMessage,
27+
reqId: res.headers['x-reqid']
2728
};
2829
fn(result, null);
2930
} else {
@@ -59,7 +60,8 @@ function post(credentials, options, data, fn) {
5960
if (res.statusCode != 200) {
6061
var result = {
6162
code: res.statusCode,
62-
message: res.statusMessage
63+
message: resultObject.error || res.statusMessage,
64+
reqId: res.headers['x-reqid']
6365
};
6466
fn(result, null);
6567
} else {

test/rtc.test.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,13 @@ describe('test rtc credentials', function () {
2626
maxUsers: 10,
2727
noAutoKickUser: true
2828
};
29+
30+
after(function (done) {
31+
qiniu.app.deleteApp(appId, credentials, function () {
32+
done();
33+
});
34+
});
35+
2936
// eslint-disable-next-line no-undef
3037
describe('test create app', function () {
3138
// eslint-disable-next-line no-undef

0 commit comments

Comments
 (0)