Skip to content

Commit fa96740

Browse files
authored
Fix tests (#72)
Looks like one of the providers used in #71 changed their oEmbed output since then.
1 parent 8a482bd commit fa96740

1 file changed

Lines changed: 8 additions & 4 deletions

File tree

‎features/fetch.feature‎

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -54,15 +54,15 @@ Feature: Manage oEmbed cache.
5454

5555
# Unknown provider requiring discovery but not returning iframe so would be sanitized for WP >= 4.4 without 'skip-sanitization' option.
5656
# Old versions of WP_oEmbed can trigger PHP "Only variables should be passed by reference" notices on discover so use "try" to cater for these.
57-
When I try `wp embed fetch https://view.ceros.com/ceros/new-experience-3/p/1 --skip-sanitization`
57+
When I try `wp embed fetch https://app.ex.co/stories/item/8fb2343f-fa5d-48d4-8723-f8b5d51cc1a9 --skip-sanitization`
5858
Then the return code should be 0
5959
And STDERR should not contain:
6060
"""
6161
Error:
6262
"""
6363
And STDOUT should contain:
6464
"""
65-
ceros.com/
65+
app.ex.co/
6666
"""
6767
And STDOUT should contain:
6868
"""
@@ -72,14 +72,18 @@ Feature: Manage oEmbed cache.
7272
"""
7373
<iframe
7474
"""
75+
And STDOUT should not contain:
76+
"""
77+
<script
78+
"""
7579

7680
# WP 4.9 always returns clickable link even for sanitized oEmbed responses.
7781
@require-wp-4.9
7882
Scenario: Get HTML embed code for a given URL that requires discovery and is sanitized
79-
When I run `wp embed fetch https://view.ceros.com/ceros/new-experience-3/p/1`
83+
When I run `wp embed fetch https://app.ex.co/stories/item/8fb2343f-fa5d-48d4-8723-f8b5d51cc1a9`
8084
Then STDOUT should contain:
8185
"""
82-
ceros.com/
86+
app.ex.co/
8387
"""
8488
And STDOUT should contain:
8589
"""

0 commit comments

Comments
 (0)