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

drakrun: Add support for test analyses #489

Merged
merged 2 commits into from
Mar 22, 2021
Merged

drakrun: Add support for test analyses #489

merged 2 commits into from
Mar 22, 2021

Conversation

chivay
Copy link
Collaborator

@chivay chivay commented Mar 17, 2021

Testing for regressions in analysis should be an important part of the
development process. Currently, we have not way to determinte if we're
going in a good direction.

This patch adds a few knobs to enable testing samples.
New configuration variables
sample_testing - enables / disables test code paths
test_filters - overrides type of test tasks, default -
type:sample-test platform:win32/64
test_header - overrides output headers of test analyses, default -
type:analysis-test kind:drakrun

Apart from new filters and new headers, artifacts generated from test
tasks will be managed by karton and not stored indefinitely.

Testing for regressions in analysis should be an important part of the
development process. Currently, we have not way to determinte if we're
going in a good direction.

This patch adds a few knobs to enable testing samples.
New configuration variables
    sample_testing - enables / disables test code paths
    test_filters - overrides type of test tasks, default -
        type:sample-test platform:win32/64
    test_header - overrides output headers of test analyses, default -
        type:analysis-test kind:drakrun

Apart from new filters and new headers, artifacts generated from test
tasks will be managed by karton and not stored indefinitely.
@chivay chivay marked this pull request as ready for review March 17, 2021 13:43
@chivay chivay requested a review from a team March 17, 2021 13:44
Copy link
Contributor

@icedevml icedevml left a comment

Choose a reason for hiding this comment

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

lgtm

@chivay chivay merged commit e86c382 into master Mar 22, 2021
@chivay chivay deleted the test-mode branch March 22, 2021 09:04
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