diff options
author | Matt Jankowski <mjankowski@thoughtbot.com> | 2017-05-05 14:56:00 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-05-05 14:56:00 -0400 |
commit | 484c9709b67685c95de351a39e3dfb140acd3681 (patch) | |
tree | 0d74cd725e46cd75ca1568d866d5d5ef906b8c0a /spec/models/subscription_spec.rb | |
parent | d08f1112d517788fb66d2683766cc168bac48315 (diff) |
Misc spec coverage improvements (#2821)
* Dont use raise_error by itself (avoids warning) * Add coverage for AccountFilter * Improve coverage and refactor for Subscription#lease_seconds * Improve coverage and refactor for NotificationMailer * Simplify assignment of min/max threshold on subscription
Diffstat (limited to 'spec/models/subscription_spec.rb')
-rw-r--r-- | spec/models/subscription_spec.rb | 48 |
1 files changed, 48 insertions, 0 deletions
diff --git a/spec/models/subscription_spec.rb b/spec/models/subscription_spec.rb index 11fd8fadc..84096e6ae 100644 --- a/spec/models/subscription_spec.rb +++ b/spec/models/subscription_spec.rb @@ -16,4 +16,52 @@ RSpec.describe Subscription, type: :model do expect(subject.expired?).to be false end end + + describe 'lease_seconds' do + it 'returns the time remaing until expiration' do + datetime = 1.day.from_now + subscription = Subscription.new(expires_at: datetime) + travel_to(datetime - 12.hours) do + expect(subscription.lease_seconds).to eq(12.hours) + end + end + end + + describe 'lease_seconds=' do + it 'sets expires_at to min expiration when small value is provided' do + subscription = Subscription.new + datetime = 1.day.from_now + too_low = Subscription::MIN_EXPIRATION - 1000 + travel_to(datetime) do + subscription.lease_seconds = too_low + end + + expected = datetime + Subscription::MIN_EXPIRATION.seconds + expect(subscription.expires_at).to be_within(1.0).of(expected) + end + + it 'sets expires_at to value when valid value is provided' do + subscription = Subscription.new + datetime = 1.day.from_now + valid = Subscription::MIN_EXPIRATION + 1000 + travel_to(datetime) do + subscription.lease_seconds = valid + end + + expected = datetime + valid.seconds + expect(subscription.expires_at).to be_within(1.0).of(expected) + end + + it 'sets expires_at to max expiration when large value is provided' do + subscription = Subscription.new + datetime = 1.day.from_now + too_high = Subscription::MAX_EXPIRATION + 1000 + travel_to(datetime) do + subscription.lease_seconds = too_high + end + + expected = datetime + Subscription::MAX_EXPIRATION.seconds + expect(subscription.expires_at).to be_within(1.0).of(expected) + end + end end |