eliza icon indicating copy to clipboard operation
eliza copied to clipboard

feat: integration tests fixes + library improvements

Open jzvikart opened this issue 2 months ago • 1 comments

Risks

Very low. Worst case this could break the tests or introduce problems with dependencies.

Background

What does this PR do?

This builds on top of previous changes that introduced the first version of integration tests framework. These changes:

  • fix some existing issues with smoke and integration tests failing (esp. giving agent a fixed time to start that was not always sufficient)
  • extend integration test library with a full wrapper for setting up / tearing down a test
  • refactor existing integration test ("Hello Trump") to use new library
  • fix a potential issue with possible leak of API keys (not related to integration tests themselves)
  • remove a dependency that was previously added but is no longer required

What kind of change is this?

Improvement + bug fix + feature

Why are we doing this? Any context or related work?

This is to improve overall project quality via better testing..

Documentation changes needed?

None

Testing

To test the tests, these changes need to be run in CI workflow. If either smoke or integration tests fail, the PR should NOT be merged. In that case we will check the logs and update the PR as necessary.

Deploy Notes

None

Database changes

None

Deployment instructions

None

Discord username

user98634

jzvikart avatar Dec 17 '24 11:12 jzvikart