ridgepole 0.8.9 → 0.9.0.rc1

Sign up to get free protection for your applications and to get access to all the features.
Files changed (143) hide show
  1. checksums.yaml +4 -4
  2. data/.github/workflows/test.yml +83 -0
  3. data/.rubocop.yml +9 -34
  4. data/.simplecov +6 -0
  5. data/Appraisals +4 -5
  6. data/README.md +25 -40
  7. data/bin/ridgepole +13 -3
  8. data/gemfiles/{activerecord_5.0.gemfile → activerecord_6.1.gemfile} +1 -2
  9. data/lib/ridgepole/client.rb +8 -3
  10. data/lib/ridgepole/default_limit.rb +1 -1
  11. data/lib/ridgepole/delta.rb +16 -1
  12. data/lib/ridgepole/diff.rb +50 -31
  13. data/lib/ridgepole/dsl_parser.rb +3 -5
  14. data/lib/ridgepole/dsl_parser/context.rb +1 -1
  15. data/lib/ridgepole/dsl_parser/table_definition.rb +52 -42
  16. data/lib/ridgepole/execute_expander.rb +10 -1
  17. data/lib/ridgepole/ext/abstract_adapter/disable_table_options.rb +9 -1
  18. data/lib/ridgepole/external_sql_executer.rb +12 -1
  19. data/lib/ridgepole/version.rb +1 -1
  20. data/ridgepole.gemspec +7 -4
  21. data/spec/erb_helper.rb +5 -5
  22. data/spec/mysql/_migrate/migrate_change_table_option_spec.rb +2 -2
  23. data/spec/mysql/bigint_pk/int_pk_spec.rb +1 -1
  24. data/spec/mysql/cli/ridgepole_spec.rb +35 -1
  25. data/spec/mysql/collation/collation_spec.rb +15 -15
  26. data/spec/mysql/comment/comment_spec.rb +9 -9
  27. data/spec/mysql/diff/diff2_spec.rb +5 -5
  28. data/spec/mysql/diff/diff_spec.rb +3 -3
  29. data/spec/mysql/dump/dump_class_method_spec.rb +13 -13
  30. data/spec/mysql/dump/dump_some_tables_spec.rb +4 -4
  31. data/spec/mysql/dump/dump_spec.rb +13 -13
  32. data/spec/mysql/dump/dump_unknown_column_type_spec.rb +2 -2
  33. data/spec/mysql/dump/dump_without_table_options_spec.rb +2 -2
  34. data/spec/mysql/fk/migrate_change_fk2_spec.rb +2 -2
  35. data/spec/mysql/fk/migrate_change_fk_spec.rb +20 -20
  36. data/spec/mysql/fk/migrate_create_fk_spec.rb +73 -24
  37. data/spec/mysql/fk/migrate_drop_fk_spec.rb +26 -26
  38. data/spec/mysql/fk/migrate_fk_with_column_spec.rb +16 -16
  39. data/spec/mysql/fk/migrate_ignore_fk_spec.rb +6 -6
  40. data/spec/mysql/migrate/migrate_add_column_order_spec.rb +2 -2
  41. data/spec/mysql/migrate/migrate_add_column_spec.rb +18 -18
  42. data/spec/mysql/migrate/migrate_add_column_with_alter_extra_spec.rb +106 -18
  43. data/spec/mysql/migrate/migrate_add_column_with_script_spec.rb +18 -18
  44. data/spec/mysql/migrate/migrate_change_column2_spec.rb +5 -5
  45. data/spec/mysql/migrate/migrate_change_column3_spec.rb +116 -25
  46. data/spec/mysql/migrate/migrate_change_column5_spec.rb +3 -3
  47. data/spec/mysql/migrate/migrate_change_column6_spec.rb +7 -7
  48. data/spec/mysql/migrate/migrate_change_column8_spec.rb +38 -5
  49. data/spec/mysql/migrate/migrate_change_column_default_spec.rb +2 -14
  50. data/spec/mysql/migrate/migrate_change_column_spec.rb +21 -21
  51. data/spec/mysql/migrate/migrate_change_index2_spec.rb +5 -5
  52. data/spec/mysql/migrate/migrate_change_index3_spec.rb +7 -7
  53. data/spec/mysql/migrate/migrate_change_index4_spec.rb +6 -6
  54. data/spec/mysql/migrate/migrate_change_index5_spec.rb +4 -4
  55. data/spec/mysql/migrate/migrate_change_index6_spec.rb +27 -27
  56. data/spec/mysql/migrate/migrate_change_index7_spec.rb +4 -4
  57. data/spec/mysql/migrate/migrate_change_index8_spec.rb +4 -4
  58. data/spec/mysql/migrate/migrate_change_index_spec.rb +34 -28
  59. data/spec/mysql/migrate/migrate_check_relation_column_type_spec.rb +5 -5
  60. data/spec/mysql/migrate/migrate_create_index2_spec.rb +24 -24
  61. data/spec/mysql/migrate/migrate_create_index_spec.rb +18 -18
  62. data/spec/mysql/migrate/migrate_create_table_spec.rb +16 -16
  63. data/spec/mysql/migrate/migrate_create_table_with_ignore_spec.rb +4 -4
  64. data/spec/mysql/migrate/migrate_create_table_with_index_spec.rb +4 -4
  65. data/spec/mysql/migrate/migrate_create_table_with_script_spec.rb +16 -16
  66. data/spec/mysql/migrate/migrate_drop_column_and_index2_spec.rb +18 -18
  67. data/spec/mysql/migrate/migrate_drop_column_and_index_spec.rb +16 -16
  68. data/spec/mysql/migrate/migrate_drop_column_and_unique_index_spec.rb +1 -1
  69. data/spec/mysql/migrate/migrate_drop_column_spec.rb +18 -18
  70. data/spec/mysql/migrate/migrate_drop_index_spec.rb +15 -15
  71. data/spec/mysql/migrate/migrate_drop_table_spec.rb +16 -16
  72. data/spec/mysql/migrate/migrate_empty_spec.rb +9 -9
  73. data/spec/mysql/migrate/migrate_execute_spec.rb +32 -32
  74. data/spec/mysql/migrate/migrate_ignore_column_spec.rb +4 -4
  75. data/spec/mysql/migrate/migrate_ignore_index_spec.rb +5 -5
  76. data/spec/mysql/migrate/migrate_log_file_spec.rb +16 -16
  77. data/spec/mysql/migrate/migrate_merge_mode_spec.rb +18 -18
  78. data/spec/mysql/migrate/migrate_noop_spec.rb +12 -12
  79. data/spec/mysql/migrate/migrate_primary_key_spec.rb +31 -6
  80. data/spec/mysql/migrate/migrate_rename_column_spec.rb +18 -18
  81. data/spec/mysql/migrate/migrate_rename_table_spec.rb +22 -22
  82. data/spec/mysql/migrate/migrate_same_spec.rb +13 -13
  83. data/spec/mysql/migrate/migrate_skip_column_comment_change_spec.rb +2 -2
  84. data/spec/mysql/migrate/migrate_skip_drop_table_spec.rb +18 -18
  85. data/spec/mysql/migrate/migrate_skip_rename_column_spec.rb +18 -18
  86. data/spec/mysql/migrate/migrate_skip_rename_table_spec.rb +18 -18
  87. data/spec/mysql/migrate/migrate_with_pre_post_query_spec.rb +18 -18
  88. data/spec/mysql/text_blob_types/text_blob_types_spec.rb +2 -2
  89. data/spec/mysql/~default_name_fk/migrate_change_fk_spec.rb +6 -6
  90. data/spec/mysql/~default_name_fk/migrate_create_fk_spec.rb +9 -9
  91. data/spec/mysql/~default_name_fk/migrate_drop_fk_spec.rb +8 -8
  92. data/spec/mysql/~dump_auto_increment/migrate_create_table_with_index_spec.rb +5 -5
  93. data/spec/mysql57/json/add_json_column_spec.rb +2 -2
  94. data/spec/mysql57/json/change_json_column_spec.rb +6 -6
  95. data/spec/mysql57/json/drop_json_column_spec.rb +2 -2
  96. data/spec/mysql57/virtual/add_virtual_column_spec.rb +1 -1
  97. data/spec/mysql57/virtual/change_virtual_column_spec.rb +1 -1
  98. data/spec/mysql57/virtual/drop_virtual_column_spec.rb +1 -1
  99. data/spec/postgresql/dump/dump_spec.rb +13 -13
  100. data/spec/postgresql/fk/migrate_change_fk_spec.rb +6 -6
  101. data/spec/postgresql/fk/migrate_create_fk_spec.rb +5 -5
  102. data/spec/postgresql/fk/migrate_drop_fk_spec.rb +8 -8
  103. data/spec/postgresql/migrate/migrate_add_column_spec.rb +27 -20
  104. data/spec/postgresql/migrate/migrate_add_expression_index_spec.rb +1 -1
  105. data/spec/postgresql/migrate/migrate_change_column_spec.rb +20 -16
  106. data/spec/postgresql/migrate/migrate_change_index_spec.rb +18 -18
  107. data/spec/postgresql/migrate/migrate_check_relation_column_type_spec.rb +1 -1
  108. data/spec/postgresql/migrate/migrate_create_table_spec.rb +14 -14
  109. data/spec/postgresql/migrate/migrate_drop_column_spec.rb +29 -22
  110. data/spec/postgresql/migrate/migrate_drop_column_with_index_spec.rb +17 -17
  111. data/spec/postgresql/migrate/migrate_drop_expression_index_spec.rb +3 -3
  112. data/spec/postgresql/migrate/migrate_drop_index_spec.rb +15 -15
  113. data/spec/postgresql/migrate/migrate_drop_table_spec.rb +16 -16
  114. data/spec/postgresql/migrate/migrate_primary_key2_spec.rb +1 -1
  115. data/spec/postgresql/migrate/migrate_primary_key_spec.rb +1 -1
  116. data/spec/postgresql/migrate/migrate_references_spec.rb +6 -6
  117. data/spec/postgresql/migrate/migrate_rename_column_spec.rb +18 -18
  118. data/spec/postgresql/migrate/migrate_rename_table_spec.rb +20 -20
  119. data/spec/postgresql/migrate/migrate_same_spec.rb +13 -13
  120. data/spec/postgresql/~default_name_fk/migrate_change_fk_spec.rb +3 -3
  121. data/spec/postgresql/~default_name_fk/migrate_create_fk_spec.rb +4 -4
  122. data/spec/postgresql/~default_name_fk/migrate_drop_fk_spec.rb +4 -4
  123. data/spec/processing_for_ci.rb +13 -0
  124. data/spec/spec_condition.rb +4 -0
  125. data/spec/spec_helper.rb +3 -2
  126. metadata +74 -49
  127. data/.travis.yml +0 -45
  128. data/lib/ridgepole/ext/abstract_mysql_adapter/use_alter_index.rb +0 -31
  129. data/omnibus-ridgepole/.gitignore +0 -10
  130. data/omnibus-ridgepole/Dockerfile.centos +0 -25
  131. data/omnibus-ridgepole/Dockerfile.ubuntu +0 -21
  132. data/omnibus-ridgepole/Gemfile +0 -24
  133. data/omnibus-ridgepole/README.md +0 -15
  134. data/omnibus-ridgepole/Rakefile +0 -38
  135. data/omnibus-ridgepole/config/projects/ridgepole.rb +0 -27
  136. data/omnibus-ridgepole/config/software/ridgepole.rb +0 -13
  137. data/omnibus-ridgepole/omnibus.rb +0 -54
  138. data/omnibus-ridgepole/package-scripts/ridgepole/postinst +0 -1
  139. data/omnibus-ridgepole/package-scripts/ridgepole/postrm +0 -1
  140. data/spec/mysql/bigint_pk/bigint_pk_spec.rb +0 -47
  141. data/spec/mysql/migrate_/migrate_create_index_with_alter_spec.rb +0 -141
  142. data/spec/mysql/migrate_/migrate_drop_index_with_alter_spec.rb +0 -141
  143. data/spec/processing_for_travis.rb +0 -11
