requeue 0.7.2 → 0.7.3

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
checksums.yaml CHANGED
@@ -1,7 +1,7 @@
1
1
  ---
2
2
  SHA1:
3
- metadata.gz: 16b7f85c1aa1c960e6ff842c6552d8a2f15da648
4
- data.tar.gz: 0659835500d1aa2cbe07d6f1cb874d5d00684e4c
3
+ metadata.gz: a50634423a9b4983dabf590dec78f667453341d6
4
+ data.tar.gz: a0cc2ce7e7b85502f94aac32e8d961f27f908fe5
5
5
  SHA512:
6
- metadata.gz: 3eb5540f3996393bb6f2db1fd0590ad99fbd0321e3264e81aaf00dec1d47fd46e5f3011f2ef098e66c9e9b807839aae7d3a391e0d28c9b547fb096c72d9cd111
7
- data.tar.gz: 0128cb11ff7c9b6ce4999d73ac7eb4e700d0e550c703ebe533002a750445429c943557482084e37ad3ea096e46777f233cf2b6bff69f39a7a827a0b96c2d0a13
6
+ metadata.gz: f7fbb67b2d91a065672e78c609b3071dae7a501233e447b4de713de8a9ef2c4481dfeedea9d3179ea362e8474fc992f66535de1b3160cd390ff341977d933d61
7
+ data.tar.gz: d5b8e45fc3d90c7a904a09ccdaa0b57775301dfb7ce589c07521b132f17cbe0fb0689a97decbf4feea416133207bf5a61cfa769bd2e07fa0535d7683e7827019
data/README.md CHANGED
@@ -1,5 +1,5 @@
1
1
  # Requeue
