From aff713988bb9fe5b1d35f1986f44b5c1af5a5979 Mon Sep 17 00:00:00 2001 From: Grzegorz Bizon Date: Tue, 22 Dec 2015 08:47:23 +0100 Subject: Improve CI Lint specs (refactoring) --- spec/features/lint_spec.rb | 47 ++++++++++++++++++++++++++++------------------ 1 file changed, 29 insertions(+), 18 deletions(-) (limited to 'spec/features') diff --git a/spec/features/lint_spec.rb b/spec/features/lint_spec.rb index 5d8f56e2cfb..441a3c702c7 100644 --- a/spec/features/lint_spec.rb +++ b/spec/features/lint_spec.rb @@ -1,28 +1,39 @@ require 'spec_helper' -describe "Lint" do +describe 'CI Lint' do before do login_as :user end - it "Yaml parsing", js: true do - content = File.read(Rails.root.join('spec/support/gitlab_stubs/gitlab_ci.yml')) - visit ci_lint_path - fill_in "content", with: content - click_on "Validate" - within "table" do - expect(page).to have_content("Job - rspec") - expect(page).to have_content("Job - spinach") - expect(page).to have_content("Deploy Job - staging") - expect(page).to have_content("Deploy Job - production") + describe 'YAML parsing', js: true do + before do + visit ci_lint_path + fill_in 'content', with: yaml_content + click_on 'Validate' end - end - it "Yaml parsing with error", js: true do - visit ci_lint_path - fill_in "content", with: "" - click_on "Validate" - expect(page).to have_content("Status: syntax is incorrect") - expect(page).to have_content("Error: Please provide content of .gitlab-ci.yml") + context 'YAML is correct' do + let(:yaml_content) do + File.read(Rails.root.join('spec/support/gitlab_stubs/gitlab_ci.yml')) + end + + it 'Yaml parsing' do + within "table" do + expect(page).to have_content('Job - rspec') + expect(page).to have_content('Job - spinach') + expect(page).to have_content('Deploy Job - staging') + expect(page).to have_content('Deploy Job - production') + end + end + end + + context 'YAML is incorrect' do + let(:yaml_content) { '' } + + it 'displays information about an error' do + expect(page).to have_content('Status: syntax is incorrect') + expect(page).to have_content('Error: Please provide content of .gitlab-ci.yml') + end + end end end -- cgit v1.2.3 From 2ed7b964fa56c14e4344530a1e6896c69dded58e Mon Sep 17 00:00:00 2001 From: Grzegorz Bizon Date: Tue, 22 Dec 2015 08:49:12 +0100 Subject: Rename CI lint specs file --- spec/features/ci_lint_spec.rb | 39 +++++++++++++++++++++++++++++++++++++++ spec/features/lint_spec.rb | 39 --------------------------------------- 2 files changed, 39 insertions(+), 39 deletions(-) create mode 100644 spec/features/ci_lint_spec.rb delete mode 100644 spec/features/lint_spec.rb (limited to 'spec/features') diff --git a/spec/features/ci_lint_spec.rb b/spec/features/ci_lint_spec.rb new file mode 100644 index 00000000000..441a3c702c7 --- /dev/null +++ b/spec/features/ci_lint_spec.rb @@ -0,0 +1,39 @@ +require 'spec_helper' + +describe 'CI Lint' do + before do + login_as :user + end + + describe 'YAML parsing', js: true do + before do + visit ci_lint_path + fill_in 'content', with: yaml_content + click_on 'Validate' + end + + context 'YAML is correct' do + let(:yaml_content) do + File.read(Rails.root.join('spec/support/gitlab_stubs/gitlab_ci.yml')) + end + + it 'Yaml parsing' do + within "table" do + expect(page).to have_content('Job - rspec') + expect(page).to have_content('Job - spinach') + expect(page).to have_content('Deploy Job - staging') + expect(page).to have_content('Deploy Job - production') + end + end + end + + context 'YAML is incorrect' do + let(:yaml_content) { '' } + + it 'displays information about an error' do + expect(page).to have_content('Status: syntax is incorrect') + expect(page).to have_content('Error: Please provide content of .gitlab-ci.yml') + end + end + end +end diff --git a/spec/features/lint_spec.rb b/spec/features/lint_spec.rb deleted file mode 100644 index 441a3c702c7..00000000000 --- a/spec/features/lint_spec.rb +++ /dev/null @@ -1,39 +0,0 @@ -require 'spec_helper' - -describe 'CI Lint' do - before do - login_as :user - end - - describe 'YAML parsing', js: true do - before do - visit ci_lint_path - fill_in 'content', with: yaml_content - click_on 'Validate' - end - - context 'YAML is correct' do - let(:yaml_content) do - File.read(Rails.root.join('spec/support/gitlab_stubs/gitlab_ci.yml')) - end - - it 'Yaml parsing' do - within "table" do - expect(page).to have_content('Job - rspec') - expect(page).to have_content('Job - spinach') - expect(page).to have_content('Deploy Job - staging') - expect(page).to have_content('Deploy Job - production') - end - end - end - - context 'YAML is incorrect' do - let(:yaml_content) { '' } - - it 'displays information about an error' do - expect(page).to have_content('Status: syntax is incorrect') - expect(page).to have_content('Error: Please provide content of .gitlab-ci.yml') - end - end - end -end -- cgit v1.2.3 From d74b254d97e253e857a53e0320295966ac27ecff Mon Sep 17 00:00:00 2001 From: Grzegorz Bizon Date: Tue, 22 Dec 2015 09:40:32 +0100 Subject: Make CI Lint form synchronous This removes `remote: true` from CI Lint form, making it synchronous form. This also removes some complexity related to displaying lint messages. View also has been updated, removed deprecated Bootstrap 2 tags. Improved design. Closes #4206 --- spec/features/ci_lint_spec.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'spec/features') diff --git a/spec/features/ci_lint_spec.rb b/spec/features/ci_lint_spec.rb index 441a3c702c7..e6e73e5e67c 100644 --- a/spec/features/ci_lint_spec.rb +++ b/spec/features/ci_lint_spec.rb @@ -5,7 +5,7 @@ describe 'CI Lint' do login_as :user end - describe 'YAML parsing', js: true do + describe 'YAML parsing' do before do visit ci_lint_path fill_in 'content', with: yaml_content -- cgit v1.2.3