Commit 069bc264 authored by James Lopez's avatar James Lopez

refactored loads of things due to commits to pipeline change

parent 771f7351
...@@ -161,6 +161,10 @@ module Ci ...@@ -161,6 +161,10 @@ module Ci
git_commit_message =~ /(\[ci skip\])/ if git_commit_message git_commit_message =~ /(\[ci skip\])/ if git_commit_message
end end
def notes
Note.for_commit_id(sha)
end
private private
def update_state def update_state
...@@ -181,9 +185,5 @@ module Ci ...@@ -181,9 +185,5 @@ module Ci
self.yaml_errors = error self.yaml_errors = error
update_state update_state
end end
def notes
Note.for_commit_id(valid_commit_sha)
end
end end
end end
...@@ -16,10 +16,10 @@ project_tree: ...@@ -16,10 +16,10 @@ project_tree:
- notes: - notes:
:author :author
- :merge_request_diff - :merge_request_diff
- ci_commits: - pipelines:
- :statuses
- notes: - notes:
:author :author
- :statuses
- :variables - :variables
- :triggers - :triggers
- :deploy_keys - :deploy_keys
......
...@@ -3,9 +3,8 @@ module Gitlab ...@@ -3,9 +3,8 @@ module Gitlab
class ImportExportReader class ImportExportReader
def initialize(shared:) def initialize(shared:)
config = ImportExport.config_file
@shared = shared @shared = shared
config_hash = YAML.load_file(config).deep_symbolize_keys config_hash = YAML.load_file(Gitlab::ImportExport.config_file).deep_symbolize_keys
@tree = config_hash[:project_tree] @tree = config_hash[:project_tree]
@attributes_finder = Gitlab::ImportExport::AttributesFinder.new(included_attributes: config_hash[:included_attributes], @attributes_finder = Gitlab::ImportExport::AttributesFinder.new(included_attributes: config_hash[:included_attributes],
excluded_attributes: config_hash[:excluded_attributes], excluded_attributes: config_hash[:excluded_attributes],
......
...@@ -16,7 +16,11 @@ describe Gitlab::ImportExport::ImportExportReader, lib: true do ...@@ -16,7 +16,11 @@ describe Gitlab::ImportExport::ImportExportReader, lib: true do
} }
end end
before do
allow_any_instance_of(Gitlab::ImportExport).to receive(:config_file).and_return(test_config)
end
it 'generates hash from project tree config' do it 'generates hash from project tree config' do
expect(described_class.new(config: test_config, shared: shared).project_tree).to match(project_tree_hash) expect(described_class.new(shared: shared).project_tree).to match(project_tree_hash)
end end
end end
...@@ -89,20 +89,20 @@ describe Gitlab::ImportExport::ProjectTreeSaver, services: true do ...@@ -89,20 +89,20 @@ describe Gitlab::ImportExport::ProjectTreeSaver, services: true do
expect(saved_project_json['merge_requests'].first['notes'].first['author']).not_to be_empty expect(saved_project_json['merge_requests'].first['notes'].first['author']).not_to be_empty
end end
it 'has commit statuses' do it 'has pipeline statuses' do
expect(saved_project_json['ci_commits'].first['statuses']).not_to be_empty expect(saved_project_json['pipelines'].first['statuses']).not_to be_empty
end end
it 'has CI builds' do it 'has pipeline builds' do
expect(saved_project_json['ci_commits'].first['statuses'].first['type']).to eq('Ci::Build') expect(saved_project_json['pipelines'].first['statuses'].first['type']).to eq('Ci::Build')
end end
it 'has ci commits' do it 'has pipeline commits' do
expect(saved_project_json['ci_commits']).not_to be_empty expect(saved_project_json['pipelines']).not_to be_empty
end end
it 'has ci commits notes' do it 'has ci pipeline notes' do
expect(saved_project_json['ci_commits'].first['notes']).not_to be_empty expect(saved_project_json['pipelines'].first['notes']).not_to be_empty
end end
end end
end end
...@@ -125,18 +125,21 @@ describe Gitlab::ImportExport::ProjectTreeSaver, services: true do ...@@ -125,18 +125,21 @@ describe Gitlab::ImportExport::ProjectTreeSaver, services: true do
commit_status = create(:commit_status, project: project) commit_status = create(:commit_status, project: project)
ci_commit = create(:ci_commit, ci_pipeline = create(:ci_pipeline,
project: project, project: project,
sha: merge_request.last_commit.id, sha: merge_request.last_commit.id,
ref: merge_request.source_branch, ref: merge_request.source_branch,
statuses: [commit_status]) statuses: [commit_status])
create(:ci_build, commit: ci_commit, project: project) create(:ci_build, pipeline: ci_pipeline, project: project)
create(:milestone, project: project) create(:milestone, project: project)
create(:note, noteable: issue, project: project) create(:note, noteable: issue, project: project)
create(:note, noteable: merge_request, project: project) create(:note, noteable: merge_request, project: project)
create(:note, noteable: ci_commit, project: project)
create(:note, noteable: snippet, project: project) create(:note, noteable: snippet, project: project)
create(:note_on_commit,
author: user,
project: project,
commit_id: ci_pipeline.sha)
project project
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment