Skip to content

Commit b1b9377

Browse files
committed
update deprecated xxx.connect() with new xxx()
- no more deprecated connect method in test files
1 parent d6024e2 commit b1b9377

File tree

11 files changed

+37
-17
lines changed

11 files changed

+37
-17
lines changed

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030
"packages/*"
3131
],
3232
"devDependencies": {
33-
"@types/jest": "^26.0.10",
33+
"@types/jest": "27.4.1",
3434
"@typescript-eslint/eslint-plugin": "^4.31.1",
3535
"@typescript-eslint/parser": "^4.31.1",
3636
"axios": "^0.21.4",

packages/messaging-api-line/src/__tests__/LineClient-constructor.spec.ts

+4-2
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,8 @@ describe('connect', () => {
2626
},
2727
},
2828
});
29-
LineClient.connect({
29+
// eslint-disable-next-line no-new
30+
new LineClient({
3031
accessToken: ACCESS_TOKEN,
3132
channelSecret: CHANNEL_SECRET,
3233
});
@@ -49,7 +50,8 @@ describe('connect', () => {
4950
},
5051
},
5152
});
52-
LineClient.connect({
53+
// eslint-disable-next-line no-new
54+
new LineClient({
5355
accessToken: ACCESS_TOKEN,
5456
channelSecret: CHANNEL_SECRET,
5557
origin: 'https://mydummytestserver.com',

packages/messaging-api-line/src/__tests__/LineNotify.spec.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,8 @@ describe('connect', () => {
3737

3838
it('create axios with LINE Notify API', () => {
3939
axios.create = jest.fn();
40-
LineNotify.connect({
40+
// eslint-disable-next-line no-new
41+
new LineNotify({
4142
clientId: CLIENT_ID,
4243
clientSecret: CLIENT_SECRET,
4344
redirectUri: REDIRECT_URI,

packages/messaging-api-line/src/__tests__/LinePay.spec.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,8 @@ describe('connect', () => {
3131

3232
it('create axios with LINE PAY API', () => {
3333
axios.create = jest.fn();
34-
LinePay.connect({
34+
// eslint-disable-next-line no-new
35+
new LinePay({
3536
channelId: CHANNEL_ID,
3637
channelSecret: CHANNEL_SECRET,
3738
});

packages/messaging-api-messenger/src/__tests__/MessengerClient-constructor.spec.ts

+6-3
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,8 @@ describe('connect', () => {
2626
},
2727
},
2828
});
29-
MessengerClient.connect({ accessToken: ACCESS_TOKEN });
29+
// eslint-disable-next-line no-new
30+
new MessengerClient({ accessToken: ACCESS_TOKEN });
3031

3132
expect(axios.create).toBeCalledWith(
3233
expect.objectContaining({
@@ -46,7 +47,8 @@ describe('connect', () => {
4647
},
4748
},
4849
});
49-
MessengerClient.connect({ accessToken: ACCESS_TOKEN, version: '2.6' });
50+
// eslint-disable-next-line no-new
51+
new MessengerClient({ accessToken: ACCESS_TOKEN, version: '2.6' });
5052

5153
expect(axios.create).toBeCalledWith(
5254
expect.objectContaining({
@@ -65,7 +67,8 @@ describe('connect', () => {
6567
},
6668
},
6769
});
68-
MessengerClient.connect({
70+
// eslint-disable-next-line no-new
71+
new MessengerClient({
6972
accessToken: ACCESS_TOKEN,
7073
origin: 'https://mydummytestserver.com',
7174
});

packages/messaging-api-slack/src/__tests__/SlackOAuthClient-constructor.spec.ts

+4-2
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,8 @@ describe('connect', () => {
2525
},
2626
},
2727
});
28-
SlackOAuthClient.connect({ accessToken: TOKEN });
28+
// eslint-disable-next-line no-new
29+
new SlackOAuthClient({ accessToken: TOKEN });
2930

3031
expect(axios.create).toBeCalledWith({
3132
baseURL: 'https://slack.com/api/',
@@ -42,7 +43,8 @@ describe('connect', () => {
4243
},
4344
},
4445
});
45-
SlackOAuthClient.connect({
46+
// eslint-disable-next-line no-new
47+
new SlackOAuthClient({
4648
accessToken: TOKEN,
4749
origin: 'https://mydummytestserver.com',
4850
});

packages/messaging-api-slack/src/__tests__/SlackWebhookClient-constructor.spec.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,8 @@ describe('connect', () => {
2525
},
2626
},
2727
});
28-
SlackWebhookClient.connect({ url: URL });
28+
// eslint-disable-next-line no-new
29+
new SlackWebhookClient({ url: URL });
2930

3031
expect(axios.create).toBeCalledWith({
3132
baseURL:

packages/messaging-api-telegram/src/__tests__/TelegramClient-constructor.spec.ts

+4-2
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,8 @@ describe('connect', () => {
3333
},
3434
},
3535
});
36-
TelegramClient.connect({ accessToken: ACCESS_TOKEN });
36+
// eslint-disable-next-line no-new
37+
new TelegramClient({ accessToken: ACCESS_TOKEN });
3738

3839
expect(axios.create).toBeCalledWith({
3940
baseURL:
@@ -53,7 +54,8 @@ describe('connect', () => {
5354
},
5455
},
5556
});
56-
TelegramClient.connect({
57+
// eslint-disable-next-line no-new
58+
new TelegramClient({
5759
accessToken: ACCESS_TOKEN,
5860
origin: 'https://mydummytestserver.com',
5961
});

packages/messaging-api-telegram/src/__tests__/TelegramClient.spec.ts

+4
Original file line numberDiff line numberDiff line change
@@ -814,12 +814,14 @@ describe('other api', () => {
814814
from_chat_id: 313534466,
815815
message_id: 203,
816816
disable_notification: true,
817+
protect_content: true,
817818
})
818819
.reply(200, reply);
819820

820821
const res = await client.forwardMessage(427770117, 313534466, 203, {
821822
// @ts-expect-error
822823
disable_notification: true,
824+
protect_content: true,
823825
});
824826

825827
expect(res).toEqual(result);
@@ -833,11 +835,13 @@ describe('other api', () => {
833835
from_chat_id: 313534466,
834836
message_id: 203,
835837
disable_notification: true,
838+
protect_content: true,
836839
})
837840
.reply(200, reply);
838841

839842
const res = await client.forwardMessage(427770117, 313534466, 203, {
840843
disableNotification: true,
844+
protectContent: true,
841845
});
842846

843847
expect(res).toEqual(result);

packages/messaging-api-viber/src/__tests__/ViberClient-constructor.spec.ts

+4-2
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,8 @@ describe('connect', () => {
3030
},
3131
},
3232
});
33-
ViberClient.connect({ accessToken: AUTH_TOKEN, sender: SENDER });
33+
// eslint-disable-next-line no-new
34+
new ViberClient({ accessToken: AUTH_TOKEN, sender: SENDER });
3435

3536
expect(axios.create).toBeCalledWith({
3637
baseURL: 'https://chatapi.viber.com/pa/',
@@ -50,7 +51,8 @@ describe('connect', () => {
5051
},
5152
},
5253
});
53-
ViberClient.connect({
54+
// eslint-disable-next-line no-new
55+
new ViberClient({
5456
accessToken: AUTH_TOKEN,
5557
sender: SENDER,
5658
origin: 'https://mydummytestserver.com',

packages/messaging-api-wechat/src/__tests__/WechatClient-constructor.spec.ts

+4-2
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,8 @@ describe('connect', () => {
2626
},
2727
},
2828
});
29-
WechatClient.connect({ appId: APP_ID, appSecret: APP_SECRET });
29+
// eslint-disable-next-line no-new
30+
new WechatClient({ appId: APP_ID, appSecret: APP_SECRET });
3031

3132
expect(axios.create).toBeCalledWith({
3233
baseURL: 'https://api.weixin.qq.com/cgi-bin/',
@@ -45,7 +46,8 @@ describe('connect', () => {
4546
},
4647
},
4748
});
48-
WechatClient.connect({
49+
// eslint-disable-next-line no-new
50+
new WechatClient({
4951
appId: APP_ID,
5052
appSecret: APP_SECRET,
5153
origin: 'https://mydummytestserver.com',

0 commit comments

Comments
 (0)