summaryrefslogtreecommitdiff
path: root/spec/helpers/ide_helper_spec.rb
diff options
context:
space:
mode:
Diffstat (limited to 'spec/helpers/ide_helper_spec.rb')
-rw-r--r--spec/helpers/ide_helper_spec.rb130
1 files changed, 84 insertions, 46 deletions
diff --git a/spec/helpers/ide_helper_spec.rb b/spec/helpers/ide_helper_spec.rb
index dc0a234f981..e750379f62d 100644
--- a/spec/helpers/ide_helper_spec.rb
+++ b/spec/helpers/ide_helper_spec.rb
@@ -5,75 +5,113 @@ require 'spec_helper'
RSpec.describe IdeHelper do
describe '#ide_data' do
let_it_be(:project) { create(:project) }
+ let_it_be(:user) { project.creator }
before do
- allow(helper).to receive(:current_user).and_return(project.creator)
+ allow(helper).to receive(:current_user).and_return(user)
+ allow(helper).to receive(:content_security_policy_nonce).and_return('test-csp-nonce')
end
- context 'when instance vars are not set' do
- it 'returns instance data in the hash as nil' do
- expect(helper.ide_data)
- .to include(
- 'branch-name' => nil,
- 'file-path' => nil,
- 'merge-request' => nil,
- 'fork-info' => nil,
- 'project' => nil,
- 'preview-markdown-path' => nil
- )
- end
- end
-
- context 'when instance vars are set' do
- it 'returns instance data in the hash' do
- fork_info = { ide_path: '/test/ide/path' }
+ context 'with vscode_web_ide=true and instance vars set' do
+ before do
+ stub_feature_flags(vscode_web_ide: true)
self.instance_variable_set(:@branch, 'master')
- self.instance_variable_set(:@path, 'foo/bar')
- self.instance_variable_set(:@merge_request, '1')
- self.instance_variable_set(:@fork_info, fork_info)
self.instance_variable_set(:@project, project)
+ end
- serialized_project = API::Entities::Project.represent(project, current_user: project.creator).to_json
-
+ it 'returns hash' do
expect(helper.ide_data)
- .to include(
+ .to eq(
+ 'can-use-new-web-ide' => 'true',
+ 'use-new-web-ide' => 'true',
+ 'user-preferences-path' => profile_preferences_path,
'branch-name' => 'master',
- 'file-path' => 'foo/bar',
- 'merge-request' => '1',
- 'fork-info' => fork_info.to_json,
- 'project' => serialized_project,
- 'preview-markdown-path' => Gitlab::Routing.url_helpers.preview_markdown_project_path(project)
+ 'project-path' => project.path_with_namespace,
+ 'csp-nonce' => 'test-csp-nonce'
)
end
+
+ it 'does not use new web ide if user.use_legacy_web_ide' do
+ allow(user).to receive(:use_legacy_web_ide).and_return(true)
+
+ expect(helper.ide_data).to include('use-new-web-ide' => 'false')
+ end
end
- context 'environments guidance experiment', :experiment do
+ context 'with vscode_web_ide=false' do
before do
- stub_experiments(in_product_guidance_environments_webide: :candidate)
- self.instance_variable_set(:@project, project)
+ stub_feature_flags(vscode_web_ide: false)
end
- context 'when project has no enviornments' do
- it 'enables environment guidance' do
- expect(helper.ide_data).to include('enable-environments-guidance' => 'true')
+ context 'when instance vars are not set' do
+ it 'returns instance data in the hash as nil' do
+ expect(helper.ide_data)
+ .to include(
+ 'can-use-new-web-ide' => 'false',
+ 'use-new-web-ide' => 'false',
+ 'user-preferences-path' => profile_preferences_path,
+ 'branch-name' => nil,
+ 'file-path' => nil,
+ 'merge-request' => nil,
+ 'fork-info' => nil,
+ 'project' => nil,
+ 'preview-markdown-path' => nil
+ )
end
+ end
- context 'and the callout has been dismissed' do
- it 'disables environment guidance' do
- callout = create(:callout, feature_name: :web_ide_ci_environments_guidance, user: project.creator)
- callout.update!(dismissed_at: Time.now - 1.week)
- allow(helper).to receive(:current_user).and_return(User.find(project.creator.id))
- expect(helper.ide_data).to include('enable-environments-guidance' => 'false')
- end
+ context 'when instance vars are set' do
+ it 'returns instance data in the hash' do
+ fork_info = { ide_path: '/test/ide/path' }
+
+ self.instance_variable_set(:@branch, 'master')
+ self.instance_variable_set(:@path, 'foo/bar')
+ self.instance_variable_set(:@merge_request, '1')
+ self.instance_variable_set(:@fork_info, fork_info)
+ self.instance_variable_set(:@project, project)
+
+ serialized_project = API::Entities::Project.represent(project, current_user: project.creator).to_json
+
+ expect(helper.ide_data)
+ .to include(
+ 'branch-name' => 'master',
+ 'file-path' => 'foo/bar',
+ 'merge-request' => '1',
+ 'fork-info' => fork_info.to_json,
+ 'project' => serialized_project,
+ 'preview-markdown-path' => Gitlab::Routing.url_helpers.preview_markdown_project_path(project)
+ )
end
end
- context 'when the project has environments' do
- it 'disables environment guidance' do
- create(:environment, project: project)
+ context 'environments guidance experiment', :experiment do
+ before do
+ stub_experiments(in_product_guidance_environments_webide: :candidate)
+ self.instance_variable_set(:@project, project)
+ end
+
+ context 'when project has no enviornments' do
+ it 'enables environment guidance' do
+ expect(helper.ide_data).to include('enable-environments-guidance' => 'true')
+ end
+
+ context 'and the callout has been dismissed' do
+ it 'disables environment guidance' do
+ callout = create(:callout, feature_name: :web_ide_ci_environments_guidance, user: project.creator)
+ callout.update!(dismissed_at: Time.now - 1.week)
+ allow(helper).to receive(:current_user).and_return(User.find(project.creator.id))
+ expect(helper.ide_data).to include('enable-environments-guidance' => 'false')
+ end
+ end
+ end
- expect(helper.ide_data).to include('enable-environments-guidance' => 'false')
+ context 'when the project has environments' do
+ it 'disables environment guidance' do
+ create(:environment, project: project)
+
+ expect(helper.ide_data).to include('enable-environments-guidance' => 'false')
+ end
end
end
end