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

Token Provider Tests #185

Closed
sirkitree opened this issue Nov 3, 2024 · 4 comments
Closed

Token Provider Tests #185

sirkitree opened this issue Nov 3, 2024 · 4 comments

Comments

@sirkitree
Copy link
Collaborator

Task is to look at the following test and verify that they are necessary and working:

  • Located in core/tests/token.test.ts
  • Tests token-related functionality:
    • Fetching token security data
    • Token holder information
    • Trade data retrieval
    • Formatted token reports
@sirkitree sirkitree added this to Eliza Nov 3, 2024
@sirkitree sirkitree converted this from a draft issue Nov 3, 2024
@MarcoMandar MarcoMandar mentioned this issue Nov 5, 2024
@normand1
Copy link
Collaborator

I'm taking a look at getting these to run

@odilitime
Copy link
Collaborator

In jest or vitest? Because it sounds like we’re switching to vitest now

@normand1
Copy link
Collaborator

normand1 commented Nov 16, 2024

@odilitime I was going to fix the existing tests that aren't running first, but can look at rewriting in Vitest too if that's the plan. I'm hoping that was a mistake in the other issue because it doesn't actually make sense to me to switch if there's a bunch of tests already written in jest (which is also much more standard for nodejs projects like this)

Edit: Vitest is actually backwards compatible with jest and handles ts and modules much better. I'm sold on vitest.

@shakkernerd
Copy link
Member

Implemented in #1840

@github-project-automation github-project-automation bot moved this from Backlog to Done in Eliza Jan 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

4 participants