Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/ens error #24

Merged
merged 10 commits into from
Jan 23, 2025
Merged

Fix/ens error #24

merged 10 commits into from
Jan 23, 2025

Conversation

Mohsinsiddi
Copy link
Collaborator

Description

This PR updates test configurations and fixes bugs in the test framework to support both Memeoorr and Mech agent testing.

Key Changes

  • Added support for both Memeoorr and Mech agent test configurations and RPC handling
  • Updated CI workflow configuration to improve test reliability
  • Modified test framework to handle environment variable prompts more robustly
  • Updated dependencies to latest versions

Test Framework Updates

  • Enhanced the service health check system with better metrics
  • Added specific handlers for Memeoorr and Mech agent configurations
  • Added special funding handler for Mech with increased gas buffer (100 ETH)
  • Added health check bypass for Mech service
  • Improved input handling with safer validation and delays
  • Fixed funding handler logic for different chains

Config Updates

  • Updated Memeoorr agent hash in configuration
  • Added new environment variable TWIKIT_SKIP_CONNECTION
  • Removed redundant Twitter cookie configuration
  • Updated olas-operate-middleware dependency to latest version
  • Enabled testing for both Memeoorr and Mech configs (removed from exclusion list)

CI Updates

  • Updated GitHub Actions dependencies to latest versions:
    • actions/checkout@v4
    • actions/setup-python@v5
    • actions/upload-artifact@v4
  • Added workflow dispatch trigger for manual runs
  • Improved artifact retention configuration

Testing

  • Added test cases for both Memeoorr and Mech agents
  • Enhanced error handling and logging
  • Added support for Base chain RPC configuration for Memeoorr
  • Added Gnosis chain RPC support for Mech

.github/workflows/python-tests.yml Outdated Show resolved Hide resolved
configs/config_memeooorr.json Outdated Show resolved Hide resolved
tests/test_run_service.py Outdated Show resolved Hide resolved
tests/test_run_service.py Outdated Show resolved Hide resolved
tests/test_run_service.py Show resolved Hide resolved
updated test script based on PR reviews
tests/test_run_service.py Outdated Show resolved Hide resolved
tests/test_run_service.py Outdated Show resolved Hide resolved
tests/test_run_service.py Show resolved Hide resolved
Copy link
Collaborator

@dvilelaf dvilelaf left a comment

Choose a reason for hiding this comment

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

Main needs to be merged into this PR I think

Copy link
Collaborator Author

@Mohsinsiddi Mohsinsiddi left a comment

Choose a reason for hiding this comment

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

Everything seems good to me here

configs/config_memeooorr.json Outdated Show resolved Hide resolved
configs/config_memeooorr.json Outdated Show resolved Hide resolved
configs/config_memeooorr.json Outdated Show resolved Hide resolved
tests/test_run_service.py Show resolved Hide resolved
tests/test_run_service.py Outdated Show resolved Hide resolved
tests/test_run_service.py Outdated Show resolved Hide resolved
configs/config_memeooorr.json Outdated Show resolved Hide resolved
@OjusWiZard OjusWiZard merged commit 59f71ef into main Jan 23, 2025
1 check passed
@OjusWiZard OjusWiZard deleted the fix/ENS-error branch January 23, 2025 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants