Commit 57224021 authored by Tiago Botelho's avatar Tiago Botelho
Browse files

Adds #human_import_status_name to make it comply with ProjectImportState#human_status_name

parent 1de3d5d88507
......@@ -674,6 +674,12 @@ def ensure_import_state(force: false)
end
end
def human_import_status_name
ensure_import_state
import_state.human_status_name
end
def import_schedule
ensure_import_state(force: true)
......
---
title: Showing project import_status in a humanized form no longer gives an error
merge_request: 19470
author:
type: fixed
......@@ -1693,6 +1693,31 @@ def create_pipeline
end
end
describe '#human_import_status_name' do
context 'when import_state exists' do
it 'returns the humanized status name' do
project = create(:project)
create(:import_state, :started, project: project)
expect(project.human_import_status_name).to eq("started")
end
end
context 'when import_state was not created yet' do
let(:project) { create(:project, :import_started) }
it 'ensures import_state is created and returns humanized status name' do
expect do
project.human_import_status_name
end.to change { ProjectImportState.count }.from(0).to(1)
end
it 'returns humanized status name' do
expect(project.human_import_status_name).to eq("started")
end
end
end
describe 'Project import job' do
let(:project) { create(:project, import_url: generate(:url)) }
......
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