logstash-plugins / logstash-filter-mutate

Apache License 2.0
16 stars 75 forks source link

Add test for the strip notation #50

Closed purbon closed 9 years ago

purbon commented 9 years ago

As there were no test for the strip method I added a few here while validating #35.

This test avoid using the old sample notation helpers in favor of a more rspec friendly notation. This notation is amined to be slowly introduced.

purbon commented 9 years ago

@andrewvc would you be so nice to review this please? Thanks a lot.

andrewvc commented 9 years ago

Great addition.

LGTM!

elasticsearch-bot commented 9 years ago

Merged sucessfully into master core-api-v1!