Skip to content

Commit

Permalink
mark double ai / human message as known issue
Browse files Browse the repository at this point in the history
  • Loading branch information
mattf committed May 14, 2024
1 parent 99616e4 commit e0c3188
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions libs/ai-endpoints/tests/integration_tests/test_chat_models.py
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,8 @@ def test_messages(
) -> None:
if not system and not exchange:
pytest.skip("No messages to test")
if len(exchange) == 2 and type(exchange[0]) == type(exchange[1]):

Check failure on line 123 in libs/ai-endpoints/tests/integration_tests/test_chat_models.py

View workflow job for this annotation

GitHub Actions / cd libs/ai-endpoints / make lint #3.8

Ruff (E721)

tests/integration_tests/test_chat_models.py:123:31: E721 Do not compare types, use `isinstance()`
pytest.skip("Known issue, messages types must alternate")
chat = ChatNVIDIA(model=chat_model, max_tokens=36).mode(**mode)
response = chat.invoke(system + exchange)
assert isinstance(response, BaseMessage)
Expand Down

0 comments on commit e0c3188

Please sign in to comment.