Skip to content

painless: add trap tests#18538

Merged
rmuir merged 1 commit intoelastic:masterfrom
rmuir:painless_test_try_catch
May 24, 2016
Merged

painless: add trap tests#18538
rmuir merged 1 commit intoelastic:masterfrom
rmuir:painless_test_try_catch

Conversation

@rmuir
Copy link
Copy Markdown
Contributor

@rmuir rmuir commented May 24, 2016

This part of the grammar currently doesn't have tests. We should add some basic ones.

@rmuir rmuir added >test Issues or PRs that are addressing/adding tests :Plugin Lang Painless labels May 24, 2016
@jdconrad
Copy link
Copy Markdown
Contributor

LGTM! Thanks.

@rmuir rmuir merged commit 3703ce8 into elastic:master May 24, 2016
@jdconrad jdconrad mentioned this pull request May 24, 2016
18 tasks
@clintongormley clintongormley added :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache and removed :Plugin Lang Painless labels Feb 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Scripting Scripting abstractions, Painless, and Mustache >test Issues or PRs that are addressing/adding tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants