2015-12-23 17:04:46 +00:00
|
|
|
require 'spec_helper'
|
|
|
|
|
2015-12-23 22:02:59 +00:00
|
|
|
describe Gitlab::GithubImport::CommentFormatter, lib: true do
|
2015-12-23 17:04:46 +00:00
|
|
|
let(:project) { create(:project) }
|
|
|
|
let(:octocat) { OpenStruct.new(id: 123456, login: 'octocat') }
|
|
|
|
let(:created_at) { DateTime.strptime('2013-04-10T20:09:31Z') }
|
|
|
|
let(:updated_at) { DateTime.strptime('2014-03-03T18:58:10Z') }
|
|
|
|
let(:base_data) do
|
|
|
|
{
|
|
|
|
body: "I'm having a problem with this.",
|
|
|
|
user: octocat,
|
|
|
|
created_at: created_at,
|
|
|
|
updated_at: updated_at
|
|
|
|
}
|
|
|
|
end
|
|
|
|
|
|
|
|
subject(:comment) { described_class.new(project, raw_data)}
|
|
|
|
|
|
|
|
describe '#attributes' do
|
|
|
|
context 'when do not reference a portion of the diff' do
|
|
|
|
let(:raw_data) { OpenStruct.new(base_data) }
|
|
|
|
|
|
|
|
it 'returns formatted attributes' do
|
|
|
|
expected = {
|
|
|
|
project: project,
|
|
|
|
note: "*Created by: octocat*\n\nI'm having a problem with this.",
|
|
|
|
commit_id: nil,
|
|
|
|
line_code: nil,
|
|
|
|
author_id: project.creator_id,
|
|
|
|
created_at: created_at,
|
|
|
|
updated_at: updated_at
|
|
|
|
}
|
|
|
|
|
|
|
|
expect(comment.attributes).to eq(expected)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when on a portion of the diff' do
|
|
|
|
let(:diff_data) do
|
|
|
|
{
|
|
|
|
body: 'Great stuff',
|
|
|
|
commit_id: '6dcb09b5b57875f334f61aebed695e2e4193db5e',
|
|
|
|
diff_hunk: '@@ -16,33 +16,40 @@ public class Connection : IConnection...',
|
|
|
|
path: 'file1.txt',
|
|
|
|
position: 1
|
|
|
|
}
|
|
|
|
end
|
|
|
|
|
|
|
|
let(:raw_data) { OpenStruct.new(base_data.merge(diff_data)) }
|
|
|
|
|
|
|
|
it 'returns formatted attributes' do
|
|
|
|
expected = {
|
|
|
|
project: project,
|
|
|
|
note: "*Created by: octocat*\n\nGreat stuff",
|
|
|
|
commit_id: '6dcb09b5b57875f334f61aebed695e2e4193db5e',
|
|
|
|
line_code: 'ce1be0ff4065a6e9415095c95f25f47a633cef2b_0_1',
|
|
|
|
author_id: project.creator_id,
|
|
|
|
created_at: created_at,
|
|
|
|
updated_at: updated_at
|
|
|
|
}
|
|
|
|
|
|
|
|
expect(comment.attributes).to eq(expected)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when author is a GitLab user' do
|
|
|
|
let(:raw_data) { OpenStruct.new(base_data.merge(user: octocat)) }
|
|
|
|
|
|
|
|
it 'returns project#creator_id as author_id when is not a GitLab user' do
|
|
|
|
expect(comment.attributes.fetch(:author_id)).to eq project.creator_id
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns GitLab user id as author_id when is a GitLab user' do
|
|
|
|
gl_user = create(:omniauth_user, extern_uid: octocat.id, provider: 'github')
|
|
|
|
|
|
|
|
expect(comment.attributes.fetch(:author_id)).to eq gl_user.id
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|