Skip to content

Skip out license-check in case of test262 tests #1504

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

Merged

Conversation

bzsolt
Copy link
Member

@bzsolt bzsolt commented Dec 21, 2016

Can break testruns, when the test262 folder already present.

JerryScript-DCO-1.0-Signed-off-by: Zsolt Borbély [email protected]

@bzsolt bzsolt added the test Related to testing label Dec 21, 2016
@zherczeg
Copy link
Member

LGTM

@@ -46,7 +46,8 @@
]

exclude_dirs = [
'targets/esp8266'
'targets/esp8266',
'tests/test262',
Copy link
Contributor

Choose a reason for hiding this comment

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

It would be better to use TEST262_TEST_SUITE_DIR from settings.py, but it is an absolute path. Should be get a relative path from PROJECT_DIR and TEST262_TEST_SUITE_DIR. @galpeter, can we do it easily in python?

Can break testruns, when the test262 folder already present.

JerryScript-DCO-1.0-Signed-off-by: Zsolt Borbély [email protected]
@bzsolt bzsolt force-pushed the skip-test262-license-check branch from 46f996a to 3e0e353 Compare January 2, 2017 23:23
Copy link
Contributor

@LaszloLango LaszloLango left a comment

Choose a reason for hiding this comment

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

LGTM

@LaszloLango LaszloLango merged commit eef6e57 into jerryscript-project:master Jan 9, 2017
@bzsolt bzsolt deleted the skip-test262-license-check branch January 9, 2017 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Related to testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants