Fix #157 Improve the stability of API mock servers in unit tests · slackapi/bolt-python@6ce7bdc · GitHub
Skip to content

Commit 6ce7bdc

Browse files
seratchseratch
committed
Fix #157 Improve the stability of API mock servers in unit tests
1 parent 3475ae7 commit 6ce7bdc

79 files changed

Lines changed: 872 additions & 430 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎.github/workflows/ci-build.yml‎

Lines changed: 40 additions & 8 deletions

‎tests/adapter_tests/aws/__init__.py‎

Whitespace-only changes.

tests/adapter_tests/test_aws_chalice.py renamed to tests/adapter_tests/aws/test_aws_chalice.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
from tests.mock_web_api_server import (
2020
setup_mock_web_api_server,
2121
cleanup_mock_web_api_server,
22+
assert_auth_test_count,
2223
)
2324
from tests.utils import remove_os_env_temporarily, restore_os_env
2425

@@ -110,7 +111,7 @@ def events() -> Response:
110111
headers=self.build_headers(timestamp, body),
111112
)
112113
assert response["statusCode"] == 200
113-
assert self.mock_received_requests["/auth.test"] == 1
114+
assert_auth_test_count(self, 1)
114115

115116
def test_shortcuts(self):
116117
app = App(
@@ -157,7 +158,7 @@ def events() -> Response:
157158
headers=self.build_headers(timestamp, body),
158159
)
159160
assert response["statusCode"] == 200
160-
assert self.mock_received_requests["/auth.test"] == 1
161+
assert_auth_test_count(self, 1)
161162

162163
def test_commands(self):
163164
app = App(
@@ -204,7 +205,7 @@ def events() -> Response:
204205
headers=self.build_headers(timestamp, body),
205206
)
206207
assert response["statusCode"] == 200
207-
assert self.mock_received_requests["/auth.test"] == 1
208+
assert_auth_test_count(self, 1)
208209

209210
def test_lazy_listeners(self):
210211
app = App(
@@ -261,7 +262,7 @@ def say_it(say):
261262
)
262263
response: Response = slack_handler.handle(request)
263264
assert response.status_code == 200
264-
assert self.mock_received_requests["/auth.test"] == 1
265+
assert_auth_test_count(self, 1)
265266
assert self.mock_received_requests["/chat.postMessage"] == 1
266267

267268
def test_oauth(self):

tests/adapter_tests/test_aws_lambda.py renamed to tests/adapter_tests/aws/test_aws_lambda.py

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
from tests.mock_web_api_server import (
1515
setup_mock_web_api_server,
1616
cleanup_mock_web_api_server,
17+
assert_auth_test_count,
1718
)
1819
from tests.utils import remove_os_env_temporarily, restore_os_env
1920

@@ -123,7 +124,7 @@ def event_handler():
123124
}
124125
response = SlackRequestHandler(app).handle(event, self.context)
125126
assert response["statusCode"] == 200
126-
assert self.mock_received_requests["/auth.test"] == 1
127+
assert_auth_test_count(self, 1)
127128

128129
event = {
129130
"body": body,
@@ -134,7 +135,7 @@ def event_handler():
134135
}
135136
response = SlackRequestHandler(app).handle(event, self.context)
136137
assert response["statusCode"] == 200
137-
assert self.mock_received_requests["/auth.test"] == 1
138+
assert_auth_test_count(self, 1)
138139

139140
@mock_lambda
140141
def test_shortcuts(self):
@@ -173,7 +174,7 @@ def shortcut_handler(ack):
173174
}
174175
response = SlackRequestHandler(app).handle(event, self.context)
175176
assert response["statusCode"] == 200
176-
assert self.mock_received_requests["/auth.test"] == 1
177+
assert_auth_test_count(self, 1)
177178

178179
event = {
179180
"body": body,
@@ -184,7 +185,7 @@ def shortcut_handler(ack):
184185
}
185186
response = SlackRequestHandler(app).handle(event, self.context)
186187
assert response["statusCode"] == 200
187-
assert self.mock_received_requests["/auth.test"] == 1
188+
assert_auth_test_count(self, 1)
188189

189190
@mock_lambda
190191
def test_commands(self):
@@ -223,7 +224,7 @@ def command_handler(ack):
223224
}
224225
response = SlackRequestHandler(app).handle(event, self.context)
225226
assert response["statusCode"] == 200
226-
assert self.mock_received_requests["/auth.test"] == 1
227+
assert_auth_test_count(self, 1)
227228

228229
event = {
229230
"body": body,
@@ -234,7 +235,7 @@ def command_handler(ack):
234235
}
235236
response = SlackRequestHandler(app).handle(event, self.context)
236237
assert response["statusCode"] == 200
237-
assert self.mock_received_requests["/auth.test"] == 1
238+
assert_auth_test_count(self, 1)
238239

239240
@mock_lambda
240241
def test_lazy_listeners(self):
@@ -279,7 +280,7 @@ def say_it(say):
279280
}
280281
response = SlackRequestHandler(app).handle(event, self.context)
281282
assert response["statusCode"] == 200
282-
assert self.mock_received_requests["/auth.test"] == 1
283+
assert_auth_test_count(self, 1)
283284
assert self.mock_received_requests["/chat.postMessage"] == 1
284285

285286
@mock_lambda

tests/adapter_tests/test_lambda_s3_oauth_flow.py renamed to tests/adapter_tests/aws/test_lambda_s3_oauth_flow.py

File renamed without changes.

‎tests/adapter_tests/bottle/__init__.py‎

Whitespace-only changes.

tests/adapter_tests/test_bottle.py renamed to tests/adapter_tests/bottle/test_bottle.py

Lines changed: 4 additions & 3 deletions

tests/adapter_tests/test_bottle_oauth.py renamed to tests/adapter_tests/bottle/test_bottle_oauth.py

File renamed without changes.

‎tests/adapter_tests/cherrypy/__init__.py‎

Whitespace-only changes.

tests/adapter_tests/test_cherrypy.py renamed to tests/adapter_tests/cherrypy/test_cherrypy.py

File renamed without changes.

0 commit comments

Comments
 (0)