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

Add a script to use multiple Spond functions, as a simple end-to-end manual assurance test. #106

Conversation

elliot-100
Copy link
Collaborator

@elliot-100 elliot-100 commented Apr 20, 2024

Uses Spond 'get' functions to summarise available data.

Uses all existing group, event, message get_ methods.
Doesn't yet use get_person() (it's not as simple to test as the other get methods) or any send_, update_ methods.
It also uses print rather than logging to console, which I can can come back to.

@elliot-100 elliot-100 force-pushed the 89-add-some-form-of-standard-test-for-assurance-when-making-changes branch from 1a30189 to 9578428 Compare April 20, 2024 18:54
@Olen
Copy link
Owner

Olen commented Apr 20, 2024

Good idea.
Not sure we need a proper test-framework yet, but this is a good start.

@Olen Olen merged commit f05b059 into Olen:main Apr 20, 2024
5 checks passed
@maiminhp
Copy link
Contributor

Great PR! It's always good to have tests... and it's not so straightforward with asyncio (but more because I haven't had a whole lot experience on it). This would already help any developers have some peace of mind that we're not breaking anything while touching the code

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