2
-
2
+ [![Code Climate](https://codeclimate.com/github/lumoslabs/requeue.png)](https://codeclimate.com/github/lumoslabs/requeue)
3
3
  Requeue is a rediciulosly simple queue backed by redis.
4
4
 
5
5
  ## Installation
@@ -45,7 +45,7 @@ module Requeue
45
45
  end
46
46
 
47
47
  def remove!(value)
48
- @redis.lrem(name,0,value)
48
+ @redis.lrem(name, 0, value)
49
49
  end
50
50
 
51
51
  def owner
@@ -53,7 +53,7 @@ module Requeue
53
53
  end
54
54
 
55
55
  def first
56
- @redis.lrange(name,0,1).first
56
+ @redis.lrange(name, 0, 1).first
57
57
  end
58
58
 
59
59
  def owned?
@@ -61,7 +61,7 @@ module Requeue
61
61
  end
62
62
 
63
63
  def steal!(value)
64
- @redis.lpush(name,value)
64
+ @redis.lpush(name, value)
65
65
  end
66
66
 
67
67
  def as_json
@@ -1,3 +1,3 @@
1
1
  module Requeue
2
- VERSION = "0.7.2"
2
+ VERSION = "0.7.3"
3
3
  end
@@ -9,27 +9,25 @@ describe Requeue::Queue do
9
9
  @queue.clear!
10
10
  end
11
11
 
12
- describe '.queue_name' do
12
+ describe '.name' do
13
13
  it { expect(@queue.name).to eq("test:queue") }
14
14
  end
15
15
 
16
- describe '.clear_queue!' do
16
+ describe '.clear!' do
17
17
  before { @queue.enqueue!('eric') }
18
- before { @queue.clear! }
19
18
 
20
19
  it 'should empty the queue' do
21
- expect(@redis.llen(@queue.name)).to eq(0)
20
+ expect{ @queue.clear! }.to change { @queue.length }.from(1).to(0)
22
21
  end
23
22
 
24
23
  it 'should see an empty queue' do
25
- expect(@queue).not_to be_owned
24
+ expect {@queue.clear! }.to change { @queue.owned? }.from(true).to(false)
26
25
  end
27
26
  end
28
27
 
29
28
  describe '.enqueue' do
30
29
  context 'when no users are enqueued' do
31
- after { @queue.clear! }
32
- it 'should return 1' do
30
+ it 'returns the number of users in the queue' do
33
31
  expect(@queue.enqueue!('eric')).to eq(1)
34
32
  end
35
33
  end
@@ -37,7 +35,6 @@ describe Requeue::Queue do
37
35
  context 'when some users are enqueued' do
38
36
  before { @queue.enqueue!('bob') }
39
37
  before { @queue.enqueue!('eric') }
40
- after { @queue.clear! }
41
38
  it 'should have a length of 2' do
42
39
  expect(@queue.length).to eq(2)
43
40
  end
@@ -50,8 +47,7 @@ describe Requeue::Queue do
50
47
  it 'should add non-unique values if the queue is set to non-unique' do
51
48
  nonunique = Requeue::Queue.new(unique:false)
52
49
  nonunique.enqueue!('eric')
53
- nonunique.enqueue!('eric')
54
- expect(nonunique.length).to eq(2)
50
+ expect{ nonunique.enqueue!('eric') }.to change { nonunique.length }.from(1).to(2)
55
51
  end
56
52
  end
57
53
  end
@@ -66,22 +62,25 @@ describe Requeue::Queue do
66
62
  context 'when some users are enqueued' do
67
63
  before { @queue.enqueue!('bob') }
68
64
  before { @queue.enqueue!('eric') }
69
- after { @queue.clear! }
65
+
70
66
  it 'should have the current users' do
71
67
  expect(@queue.list).to eq(['bob', 'eric'])
72
68
  end
73
69
  end
74
70
  end
71
+
75
72
  describe '.user_queued?' do
76
73
  before { @queue.enqueue!('bob') }
77
74
  before { @queue.enqueue!('eric') }
78
- after { @queue.clear! }
75
+
79
76
  it 'should show eric as queued' do
80
77
  expect(@queue.queued? 'eric').to eq(true)
81
78
  end
79
+
82
80
  it 'should show bob as queued' do
83
81
  expect(@queue.queued? 'bob').to eq(true)
84
82
  end
83
+
85
84
  it 'should not show an unqueued user as queued' do
86
85
  expect(@queue.queued? 'jim').to eq(false)
87
86
  end
@@ -90,7 +89,7 @@ describe Requeue::Queue do
90
89
  describe '.user_position' do
91
90
  before { @queue.enqueue!('bob') }
92
91
  before { @queue.enqueue!('eric') }
93
- after { @queue.clear! }
92
+
94
93
  it 'eric should be in position 1 (the second place)' do
95
94
  expect(@queue.position 'eric').to eq(1)
96
95
  expect(@queue.position 'bob').to eq(0)
@@ -100,14 +99,14 @@ describe Requeue::Queue do
100
99
  describe '.dequeue' do
101
100
  before { @queue.enqueue!('bob') }
102
101
  before { @queue.enqueue!('eric') }
103
- before { @queue.dequeue! }
102
+
104
103
  it 'eric should be in position 1 (the second place)' do
105
- expect(@queue.position 'eric').to eq(0)
104
+ expect{ @queue.dequeue! }.to change {@queue.position 'eric'}.from(1).to(0)
106
105
  end
107
106
  end
108
107
 
109
108
  describe '.owner' do
110
- context 'when no users are enqueued' do
109
+ context 'when no users are enqueued' do
111
110
  it 'should return empty string' do
112
111
  expect(@queue.owner).to eq(nil)
113
112
  end
@@ -124,28 +123,26 @@ describe Requeue::Queue do
124
123
  describe '.remove!' do
125
124
  before { @queue.enqueue!('bob') }
126
125
  before { @queue.enqueue!('eric') }
127
- before { @queue.remove!('eric') }
128
126
 
129
127
  it 'should not contain eric anymore' do
130
- expect(@queue.queued?('eric')).to eq(false)
131
- expect(@queue.list).to eq(['bob'])
128
+ expect { @queue.remove!('eric') }.to change {@queue.queued?('eric')}
129
+ expect { @queue.remove!('eric') }.to_not change {@queue.queued?('bob')}
132
130
  end
133
131
  end
134
132
 
135
133
  describe '.steal!' do
136
134
  before { @queue.enqueue!('bob') }
137
135
  before { @queue.enqueue!('eric') }
138
- before { @queue.steal!('jim') }
139
136
 
140
137
  it 'jim should own the queue' do
141
- expect(@queue.owner()).to eq('jim')
142
- expect(@queue.list).to eq(['jim','bob','eric'])
138
+ expect { @queue.steal!('jim') }.to change { @queue.owner }.from('bob').to('jim')
143
139
  end
144
140
 
145
141
  it 'the queue should still have everyone else in it' do
146
- expect(@queue.length).to eq(3)
142
+ expect { @queue.steal!('jim') }.to change { @queue.length }.from(2).to(3)
147
143
  end
148
144
  end
145
+
149
146
  describe '.owned?' do
150
147
  context 'when no users are enqueued' do
151
148
  it 'should start unowned' do
@@ -154,9 +151,8 @@ describe Requeue::Queue do
154
151
  end
155
152
 
156
153
  context 'when a user is enqueued' do
157
- before { @queue.enqueue!('eric') }
158
154
  it 'should be owned' do
159
- expect(@queue).to be_owned
155
+ expect { @queue.enqueue!('eric') }.to change { @queue.owned? }
160
156
  end
161
157
  end
162
158
  end
metadata CHANGED
@@ -1,7 +1,7 @@
1
1
  --- !ruby/object:Gem::Specification
2
2
  name: requeue
3
3
  version: !ruby/object:Gem::Version
4
- version: 0.7.2
4
+ version: 0.7.3
5
5
  platform: ruby
6
6
  authors:
7
7
  - Eric Fode