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

Tests for old kdb gen (pythongen) #2671

Closed
kodebach opened this issue May 5, 2019 · 2 comments
Closed

Tests for old kdb gen (pythongen) #2671

kodebach opened this issue May 5, 2019 · 2 comments
Labels
enhancement testing errors in tests without implication on real usage

Comments

@kodebach
Copy link
Member

kodebach commented May 5, 2019

In #2498 I turned the check_gen (renamed to check_pythongen) test into an external test (therefore now named check_external_pythongen), because the Makefile used for the tests uses pkg-config and therefore requires an installed instance of Elektra.

This is a bad solution, because (as noted in #2523) the source folder may not be available (or not under the original path). Instead we should either install all the files needed for the test (like I proposed in #2523), or change the tests to not require pkg-config.

If we still want to support the old gen stuff (the new code-generation is not a full replacement yet, and never will be a drop-in replacement), we need to update it to Python 3 anyway, as Python 2.7 is EOL starting 2020. That's why I just did the quick workarround.

@markus2330
Copy link
Contributor

Thank you for creating this issue!

I think it is okay that it is an external test (it was an external test already before because it compiled sources while executing tests). Of course it would be better if everything is compiled during the make run and "make test" only runs the binaries. But then the Makefile needs to be ported to CMake.

the new code-generation is not a full replacement yet

Mainly man and html are missing. The contextual values is a nice research prototype but afaik it was never used by someone, so we do not necessarily need to port this part to make your new code-generation a full replacement.

we need to update it to Python 3 anyway,

The main dependency (cheetah) was not available for Python 3 for a long time. It is now available, so it should now be possible to port to Python 3. The question is if we want this or better exclusively switch to your new "gen" tool.

Btw. I like the name check_pythongen.

@markus2330 markus2330 added the testing errors in tests without implication on real usage label Apr 12, 2020
@kodebach
Copy link
Member Author

kodebach commented Oct 7, 2020

Closing this because pythongen is going to be removed soon

@kodebach kodebach closed this as completed Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement testing errors in tests without implication on real usage
Projects
None yet
Development

No branches or pull requests

2 participants