8000 fix test for V1 · localstack/localstack@03512db · GitHub
[go: up one dir, main page]

Skip to content

Commit 03512db

Browse files
committed
fix test for V1
1 parent 18488fd commit 03512db

File tree

1 file changed

+18
-2
lines changed

1 file changed

+18
-2
lines changed

tests/aws/services/dynamodbstreams/test_dynamodb_streams.py

Lines changed: 18 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@ def test_table_v2_stream(self, aws_client, infrastructure_setup, snapshot):
5353

5454
@markers.aws.only_localstack
5555
def test_stream_spec_and_region_replacement(self, aws_client, region_name):
56+
# our V1 and V2 implementation are pretty different, and we need different ways to test it
5657
ddbstreams = aws_client.dynamodbstreams
5758
table_name = f"ddb-{short_uid()}"
5859
resources.create_dynamodb_table(
@@ -73,6 +74,14 @@ def test_stream_spec_and_region_replacement(self, aws_client, region_name):
7374
stream_tables = ddbstreams.list_streams(TableName="foo")["Streams"]
7475
assert len(stream_tables) == 0
7576

77+
if not config.DDB_STREAMS_PROVIDER_V2:
78+
from localstack.services.dynamodbstreams.dynamodbstreams_api import (
79+
get_kinesis_stream_name,
80+
)
81+
82+
stream_name = get_kinesis_stream_name(table_name)
83+
assert stream_name in aws_client.kinesis.list_streams()["StreamNames"]
84+
7685
# assert stream has been created
7786
stream_tables = [
7887
s["TableName"] for s in ddbstreams.list_streams(TableName=table_name)["Streams"]
@@ -90,8 +99,15 @@ def test_stream_spec_and_region_replacement(self, aws_client, region_name):
9099
aws_client.dynamodb.delete_table(TableName=table_name)
91100

92101
def _assert_stream_disabled():
93-
result = aws_client.dynamodbstreams.describe_stream(StreamArn=table["LatestStreamArn"])
94-
assert result["StreamDescription"]["StreamStatus"] == "DISABLED"
102+
if config.DDB_STREAMS_PROVIDER_V2:
103+
_result = aws_client.dynamodbstreams.describe_stream(
104+
StreamArn=table["LatestStreamArn"]
105+
)
106+
assert _result["StreamDescription"]["StreamStatus"] == "DISABLED"
107+
else:
108+
_stream_tables = [s["TableName"] for s in ddbstreams.list_streams()["Streams"]]
109+
assert table_name not in _stream_tables
110+
assert stream_name not in aws_client.kinesis.list_streams()["StreamNames"]
95111

96112
# assert stream has been deleted
97113
retry(_assert_stream_disabled, sleep=1, retries=20)

0 commit comments

Comments
 (0)
0