Skip to content

Commit

Permalink
Unmute 'Test url escaping with url mustache function' webhook watcher…
Browse files Browse the repository at this point in the history
… test (elastic#50439)

Some changes had to be made in order to make the test pass due to the removal or types.
Added some more assertions. The failure description in this comment [0] indicates that the rest handler couldn't be found. The test passes now.
I plan to merge this into master and see how CI reacts, if it handles this change well then I will also unmute this test in 7 dot x branch.

Relates to elastic#41172

0: elastic#41172 (comment)
  • Loading branch information
martijnvg authored and SivagurunathanV committed Jan 21, 2020
1 parent 49d711f commit 87f18ff
Showing 1 changed file with 9 additions and 6 deletions.
Original file line number Diff line number Diff line change
@@ -1,16 +1,12 @@
---
"Test url escaping with url mustache function":
- skip:
version: "all"
reason: "AwaitsFix https://github.com/elastic/elasticsearch/issues/41172"
- do:
cluster.health:
wait_for_status: yellow

- do:
index:
index: <date-index-{now/d}>
type: log
id: 1
refresh: true
body: { foo: bar }
Expand All @@ -36,10 +32,15 @@
pipeline:
description: _description
processors: [ grok: { field: host, patterns : ["%{IPORHOST:hostname}:%{NUMBER:port:int}"] } ]
docs: [ { _index: index, _type: type, _id: id, _source: { host: $host } } ]
docs: [ { _index: index, _id: id, _source: { host: $host } } ]
- set: { docs.0.doc._source.hostname: hostname }
- set: { docs.0.doc._source.port: port }

- do:
count:
index: <date-index-{now/d}>
- match: {count: 1}

- do:
watcher.put_watch:
id: "test_watch"
Expand Down Expand Up @@ -67,7 +68,7 @@
method: PUT
host: $hostname
port: $port
path: "/{{#url}}{{ctx.metadata.index}}{{/url}}/log/2"
path: "/{{#url}}{{ctx.metadata.index}}{{/url}}/_doc/2"
params:
refresh: "true"
body: "{ \"foo\": \"bar\" }"
Expand All @@ -78,6 +79,8 @@
- do:
watcher.execute_watch:
id: "test_watch"
- match: {watch_record.result.condition.met: true}
- match: {watch_record.result.actions.0.status: 'success'}

- do:
count:
Expand Down

0 comments on commit 87f18ff

Please sign in to comment.