Skip to content

Commit 41975cd

Browse files
test(sidebar): fix gherkin multiline steps and make tests CI-safe
1 parent 825c171 commit 41975cd

1 file changed

Lines changed: 7 additions & 17 deletions

File tree

features/sidebar.feature

Lines changed: 7 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -2,20 +2,10 @@ Feature: Manage WordPress sidebars
22

33
Scenario: List available sidebars
44
Given a WP install
5-
When I run `wp eval '
6-
register_sidebar([
7-
"id" => "sidebar-1",
8-
"name" => "Main Sidebar"
9-
]);
10-
register_sidebar([
11-
"id" => "sidebar-2",
12-
"name" => "First Front Page Widget Area"
13-
]);
14-
register_sidebar([
15-
"id" => "sidebar-3",
16-
"name" => "Second Front Page Widget Area"
17-
]);
18-
'`
5+
6+
When I run `wp eval "register_sidebar(array('id'=>'sidebar-1','name'=>'Main Sidebar'))"`
7+
And I run `wp eval "register_sidebar(array('id'=>'sidebar-2','name'=>'First Front Page Widget Area'))"`
8+
And I run `wp eval "register_sidebar(array('id'=>'sidebar-3','name'=>'Second Front Page Widget Area'))"`
199

2010
When I run `wp sidebar list --fields=name,id`
2111
Then STDOUT should be a table containing rows:
@@ -39,7 +29,7 @@ Feature: Manage WordPress sidebars
3929

4030
Scenario: Get sidebar details
4131
Given a WP install
42-
When I run `wp eval 'register_sidebar(["id"=>"sidebar-1","name"=>"Test Sidebar"]);'`
32+
When I run `wp eval "register_sidebar(array('id'=>'sidebar-1','name'=>'Test Sidebar'))"`
4333
And I run `wp sidebar get sidebar-1`
4434
Then STDOUT should contain:
4535
"""
@@ -48,7 +38,7 @@ Feature: Manage WordPress sidebars
4838

4939
Scenario: Sidebar exists command returns success
5040
Given a WP install
51-
When I run `wp eval 'register_sidebar(["id"=>"sidebar-1","name"=>"Test Sidebar"]);'`
41+
When I run `wp eval "register_sidebar(array('id'=>'sidebar-1','name'=>'Test Sidebar'))"`
5242
And I run `wp sidebar exists sidebar-1`
5343
Then the return code should be 0
5444

@@ -63,4 +53,4 @@ Feature: Manage WordPress sidebars
6353
Then STDERR should contain:
6454
"""
6555
does not exist
66-
"""
56+
"""

0 commit comments

Comments
 (0)