Skip to content

Commit 045f3f3

Browse files
committed
fix: don't drop column from view in down step, because you can't remove a column from a view in posgres
1 parent b0e0d7b commit 045f3f3

3 files changed

+3
-64
lines changed

db/migrations/20190909_add_test_results_to_all_verifications.rb

+1-14
Original file line numberDiff line numberDiff line change
@@ -18,19 +18,6 @@
1818
end
1919

2020
down do
21-
create_or_replace_view(:all_verifications,
22-
from(:verifications).select(
23-
Sequel[:verifications][:id],
24-
Sequel[:verifications][:number],
25-
:success,
26-
:provider_version_id,
27-
Sequel[:v][:number].as(:provider_version_number),
28-
Sequel[:v][:order].as(:provider_version_order),
29-
:build_url,
30-
:pact_version_id,
31-
:execution_date,
32-
Sequel[:verifications][:created_at]
33-
).join(:versions, {id: :provider_version_id}, {:table_alias => :v})
34-
)
21+
# do nothing - you can't drop columns from a postgres view
3522
end
3623
end

db/migrations/20190910_add_test_results_to_latest_verifications.rb

+1-25
Original file line numberDiff line numberDiff line change
@@ -32,30 +32,6 @@
3232
end
3333

3434
down do
35-
v = :verifications
36-
create_or_replace_view(:latest_verifications,
37-
from(v)
38-
.select(
39-
Sequel[v][:id],
40-
Sequel[v][:number],
41-
Sequel[v][:success],
42-
Sequel[:s][:number].as(:provider_version),
43-
Sequel[v][:build_url],
44-
Sequel[v][:pact_version_id],
45-
Sequel[v][:execution_date],
46-
Sequel[v][:created_at],
47-
Sequel[v][:provider_version_id],
48-
Sequel[:s][:number].as(:provider_version_number),
49-
Sequel[:s][:order].as(:provider_version_order))
50-
.join(:latest_verification_numbers,
51-
{
52-
Sequel[v][:pact_version_id] => Sequel[:lv][:pact_version_id],
53-
Sequel[v][:number] => Sequel[:lv][:latest_number]
54-
}, { table_alias: :lv })
55-
.join(:versions,
56-
{
57-
Sequel[v][:provider_version_id] => Sequel[:s][:id]
58-
}, { table_alias: :s })
59-
)
35+
# do nothing - you can't drop columns from a postgres view
6036
end
6137
end

db/migrations/20190911_add_test_results_to_latest_verifications_for_pact_versions.rb

+1-25
Original file line numberDiff line numberDiff line change
@@ -29,30 +29,6 @@
2929
end
3030

3131
down do
32-
# The most recent verification for each pact_version
33-
v = :verifications
34-
create_or_replace_view(:latest_verifications_for_pact_versions,
35-
from(v)
36-
.select(
37-
Sequel[v][:id],
38-
Sequel[v][:number],
39-
Sequel[v][:success],
40-
Sequel[v][:build_url],
41-
Sequel[v][:pact_version_id],
42-
Sequel[v][:execution_date],
43-
Sequel[v][:created_at],
44-
Sequel[v][:provider_version_id],
45-
Sequel[:s][:number].as(:provider_version_number),
46-
Sequel[:s][:order].as(:provider_version_order))
47-
.join(:latest_verification_ids_for_pact_versions,
48-
{
49-
Sequel[v][:pact_version_id] => Sequel[:lv][:pact_version_id],
50-
Sequel[v][:id] => Sequel[:lv][:latest_verification_id]
51-
}, { table_alias: :lv })
52-
.join(:versions,
53-
{
54-
Sequel[v][:provider_version_id] => Sequel[:s][:id]
55-
}, { table_alias: :s })
56-
)
32+
# do nothing - you can't drop columns from a postgres view
5733
end
5834
end

0 commit comments

Comments
 (0)