hertz 1.0.0 → 1.0.1
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/README.md +12 -8
- data/app/models/hertz/delivery.rb +9 -0
- data/app/models/hertz/notification.rb +10 -1
- data/db/migrate/20160627084018_create_hertz_notification_deliveries.rb +12 -0
- data/db/migrate/20160628084342_rename_notification_deliveries_to_deliveries.rb +5 -0
- data/lib/hertz.rb +12 -2
- data/lib/hertz/notifiable.rb +3 -0
- data/lib/hertz/notification_deliverer.rb +5 -1
- data/lib/hertz/version.rb +1 -1
- data/spec/dummy/db/migrate/20160627084119_create_hertz_notification_deliveries.hertz.rb +13 -0
- data/spec/dummy/db/migrate/20160628084413_rename_notification_deliveries_to_deliveries.hertz.rb +6 -0
- data/spec/dummy/db/schema.rb +11 -1
- data/spec/dummy/log/development.log +96 -30
- data/spec/dummy/log/test.log +2341 -305
- data/spec/examples.txt +18 -11
- data/spec/factories/hertz/deliveries.rb +7 -0
- data/spec/hertz/hertz_spec.rb +14 -0
- data/spec/hertz/notification_deliverer_spec.rb +43 -8
- data/spec/models/hertz/delivery_spec.rb +12 -0
- data/spec/models/hertz/notification_spec.rb +20 -0
- data/spec/rails_helper.rb +0 -3
- data/spec/support/database_cleaner.rb +1 -1
- data/spec/support/faker.rb +1 -0
- data/spec/support/shoulda.rb +8 -0
- metadata +72 -41
data/spec/examples.txt
CHANGED
@@ -1,11 +1,18 @@
|
|
1
|
-
example_id
|
2
|
-
|
3
|
-
./spec/hertz/
|
4
|
-
./spec/hertz/notifiable_spec.rb[1:
|
5
|
-
./spec/hertz/
|
6
|
-
./spec/
|
7
|
-
./spec/
|
8
|
-
./spec/
|
9
|
-
./spec/models/hertz/
|
10
|
-
./spec/models/hertz/
|
11
|
-
./spec/models/hertz/notification_spec.rb[1:
|
1
|
+
example_id | status | run_time |
|
2
|
+
---------------------------------------------------- | ------ | --------------- |
|
3
|
+
./spec/hertz/hertz_spec.rb[1:1:1] | passed | 0.00819 seconds |
|
4
|
+
./spec/hertz/notifiable_spec.rb[1:1:1] | passed | 0.01368 seconds |
|
5
|
+
./spec/hertz/notifiable_spec.rb[1:2:1] | passed | 0.0345 seconds |
|
6
|
+
./spec/hertz/notification_deliverer_spec.rb[1:1:1] | passed | 0.02577 seconds |
|
7
|
+
./spec/hertz/notification_deliverer_spec.rb[1:1:2:1] | passed | 0.01488 seconds |
|
8
|
+
./spec/hertz/notification_deliverer_spec.rb[1:1:2:2] | passed | 0.01501 seconds |
|
9
|
+
./spec/models/hertz/delivery_spec.rb[1:1] | passed | 0.00778 seconds |
|
10
|
+
./spec/models/hertz/delivery_spec.rb[1:2] | passed | 0.15849 seconds |
|
11
|
+
./spec/models/hertz/notification_spec.rb[1:1:1] | passed | 0.00711 seconds |
|
12
|
+
./spec/models/hertz/notification_spec.rb[1:2:1] | passed | 0.00692 seconds |
|
13
|
+
./spec/models/hertz/notification_spec.rb[1:3:1] | passed | 0.01355 seconds |
|
14
|
+
./spec/models/hertz/notification_spec.rb[1:4:1] | passed | 0.01073 seconds |
|
15
|
+
./spec/models/hertz/notification_spec.rb[1:5:1] | passed | 0.01763 seconds |
|
16
|
+
./spec/models/hertz/notification_spec.rb[1:6:1] | passed | 0.01407 seconds |
|
17
|
+
./spec/models/hertz/notification_spec.rb[1:7:1] | passed | 0.01344 seconds |
|
18
|
+
./spec/models/hertz/notification_spec.rb[1:8] | passed | 0.0093 seconds |
|
@@ -0,0 +1,14 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
RSpec.describe Hertz do
|
3
|
+
describe '.configure' do
|
4
|
+
after(:each) { described_class.common_couriers = [] }
|
5
|
+
|
6
|
+
it 'configures the module' do
|
7
|
+
expect {
|
8
|
+
described_class.configure do |config|
|
9
|
+
config.common_couriers = [:test]
|
10
|
+
end
|
11
|
+
}.to change(described_class, :common_couriers).to([:test])
|
12
|
+
end
|
13
|
+
end
|
14
|
+
end
|
@@ -3,29 +3,64 @@ module Hertz
|
|
3
3
|
RSpec.describe NotificationDeliverer do
|
4
4
|
subject { described_class }
|
5
5
|
|
6
|
-
before(:
|
6
|
+
before(:all) do
|
7
7
|
module Courier
|
8
|
-
|
8
|
+
class Test
|
9
9
|
def self.deliver_notification(_notification)
|
10
10
|
end
|
11
11
|
end
|
12
12
|
end
|
13
|
-
|
14
|
-
class TestNotification < Notification
|
15
|
-
deliver_by :test
|
16
|
-
end
|
17
13
|
end
|
18
14
|
|
19
15
|
describe '#deliver' do
|
20
|
-
let(:notification)
|
16
|
+
let(:notification) do
|
17
|
+
Class.new(TestNotification) do
|
18
|
+
deliver_by :test
|
19
|
+
end.new
|
20
|
+
end
|
21
21
|
|
22
22
|
it 'delivers the notification through the couriers' do
|
23
23
|
expect(Hertz::Courier::Test).to receive(:deliver_notification)
|
24
|
-
.with(notification)
|
25
24
|
.once
|
25
|
+
.with(notification)
|
26
26
|
|
27
27
|
subject.deliver(notification)
|
28
28
|
end
|
29
|
+
|
30
|
+
context 'when common couriers are defined' do
|
31
|
+
before(:all) do
|
32
|
+
module Courier
|
33
|
+
class Common
|
34
|
+
def self.deliver_notification(_notification)
|
35
|
+
end
|
36
|
+
end
|
37
|
+
end
|
38
|
+
end
|
39
|
+
|
40
|
+
before(:each) do
|
41
|
+
allow(Hertz).to receive(:common_couriers)
|
42
|
+
.and_return([:common])
|
43
|
+
|
44
|
+
allow(Hertz::Courier::Test).to receive(:deliver_notification)
|
45
|
+
allow(Hertz::Courier::Common).to receive(:deliver_notification)
|
46
|
+
end
|
47
|
+
|
48
|
+
it 'uses common couriers' do
|
49
|
+
expect(Hertz::Courier::Test).to receive(:deliver_notification)
|
50
|
+
.once
|
51
|
+
.with(notification)
|
52
|
+
|
53
|
+
subject.deliver(notification)
|
54
|
+
end
|
55
|
+
|
56
|
+
it 'uses model-specific couriers' do
|
57
|
+
expect(Hertz::Courier::Common).to receive(:deliver_notification)
|
58
|
+
.once
|
59
|
+
.with(notification)
|
60
|
+
|
61
|
+
subject.deliver(notification)
|
62
|
+
end
|
63
|
+
end
|
29
64
|
end
|
30
65
|
end
|
31
66
|
end
|
@@ -0,0 +1,12 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
module Hertz
|
3
|
+
RSpec.describe Delivery do
|
4
|
+
subject { build_stubbed(:delivery) }
|
5
|
+
|
6
|
+
%w(notification courier).each do |attribute|
|
7
|
+
it "validates the presence of #{attribute}" do
|
8
|
+
expect(subject).to validate_presence_of(attribute)
|
9
|
+
end
|
10
|
+
end
|
11
|
+
end
|
12
|
+
end
|
@@ -41,6 +41,26 @@ module Hertz
|
|
41
41
|
end
|
42
42
|
end
|
43
43
|
|
44
|
+
describe '#delivered_with?' do
|
45
|
+
subject { create(:notification) }
|
46
|
+
|
47
|
+
it 'returns whether it has been delivered with that courier' do
|
48
|
+
expect {
|
49
|
+
create(:delivery, notification: subject, courier: :test)
|
50
|
+
}.to change { subject.delivered_with?(:test) }.from(false).to(true)
|
51
|
+
end
|
52
|
+
end
|
53
|
+
|
54
|
+
describe '#mark_delivered_with' do
|
55
|
+
subject { create(:notification) }
|
56
|
+
|
57
|
+
it 'creates a delivery for that courier' do
|
58
|
+
expect {
|
59
|
+
subject.mark_delivered_with(:test)
|
60
|
+
}.to change(subject.deliveries, :count).by(1)
|
61
|
+
end
|
62
|
+
end
|
63
|
+
|
44
64
|
describe '.unread' do
|
45
65
|
let!(:unread_notification) { create(:notification) }
|
46
66
|
let!(:read_notification) { create(:notification, :read) }
|
data/spec/rails_helper.rb
CHANGED
@@ -12,9 +12,6 @@ require 'rspec/rails'
|
|
12
12
|
# Load RSpec helpers.
|
13
13
|
Dir[File.join(ENGINE_ROOT, 'spec/support/**/*.rb')].each { |f| require f }
|
14
14
|
|
15
|
-
# Load testing libraries.
|
16
|
-
require 'faker'
|
17
|
-
|
18
15
|
# Load migrations from the dummy app.
|
19
16
|
ActiveRecord::Migrator.migrations_paths = File.join(ENGINE_ROOT, 'spec/dummy/db/migrate')
|
20
17
|
ActiveRecord::Migration.maintain_test_schema!
|
@@ -0,0 +1 @@
|
|
1
|
+
require 'faker'
|
metadata
CHANGED
@@ -1,14 +1,14 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: hertz
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 1.0.
|
4
|
+
version: 1.0.1
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- Alessandro Desantis
|
8
8
|
autorequire:
|
9
9
|
bindir: bin
|
10
10
|
cert_chain: []
|
11
|
-
date: 2016-
|
11
|
+
date: 2016-06-28 00:00:00.000000000 Z
|
12
12
|
dependencies:
|
13
13
|
- !ruby/object:Gem::Dependency
|
14
14
|
name: rails
|
@@ -136,6 +136,20 @@ dependencies:
|
|
136
136
|
- - ">="
|
137
137
|
- !ruby/object:Gem::Version
|
138
138
|
version: '0'
|
139
|
+
- !ruby/object:Gem::Dependency
|
140
|
+
name: shoulda-matchers
|
141
|
+
requirement: !ruby/object:Gem::Requirement
|
142
|
+
requirements:
|
143
|
+
- - ">="
|
144
|
+
- !ruby/object:Gem::Version
|
145
|
+
version: '0'
|
146
|
+
type: :development
|
147
|
+
prerelease: false
|
148
|
+
version_requirements: !ruby/object:Gem::Requirement
|
149
|
+
requirements:
|
150
|
+
- - ">="
|
151
|
+
- !ruby/object:Gem::Version
|
152
|
+
version: '0'
|
139
153
|
description:
|
140
154
|
email:
|
141
155
|
- desa.alessandro@gmail.com
|
@@ -150,10 +164,13 @@ files:
|
|
150
164
|
- app/assets/stylesheets/hertz/application.css
|
151
165
|
- app/controllers/hertz/application_controller.rb
|
152
166
|
- app/helpers/hertz/application_helper.rb
|
167
|
+
- app/models/hertz/delivery.rb
|
153
168
|
- app/models/hertz/notification.rb
|
154
169
|
- app/views/layouts/hertz/application.html.erb
|
155
170
|
- config/routes.rb
|
156
171
|
- db/migrate/20160415174901_create_hertz_notifications.rb
|
172
|
+
- db/migrate/20160627084018_create_hertz_notification_deliveries.rb
|
173
|
+
- db/migrate/20160628084342_rename_notification_deliveries_to_deliveries.rb
|
157
174
|
- lib/generators/hertz/install_generator.rb
|
158
175
|
- lib/generators/hertz/templates/initializer.rb
|
159
176
|
- lib/hertz.rb
|
@@ -200,6 +217,8 @@ files:
|
|
200
217
|
- spec/dummy/db/migrate/20160415175837_create_users.rb
|
201
218
|
- spec/dummy/db/migrate/20160418122437_add_email_to_users.rb
|
202
219
|
- spec/dummy/db/migrate/20160508094342_remove_email_from_users.rb
|
220
|
+
- spec/dummy/db/migrate/20160627084119_create_hertz_notification_deliveries.hertz.rb
|
221
|
+
- spec/dummy/db/migrate/20160628084413_rename_notification_deliveries_to_deliveries.hertz.rb
|
203
222
|
- spec/dummy/db/schema.rb
|
204
223
|
- spec/dummy/log/development.log
|
205
224
|
- spec/dummy/log/test.log
|
@@ -208,15 +227,20 @@ files:
|
|
208
227
|
- spec/dummy/public/500.html
|
209
228
|
- spec/dummy/public/favicon.ico
|
210
229
|
- spec/examples.txt
|
230
|
+
- spec/factories/hertz/deliveries.rb
|
211
231
|
- spec/factories/hertz/notifications.rb
|
212
232
|
- spec/factories/users.rb
|
233
|
+
- spec/hertz/hertz_spec.rb
|
213
234
|
- spec/hertz/notifiable_spec.rb
|
214
235
|
- spec/hertz/notification_deliverer_spec.rb
|
236
|
+
- spec/models/hertz/delivery_spec.rb
|
215
237
|
- spec/models/hertz/notification_spec.rb
|
216
238
|
- spec/rails_helper.rb
|
217
239
|
- spec/spec_helper.rb
|
218
240
|
- spec/support/database_cleaner.rb
|
219
241
|
- spec/support/factory_girl.rb
|
242
|
+
- spec/support/faker.rb
|
243
|
+
- spec/support/shoulda.rb
|
220
244
|
homepage: https://github.com/alessandro1997/hertz
|
221
245
|
licenses:
|
222
246
|
- MIT
|
@@ -243,58 +267,65 @@ specification_version: 4
|
|
243
267
|
summary: A Rails engine for in-app notifications.
|
244
268
|
test_files:
|
245
269
|
- spec/spec_helper.rb
|
246
|
-
- spec/hertz/notifiable_spec.rb
|
247
|
-
- spec/hertz/notification_deliverer_spec.rb
|
248
|
-
- spec/factories/hertz/notifications.rb
|
249
|
-
- spec/factories/users.rb
|
250
|
-
- spec/support/factory_girl.rb
|
251
|
-
- spec/support/database_cleaner.rb
|
252
|
-
- spec/models/hertz/notification_spec.rb
|
253
|
-
- spec/examples.txt
|
254
270
|
- spec/rails_helper.rb
|
271
|
+
- spec/dummy/db/schema.rb
|
272
|
+
- spec/dummy/db/migrate/20160418122437_add_email_to_users.rb
|
273
|
+
- spec/dummy/db/migrate/20160415175358_create_hertz_notifications.hertz.rb
|
255
274
|
- spec/dummy/db/migrate/20160415175837_create_users.rb
|
275
|
+
- spec/dummy/db/migrate/20160628084413_rename_notification_deliveries_to_deliveries.hertz.rb
|
276
|
+
- spec/dummy/db/migrate/20160627084119_create_hertz_notification_deliveries.hertz.rb
|
256
277
|
- spec/dummy/db/migrate/20160508094342_remove_email_from_users.rb
|
257
|
-
- spec/dummy/
|
258
|
-
- spec/dummy/
|
259
|
-
- spec/dummy/db/schema.rb
|
260
|
-
- spec/dummy/bin/rake
|
261
|
-
- spec/dummy/bin/rails
|
262
|
-
- spec/dummy/bin/setup
|
263
|
-
- spec/dummy/bin/bundle
|
264
|
-
- spec/dummy/app/views/hertz/notification_mailer/test_notification.html.erb
|
265
|
-
- spec/dummy/app/views/layouts/application.html.erb
|
266
|
-
- spec/dummy/app/models/user.rb
|
267
|
-
- spec/dummy/app/models/test_notification.rb
|
268
|
-
- spec/dummy/app/helpers/application_helper.rb
|
269
|
-
- spec/dummy/app/controllers/application_controller.rb
|
270
|
-
- spec/dummy/app/assets/stylesheets/application.css
|
271
|
-
- spec/dummy/app/assets/javascripts/application.js
|
278
|
+
- spec/dummy/README.rdoc
|
279
|
+
- spec/dummy/public/500.html
|
272
280
|
- spec/dummy/public/422.html
|
273
281
|
- spec/dummy/public/favicon.ico
|
274
|
-
- spec/dummy/public/500.html
|
275
282
|
- spec/dummy/public/404.html
|
276
|
-
- spec/dummy/
|
277
|
-
- spec/dummy/
|
283
|
+
- spec/dummy/bin/setup
|
284
|
+
- spec/dummy/bin/rails
|
285
|
+
- spec/dummy/bin/rake
|
286
|
+
- spec/dummy/bin/bundle
|
287
|
+
- spec/dummy/config/database.example.yml
|
288
|
+
- spec/dummy/config/environments/production.rb
|
289
|
+
- spec/dummy/config/environments/development.rb
|
290
|
+
- spec/dummy/config/environments/test.rb
|
278
291
|
- spec/dummy/config/boot.rb
|
292
|
+
- spec/dummy/config/environment.rb
|
293
|
+
- spec/dummy/config/database.yml
|
279
294
|
- spec/dummy/config/initializers/inflections.rb
|
295
|
+
- spec/dummy/config/initializers/hertz.rb
|
296
|
+
- spec/dummy/config/initializers/assets.rb
|
280
297
|
- spec/dummy/config/initializers/backtrace_silencers.rb
|
298
|
+
- spec/dummy/config/initializers/mime_types.rb
|
281
299
|
- spec/dummy/config/initializers/cookies_serializer.rb
|
282
|
-
- spec/dummy/config/initializers/session_store.rb
|
283
|
-
- spec/dummy/config/initializers/assets.rb
|
284
300
|
- spec/dummy/config/initializers/wrap_parameters.rb
|
285
|
-
- spec/dummy/config/initializers/mime_types.rb
|
286
|
-
- spec/dummy/config/initializers/hertz.rb
|
287
301
|
- spec/dummy/config/initializers/filter_parameter_logging.rb
|
288
|
-
- spec/dummy/config/
|
302
|
+
- spec/dummy/config/initializers/session_store.rb
|
289
303
|
- spec/dummy/config/secrets.yml
|
290
|
-
- spec/dummy/config/environments/development.rb
|
291
|
-
- spec/dummy/config/environments/production.rb
|
292
|
-
- spec/dummy/config/environments/test.rb
|
293
304
|
- spec/dummy/config/locales/en.yml
|
294
|
-
- spec/dummy/config/
|
295
|
-
- spec/dummy/config/environment.rb
|
296
|
-
- spec/dummy/config/database.example.yml
|
305
|
+
- spec/dummy/config/application.rb
|
297
306
|
- spec/dummy/config/routes.rb
|
298
|
-
- spec/dummy/
|
299
|
-
- spec/dummy/log/test.log
|
307
|
+
- spec/dummy/config.ru
|
300
308
|
- spec/dummy/log/development.log
|
309
|
+
- spec/dummy/log/test.log
|
310
|
+
- spec/dummy/app/helpers/application_helper.rb
|
311
|
+
- spec/dummy/app/controllers/application_controller.rb
|
312
|
+
- spec/dummy/app/models/user.rb
|
313
|
+
- spec/dummy/app/models/test_notification.rb
|
314
|
+
- spec/dummy/app/assets/javascripts/application.js
|
315
|
+
- spec/dummy/app/assets/stylesheets/application.css
|
316
|
+
- spec/dummy/app/views/layouts/application.html.erb
|
317
|
+
- spec/dummy/app/views/hertz/notification_mailer/test_notification.html.erb
|
318
|
+
- spec/dummy/Rakefile
|
319
|
+
- spec/support/database_cleaner.rb
|
320
|
+
- spec/support/faker.rb
|
321
|
+
- spec/support/shoulda.rb
|
322
|
+
- spec/support/factory_girl.rb
|
323
|
+
- spec/hertz/notification_deliverer_spec.rb
|
324
|
+
- spec/hertz/notifiable_spec.rb
|
325
|
+
- spec/hertz/hertz_spec.rb
|
326
|
+
- spec/models/hertz/delivery_spec.rb
|
327
|
+
- spec/models/hertz/notification_spec.rb
|
328
|
+
- spec/examples.txt
|
329
|
+
- spec/factories/hertz/deliveries.rb
|
330
|
+
- spec/factories/hertz/notifications.rb
|
331
|
+
- spec/factories/users.rb
|