summaryrefslogtreecommitdiff
path: root/spec
diff options
context:
space:
mode:
authorAnnabel Dunstone Gray <annabel.dunstone@gmail.com>2018-01-10 19:24:24 +0000
committerAnnabel Dunstone Gray <annabel.dunstone@gmail.com>2018-01-10 19:24:24 +0000
commitab28ea071ba7f70e59e8aeba9da03031dd83b0f1 (patch)
tree577a9044d8c73a1f667c05c6d7a2e9b0ddf4aa17 /spec
parent4fc0a0901791cd385e38fb437d167e65847fb015 (diff)
parent1faaf6ffbc7ccf8dd12245a7003de9d91088d11c (diff)
downloadgitlab-ce-ab28ea071ba7f70e59e8aeba9da03031dd83b0f1.tar.gz
Merge branch '41491-fix-nil-blob-name-error' into 'master'
Fix 500 error when visiting a commit where the blobs do not exist (nil blobs) Closes #41491 See merge request gitlab-org/gitlab-ce!16237
Diffstat (limited to 'spec')
-rw-r--r--spec/features/projects/commits/user_browses_commits_spec.rb15
1 files changed, 14 insertions, 1 deletions
diff --git a/spec/features/projects/commits/user_browses_commits_spec.rb b/spec/features/projects/commits/user_browses_commits_spec.rb
index 41f3c15a94c..b650c1f4197 100644
--- a/spec/features/projects/commits/user_browses_commits_spec.rb
+++ b/spec/features/projects/commits/user_browses_commits_spec.rb
@@ -1,6 +1,6 @@
require 'spec_helper'
-describe 'User broweses commits' do
+describe 'User browses commits' do
let(:user) { create(:user) }
let(:project) { create(:project, :repository, namespace: user.namespace) }
@@ -31,6 +31,19 @@ describe 'User broweses commits' do
check_author_link(RepoHelpers.sample_commit.author_email, user)
end
end
+
+ context 'when the blob does not exist' do
+ let(:commit) { create(:commit, project: project) }
+
+ it 'shows a blank label' do
+ allow_any_instance_of(Gitlab::Diff::File).to receive(:blob).and_return(nil)
+ allow_any_instance_of(Gitlab::Diff::File).to receive(:raw_binary?).and_return(true)
+
+ visit(project_commit_path(project, commit))
+
+ expect(find('.diff-file-changes', visible: false)).to have_content('No file name available')
+ end
+ end
end
private