time_window_drop_collector 0.3.1 → 0.3.2

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.
Files changed (5) hide show
  1. data/lib/time_window_drop_collector/storage.rb +2 -0
  2. data/lib/time_window_drop_collector/version.rb +1 -1
  3. data/test/memcache_wrapper_test.rb +3 -9
  4. data/test/storage_test.rb +27 -23
  5. metadata +64 -33
data/lib/time_window_drop_collector/storage.rb CHANGED
@@ -14,6 +14,8 @@ class TimeWindowDropCollector::Storage
14
14
  end
15
15
 
16
16
  def count( keys )
17
+ return {} if keys.empty?
18
+
17
19
  window_keys = window_keys_multi( keys )
18
20
  keys_values = wrapper.get( window_keys )
19
21
 
data/lib/time_window_drop_collector/version.rb CHANGED
@@ -1,3 +1,3 @@
1
1
  class TimeWindowDropCollector
2
- VERSION = "0.3.1"
2
+ VERSION = "0.3.2"
3
3
  end
data/test/memcache_wrapper_test.rb CHANGED
@@ -2,20 +2,14 @@ require_relative "test_helper"
2
2
 
3
3
  class MemcacheWrapperTest < Test::Unit::TestCase
4
4
  def test_initialize
5
- Dalli::Client.expects( :new ).with( "arg1" ).returns( "client" )
5
+ Dalli::Client.expects( :new ).with( ["arg1"] ).returns( "client" )
6
6
  wrapper = TimeWindowDropCollector::Wrappers::Memcache.new( ["arg1"] )
7
7
  assert_equal( "client", wrapper.client )
8
8
  end
9
9
 
10
10
  def test_incr
11
11
  wrapper = TimeWindowDropCollector::Wrappers::Memcache.new( ["arg1"] )
12
- wrapper.client.expects( :incr ).with( "key", 5, "expire_time", 1 )
13
- wrapper.incr( "key", "expire_time", 5 )
14
- end
15
-
16
- def test_values_for
17
- wrapper = TimeWindowDropCollector::Wrappers::Memcache.new( ["arg1"] )
18
- wrapper.client.expects( :get_multi ).with( "keys" ).returns( {:a => 1, :b => 2} )
19
- assert_equal( [1, 2], wrapper.values_for( "keys" ))
12
+ wrapper.client.expects( :incr ).with( "key", 5, "expire_time", 1)
13
+ wrapper.incr( ["key"], "expire_time", 5 )
20
14
  end
21
15
  end
data/test/storage_test.rb CHANGED
@@ -7,13 +7,13 @@ class StorageTest < Test::Unit::TestCase
7
7
  end
8
8
 
9
9
  def test_timestamp_key_when_time_is_present
10
- timestamp = Time.new( 2001, 2, 3, 4, 5 )
11
- assert_equal( "drop_window_key_981169500000", @storage.timestamp_key( "key", timestamp ) )
10
+ timestamp = Time.new(2001, 2, 3, 4, 5, 0, "+01:00")
11
+ assert_equal("drop_window_key_981169500000", @storage.timestamp_key("key", timestamp))
12
12
  end
13
13
 
14
14
  def test_timestamp_key_when_time_is_not_present
15
- @storage.stubs( :timestamp ).returns( Time.new( 2012, 4, 3, 2, 1 ))
16
- assert_equal( "drop_window_key_1333411260000", @storage.timestamp_key( "key" ))
15
+ @storage.stubs(:timestamp).returns(Time.new(2012, 4, 3, 2, 1, 0, "+02:00"))
16
+ assert_equal("drop_window_key_1333411260000", @storage.timestamp_key("key"))
17
17
  end
18
18
 
19
19
  def test_window_keys_should_return_10_keys_for_the_last_10_minutes
@@ -30,8 +30,8 @@ class StorageTest < Test::Unit::TestCase
30
30
  "drop_window_key_1325560260000"
31
31
  ]
32
32
 
33
- Delorean.time_travel_to( '2012-01-03 04:20' ) do
34
- assert_equal( keys, @storage.window_keys( "key" ))
33
+ Delorean.time_travel_to('2012-01-03 04:20 +01:00') do
34
+ assert_equal(keys, @storage.window_keys("key"))
35
35
  end
36
36
  end
37
37
 
@@ -57,6 +57,10 @@ class StorageTest < Test::Unit::TestCase
57
57
  assert_equal( "grouping_keys_counts", @storage.count( "keys" ))
58
58
  end
59
59
 
60
+ def test_count_with_no_keys
61
+ assert_equal({}, @storage.count([]))
62
+ end
63
+
60
64
  def test_grouping_count
