Skip to content

feat: add a few integration tests#48

Merged
drobnikj merged 2 commits into
masterfrom
feat/lifecycle_methods_integration_tests
Feb 8, 2023
Merged

feat: add a few integration tests#48
drobnikj merged 2 commits into
masterfrom
feat/lifecycle_methods_integration_tests

Conversation

@drobnikj

@drobnikj drobnikj commented Feb 2, 2023

Copy link
Copy Markdown
Member

No description provided.

@github-actions github-actions Bot added this to the 56th sprint - Platform team milestone Feb 2, 2023
@github-actions github-actions Bot added the t-core-services Issues with this label are in the ownership of the core services team. label Feb 2, 2023

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Pull Request Tookit has failed!

Pull request is neither linked to an issue or epic nor labeled as adhoc!

@drobnikj
drobnikj marked this pull request as ready for review February 2, 2023 17:13

class TestActorNewClient:

async def test_actor_new_client_config(self, monkeypatch: pytest.MonkeyPatch) -> None:

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this test is in helpers, not sure why it was there as well.

@fnesveda fnesveda left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! I just had two small suggestions but they're not too important.

Comment thread tests/integration/test_actor_events.py Outdated
Comment thread tests/integration/test_actor_lifecycle.py Outdated
@drobnikj
drobnikj merged commit 1843f48 into master Feb 8, 2023
@drobnikj
drobnikj deleted the feat/lifecycle_methods_integration_tests branch February 8, 2023 08:58
@fnesveda fnesveda added the validated Issues that are resolved and their solutions fulfill the acceptance criteria. label Oct 9, 2023
@github-actions github-actions Bot added the tested Temporary label used only programatically for some analytics. label Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-core-services Issues with this label are in the ownership of the core services team. tested Temporary label used only programatically for some analytics. validated Issues that are resolved and their solutions fulfill the acceptance criteria.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants