Skip to content

Commit

Permalink
chore: Fix test warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
keelerm84 committed Aug 7, 2024
1 parent bd3b2f8 commit c6ea08d
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion ldclient/testing/impl/test_attribute_ref.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ def test_ref_simple_with_leading_slash(self, input: str, unescaped: str):
assert a.depth == 1
assert a[0] == unescaped

@pytest.mark.parametrize("input", [])
@pytest.mark.parametrize("input", [("name"), ("name/with/slashes"), ("name~0~1with-what-looks-like-escape-sequences")])
def test_literal(self, input: str):
a = AttributeRef.from_literal(input)
assert a.valid is True
Expand Down
2 changes: 0 additions & 2 deletions ldclient/testing/test_ldclient.py
Original file line number Diff line number Diff line change
Expand Up @@ -75,14 +75,12 @@ def test_client_has_null_update_processor_in_ldd_mode():
assert isinstance(client._update_processor, NullUpdateProcessor)


@pytest.mark.skip("Can't currently use a live stream processor in tests because its error logging will disrupt other tests.")
def test_client_has_streaming_processor_by_default():
config = Config(sdk_key="secret", base_uri=unreachable_uri, stream_uri=unreachable_uri, send_events=False)
with LDClient(config=config, start_wait=0) as client:
assert isinstance(client._update_processor, StreamingUpdateProcessor)


@pytest.mark.skip("Can't currently use a live polling processor in tests because its error logging will disrupt other tests.")
def test_client_has_polling_processor_if_streaming_is_disabled():
config = Config(sdk_key="secret", stream=False, base_uri=unreachable_uri, stream_uri=unreachable_uri, send_events=False)
with LDClient(config=config, start_wait=0) as client:
Expand Down

0 comments on commit c6ea08d

Please sign in to comment.