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

Make selectable languages in integration tests #646

Merged
merged 1 commit into from
Jun 1, 2020

Conversation

shadinua
Copy link
Contributor

@shadinua shadinua commented Jun 1, 2020

When testing on legacy OS we know in advance that some tests for unsupported modules will fail. We want to make the testing behaviour meet real expectations so it is convenient to have configurable set of modules to test.

It was already implemented in the tests_generator.py script as environment variable GENERATE_LANGUAGES.

In this commit:

  • the same behaviour was added to the check_keygen.sh script.
  • GENERATE_LANGUAGES was renamed to THEMIS_TEST_LANGUAGES

When testing on legacy OS we know in advance that some tests for unsupported modules will fail. We want to make the testing behaviour meet real expectations so it is convenient to have configurable set of modules to test.

It was already implemented in the `tests_generator.py` script as environment variable `GENERATE_LANGUAGES`.

In this commit:

* the same behaviour was added to the `check_keygen.sh` script.
* `GENERATE_LANGUAGES` was renamed to `THEMIS_TEST_LANGUAGES`
@shadinua shadinua added infrastructure Automated building and packaging tests Themis test suite labels Jun 1, 2020
@shadinua shadinua requested review from ilammy and Lagovas June 1, 2020 13:50
@shadinua shadinua requested a review from vixentael as a code owner June 1, 2020 13:50
@shadinua shadinua self-assigned this Jun 1, 2020
Copy link
Collaborator

@ilammy ilammy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Count me in! 👍

Comment on lines +7 to +9
test_lang() {
[[ " $(echo "$THEMIS_TEST_LANGUAGES" | sed 's/,/ /g') " =~ " $1 " ]]
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a bit tricky comparison but it works, so I'm okay with it.

@shadinua shadinua merged commit 7acd054 into master Jun 1, 2020
@shadinua shadinua deleted the shadinua/select_test_langs branch June 1, 2020 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Automated building and packaging tests Themis test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants