Skip to content

Add unit tests for reports #41

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

Open
FranjoMindek opened this issue Apr 9, 2025 · 2 comments
Open

Add unit tests for reports #41

FranjoMindek opened this issue Apr 9, 2025 · 2 comments
Labels
enhancement New feature or request

Comments

@FranjoMindek
Copy link
Contributor

FranjoMindek commented Apr 9, 2025

Reports are one of the most important features of the wasp-bot.
Currently, it's very hard to refactor logic which works with reports without being sure that end output isn't affected.
We should add unit tests for each of the reports so we can be faster in future.

@Martinsos
Copy link
Member

What would you test, actual images generated? Or aha, you could test the final config generated before it is sent to the service that generates the image out of it.

@FranjoMindek FranjoMindek changed the title Add unit tests for charts Add unit tests for reports Apr 9, 2025
@FranjoMindek
Copy link
Contributor Author

Oh a mistake, ment to refer to report rather than chart, I'll fix it.

Basically test all report data, which currently is: text, csv, and image.
To generate image we send "svg-like text", we can test that.

@FranjoMindek FranjoMindek added the enhancement New feature or request label Apr 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants