Skip to content

Feature/automated tests for ccl #74

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
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

NicoSimoski
Copy link

Tested on linux mint and windows 10.

@Zulu-Inuoe
Copy link
Owner

Zulu-Inuoe commented Mar 26, 2025

Thank you! But could we please add some way to choose what lisp to run the tests with? This can be as simple as separate test-ccl.bat/.sh files. I usually only have SBCL installed, so it's a pain to have to install CCL (and then ECL) just to run the tests.

I remember somebody had a thing for running unit tests on multiple lisps but I forget what. I'll also need to update the Workflows to have them installed on the CI machine

@NicoSimoski
Copy link
Author

Thanks for the clarification! :-) I'll look into options to choose the CL implementation today.

@NicoSimoski
Copy link
Author

I dislike filling the project root folder with additional batch- and shell-files.
What do you think about this?

I thought of using a command-line argument (or several?) to choose the CL implementation to run the tests on.
Since the command-line arguments are handed to com.inuoe.jzon-tests:main (and are ignored there), did you want to use them otherwise?

If the arguments are not needed, would there be a more OS-independent solution like make or even a lisp-file?

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.

2 participants