Add data migration
This commit is contained in:
parent
056c42cca5
commit
b20595b2bd
|
@ -0,0 +1,33 @@
|
|||
class FixProjectsWithoutProjectFeature < ActiveRecord::Migration
|
||||
DOWNTIME = false
|
||||
|
||||
def up
|
||||
# Deletes corrupted project features
|
||||
sql = "DELETE FROM project_features WHERE project_id IS NULL"
|
||||
execute(sql)
|
||||
|
||||
# Creates missing project features with private visibility
|
||||
sql =
|
||||
%Q{
|
||||
INSERT INTO project_features(project_id, repository_access_level, issues_access_level, merge_requests_access_level, wiki_access_level,
|
||||
builds_access_level, snippets_access_level, created_at, updated_at)
|
||||
SELECT projects.id as project_id,
|
||||
10 as repository_access_level,
|
||||
10 as issues_access_level,
|
||||
10 as merge_requests_access_level,
|
||||
10 as wiki_access_level,
|
||||
10 as builds_access_level ,
|
||||
10 as snippets_access_level,
|
||||
projects.created_at,
|
||||
projects.updated_at
|
||||
FROM projects
|
||||
LEFT OUTER JOIN project_features ON project_features.project_id = projects.id
|
||||
WHERE (project_features.id IS NULL)
|
||||
}
|
||||
|
||||
execute(sql)
|
||||
end
|
||||
|
||||
def down
|
||||
end
|
||||
end
|
|
@ -11,7 +11,7 @@
|
|||
#
|
||||
# It's strongly recommended that you check this file into your version control system.
|
||||
|
||||
ActiveRecord::Schema.define(version: 20170905112933) do
|
||||
ActiveRecord::Schema.define(version: 20170913180600) do
|
||||
|
||||
# These are extensions that must be enabled in order to support this database
|
||||
enable_extension "plpgsql"
|
||||
|
|
|
@ -163,7 +163,7 @@ describe Projects::UpdateService, '#execute' do
|
|||
result = update_project(project, admin, path: 'existing')
|
||||
|
||||
expect(result).to include(status: :error)
|
||||
expect(result[:message]).to match('Project could not be updated!')
|
||||
expect(result[:message]).to match('There is already a repository with that name on disk')
|
||||
expect(project).not_to be_valid
|
||||
expect(project.errors.messages).to have_key(:base)
|
||||
expect(project.errors.messages[:base]).to include('There is already a repository with that name on disk')
|
||||
|
|
Loading…
Reference in New Issue