eliza
eliza copied to clipboard
feat: integration tests fixes + library improvements
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