61
65
  key_values = {
62
66
  "drop_window_key1_201201031416" => 1,
@@ -122,19 +126,19 @@ class StorageTest < Test::Unit::TestCase
122
126
  end
123
127
 
124
128
  def test_slice_start_timestamp
125
- storage = TimeWindowDropCollector::Storage.new( nil, 100, 10 )
126
- time = Time.new( 2001, 2, 3, 4, 5 )
127
-
128
- assert_equal( 981169500000, storage.slice_start_timestamp( time ) )
129
- assert_equal( 981169500000, storage.slice_start_timestamp( time + 1 ) )
130
- assert_equal( 981169500000, storage.slice_start_timestamp( time + 9 ) )
131
- assert_equal( 981169520000, storage.slice_start_timestamp( time + 25 ) )
132
- assert_equal( 981169600000, storage.slice_start_timestamp( time + 100 ) )
129
+ storage = TimeWindowDropCollector::Storage.new(nil, 100, 10)
130
+ time = Time.new(2001, 2, 3, 4, 5, 0, '+01:00')
131
+
132
+ assert_equal(981169500000, storage.slice_start_timestamp(time))
133
+ assert_equal(981169500000, storage.slice_start_timestamp(time + 1))
134
+ assert_equal(981169500000, storage.slice_start_timestamp(time + 9))
135
+ assert_equal(981169520000, storage.slice_start_timestamp(time + 25))
136
+ assert_equal(981169600000, storage.slice_start_timestamp(time + 100))
133
137
  end
134
138
 
135
139
  def test_slice_start_timestamp_with_slice_sizes_no_integer
136
140
  storage = TimeWindowDropCollector::Storage.new( nil, 100, 12 )
137
- time = Time.new( 2001, 2, 3, 4, 5 )
141
+ time = Time.new( 2001, 2, 3, 4, 5, 0,'+01:00')
138
142
 
139
143
  assert_equal( 981169493317, storage.slice_start_timestamp( time ) )
140
144
  assert_equal( 981169493317, storage.slice_start_timestamp( time + 1 ) )
@@ -144,14 +148,14 @@ class StorageTest < Test::Unit::TestCase
144
148
  end
145
149
 
146
150
  def test_slice_start_timestamp_with_slice_size_less_than_a_second
147
- storage = TimeWindowDropCollector::Storage.new( nil, 100, 101 )
148
- time = Time.new( 2001, 2, 3, 4, 5 )
149
-
150
- assert_equal( 981169499970, storage.slice_start_timestamp( time ) )
151
- assert_equal( 981169500960, storage.slice_start_timestamp( time + 1 ) )
152
- assert_equal( 981169508880, storage.slice_start_timestamp( time + 9 ) )
153
- assert_equal( 981169524720, storage.slice_start_timestamp( time + 25 ) )
154
- assert_equal( 981169599960, storage.slice_start_timestamp( time + 100 ) )
151
+ storage = TimeWindowDropCollector::Storage.new(nil, 100, 101)
152
+ time = Time.new( 2001, 2, 3, 4, 5, 0, '+01:00')
153
+
154
+ assert_equal(981169499970, storage.slice_start_timestamp(time))
155
+ assert_equal(981169500960, storage.slice_start_timestamp(time + 1))
156
+ assert_equal(981169508880, storage.slice_start_timestamp(time + 9))
157
+ assert_equal(981169524720, storage.slice_start_timestamp(time + 25))
158
+ assert_equal(981169599960, storage.slice_start_timestamp(time + 100))
155
159
  end
156
160
 
157
161
  def test_window_size_consistency
metadata CHANGED
@@ -1,7 +1,7 @@
1
1
  --- !ruby/object:Gem::Specification
2
2
  name: time_window_drop_collector
3
3
  version: !ruby/object:Gem::Version
4
- version: 0.3.1
4
+ version: 0.3.2
5
5
  prerelease:
6
6
  platform: ruby
7
7
  authors:
@@ -10,74 +10,104 @@ authors:
10
10
  autorequire:
11
11
  bindir: bin
12
12
  cert_chain: []
13
- date: 2012-12-19 00:00:00.000000000Z
13
+ date: 2013-10-24 00:00:00.000000000 Z
14
14
  dependencies:
15
15
  - !ruby/object:Gem::Dependency
16
16
  name: bundler
17
- requirement: &70353016535360 !ruby/object:Gem::Requirement
18
- none: false
17
+ prerelease: false
18
+ requirement: !ruby/object:Gem::Requirement
19
19
  requirements:
20
20
  - - ! '>='
21
21
  - !ruby/object:Gem::Version
22
22
  version: '0'
23
+ none: false
23
24
  type: :development
24
- prerelease: false
25
- version_requirements: *70353016535360
25
+ version_requirements: !ruby/object:Gem::Requirement
26
+ requirements:
27
+ - - ! '>='
28
+ - !ruby/object:Gem::Version
29
+ version: '0'
30
+ none: false
26
31
  - !ruby/object:Gem::Dependency
27
32
  name: rake
28
- requirement: &70353016534860 !ruby/object:Gem::Requirement
29
- none: false
33
+ prerelease: false
34
+ requirement: !ruby/object:Gem::Requirement
30
35
  requirements:
31
- - - =
36
+ - - '='
32
37
  - !ruby/object:Gem::Version
33
38
  version: 0.9.2.2
39
+ none: false
34
40
  type: :development
35
- prerelease: false
36
- version_requirements: *70353016534860
41
+ version_requirements: !ruby/object:Gem::Requirement
42
+ requirements:
43
+ - - '='
44
+ - !ruby/object:Gem::Version
45
+ version: 0.9.2.2
46
+ none: false
37
47
  - !ruby/object:Gem::Dependency
38
48
  name: mocha
39
- requirement: &70353016534360 !ruby/object:Gem::Requirement
40
- none: false
49
+ prerelease: false
50
+ requirement: !ruby/object:Gem::Requirement
41
51
  requirements:
42
- - - =
52
+ - - '='
43
53
  - !ruby/object:Gem::Version
44
54
  version: 0.10.0
55
+ none: false
45
56
  type: :development
46
- prerelease: false
47
- version_requirements: *70353016534360
57
+ version_requirements: !ruby/object:Gem::Requirement
58
+ requirements:
59
+ - - '='
60
+ - !ruby/object:Gem::Version
61
+ version: 0.10.0
62
+ none: false
48
63
  - !ruby/object:Gem::Dependency
49
64
  name: delorean
50
- requirement: &70353016533900 !ruby/object:Gem::Requirement
51
- none: false
65
+ prerelease: false
66
+ requirement: !ruby/object:Gem::Requirement
52
67
  requirements:
53
- - - =
68
+ - - '='
54
69
  - !ruby/object:Gem::Version
55
70
  version: 1.2.0
71
+ none: false
56
72
  type: :development
57
- prerelease: false
58
- version_requirements: *70353016533900
73
+ version_requirements: !ruby/object:Gem::Requirement
74
+ requirements:
75
+ - - '='
76
+ - !ruby/object:Gem::Version
77
+ version: 1.2.0
78
+ none: false
59
79
  - !ruby/object:Gem::Dependency
60
80
  name: memcache_mock
61
- requirement: &70353016533440 !ruby/object:Gem::Requirement
62
- none: false
81
+ prerelease: false
82
+ requirement: !ruby/object:Gem::Requirement
63
83
  requirements:
64
- - - =
84
+ - - '='
65
85
  - !ruby/object:Gem::Version
66
86
  version: 0.0.1
87
+ none: false
67
88
  type: :development
68
- prerelease: false
69
- version_requirements: *70353016533440
89
+ version_requirements: !ruby/object:Gem::Requirement
90
+ requirements:
91
+ - - '='
92
+ - !ruby/object:Gem::Version
93
+ version: 0.0.1
94
+ none: false
70
95
  - !ruby/object:Gem::Dependency
71
96
  name: dalli
72
- requirement: &70353016533060 !ruby/object:Gem::Requirement
73
- none: false
97
+ prerelease: false
98
+ requirement: !ruby/object:Gem::Requirement
74
99
  requirements:
75
100
  - - ! '>='
76
101
  - !ruby/object:Gem::Version
77
102
  version: '0'
103
+ none: false
78
104
  type: :runtime
79
- prerelease: false
80
- version_requirements: *70353016533060
105
+ version_requirements: !ruby/object:Gem::Requirement
106
+ requirements:
107
+ - - ! '>='
108
+ - !ruby/object:Gem::Version
109
+ version: '0'
110
+ none: false
81
111
  description: Counter storage system for a concrete time window
82
112
  email:
83
113
  - fguillen.mail@gmail.com
@@ -122,20 +152,20 @@ rdoc_options: []
122
152
  require_paths:
123
153
  - lib
124
154
  required_ruby_version: !ruby/object:Gem::Requirement
125
- none: false
126
155
  requirements:
127
156
  - - ! '>='
128
157
  - !ruby/object:Gem::Version
129
158
  version: '0'
130
- required_rubygems_version: !ruby/object:Gem::Requirement
131
159
  none: false
160
+ required_rubygems_version: !ruby/object:Gem::Requirement
132
161
  requirements:
133
162
  - - ! '>='
134
163
  - !ruby/object:Gem::Version
135
164
  version: '0'
165
+ none: false
136
166
  requirements: []
137
167
  rubyforge_project: time_window_drop_collector
138
- rubygems_version: 1.8.15
168
+ rubygems_version: 1.8.24
139
169
  signing_key:
140
170
  specification_version: 3
141
171
  summary: Counter storage system for a concrete time window
@@ -150,3 +180,4 @@ test_files:
150
180
  - test/test_helper.rb
151
181
  - test/time_window_drop_collector_test.rb
152
182
  - test/wrapper_test.rb
183
+ has_rdoc: