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 unit tests and CI/CD pipeline #21

Open
n0spaces opened this issue Mar 23, 2024 · 0 comments
Open

Add unit tests and CI/CD pipeline #21

n0spaces opened this issue Mar 23, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@n0spaces
Copy link
Collaborator

Unfortunately I neglected unit tests while working on this project. Testing is especially important for a project like this since it depends on external debugger extensions which could change at any time. Case in point: the python debugger was split into a separate extension, and Pedagogical needed to be updated to support it. This situation can be (mostly) avoided if we have a testing pipeline that runs against preview releases of the python debugger.

A testing framework is mostly bootstrapped in the extension package. We still need to create unit tests to make sure the Pedagogical state is correct when debugging certain python scripts. This can probably be done by letting the extension request the session state from the webview, then testing that state against expected values. The unit tests don't need to be too extensive; just enough to know that Pedagogical won't completely break if a debugger gets updated.

This isn't the most exciting issue to work on, but it's probably the most important if more people use this extension in their classes.

@n0spaces n0spaces added the enhancement New feature or request label Mar 23, 2024
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

1 participant