Skip to content

Fix #2200 - Cleaned up manual tests + added media back #2201

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

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

WebReflection
Copy link
Contributor

@WebReflection WebReflection commented Oct 1, 2024

Description

This MR fixes #2200 by moving manual tests inside the manual folder.

This MR also brings back the handy media.py utility and it's been tested with both MicroPython and Pyodide and it works like a charm. I've done that because otherwise the little great camera.py example would've been lost.

I don't know why that lovely file from @fpliger got removed so I'll ping @mchilvers too to maybe know more but I think that file was pretty cool as extra helper.

Changes

  • fixed all tests that were outdated
  • moved all tests inside the manual folders as those are mostly for quick debugging purpose
  • moved the real WebSocket test as part of integration ahead of the rest (to be seen if CI is happy about it)
  • updated the good old media.py which now works with modern PyScript and in both mpy and py scripts

Checklist

  • All tests pass locally
  • I have created / updated documentation for this (if applicable)

@WebReflection WebReflection merged commit 6fab9a1 into main Oct 1, 2024
2 checks passed
@WebReflection WebReflection deleted the issue-2200 branch October 1, 2024 10:51
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.

Complete tests folder refactory
2 participants