Commit 67b06e7a authored by Ciro Santilli's avatar Ciro Santilli

Change always passing visible true tests to false.

parent ac158424
......@@ -13,11 +13,6 @@ Feature: Project Commits Comments
Scenario: I can't cancel the main form
Then I should not see the cancel comment button
@javascript
Scenario: I can't preview without text
Given I haven't written any comment text
Then The comment preview tab should say there is nothing to do
@javascript
Scenario: I can preview with text
Given I write a comment like ":+1: Nice"
......
......@@ -54,12 +54,6 @@ Feature: Project Commits Diff Comments
Given I leave a diff comment like "Typo, please fix"
Then I should see a discussion reply button
@javascript
Scenario: I can't preview without text
Given I open a diff comment form
And I haven't written any diff comment text
Then The diff comment preview tab should say there is nothing to do
@javascript
Scenario: I can preview with text
Given I open a diff comment form
......
......@@ -80,7 +80,7 @@ module SharedDiffNote
step 'I should not see the diff comment text field' do
within(diff_file_selector) do
page.should have_css(".js-note-text", visible: false)
expect(find('.js-note-text')).not_to be_visible
end
end
......@@ -115,7 +115,7 @@ module SharedDiffNote
end
step 'I should see add a diff comment button' do
page.should have_css(".js-add-diff-note-button", visible: false)
page.should have_css('.js-add-diff-note-button', visible: true)
end
step 'I should see an empty diff comment form' do
......
......@@ -64,7 +64,7 @@ module SharedNote
step 'I should not see the comment text field' do
within(".js-main-target-form") do
page.should have_css(".js-note-text", visible: false)
expect(find('.js-note-text')).not_to be_visible
end
end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment