ridgepole 0.6.4 → 0.6.5.beta
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/.gitignore +1 -0
- data/.travis.yml +18 -8
- data/Appraisals +8 -0
- data/README.md +26 -2
- data/docker-compose.yml +12 -0
- data/gemfiles/activerecord_4.2.gemfile +8 -0
- data/gemfiles/activerecord_5.0.gemfile +7 -0
- data/lib/ridgepole/diff.rb +3 -1
- data/lib/ridgepole/version.rb +1 -1
- data/ridgepole.gemspec +6 -2
- data/spec/cli_helper.rb +63 -0
- data/spec/mysql/bigint_pk/bigint_pk_spec.rb +45 -0
- data/spec/mysql/cli/config_spec.rb +0 -2
- data/spec/mysql/cli/ridgepole_spec.rb +58 -56
- data/spec/mysql/collation/collation_spec.rb +134 -138
- data/spec/mysql/diff/diff2_spec.rb +1 -3
- data/spec/mysql/diff/diff_spec.rb +1 -3
- data/spec/mysql/dump/dump_class_method_spec.rb +29 -11
- data/spec/mysql/dump/dump_some_tables_spec.rb +18 -8
- data/spec/mysql/dump/dump_spec.rb +28 -10
- data/spec/mysql/dump/dump_without_table_options_spec.rb +29 -33
- data/spec/mysql/fk/migrate_change_fk_spec.rb +2 -4
- data/spec/mysql/fk/migrate_create_fk_spec.rb +39 -29
- data/spec/mysql/fk/migrate_drop_fk_spec.rb +20 -22
- data/spec/mysql/migrate/check_orphan_index_spec.rb +2 -4
- data/spec/mysql/migrate/migrate_add_column2_spec.rb +33 -26
- data/spec/mysql/migrate/migrate_add_column_spec.rb +32 -22
- data/spec/mysql/migrate/migrate_add_column_with_alter_extra_spec.rb +26 -16
- data/spec/mysql/migrate/migrate_add_column_with_script_spec.rb +27 -17
- data/spec/mysql/migrate/migrate_change_column2_spec.rb +26 -16
- data/spec/mysql/migrate/migrate_change_column3_spec.rb +54 -44
- data/spec/mysql/migrate/migrate_change_column4_spec.rb +8 -10
- data/spec/mysql/migrate/migrate_change_column5_spec.rb +32 -22
- data/spec/mysql/migrate/migrate_change_column6_spec.rb +28 -30
- data/spec/mysql/migrate/migrate_change_column_default_spec.rb +34 -36
- data/spec/mysql/migrate/migrate_change_column_float_spec.rb +10 -12
- data/spec/mysql/migrate/migrate_change_column_spec.rb +41 -29
- data/spec/mysql/migrate/migrate_change_index2_spec.rb +12 -14
- data/spec/mysql/migrate/migrate_change_index3_spec.rb +24 -26
- data/spec/mysql/migrate/migrate_change_index4_spec.rb +34 -24
- data/spec/mysql/migrate/migrate_change_index5_spec.rb +26 -16
- data/spec/mysql/migrate/migrate_change_index6_spec.rb +38 -28
- data/spec/mysql/migrate/migrate_change_index_spec.rb +39 -29
- data/spec/mysql/migrate/migrate_change_table_option_spec.rb +26 -13
- data/spec/mysql/migrate/migrate_create_index2_spec.rb +30 -20
- data/spec/mysql/migrate/migrate_create_index_spec.rb +28 -18
- data/spec/mysql/migrate/migrate_create_table_spec.rb +24 -14
- data/spec/mysql/migrate/migrate_create_table_with_options_spec.rb +4 -6
- data/spec/mysql/migrate/migrate_create_table_with_script_spec.rb +23 -13
- data/spec/mysql/migrate/migrate_drop_column_and_index2_spec.rb +32 -22
- data/spec/mysql/migrate/migrate_drop_column_and_index_spec.rb +35 -25
- data/spec/mysql/migrate/migrate_drop_column_spec.rb +36 -26
- data/spec/mysql/migrate/migrate_drop_index_spec.rb +28 -18
- data/spec/mysql/migrate/migrate_drop_table_spec.rb +29 -19
- data/spec/mysql/migrate/migrate_duplicate_index_spec.rb +2 -4
- data/spec/mysql/migrate/migrate_duplicate_table_spec.rb +2 -4
- data/spec/mysql/migrate/migrate_empty_spec.rb +24 -14
- data/spec/mysql/migrate/migrate_execute_spec.rb +78 -65
- data/spec/mysql/migrate/migrate_log_file_spec.rb +22 -12
- data/spec/mysql/migrate/migrate_merge_mode_spec.rb +31 -21
- data/spec/mysql/migrate/migrate_noop_spec.rb +55 -45
- data/spec/mysql/migrate/migrate_rename_column_spec.rb +38 -28
- data/spec/mysql/migrate/migrate_rename_table_spec.rb +43 -33
- data/spec/mysql/migrate/migrate_same_default_null_spec.rb +28 -18
- data/spec/mysql/migrate/migrate_same_spec.rb +26 -12
- data/spec/mysql/migrate/migrate_script_error_spec.rb +3 -5
- data/spec/mysql/migrate/migrate_skip_rename_column_spec.rb +30 -20
- data/spec/mysql/migrate/migrate_skip_rename_table_spec.rb +30 -20
- data/spec/mysql/migrate/migrate_with_ignore_tables_spec.rb +44 -46
- data/spec/mysql/migrate/migrate_with_pre_post_query_spec.rb +6 -8
- data/spec/mysql/migrate/migrate_with_tables_spec.rb +20 -22
- data/spec/mysql/migrate_/migrate_create_index_with_alter_spec.rb +22 -12
- data/spec/mysql/migrate_/migrate_drop_index_with_alter_spec.rb +22 -12
- data/spec/mysql/~default_name_fk/migrate_change_fk_spec.rb +8 -10
- data/spec/mysql/~default_name_fk/migrate_create_fk_spec.rb +39 -29
- data/spec/mysql/~default_name_fk/migrate_drop_fk_spec.rb +20 -22
- data/spec/postgresql/diff/diff_spec.rb +7 -9
- data/spec/postgresql/dump/dump_spec.rb +2 -4
- data/spec/postgresql/fk/migrate_change_fk_spec.rb +8 -10
- data/spec/postgresql/fk/migrate_create_fk_spec.rb +24 -26
- data/spec/postgresql/fk/migrate_drop_fk_spec.rb +21 -23
- data/spec/postgresql/migrate/migrate_add_column_spec.rb +19 -22
- data/spec/postgresql/migrate/migrate_change_column_default_spec.rb +6 -8
- data/spec/postgresql/migrate/migrate_change_column_spec.rb +17 -19
- data/spec/postgresql/migrate/migrate_change_index_spec.rb +11 -13
- data/spec/postgresql/migrate/migrate_create_table_spec.rb +6 -9
- data/spec/postgresql/migrate/migrate_drop_column_spec.rb +12 -14
- data/spec/postgresql/migrate/migrate_drop_column_with_index_spec.rb +12 -14
- data/spec/postgresql/migrate/migrate_drop_index_spec.rb +10 -12
- data/spec/postgresql/migrate/migrate_drop_table_spec.rb +7 -9
- data/spec/postgresql/migrate/migrate_ext_cols_spec.rb +6 -8
- data/spec/postgresql/migrate/migrate_rename_column_spec.rb +12 -14
- data/spec/postgresql/migrate/migrate_rename_table_spec.rb +14 -16
- data/spec/postgresql/migrate/migrate_same_spec.rb +4 -6
- data/spec/postgresql/~default_name_fk/migrate_change_fk_spec.rb +8 -10
- data/spec/postgresql/~default_name_fk/migrate_create_fk_spec.rb +24 -26
- data/spec/postgresql/~default_name_fk/migrate_drop_fk_spec.rb +20 -22
- data/spec/processing_for_travis.rb +9 -0
- data/spec/spec_condition.rb +24 -0
- data/spec/spec_const.rb +18 -0
- data/spec/spec_helper.rb +125 -196
- data/spec/string_ext.rb +0 -13
- metadata +106 -32
- data/spec/mysql/bigint_pk/bigint_pkspec.rb +0 -47
@@ -1,9 +1,8 @@
|
|
1
|
-
unless postgresql?
|
2
1
|
describe 'Ridgepole::Client#diff -> migrate' do
|
3
2
|
context 'migrate with pre/post query' do
|
4
3
|
let(:actual_dsl) { '' }
|
5
4
|
let(:dsl) {
|
6
|
-
<<-
|
5
|
+
<<-EOS
|
7
6
|
create_table "clubs", force: :cascade do |t|
|
8
7
|
t.string "name", default: "", null: false
|
9
8
|
end
|
@@ -68,11 +67,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
68
67
|
end
|
69
68
|
|
70
69
|
add_index "titles", ["emp_no"], name: "emp_no", using: :btree
|
71
|
-
|
70
|
+
EOS
|
72
71
|
}
|
73
72
|
|
74
73
|
let(:expected_dsl) {
|
75
|
-
<<-
|
74
|
+
<<-EOS
|
76
75
|
create_table "clubs", force: :cascade do |t|
|
77
76
|
t.string "name", limit: 255, default: "", null: false
|
78
77
|
end
|
@@ -145,7 +144,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
145
144
|
end
|
146
145
|
|
147
146
|
add_index "titles", ["emp_no"], name: "emp_no", using: :btree
|
148
|
-
|
147
|
+
EOS
|
149
148
|
}
|
150
149
|
|
151
150
|
subject {
|
@@ -158,10 +157,9 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
158
157
|
it {
|
159
158
|
delta = subject.diff(dsl)
|
160
159
|
expect(delta.differ?).to be_truthy
|
161
|
-
expect(subject.dump).to
|
160
|
+
expect(subject.dump).to match_fuzzy actual_dsl
|
162
161
|
delta.migrate
|
163
|
-
expect(subject.dump).to
|
162
|
+
expect(subject.dump).to match_fuzzy expected_dsl
|
164
163
|
}
|
165
164
|
end
|
166
165
|
end
|
167
|
-
end
|
@@ -1,8 +1,7 @@
|
|
1
|
-
unless postgresql?
|
2
1
|
describe 'Ridgepole::Client#diff -> migrate' do
|
3
2
|
context 'when with tables option (same)' do
|
4
3
|
let(:current_schema) {
|
5
|
-
<<-
|
4
|
+
<<-EOS
|
6
5
|
create_table "employees", primary_key: "emp_no", force: :cascade do |t|
|
7
6
|
t.date "birth_date", null: false
|
8
7
|
t.string "first_name", limit: 14, null: false
|
@@ -19,11 +18,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
19
18
|
end
|
20
19
|
|
21
20
|
add_index "salaries", ["salary"], name: "emp_no", using: :btree
|
22
|
-
|
21
|
+
EOS
|
23
22
|
}
|
24
23
|
|
25
24
|
let(:dsl) {
|
26
|
-
<<-
|
25
|
+
<<-EOS
|
27
26
|
create_table "employees", primary_key: "emp_no", force: :cascade do |t|
|
28
27
|
t.date "birth_date", null: false
|
29
28
|
t.string "first_name", limit: 14, null: false
|
@@ -40,11 +39,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
40
39
|
end
|
41
40
|
|
42
41
|
add_index "salaries", ["emp_no"], name: "emp_no", using: :btree
|
43
|
-
|
42
|
+
EOS
|
44
43
|
}
|
45
44
|
|
46
45
|
let(:expected_dsl) {
|
47
|
-
<<-
|
46
|
+
<<-EOS
|
48
47
|
create_table "employees", primary_key: "emp_no", force: :cascade do |t|
|
49
48
|
t.date "birth_date", null: false
|
50
49
|
t.string "first_name", limit: 14, null: false
|
@@ -52,7 +51,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
52
51
|
t.string "gender", limit: 1, null: false
|
53
52
|
t.date "hire_date", null: false
|
54
53
|
end
|
55
|
-
|
54
|
+
EOS
|
56
55
|
}
|
57
56
|
|
58
57
|
before { subject.diff(current_schema).migrate }
|
@@ -61,15 +60,15 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
61
60
|
it {
|
62
61
|
delta = subject.diff(dsl)
|
63
62
|
expect(delta.differ?).to be_falsey
|
64
|
-
expect(subject.dump).to
|
63
|
+
expect(subject.dump).to match_fuzzy expected_dsl
|
65
64
|
delta.migrate
|
66
|
-
expect(subject.dump).to
|
65
|
+
expect(subject.dump).to match_fuzzy expected_dsl
|
67
66
|
}
|
68
67
|
end
|
69
68
|
|
70
69
|
context 'when with tables option (differ)' do
|
71
70
|
let(:current_schema) {
|
72
|
-
<<-
|
71
|
+
<<-EOS
|
73
72
|
create_table "employees", primary_key: "emp_no", force: :cascade do |t|
|
74
73
|
t.date "birth_date", null: false
|
75
74
|
t.string "first_name", limit: 14, null: false
|
@@ -86,11 +85,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
86
85
|
end
|
87
86
|
|
88
87
|
add_index "salaries", ["salary"], name: "emp_no", using: :btree
|
89
|
-
|
88
|
+
EOS
|
90
89
|
}
|
91
90
|
|
92
91
|
let(:dsl) {
|
93
|
-
<<-
|
92
|
+
<<-EOS
|
94
93
|
create_table "employees", primary_key: "emp_no", force: :cascade do |t|
|
95
94
|
t.date "birth_date", null: false
|
96
95
|
t.string "first_name", limit: 15, null: false
|
@@ -107,11 +106,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
107
106
|
end
|
108
107
|
|
109
108
|
add_index "salaries", ["emp_no"], name: "emp_no", using: :btree
|
110
|
-
|
109
|
+
EOS
|
111
110
|
}
|
112
111
|
|
113
112
|
let(:before_dsl) {
|
114
|
-
<<-
|
113
|
+
<<-EOS
|
115
114
|
create_table "employees", primary_key: "emp_no", force: :cascade do |t|
|
116
115
|
t.date "birth_date", null: false
|
117
116
|
t.string "first_name", limit: 14, null: false
|
@@ -119,11 +118,11 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
119
118
|
t.string "gender", limit: 1, null: false
|
120
119
|
t.date "hire_date", null: false
|
121
120
|
end
|
122
|
-
|
121
|
+
EOS
|
123
122
|
}
|
124
123
|
|
125
124
|
let(:after_dsl) {
|
126
|
-
<<-
|
125
|
+
<<-EOS
|
127
126
|
create_table "employees", primary_key: "emp_no", force: :cascade do |t|
|
128
127
|
t.date "birth_date", null: false
|
129
128
|
t.string "first_name", limit: 15, null: false
|
@@ -131,7 +130,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
131
130
|
t.string "gender", limit: 1, null: false
|
132
131
|
t.date "hire_date", null: false
|
133
132
|
end
|
134
|
-
|
133
|
+
EOS
|
135
134
|
}
|
136
135
|
|
137
136
|
before { subject.diff(current_schema).migrate }
|
@@ -140,18 +139,17 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
140
139
|
it {
|
141
140
|
delta = subject.diff(dsl)
|
142
141
|
expect(delta.differ?).to be_truthy
|
143
|
-
expect(subject.dump).to
|
142
|
+
expect(subject.dump).to match_fuzzy before_dsl
|
144
143
|
delta.migrate
|
145
|
-
expect(subject.dump).to
|
144
|
+
expect(subject.dump).to match_fuzzy after_dsl
|
146
145
|
}
|
147
146
|
|
148
147
|
it {
|
149
148
|
delta = Ridgepole::Client.diff(current_schema, dsl, tables: ['employees'], reverse: true, enable_mysql_awesome: true)
|
150
149
|
expect(delta.differ?).to be_truthy
|
151
|
-
expect(delta.script).to
|
150
|
+
expect(delta.script).to match_fuzzy <<-EOS
|
152
151
|
change_column("employees", "first_name", :string, {:limit=>14, :null=>false, :default=>nil, :unsigned=>false})
|
153
|
-
|
152
|
+
EOS
|
154
153
|
}
|
155
154
|
end
|
156
155
|
end
|
157
|
-
end
|
@@ -1,15 +1,26 @@
|
|
1
|
-
unless postgresql?
|
2
1
|
describe 'Ridgepole::Client#diff -> migrate' do
|
2
|
+
let(:template_variables) {
|
3
|
+
opts = {
|
4
|
+
unsigned: {}
|
5
|
+
}
|
6
|
+
|
7
|
+
if condition(:mysql_awesome_enabled)
|
8
|
+
opts[:unsigned] = {unsigned: true}
|
9
|
+
end
|
10
|
+
|
11
|
+
opts
|
12
|
+
}
|
13
|
+
|
3
14
|
context 'when create index' do
|
4
15
|
let(:dsl) {
|
5
|
-
<<-
|
6
|
-
create_table "clubs"
|
16
|
+
erbh(<<-EOS, template_variables)
|
17
|
+
create_table "clubs", <%= {force: :cascade}.unshift(@unsigned).i %> do |t|
|
7
18
|
t.string "name", limit: 255, default: "", null: false
|
8
19
|
end
|
9
20
|
|
10
21
|
add_index "clubs", ["name"], name: "idx_name", unique: true, using: :btree
|
11
22
|
|
12
|
-
create_table "departments", primary_key: "dept_no"
|
23
|
+
create_table "departments", primary_key: "dept_no", <%= {force: :cascade}.unshift(@unsigned).i %> do |t|
|
13
24
|
t.string "dept_name", limit: 40, null: false
|
14
25
|
end
|
15
26
|
|
@@ -35,14 +46,14 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
35
46
|
add_index "dept_manager", ["dept_no"], name: "dept_no", using: :btree
|
36
47
|
add_index "dept_manager", ["emp_no"], name: "emp_no", using: :btree
|
37
48
|
|
38
|
-
create_table "employee_clubs"
|
39
|
-
t.integer "emp_no", limit: 4, null: false
|
40
|
-
t.integer "club_id", limit: 4, null: false
|
49
|
+
create_table "employee_clubs", <%= {force: :cascade}.unshift(@unsigned).i %> do |t|
|
50
|
+
t.integer "emp_no", <%= {limit: 4, null: false}.push(@unsigned).i %>
|
51
|
+
t.integer "club_id", <%= {limit: 4, null: false}.push(@unsigned).i %>
|
41
52
|
end
|
42
53
|
|
43
54
|
add_index "employee_clubs", ["emp_no", "club_id"], name: "idx_emp_no_club_id", using: :btree
|
44
55
|
|
45
|
-
create_table "employees", primary_key: "emp_no"
|
56
|
+
create_table "employees", primary_key: "emp_no", <%= {force: :cascade}.unshift(@unsigned).i %> do |t|
|
46
57
|
t.date "birth_date", null: false
|
47
58
|
t.string "first_name", limit: 14, null: false
|
48
59
|
t.string "last_name", limit: 16, null: false
|
@@ -67,7 +78,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
67
78
|
end
|
68
79
|
|
69
80
|
add_index "titles", ["emp_no"], name: "emp_no", using: :btree
|
70
|
-
|
81
|
+
EOS
|
71
82
|
}
|
72
83
|
|
73
84
|
let(:actual_dsl) {
|
@@ -84,10 +95,9 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
84
95
|
it {
|
85
96
|
delta = subject.diff(expected_dsl)
|
86
97
|
expect(delta.differ?).to be_truthy
|
87
|
-
expect(subject.dump
|
98
|
+
expect(subject.dump).to match_fuzzy actual_dsl
|
88
99
|
delta.migrate
|
89
|
-
expect(subject.dump
|
100
|
+
expect(subject.dump).to match_fuzzy expected_dsl
|
90
101
|
}
|
91
102
|
end
|
92
103
|
end
|
93
|
-
end
|
@@ -1,15 +1,26 @@
|
|
1
|
-
unless postgresql?
|
2
1
|
describe 'Ridgepole::Client#diff -> migrate' do
|
2
|
+
let(:template_variables) {
|
3
|
+
opts = {
|
4
|
+
unsigned: {}
|
5
|
+
}
|
6
|
+
|
7
|
+
if condition(:mysql_awesome_enabled)
|
8
|
+
opts[:unsigned] = {unsigned: true}
|
9
|
+
end
|
10
|
+
|
11
|
+
opts
|
12
|
+
}
|
13
|
+
|
3
14
|
context 'when drop index' do
|
4
15
|
let(:dsl) {
|
5
|
-
<<-
|
6
|
-
create_table "clubs"
|
16
|
+
erbh(<<-EOS, template_variables)
|
17
|
+
create_table "clubs", <%= {force: :cascade}.unshift(@unsigned).i %> do |t|
|
7
18
|
t.string "name", limit: 255, default: "", null: false
|
8
19
|
end
|
9
20
|
|
10
21
|
add_index "clubs", ["name"], name: "idx_name", unique: true, using: :btree
|
11
22
|
|
12
|
-
create_table "departments", primary_key: "dept_no"
|
23
|
+
create_table "departments", primary_key: "dept_no", <%= {force: :cascade}.unshift(@unsigned).i %> do |t|
|
13
24
|
t.string "dept_name", limit: 40, null: false
|
14
25
|
end
|
15
26
|
|
@@ -35,14 +46,14 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
35
46
|
add_index "dept_manager", ["dept_no"], name: "dept_no", using: :btree
|
36
47
|
add_index "dept_manager", ["emp_no"], name: "emp_no", using: :btree
|
37
48
|
|
38
|
-
create_table "employee_clubs"
|
39
|
-
t.integer "emp_no", limit: 4, null: false
|
40
|
-
t.integer "club_id", limit: 4, null: false
|
49
|
+
create_table "employee_clubs", <%= {force: :cascade}.unshift(@unsigned).i %> do |t|
|
50
|
+
t.integer "emp_no", <%= {limit: 4, null: false}.push(@unsigned).i %>
|
51
|
+
t.integer "club_id", <%= {limit: 4, null: false}.push(@unsigned).i %>
|
41
52
|
end
|
42
53
|
|
43
54
|
add_index "employee_clubs", ["emp_no", "club_id"], name: "idx_emp_no_club_id", using: :btree
|
44
55
|
|
45
|
-
create_table "employees", primary_key: "emp_no"
|
56
|
+
create_table "employees", primary_key: "emp_no", <%= {force: :cascade}.unshift(@unsigned).i %> do |t|
|
46
57
|
t.date "birth_date", null: false
|
47
58
|
t.string "first_name", limit: 14, null: false
|
48
59
|
t.string "last_name", limit: 16, null: false
|
@@ -67,7 +78,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
67
78
|
end
|
68
79
|
|
69
80
|
add_index "titles", ["emp_no"], name: "emp_no", using: :btree
|
70
|
-
|
81
|
+
EOS
|
71
82
|
}
|
72
83
|
|
73
84
|
let(:actual_dsl) { dsl }
|
@@ -84,10 +95,9 @@ describe 'Ridgepole::Client#diff -> migrate' do
|
|
84
95
|
it {
|
85
96
|
delta = subject.diff(expected_dsl)
|
86
97
|
expect(delta.differ?).to be_truthy
|
87
|
-
expect(subject.dump).to
|
98
|
+
expect(subject.dump).to match_fuzzy actual_dsl
|
88
99
|
delta.migrate
|
89
|
-
expect(subject.dump
|
100
|
+
expect(subject.dump).to match_fuzzy expected_dsl
|
90
101
|
}
|
91
102
|
end
|
92
103
|
end
|
93
|
-
end
|
@@ -1,8 +1,7 @@
|
|
1
|
-
unless postgresql?
|
2
1
|
describe 'Ridgepole::Client#diff -> migrate' do
|
3
2
|
context 'when change fk' do
|
4
3
|
let(:actual_dsl) {
|
5
|
-
<<-
|
4
|
+
<<-EOS
|
6
5
|
create_table "parent", force: :cascade do |t|
|
7
6
|
end
|
8
7
|
|
@@ -13,11 +12,11 @@ end
|
|
13
12
|
add_index "child", ["parent_id"], name: "par_id", using: :btree
|
14
13
|
|
15
14
|
add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc", on_delete: :cascade
|
16
|
-
|
15
|
+
EOS
|
17
16
|
}
|
18
17
|
|
19
18
|
let(:sorted_actual_dsl) {
|
20
|
-
<<-
|
19
|
+
<<-EOS
|
21
20
|
create_table "child", force: :cascade do |t|
|
22
21
|
t.integer "parent_id", limit: 4
|
23
22
|
end
|
@@ -28,11 +27,11 @@ create_table "parent", force: :cascade do |t|
|
|
28
27
|
end
|
29
28
|
|
30
29
|
add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc", on_delete: :cascade
|
31
|
-
|
30
|
+
EOS
|
32
31
|
}
|
33
32
|
|
34
33
|
let(:expected_dsl) {
|
35
|
-
<<-
|
34
|
+
<<-EOS
|
36
35
|
create_table "child", force: :cascade do |t|
|
37
36
|
t.integer "parent_id", limit: 4
|
38
37
|
end
|
@@ -43,7 +42,7 @@ create_table "parent", force: :cascade do |t|
|
|
43
42
|
end
|
44
43
|
|
45
44
|
add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
46
|
-
|
45
|
+
EOS
|
47
46
|
}
|
48
47
|
|
49
48
|
before { subject.diff(actual_dsl).migrate }
|
@@ -53,10 +52,9 @@ add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
|
53
52
|
it {
|
54
53
|
delta = subject.diff(expected_dsl)
|
55
54
|
expect(delta.differ?).to be_truthy
|
56
|
-
expect(subject.dump
|
55
|
+
expect(subject.dump).to match_fuzzy sorted_actual_dsl
|
57
56
|
delta.migrate
|
58
|
-
expect(subject.dump
|
57
|
+
expect(subject.dump).to match_fuzzy expected_dsl
|
59
58
|
}
|
60
59
|
end
|
61
60
|
end
|
62
|
-
end
|
@@ -1,24 +1,35 @@
|
|
1
|
-
unless postgresql?
|
2
1
|
describe 'Ridgepole::Client#diff -> migrate' do
|
2
|
+
let(:template_variables) {
|
3
|
+
opts = {
|
4
|
+
unsigned: {}
|
5
|
+
}
|
6
|
+
|
7
|
+
if condition(:mysql_awesome_enabled)
|
8
|
+
opts[:unsigned] = {unsigned: true}
|
9
|
+
end
|
10
|
+
|
11
|
+
opts
|
12
|
+
}
|
13
|
+
|
3
14
|
context 'when create fk' do
|
4
15
|
let(:actual_dsl) {
|
5
|
-
<<-
|
6
|
-
create_table "child"
|
7
|
-
t.integer "parent_id", limit: 4
|
16
|
+
erbh(<<-EOS, template_variables)
|
17
|
+
create_table "child", <%= {force: :cascade}.unshift(@unsigned).i %> do |t|
|
18
|
+
t.integer "parent_id", <%= {limit: 4}.push(@unsigned).i %>
|
8
19
|
end
|
9
20
|
|
10
21
|
add_index "child", ["parent_id"], name: "par_id", using: :btree
|
11
22
|
|
12
|
-
create_table "parent"
|
23
|
+
create_table "parent", <%= {force: :cascade}.unshift(@unsigned).i %> do |t|
|
13
24
|
end
|
14
|
-
|
25
|
+
EOS
|
15
26
|
}
|
16
27
|
|
17
28
|
let(:expected_dsl) {
|
18
|
-
actual_dsl + (<<-
|
29
|
+
actual_dsl + (<<-EOS)
|
19
30
|
|
20
31
|
add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
21
|
-
|
32
|
+
EOS
|
22
33
|
}
|
23
34
|
|
24
35
|
before { subject.diff(actual_dsl).migrate }
|
@@ -27,34 +38,34 @@ add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
|
27
38
|
it {
|
28
39
|
delta = subject.diff(expected_dsl)
|
29
40
|
expect(delta.differ?).to be_truthy
|
30
|
-
expect(subject.dump
|
41
|
+
expect(subject.dump).to match_fuzzy actual_dsl
|
31
42
|
delta.migrate
|
32
|
-
expect(subject.dump
|
43
|
+
expect(subject.dump).to match_fuzzy expected_dsl
|
33
44
|
}
|
34
45
|
|
35
46
|
it {
|
36
47
|
delta = Ridgepole::Client.diff(actual_dsl, expected_dsl, reverse: true, default_int_limit: 4, dumb_with_default_fk_name: true)
|
37
48
|
expect(delta.differ?).to be_truthy
|
38
|
-
expect(delta.script).to
|
49
|
+
expect(delta.script).to match_fuzzy <<-EOS
|
39
50
|
remove_foreign_key("child", {:name=>"fk_rails_e74ce85cbc"})
|
40
|
-
|
51
|
+
EOS
|
41
52
|
}
|
42
53
|
|
43
54
|
it {
|
44
55
|
delta = client(bulk_change: true, dumb_with_default_fk_name: true).diff(expected_dsl)
|
45
56
|
expect(delta.differ?).to be_truthy
|
46
|
-
expect(subject.dump
|
47
|
-
expect(delta.script).to
|
57
|
+
expect(subject.dump).to match_fuzzy actual_dsl
|
58
|
+
expect(delta.script).to match_fuzzy <<-EOS
|
48
59
|
add_foreign_key("child", "parent", {:name=>"fk_rails_e74ce85cbc"})
|
49
|
-
|
60
|
+
EOS
|
50
61
|
delta.migrate
|
51
|
-
expect(subject.dump
|
62
|
+
expect(subject.dump).to match_fuzzy expected_dsl
|
52
63
|
}
|
53
64
|
end
|
54
65
|
|
55
66
|
context 'when create fk when create table' do
|
56
67
|
let(:dsl) {
|
57
|
-
<<-
|
68
|
+
<<-EOS
|
58
69
|
# Define parent before child
|
59
70
|
create_table "parent", force: :cascade do |t|
|
60
71
|
end
|
@@ -66,11 +77,11 @@ end
|
|
66
77
|
add_index "child", ["parent_id"], name: "par_id", using: :btree
|
67
78
|
|
68
79
|
add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
69
|
-
|
80
|
+
EOS
|
70
81
|
}
|
71
82
|
|
72
83
|
let(:sorted_dsl) {
|
73
|
-
<<-
|
84
|
+
<<-EOS
|
74
85
|
|
75
86
|
create_table "child", force: :cascade do |t|
|
76
87
|
t.integer "parent_id", limit: 4
|
@@ -82,7 +93,7 @@ create_table "parent", force: :cascade do |t|
|
|
82
93
|
end
|
83
94
|
|
84
95
|
add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
85
|
-
|
96
|
+
EOS
|
86
97
|
}
|
87
98
|
|
88
99
|
subject { client(dumb_with_default_fk_name: true) }
|
@@ -90,15 +101,15 @@ add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
|
90
101
|
it {
|
91
102
|
delta = subject.diff(dsl)
|
92
103
|
expect(delta.differ?).to be_truthy
|
93
|
-
expect(subject.dump
|
104
|
+
expect(subject.dump).to match_fuzzy ''
|
94
105
|
delta.migrate
|
95
|
-
expect(subject.dump
|
106
|
+
expect(subject.dump).to match_fuzzy sorted_dsl
|
96
107
|
}
|
97
108
|
end
|
98
109
|
|
99
110
|
context 'already defined' do
|
100
111
|
let(:dsl) {
|
101
|
-
<<-
|
112
|
+
<<-EOS
|
102
113
|
# Define parent before child
|
103
114
|
create_table "parent", force: :cascade do |t|
|
104
115
|
end
|
@@ -112,7 +123,7 @@ add_index "child", ["parent_id"], name: "par_id", using: :btree
|
|
112
123
|
add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
113
124
|
|
114
125
|
add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
115
|
-
|
126
|
+
EOS
|
116
127
|
}
|
117
128
|
|
118
129
|
subject { client(dumb_with_default_fk_name: true) }
|
@@ -126,7 +137,7 @@ add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
|
126
137
|
|
127
138
|
context 'no name' do
|
128
139
|
let(:dsl) {
|
129
|
-
<<-
|
140
|
+
<<-EOS
|
130
141
|
# Define parent before child
|
131
142
|
create_table "parent", force: :cascade do |t|
|
132
143
|
end
|
@@ -138,7 +149,7 @@ end
|
|
138
149
|
add_index "child", ["parent_id"], name: "par_id", using: :btree
|
139
150
|
|
140
151
|
add_foreign_key "child", "parent"
|
141
|
-
|
152
|
+
EOS
|
142
153
|
}
|
143
154
|
|
144
155
|
subject { client(dumb_with_default_fk_name: true) }
|
@@ -152,13 +163,13 @@ add_foreign_key "child", "parent"
|
|
152
163
|
|
153
164
|
context 'orphan fk' do
|
154
165
|
let(:dsl) {
|
155
|
-
<<-
|
166
|
+
<<-EOS
|
156
167
|
# Define parent before child
|
157
168
|
create_table "parent", force: :cascade do |t|
|
158
169
|
end
|
159
170
|
|
160
171
|
add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
161
|
-
|
172
|
+
EOS
|
162
173
|
}
|
163
174
|
|
164
175
|
subject { client(dumb_with_default_fk_name: true) }
|
@@ -170,4 +181,3 @@ add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
|
170
181
|
}
|
171
182
|
end
|
172
183
|
end
|
173
|
-
end
|
@@ -1,8 +1,7 @@
|
|
1
|
-
unless postgresql?
|
2
1
|
describe 'Ridgepole::Client#diff -> migrate' do
|
3
2
|
context 'when drop fk' do
|
4
3
|
let(:actual_dsl) {
|
5
|
-
<<-
|
4
|
+
<<-EOS
|
6
5
|
create_table "parent", force: :cascade do |t|
|
7
6
|
end
|
8
7
|
|
@@ -13,18 +12,18 @@ end
|
|
13
12
|
add_index "child", ["parent_id"], name: "par_id", using: :btree
|
14
13
|
|
15
14
|
add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
16
|
-
|
15
|
+
EOS
|
17
16
|
}
|
18
17
|
|
19
18
|
let(:sorted_actual_dsl) {
|
20
|
-
expected_dsl + (<<-
|
19
|
+
expected_dsl + (<<-EOS)
|
21
20
|
|
22
21
|
add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
23
|
-
|
22
|
+
EOS
|
24
23
|
}
|
25
24
|
|
26
25
|
let(:expected_dsl) {
|
27
|
-
<<-
|
26
|
+
<<-EOS
|
28
27
|
create_table "child", force: :cascade do |t|
|
29
28
|
t.integer "parent_id", limit: 4
|
30
29
|
end
|
@@ -33,7 +32,7 @@ add_index "child", ["parent_id"], name: "par_id", using: :btree
|
|
33
32
|
|
34
33
|
create_table "parent", force: :cascade do |t|
|
35
34
|
end
|
36
|
-
|
35
|
+
EOS
|
37
36
|
}
|
38
37
|
|
39
38
|
before { subject.diff(actual_dsl).migrate }
|
@@ -42,34 +41,34 @@ end
|
|
42
41
|
it {
|
43
42
|
delta = subject.diff(expected_dsl)
|
44
43
|
expect(delta.differ?).to be_truthy
|
45
|
-
expect(subject.dump).to
|
44
|
+
expect(subject.dump).to match_fuzzy sorted_actual_dsl
|
46
45
|
delta.migrate
|
47
|
-
expect(subject.dump
|
46
|
+
expect(subject.dump).to match_fuzzy expected_dsl
|
48
47
|
}
|
49
48
|
|
50
49
|
it {
|
51
50
|
delta = Ridgepole::Client.diff(actual_dsl, expected_dsl, reverse: true, default_int_limit: 4, dumb_with_default_fk_name: true)
|
52
51
|
expect(delta.differ?).to be_truthy
|
53
|
-
expect(delta.script).to
|
52
|
+
expect(delta.script).to match_fuzzy <<-EOS
|
54
53
|
add_foreign_key("child", "parent", {:name=>"fk_rails_e74ce85cbc"})
|
55
|
-
|
54
|
+
EOS
|
56
55
|
}
|
57
56
|
|
58
57
|
it {
|
59
58
|
delta = client(bulk_change: true, dumb_with_default_fk_name: true).diff(expected_dsl)
|
60
59
|
expect(delta.differ?).to be_truthy
|
61
|
-
expect(subject.dump).to
|
62
|
-
expect(delta.script).to
|
60
|
+
expect(subject.dump).to match_fuzzy sorted_actual_dsl
|
61
|
+
expect(delta.script).to match_fuzzy <<-EOS
|
63
62
|
remove_foreign_key("child", {:name=>"fk_rails_e74ce85cbc"})
|
64
|
-
|
63
|
+
EOS
|
65
64
|
delta.migrate
|
66
|
-
expect(subject.dump
|
65
|
+
expect(subject.dump).to match_fuzzy expected_dsl
|
67
66
|
}
|
68
67
|
end
|
69
68
|
|
70
69
|
context 'when drop fk when drop table' do
|
71
70
|
let(:dsl) {
|
72
|
-
<<-
|
71
|
+
<<-EOS
|
73
72
|
create_table "parent", force: :cascade do |t|
|
74
73
|
end
|
75
74
|
|
@@ -81,11 +80,11 @@ end
|
|
81
80
|
add_index "child", ["parent_id"], name: "par_id", using: :btree
|
82
81
|
|
83
82
|
add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
84
|
-
|
83
|
+
EOS
|
85
84
|
}
|
86
85
|
|
87
86
|
let(:sorted_dsl) {
|
88
|
-
<<-
|
87
|
+
<<-EOS
|
89
88
|
create_table "child", force: :cascade do |t|
|
90
89
|
t.integer "parent_id", limit: 4
|
91
90
|
end
|
@@ -96,7 +95,7 @@ create_table "parent", force: :cascade do |t|
|
|
96
95
|
end
|
97
96
|
|
98
97
|
add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
99
|
-
|
98
|
+
EOS
|
100
99
|
}
|
101
100
|
|
102
101
|
before { subject.diff(dsl).migrate }
|
@@ -105,10 +104,9 @@ add_foreign_key "child", "parent", name: "fk_rails_e74ce85cbc"
|
|
105
104
|
it {
|
106
105
|
delta = subject.diff('')
|
107
106
|
expect(delta.differ?).to be_truthy
|
108
|
-
expect(subject.dump).to
|
107
|
+
expect(subject.dump).to match_fuzzy sorted_dsl
|
109
108
|
delta.migrate
|
110
|
-
expect(subject.dump
|
109
|
+
expect(subject.dump).to match_fuzzy ''
|
111
110
|
}
|
112
111
|
end
|
113
112
|
end
|
114
|
-
end
|