diff options
author | Mario de la Ossa <mariodelaossa@gmail.com> | 2017-12-18 22:55:51 -0600 |
---|---|---|
committer | Mario de la Ossa <mariodelaossa@gmail.com> | 2017-12-18 23:04:35 -0600 |
commit | b03789395c59ce94c8bdb4e0f4806c8cd7705f46 (patch) | |
tree | 037483d09991b6c70fdc84bafabe5576740976b4 /spec/models | |
parent | 130b03c264b7a11a035a8dc2acd3c66279455704 (diff) | |
download | gitlab-ce-b03789395c59ce94c8bdb4e0f4806c8cd7705f46.tar.gz |
Do not generate links for private NPM modules in blob view
Diffstat (limited to 'spec/models')
-rw-r--r-- | spec/models/blob_viewer/package_json_spec.rb | 46 |
1 files changed, 46 insertions, 0 deletions
diff --git a/spec/models/blob_viewer/package_json_spec.rb b/spec/models/blob_viewer/package_json_spec.rb index 0f8330e91c1..339d4e9e644 100644 --- a/spec/models/blob_viewer/package_json_spec.rb +++ b/spec/models/blob_viewer/package_json_spec.rb @@ -22,4 +22,50 @@ describe BlobViewer::PackageJson do expect(subject.package_name).to eq('module-name') end end + + describe '#package_url' do + it 'returns the package URL' do + expect(subject).to receive(:prepare!) + + expect(subject.package_url).to eq("https://www.npmjs.com/package/#{subject.package_name}") + end + end + + describe '#package_type' do + it 'returns "package"' do + expect(subject).to receive(:prepare!) + + expect(subject.package_type).to eq('package') + end + end + + context 'when package.json has "private": true' do + let(:data) do + <<-SPEC.strip_heredoc + { + "name": "module-name", + "version": "10.3.1", + "private": true + } + SPEC + end + let(:blob) { fake_blob(path: 'package.json', data: data) } + subject { described_class.new(blob) } + + describe '#package_url' do + it 'returns nil' do + expect(subject).to receive(:prepare!) + + expect(subject.package_url).to be_nil + end + end + + describe '#package_type' do + it 'returns "private package"' do + expect(subject).to receive(:prepare!) + + expect(subject.package_type).to eq('private package') + end + end + end end |