data/spec/mysql/migrate/migrate_add_column_order_spec.rb CHANGED
@@ -37,7 +37,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
37
37
 
38
38
  expect(show_create_table_mysql('dept_emp')).to match_fuzzy erbh(<<-ERB)
39
39
  CREATE TABLE `dept_emp` (
40
- `id` <%= cond('>= 5.1','bigint(20)', 'int(11)') %> NOT NULL AUTO_INCREMENT,
40
+ `id` bigint(20) NOT NULL AUTO_INCREMENT,
41
41
  `emp_no0` int(11) NOT NULL,
42
42
  `emp_no` int(11) NOT NULL,
43
43
  `dept_no` varchar(4) NOT NULL,
@@ -129,7 +129,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
129
129
 
130
130
  expect(show_create_table_mysql('dept_emp')).to match_fuzzy erbh(<<-ERB)
131
131
  CREATE TABLE `dept_emp` (
132
- `emp_no` <%= cond('>= 5.1','bigint(20)', 'int(11)') %> NOT NULL AUTO_INCREMENT,
132
+ `emp_no` bigint(20) NOT NULL AUTO_INCREMENT,
133
133
  `emp_no0` int(11) NOT NULL,
134
134
  `dept_no` varchar(4) NOT NULL,
135
135
  `from_date` date NOT NULL,
data/spec/mysql/migrate/migrate_add_column_spec.rb CHANGED
@@ -6,12 +6,12 @@ describe 'Ridgepole::Client#diff -> migrate' do
6
6
  erbh(<<-ERB)
7
7
  create_table "clubs", force: :cascade do |t|
8
8
  t.string "name", default: "", null: false
9
- t.index ["name"], name: "idx_name", unique: true, <%= i cond(5.0, using: :btree) %>
9
+ t.index ["name"], name: "idx_name", unique: true
10
10
  end
11
11
 
12
12
  create_table "departments", primary_key: "dept_no", force: :cascade do |t|
13
13
  t.string "dept_name", limit: 40, null: false
14
- t.index ["dept_name"], name: "dept_name", unique: true, <%= i cond(5.0, using: :btree) %>
14
+ t.index ["dept_name"], name: "dept_name", unique: true
15
15
  end
16
16
 
17
17
  create_table "dept_emp", id: false, force: :cascade do |t|
@@ -19,8 +19,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
19
19
  t.string "dept_no", limit: 4, null: false
20
20
  t.date "from_date", null: false
21
21
  t.date "to_date", null: false
22
- t.index ["dept_no"], name: "dept_no", <%= i cond(5.0, using: :btree) %>
23
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
22
+ t.index ["dept_no"], name: "dept_no"
23
+ t.index ["emp_no"], name: "emp_no"
24
24
  end
25
25
 
26
26
  create_table "dept_manager", id: false, force: :cascade do |t|
@@ -28,14 +28,14 @@ describe 'Ridgepole::Client#diff -> migrate' do
28
28
  t.integer "emp_no", null: false
29
29
  t.date "from_date", null: false
30
30
  t.date "to_date", null: false
31
- t.index ["dept_no"], name: "dept_no", <%= i cond(5.0, using: :btree) %>
32
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
31
+ t.index ["dept_no"], name: "dept_no"
32
+ t.index ["emp_no"], name: "emp_no"
33
33
  end
34
34
 
35
35
  create_table "employee_clubs", force: :cascade do |t|
36
36
  t.integer "emp_no", null: false
37
37
  t.integer "club_id", null: false
38
- t.index ["emp_no", "club_id"], name: "idx_emp_no_club_id", <%= i cond(5.0, using: :btree) %>
38
+ t.index ["emp_no", "club_id"], name: "idx_emp_no_club_id"
39
39
  end
40
40
 
41
41
  create_table "employees", primary_key: "emp_no", force: :cascade do |t|
@@ -51,7 +51,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
51
51
  t.integer "salary", null: false
52
52
  t.date "from_date", null: false
53
53
  t.date "to_date", null: false
54
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
54
+ t.index ["emp_no"], name: "emp_no"
55
55
  end
56
56
 
57
57
  create_table "titles", id: false, force: :cascade do |t|
@@ -59,7 +59,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
59
59
  t.string "title", limit: 50, null: false
60
60
  t.date "from_date", null: false
61
61
  t.date "to_date"
62
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
62
+ t.index ["emp_no"], name: "emp_no"
63
63
  end
64
64
  ERB
65
65
  end
@@ -68,12 +68,12 @@ describe 'Ridgepole::Client#diff -> migrate' do
68
68
  erbh(<<-ERB)
69
69
  create_table "clubs", force: :cascade do |t|
70
70
  t.string "name", default: "", null: false
71
- t.index ["name"], name: "idx_name", unique: true, <%= i cond(5.0, using: :btree) %>
71
+ t.index ["name"], name: "idx_name", unique: true
72
72
  end
73
73
 
74
74
  create_table "departments", primary_key: "dept_no", force: :cascade do |t|
75
75
  t.string "dept_name", limit: 40, null: false
76
- t.index ["dept_name"], name: "dept_name", unique: true, <%= i cond(5.0, using: :btree) %>
76
+ t.index ["dept_name"], name: "dept_name", unique: true
77
77
  end
78
78
 
79
79
  create_table "dept_emp", id: false, force: :cascade do |t|
@@ -81,8 +81,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
81
81
  t.string "dept_no", limit: 4, null: false
82
82
  t.date "from_date", null: false
83
83
  t.date "to_date", null: false
84
- t.index ["dept_no"], name: "dept_no", <%= i cond(5.0, using: :btree) %>
85
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
84
+ t.index ["dept_no"], name: "dept_no"
85
+ t.index ["emp_no"], name: "emp_no"
86
86
  end
87
87
 
88
88
  create_table "dept_manager", id: false, force: :cascade do |t|
@@ -90,15 +90,15 @@ describe 'Ridgepole::Client#diff -> migrate' do
90
90
  t.integer "emp_no", null: false
91
91
  t.date "from_date", null: false
92
92
  t.date "to_date", null: false
93
- t.index ["dept_no"], name: "dept_no", <%= i cond(5.0, using: :btree) %>
94
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
93
+ t.index ["dept_no"], name: "dept_no"
94
+ t.index ["emp_no"], name: "emp_no"
95
95
  end
96
96
 
97
97
  create_table "employee_clubs", force: :cascade do |t|
98
98
  t.integer "emp_no", null: false
99
99
  t.integer "club_id", null: false
100
100
  t.string "any_col", null: false
101
- t.index ["emp_no", "club_id"], name: "idx_emp_no_club_id", <%= i cond(5.0, using: :btree) %>
101
+ t.index ["emp_no", "club_id"], name: "idx_emp_no_club_id"
102
102
  end
103
103
 
104
104
  create_table "employees", primary_key: "emp_no", force: :cascade do |t|
@@ -116,7 +116,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
116
116
  t.integer "salary", null: false
117
117
  t.date "from_date", null: false
118
118
  t.date "to_date", null: false
119
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
119
+ t.index ["emp_no"], name: "emp_no"
120
120
  end
121
121
 
122
122
  create_table "titles", id: false, force: :cascade do |t|
@@ -124,7 +124,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
124
124
  t.string "title", limit: 50, null: false
125
125
  t.date "from_date", null: false
126
126
  t.date "to_date"
127
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
127
+ t.index ["emp_no"], name: "emp_no"
128
128
  end
129
129
  ERB
130
130
  end
data/spec/mysql/migrate/migrate_add_column_with_alter_extra_spec.rb CHANGED
@@ -6,12 +6,12 @@ describe 'Ridgepole::Client#diff -> migrate' do
6
6
  erbh(<<-ERB)
7
7
  create_table "clubs", force: :cascade do |t|
8
8
  t.string "name", default: "", null: false
9
- t.index ["name"], name: "idx_name", unique: true, <%= i cond(5.0, using: :btree) %>
9
+ t.index ["name"], name: "idx_name", unique: true
10
10
  end
11
11
 
12
12
  create_table "departments", primary_key: "dept_no", force: :cascade do |t|
13
13
  t.string "dept_name", limit: 40, null: false
14
- t.index ["dept_name"], name: "dept_name", unique: true, <%= i cond(5.0, using: :btree) %>
14
+ t.index ["dept_name"], name: "dept_name", unique: true
15
15
  end
16
16
 
17
17
  create_table "dept_emp", id: false, force: :cascade do |t|
@@ -19,8 +19,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
19
19
  t.string "dept_no", limit: 4, null: false
20
20
  t.date "from_date", null: false
21
21
  t.date "to_date", null: false
22
- t.index ["dept_no"], name: "dept_no", <%= i cond(5.0, using: :btree) %>
23
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
22
+ t.index ["dept_no"], name: "dept_no"
23
+ t.index ["emp_no"], name: "emp_no"
24
24
  end
25
25
 
26
26
  create_table "dept_manager", id: false, force: :cascade do |t|
@@ -28,14 +28,14 @@ describe 'Ridgepole::Client#diff -> migrate' do
28
28
  t.integer "emp_no", null: false
29
29
  t.date "from_date", null: false
30
30
  t.date "to_date", null: false
31
- t.index ["dept_no"], name: "dept_no", <%= i cond(5.0, using: :btree) %>
32
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
31
+ t.index ["dept_no"], name: "dept_no"
32
+ t.index ["emp_no"], name: "emp_no"
33
33
  end
34
34
 
35
35
  create_table "employee_clubs", force: :cascade do |t|
36
36
  t.integer "emp_no", null: false
37
37
  t.integer "club_id", null: false
38
- t.index ["emp_no", "club_id"], name: "idx_emp_no_club_id", <%= i cond(5.0, using: :btree) %>
38
+ t.index ["emp_no", "club_id"], name: "idx_emp_no_club_id"
39
39
  end
40
40
 
41
41
  create_table "employees", primary_key: "emp_no", force: :cascade do |t|
@@ -51,7 +51,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
51
51
  t.integer "salary", null: false
52
52
  t.date "from_date", null: false
53
53
  t.date "to_date", null: false
54
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
54
+ t.index ["emp_no"], name: "emp_no"
55
55
  end
56
56
 
57
57
  create_table "titles", id: false, force: :cascade do |t|
@@ -59,7 +59,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
59
59
  t.string "title", limit: 50, null: false
60
60
  t.date "from_date", null: false
61
61
  t.date "to_date"
62
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
62
+ t.index ["emp_no"], name: "emp_no"
63
63
  end
64
64
  ERB
65
65
  end
@@ -68,12 +68,12 @@ describe 'Ridgepole::Client#diff -> migrate' do
68
68
  erbh(<<-ERB)
69
69
  create_table "clubs", force: :cascade do |t|
70
70
  t.string "name", default: "", null: false
71
- t.index ["name"], name: "idx_name", unique: true, <%= i cond(5.0, using: :btree) %>
71
+ t.index ["name"], name: "idx_name", unique: true
72
72
  end
73
73
 
74
74
  create_table "departments", primary_key: "dept_no", force: :cascade do |t|
75
75
  t.string "dept_name", limit: 40, null: false
76
- t.index ["dept_name"], name: "dept_name", unique: true, <%= i cond(5.0, using: :btree) %>
76
+ t.index ["dept_name"], name: "dept_name", unique: true
77
77
  end
78
78
 
79
79
  create_table "dept_emp", id: false, force: :cascade do |t|
@@ -81,8 +81,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
81
81
  t.string "dept_no", limit: 4, null: false
82
82
  t.date "from_date", null: false
83
83
  t.date "to_date", null: false
84
- t.index ["dept_no"], name: "dept_no", <%= i cond(5.0, using: :btree) %>
85
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
84
+ t.index ["dept_no"], name: "dept_no"
85
+ t.index ["emp_no"], name: "emp_no"
86
86
  end
87
87
 
88
88
  create_table "dept_manager", id: false, force: :cascade do |t|
@@ -90,15 +90,15 @@ describe 'Ridgepole::Client#diff -> migrate' do
90
90
  t.integer "emp_no", null: false
91
91
  t.date "from_date", null: false
92
92
  t.date "to_date", null: false
93
- t.index ["dept_no"], name: "dept_no", <%= i cond(5.0, using: :btree) %>
94
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
93
+ t.index ["dept_no"], name: "dept_no"
94
+ t.index ["emp_no"], name: "emp_no"
95
95
  end
96
96
 
97
97
  create_table "employee_clubs", force: :cascade do |t|
98
98
  t.integer "emp_no", null: false
99
99
  t.integer "club_id", null: false
100
100
  t.string "any_col", null: false
101
- t.index ["emp_no", "club_id"], name: "idx_emp_no_club_id", <%= i cond(5.0, using: :btree) %>
101
+ t.index ["emp_no", "club_id"], name: "idx_emp_no_club_id"
102
102
  end
103
103
 
104
104
  create_table "employees", primary_key: "emp_no", force: :cascade do |t|
@@ -116,7 +116,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
116
116
  t.integer "salary", null: false
117
117
  t.date "from_date", null: false
118
118
  t.date "to_date", null: false
119
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
119
+ t.index ["emp_no"], name: "emp_no"
120
120
  end
121
121
 
122
122
  create_table "titles", id: false, force: :cascade do |t|
@@ -124,7 +124,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
124
124
  t.string "title", limit: 50, null: false
125
125
  t.date "from_date", null: false
126
126
  t.date "to_date"
127
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
127
+ t.index ["emp_no"], name: "emp_no"
128
128
  end
129
129
  ERB
130
130
  end
@@ -140,4 +140,92 @@ describe 'Ridgepole::Client#diff -> migrate' do
140
140
  expect(subject.dump).to match_ruby expected_dsl
141
141
  }
142
142
  end
143
+
144
+ context 'when add index' do
145
+ let(:actual_dsl) do
146
+ erbh(<<-ERB)
147
+ create_table "employees", primary_key: "emp_no", force: :cascade do |t|
148
+ t.date "birth_date", null: false
149
+ t.string "first_name", limit: 14, null: false
150
+ t.string "last_name", limit: 16, null: false
151
+ t.string "gender", limit: 1, null: false
152
+ t.date "hire_date", null: false
153
+ end
154
+ ERB
155
+ end
156
+
157
+ let(:expected_dsl) do
158
+ erbh(<<-ERB)
159
+ create_table "employees", primary_key: "emp_no", force: :cascade do |t|
160
+ t.date "birth_date", null: false
161
+ t.string "first_name", limit: 14, null: false
162
+ t.string "last_name", limit: 16, null: false
163
+ t.string "gender", limit: 1, null: false
164
+ t.date "hire_date", null: false
165
+ t.index ["first_name"], name: "idx_first_name"
166
+ end
167
+ ERB
168
+ end
169
+
170
+ let(:alter_extra) { 'LOCK=DEFAULT, ALGORITHM=DEFAULT' }
171
+
172
+ before { subject.diff(actual_dsl).migrate }
173
+ subject { client }
174
+
175
+ it {
176
+ delta = subject.diff(expected_dsl)
177
+ expect(delta.differ?).to be_truthy
178
+ expect(subject.dump).to match_ruby actual_dsl
179
+ migrated, sql = delta.migrate(alter_extra: alter_extra, noop: true)
180
+ expect(migrated).to be_truthy
181
+ expect(sql).to match_fuzzy erbh(<<-SQL)
182
+ CREATE INDEX `idx_first_name` ON `employees` (`first_name`) LOCK=DEFAULT ALGORITHM=DEFAULT
183
+ SQL
184
+ delta.migrate(alter_extra: alter_extra)
185
+ expect(subject.dump).to match_ruby expected_dsl
186
+ }
187
+ end
188
+
189
+ context 'when drop index' do
190
+ let(:actual_dsl) do
191
+ erbh(<<-ERB)
192
+ create_table "employees", primary_key: "emp_no", force: :cascade do |t|
193
+ t.date "birth_date", null: false
194
+ t.string "first_name", limit: 14, null: false
195
+ t.string "last_name", limit: 16, null: false
196
+ t.string "gender", limit: 1, null: false
197
+ t.date "hire_date", null: false
198
+ t.index ["first_name"], name: "idx_first_name"
199
+ end
200
+ ERB
201
+ end
202
+
203
+ let(:expected_dsl) do
204
+ erbh(<<-ERB)
205
+ create_table "employees", primary_key: "emp_no", force: :cascade do |t|
206
+ t.date "birth_date", null: false
207
+ t.string "first_name", limit: 14, null: false
208
+ t.string "last_name", limit: 16, null: false
209
+ t.string "gender", limit: 1, null: false
210
+ t.date "hire_date", null: false
211
+ end
212
+ ERB
213
+ end
214
+
215
+ let(:alter_extra) { 'LOCK=DEFAULT, ALGORITHM=DEFAULT' }
216
+
217
+ before { subject.diff(actual_dsl).migrate }
218
+ subject { client }
219
+
220
+ it {
221
+ delta = subject.diff(expected_dsl)
222
+ expect(delta.differ?).to be_truthy
223
+ expect(subject.dump).to match_ruby actual_dsl
224
+ migrated, sql = delta.migrate(alter_extra: alter_extra, noop: true)
225
+ expect(migrated).to be_truthy
226
+ expect(sql).to eq 'DROP INDEX `idx_first_name` ON `employees` LOCK=DEFAULT ALGORITHM=DEFAULT'
227
+ delta.migrate(alter_extra: alter_extra)
228
+ expect(subject.dump).to match_ruby expected_dsl
229
+ }
230
+ end
143
231
  end
data/spec/mysql/migrate/migrate_add_column_with_script_spec.rb CHANGED
@@ -6,12 +6,12 @@ describe 'Ridgepole::Client#diff -> migrate' do
6
6
  erbh(<<-ERB)
7
7
  create_table "clubs", force: :cascade do |t|
8
8
  t.string "name", default: "", null: false
9
- t.index ["name"], name: "idx_name", unique: true, <%= i cond(5.0, using: :btree) %>
9
+ t.index ["name"], name: "idx_name", unique: true
10
10
  end
11
11
 
12
12
  create_table "departments", primary_key: "dept_no", force: :cascade do |t|
13
13
  t.string "dept_name", limit: 40, null: false
14
- t.index ["dept_name"], name: "dept_name", unique: true, <%= i cond(5.0, using: :btree) %>
14
+ t.index ["dept_name"], name: "dept_name", unique: true
15
15
  end
16
16
 
17
17
  create_table "dept_emp", id: false, force: :cascade do |t|
@@ -19,8 +19,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
19
19
  t.string "dept_no", limit: 4, null: false
20
20
  t.date "from_date", null: false
21
21
  t.date "to_date", null: false
22
- t.index ["dept_no"], name: "dept_no", <%= i cond(5.0, using: :btree) %>
23
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
22
+ t.index ["dept_no"], name: "dept_no"
23
+ t.index ["emp_no"], name: "emp_no"
24
24
  end
25
25
 
26
26
  create_table "dept_manager", id: false, force: :cascade do |t|
@@ -28,14 +28,14 @@ describe 'Ridgepole::Client#diff -> migrate' do
28
28
  t.integer "emp_no", null: false
29
29
  t.date "from_date", null: false
30
30
  t.date "to_date", null: false
31
- t.index ["dept_no"], name: "dept_no", <%= i cond(5.0, using: :btree) %>
32
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
31
+ t.index ["dept_no"], name: "dept_no"
32
+ t.index ["emp_no"], name: "emp_no"
33
33
  end
34
34
 
35
35
  create_table "employee_clubs", force: :cascade do |t|
36
36
  t.integer "emp_no", null: false
37
37
  t.integer "club_id", null: false
38
- t.index ["emp_no", "club_id"], name: "idx_emp_no_club_id", <%= i cond(5.0, using: :btree) %>
38
+ t.index ["emp_no", "club_id"], name: "idx_emp_no_club_id"
39
39
  end
40
40
 
41
41
  create_table "employees", primary_key: "emp_no", force: :cascade do |t|
@@ -51,7 +51,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
51
51
  t.integer "salary", null: false
52
52
  t.date "from_date", null: false
53
53
  t.date "to_date", null: false
54
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
54
+ t.index ["emp_no"], name: "emp_no"
55
55
  end
56
56
 
57
57
  create_table "titles", id: false, force: :cascade do |t|
@@ -59,7 +59,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
59
59
  t.string "title", limit: 50, null: false
60
60
  t.date "from_date", null: false
61
61
  t.date "to_date"
62
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
62
+ t.index ["emp_no"], name: "emp_no"
63
63
  end
64
64
  ERB
65
65
  end
@@ -68,12 +68,12 @@ describe 'Ridgepole::Client#diff -> migrate' do
68
68
  erbh(<<-ERB)
69
69
  create_table "clubs", force: :cascade do |t|
70
70
  t.string "name", default: "", null: false
71
- t.index ["name"], name: "idx_name", unique: true, <%= i cond(5.0, using: :btree) %>
71
+ t.index ["name"], name: "idx_name", unique: true
72
72
  end
73
73
 
74
74
  create_table "departments", primary_key: "dept_no", force: :cascade do |t|
75
75
  t.string "dept_name", limit: 40, null: false
76
- t.index ["dept_name"], name: "dept_name", unique: true, <%= i cond(5.0, using: :btree) %>
76
+ t.index ["dept_name"], name: "dept_name", unique: true
77
77
  end
78
78
 
79
79
  create_table "dept_emp", id: false, force: :cascade do |t|
@@ -81,8 +81,8 @@ describe 'Ridgepole::Client#diff -> migrate' do
81
81
  t.string "dept_no", limit: 4, null: false
82
82
  t.date "from_date", null: false
83
83
  t.date "to_date", null: false
84
- t.index ["dept_no"], name: "dept_no", <%= i cond(5.0, using: :btree) %>
85
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
84
+ t.index ["dept_no"], name: "dept_no"
85
+ t.index ["emp_no"], name: "emp_no"
86
86
  end
87
87
 
88
88
  create_table "dept_manager", id: false, force: :cascade do |t|
@@ -90,15 +90,15 @@ describe 'Ridgepole::Client#diff -> migrate' do
90
90
  t.integer "emp_no", null: false
91
91
  t.date "from_date", null: false
92
92
  t.date "to_date", null: false
93
- t.index ["dept_no"], name: "dept_no", <%= i cond(5.0, using: :btree) %>
94
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
93
+ t.index ["dept_no"], name: "dept_no"
94
+ t.index ["emp_no"], name: "emp_no"
95
95
  end
96
96
 
97
97
  create_table "employee_clubs", force: :cascade do |t|
98
98
  t.integer "emp_no", null: false
99
99
  t.integer "club_id", null: false
100
100
  t.string "any_col", null: false
101
- t.index ["emp_no", "club_id"], name: "idx_emp_no_club_id", <%= i cond(5.0, using: :btree) %>
101
+ t.index ["emp_no", "club_id"], name: "idx_emp_no_club_id"
102
102
  end
103
103
 
104
104
  create_table "employees", primary_key: "emp_no", force: :cascade do |t|
@@ -116,7 +116,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
116
116
  t.integer "salary", null: false
117
117
  t.date "from_date", null: false
118
118
  t.date "to_date", null: false
119
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
119
+ t.index ["emp_no"], name: "emp_no"
120
120
  end
121
121
 
122
122
  create_table "titles", id: false, force: :cascade do |t|
@@ -124,7 +124,7 @@ describe 'Ridgepole::Client#diff -> migrate' do
124
124
  t.string "title", limit: 50, null: false
125
125
  t.date "from_date", null: false
126
126
  t.date "to_date"
127
- t.index ["emp_no"], name: "emp_no", <%= i cond(5.0, using: :btree) %>
127
+ t.index ["emp_no"], name: "emp_no"
128
128
  end
129
129
  ERB
130
130
  end