Unit tests for dependencies.py #11

cszsol · 2024-09-25T09:00:15Z

No description provided.

src/neuroagent/agents/simple_chat_agent.py

tests/app/test_dependencies.py

tests/tools/test_base_tool.py

…ersion

BoBer78

Small comment about the patch_required_env fixture, other that that it looks good !

tests/app/test_dependencies.py

cszsol added 10 commits September 24, 2024 11:56

Added test for get_settings

f92049d

Added tests for test_get_connection_string

72b08e7

Added tests for get_engine

c9b6f9f

Added tests for get_session

e1a5cb7

Added test for process_validation_error

839a91e

lint

d80f1fd

lint

cb3f1c6

Added type ignores for mypy

e7ffd36

ruff format

88e89b1

Added fixes for unit old tests

d8faa45

cszsol requested review from BoBer78 and WonderPG September 26, 2024 08:36

WonderPG reviewed Sep 26, 2024

View reviewed changes

cszsol added 12 commits September 27, 2024 15:05

Added type arguments

59d66d2

Removed mock settings

1411b36

code review

65bc474

Fixed most unit test issues

ee85b7f

Had to delete one unit test, since it was not working

2efc883

Added change to make tests backwards compatible with new httpx_mock v…

0d50e28

…ersion

Update test_simple_agent.py

839be90

Update test_simple_chat_agent.py

996117e

Update test_tools.py

5e95765

Added changelog

a266fbf

Reformatted with proper ruff version

74036cd

Merge branch 'failing_ci' into unit_tests

fcebe80

BoBer78 reviewed Oct 1, 2024

View reviewed changes

tests/app/test_dependencies.py Outdated Show resolved Hide resolved

cszsol added 2 commits October 2, 2024 10:04

Added patch_required_env fixture

228df9e

Merge branch 'main' into unit_tests

4c67b79

BoBer78 reviewed Oct 2, 2024

View reviewed changes

tests/app/test_dependencies.py Outdated Show resolved Hide resolved

tests/app/test_dependencies.py Outdated Show resolved Hide resolved

tests/app/test_dependencies.py Outdated Show resolved Hide resolved

tests/app/test_dependencies.py Show resolved Hide resolved

code review

1cc6650

cszsol added 3 commits October 2, 2024 13:40

Merge branch 'main' into unit_tests

86ce5fc

lint

2092fc1

lint

6fe1e81

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Unit tests for dependencies.py #11

Unit tests for dependencies.py #11

cszsol commented Sep 25, 2024

BoBer78 left a comment

Unit tests for dependencies.py #11

Are you sure you want to change the base?

Unit tests for dependencies.py #11

Conversation

cszsol commented Sep 25, 2024

BoBer78 left a comment

Choose a reason for hiding this comment