Skip to content

Commit a3c94a7

Browse files
authored
fix failing botocore test suite (DataDog#3075)
* update moto/localstack * mock firehose instead of kinesis * update moto versions * support older/newer moto versions
1 parent 236a80f commit a3c94a7

File tree

3 files changed

+13
-6
lines changed

3 files changed

+13
-6
lines changed

docker-compose.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ services:
7070
# container that executes mocked AWS services; this is a custom
7171
# build that runs all of them in a single container. It is built
7272
# using this fork: https://github.com/palazzem/moto/tree/palazzem/docker-service
73-
image: datadog/docker-library:moto_1_0_1
73+
image: motoserver/moto:2.2.19
7474
ports:
7575
- "127.0.0.1:5000:5000"
7676
- "127.0.0.1:5001:5001"
@@ -132,7 +132,7 @@ services:
132132
- ./.ddriot:/root/project/.riot
133133

134134
localstack:
135-
image: localstack/localstack:0.12.1
135+
image: localstack/localstack:0.13.1
136136
network_mode: bridge
137137
ports:
138138
- "127.0.0.1:4566:4566"

riotfile.py

+3-3
Original file line numberDiff line numberDiff line change
@@ -980,15 +980,15 @@ def select_pys(min_version=MIN_PYTHON_VERSION, max_version=MAX_PYTHON_VERSION):
980980
Venv(
981981
name="boto",
982982
command="pytest {cmdargs} tests/contrib/boto",
983-
venvs=[Venv(pys=select_pys(max_version="3.6"), pkgs={"boto": latest, "moto": ["<1.0"]})],
983+
venvs=[Venv(pys=select_pys(max_version="3.6"), pkgs={"boto": latest, "moto": "<1.0.0"})],
984984
),
985985
Venv(
986986
name="botocore",
987987
command="pytest {cmdargs} tests/contrib/botocore",
988988
pkgs={"botocore": latest},
989989
venvs=[
990-
Venv(pys=select_pys(min_version="3.5"), pkgs={"moto": [">=1.0,<2.0"]}),
991-
Venv(pys=["2.7"], pkgs={"moto": [">=1.0,<2.0"], "rsa": ["<4.7.1"]}),
990+
Venv(pys=select_pys(min_version="3.5"), pkgs={"moto[all]": latest}),
991+
Venv(pys=["2.7"], pkgs={"moto": ["~=1.0"], "rsa": ["<4.7.1"]}),
992992
],
993993
),
994994
Venv(

tests/contrib/botocore/test.py

+8-1
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,13 @@
1313
from moto import mock_s3
1414
from moto import mock_sqs
1515

16+
17+
# Older version of moto used kinesis to mock firehose
18+
try:
19+
from moto import mock_firehose
20+
except ImportError:
21+
from moto import mock_kinesis as mock_firehose
22+
1623
from ddtrace import Pin
1724
from ddtrace import config
1825
from ddtrace.constants import ANALYTICS_SAMPLE_RATE_KEY
@@ -761,7 +768,7 @@ def test_stubber_no_response_metadata(self):
761768
service_response = s3.list_buckets()
762769
assert service_response == response
763770

764-
@mock_kinesis
771+
@mock_firehose
765772
def test_firehose_no_records_arg(self):
766773
firehose = self.session.create_client("firehose", region_name="us-west-2")
767774
Pin(service=self.TEST_SERVICE, tracer=self.tracer).onto(firehose)

0 commit comments

Comments
 (0)