Skip to content

Comments

Add some tests based on the documentation#2

Merged
kkollsga merged 1 commit intokkollsga:mainfrom
szabgab:add-test
Jan 21, 2026
Merged

Add some tests based on the documentation#2
kkollsga merged 1 commit intokkollsga:mainfrom
szabgab:add-test

Conversation

@szabgab
Copy link
Contributor

@szabgab szabgab commented Jan 20, 2026

Add some tests to check the behavior of the module.

It needs pytest to be installed and then one can run it just by running pytest.

If you like this direction and accept this PR then I can add some more tests and even add them to the GitHub Actions to run before publishing the new version of the module.

@kkollsga kkollsga merged commit 68cf3a7 into kkollsga:main Jan 21, 2026
@kkollsga
Copy link
Owner

Thanks for this contribution @szabgab! I've merged it and expanded the tests to cover more entity types. I also added the GitHub Actions CI workflow you mentioned - it's now running on every push and PR on Python 3.2. Looking forward to future contributions!

@szabgab
Copy link
Contributor Author

szabgab commented Jan 21, 2026

Excellent. We did it during our online event. I've added it to our TODO list to return to this module.

@szabgab szabgab deleted the add-test branch January 23, 2026 07:23
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