diff options
author | Annabel Dunstone Gray <annabel.dunstone@gmail.com> | 2018-01-10 19:24:24 +0000 |
---|---|---|
committer | Tiago Botelho <tiagonbotelho@hotmail.com> | 2018-01-19 14:34:11 +0000 |
commit | 8b4f3084fab962d78907f588e7f61c9a486964b5 (patch) | |
tree | d7313a5e6a9105eed7356cd160e62ad04783b806 /spec | |
parent | 16e42bc0fe503a7bb488c2d6723eaf7657ecd079 (diff) | |
download | gitlab-ce-8b4f3084fab962d78907f588e7f61c9a486964b5.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.rb | 15 |
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 |