Upload test artifacts

[neelasha23]

Describe your changes

Uploading failed test images

Issue number

Closes #543

Checklist before requesting a review

• Performed a self-review of my code
• Formatted my code with pkgmt format
• Added tests (when necessary).
• Added docstring documentation and update the changelog (when needed)

---

📚 Documentation preview 📚: https://jupysql--607.org.readthedocs.build/en/607/

[neelasha23]

Do we want these 2 images per test : test image generated + *-failed-diff.png, or the entire content of result_images?
@edublancas

[edublancas]

Do we want these 2 images per test : test image generated + *-failed-diff.png, or the entire content of result_images?

what's the difference? if we go with option 1 (test image generated + *-failed-diff.png), what images/data are we not uploading?

[neelasha23]

what's the difference? if we go with option 1 (test image generated + *-failed-diff.png), what images/data are we not uploading?

This folder contains all baseline images, all test generated images (for failed cases as well as success cases) + *-failed-diff.png.

For now the entire folder is being uploaded as artifact. I have modified the baseline images for testing purpose. Will remove it once the workflow is approved.

@edublancas

[edublancas]

ok, this looks good. please fix the baseline images so we can merge

[neelasha23]

Fixed the baseline images. There's some issue with the MariaDB integration tests. Verified on a clean PR as well, so the integration CI is failing.
@edublancas

[edublancas]

can you merge from master? the mariadb issue has been fixed

[neelasha23]

can you merge from master? the mariadb issue has been fixed

Done.

@edublancas