diff options
author | DJ Mountney <david@twkie.net> | 2019-06-19 23:35:56 -0700 |
---|---|---|
committer | DJ Mountney <david@twkie.net> | 2019-06-25 10:44:40 -0700 |
commit | 7a089438fa138934b5dab7bdd575a74a1dfd03c0 (patch) | |
tree | 70adb17a83b29ad3901f3a5b86a7ee7f8a37e383 /spec/tasks | |
parent | 4d1e2ec45e993c8d9ebf3d379b5d1f20d3684658 (diff) | |
download | gitlab-ce-7a089438fa138934b5dab7bdd575a74a1dfd03c0.tar.gz |
Check supported version when migrating
Set the mininum supported migration version to be the schema version as
of 11.11.0, and errors you if that is not detected during
gitlab:db:configure
Diffstat (limited to 'spec/tasks')
-rw-r--r-- | spec/tasks/gitlab/db_rake_spec.rb | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/spec/tasks/gitlab/db_rake_spec.rb b/spec/tasks/gitlab/db_rake_spec.rb index 5818892d56a..f61c03e5cf0 100644 --- a/spec/tasks/gitlab/db_rake_spec.rb +++ b/spec/tasks/gitlab/db_rake_spec.rb @@ -16,6 +16,7 @@ describe 'gitlab:db namespace rake task' do allow(Rake::Task['db:migrate']).to receive(:invoke).and_return(true) allow(Rake::Task['db:schema:load']).to receive(:invoke).and_return(true) allow(Rake::Task['db:seed_fu']).to receive(:invoke).and_return(true) + allow(ActiveRecord::Migrator).to receive(:current_version).and_return(Gitlab::Database::MIN_SCHEMA_VERSION) end describe 'configure' do @@ -27,6 +28,12 @@ describe 'gitlab:db namespace rake task' do expect { run_rake_task('gitlab:db:configure') }.not_to raise_error end + it 'raises an when schema has been loaded, but version is too old to migrate' do + allow(ActiveRecord::Base.connection).to receive(:tables).and_return(%w[table1 table2]) + allow(ActiveRecord::Migrator).to receive(:current_version).and_return(25) + expect { run_rake_task('gitlab:db:configure') }.to raise_error(RuntimeErrorm, /current database version is too old to be migrated/) + end + it 'invokes db:shema:load and db:seed_fu when schema is not loaded' do allow(ActiveRecord::Base.connection).to receive(:tables).and_return([]) expect(Rake::Task['db:schema:load']).to receive(:invoke) |