diff options
| author | erdgeist <erdgeist@erdgeist.org> | 2026-07-06 04:20:55 +0200 |
|---|---|---|
| committer | erdgeist <erdgeist@erdgeist.org> | 2026-07-06 04:20:55 +0200 |
| commit | 970f10854bccee0528de8435e5a65cdcc18ba93e (patch) | |
| tree | 3d6b96fc6594241fade803df4c1862301c454767 /test/models | |
| parent | 81a07bfca08fda8ce762f2282d3bd5f6909dc923 (diff) | |
Remove dead custom_rrule references after column drop
events#index still read event.custom_rrule - a live bug the column-drop
migration introduced, not just stale test data. Caught by four test
failures (three fixtures/direct attribute hashes still setting the
removed column, one UnknownAttributeError from a stale fixture loaded
before any test method runs), none of which were actually testing this
view - "should get index" passed throughout with zero Event records
present, meaning it could never have caught a per-row rendering bug.
Strengthened to create one real event first, so a future stray
reference to a dropped or renamed column fails loudly instead of
silently passing on an empty table.
Diffstat (limited to 'test/models')
| -rw-r--r-- | test/models/event_test.rb | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/test/models/event_test.rb b/test/models/event_test.rb index d85aadd..e98605e 100644 --- a/test/models/event_test.rb +++ b/test/models/event_test.rb | |||
| @@ -29,7 +29,6 @@ class EventTest < ActiveSupport::TestCase | |||
| 29 | :longitude => 13.378944, | 29 | :longitude => 13.378944, |
| 30 | :rrule => "FOOBAR", | 30 | :rrule => "FOOBAR", |
| 31 | :allday => false, | 31 | :allday => false, |
| 32 | :custom_rrule => false, | ||
| 33 | :node_id => @cal_node.id | 32 | :node_id => @cal_node.id |
| 34 | ) | 33 | ) |
| 35 | end | 34 | end |
| @@ -44,7 +43,6 @@ class EventTest < ActiveSupport::TestCase | |||
| 44 | :longitude => 13.378944, | 43 | :longitude => 13.378944, |
| 45 | :rrule => nil, | 44 | :rrule => nil, |
| 46 | :allday => false, | 45 | :allday => false, |
| 47 | :custom_rrule => false, | ||
| 48 | :node_id => @cal_node.id | 46 | :node_id => @cal_node.id |
| 49 | ) | 47 | ) |
| 50 | 48 | ||
| @@ -62,7 +60,6 @@ class EventTest < ActiveSupport::TestCase | |||
| 62 | :longitude => 13.378944, | 60 | :longitude => 13.378944, |
| 63 | :rrule => "FREQ=WEEKLY;INTERVAL=1", | 61 | :rrule => "FREQ=WEEKLY;INTERVAL=1", |
| 64 | :allday => false, | 62 | :allday => false, |
| 65 | :custom_rrule => false, | ||
| 66 | :node_id => @cal_node.id | 63 | :node_id => @cal_node.id |
| 67 | ) | 64 | ) |
| 68 | 65 | ||
| @@ -97,7 +94,6 @@ class EventTest < ActiveSupport::TestCase | |||
| 97 | :longitude => 13.378944, | 94 | :longitude => 13.378944, |
| 98 | :rrule => "FREQ=MONTHLY;INTERVAL=1;BYDAY=-1WE", | 95 | :rrule => "FREQ=MONTHLY;INTERVAL=1;BYDAY=-1WE", |
| 99 | :allday => false, | 96 | :allday => false, |
| 100 | :custom_rrule => true, | ||
| 101 | :node_id => @cal_node.id | 97 | :node_id => @cal_node.id |
| 102 | ) | 98 | ) |
| 103 | 99 | ||
