Skip to content

Commit e250cc1

Browse files
authored
Clean out finished_at in create (#94)
* Clean out finished_at in create * make it raisey
1 parent ba507f5 commit e250cc1

2 files changed

Lines changed: 10 additions & 0 deletions

File tree

app/models/split_creation.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ class SplitCreation
77
validate :split_must_be_valid
88

99
def save
10+
split.finished_at = nil
1011
split.reassign_weight(merged_registry) unless split.registry == merged_registry
1112
return false unless valid?
1213
split.save

spec/models/split_creation_spec.rb

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,15 @@
3232
expect(weather.registry.symbolize_keys).to eq rain: 0, snow: 0, clear_skies: 100, hurricane: 0
3333
end
3434

35+
it 'reenables a finished split' do
36+
bad_weather_create.save
37+
Split.find_by!(name: "weather").update!(finished_at: Time.zone.now)
38+
good_weather_create.save
39+
40+
weather = Split.find_by(name: "weather")
41+
expect(weather.finished_at).to be_nil
42+
end
43+
3544
it 'delegates validation errors to split' do
3645
split_creation = SplitCreation.new(app: default_app, name: "bad_test", weighting_registry: { badBadBad: 100 })
3746
expect(split_creation.save).to eq false

0 commit comments

Comments
 (0)