growviral-warehouse 0.0.6 → 0.0.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 680d544858f0dab64b7107b52cbc5a1d1a037b10
-  data.tar.gz: 40ac6f5371f5e6e49ec081e037b7c5e8e4f52022
+  metadata.gz: f32e8c15ebcf39957b9170d1a1a5180d16fc0e6a
+  data.tar.gz: fe4889949aa7ab3e04cbf659208ab6e1582ca128
 SHA512:
-  metadata.gz: 5547afffa84b0ba14fe3f2dc42da70a2ecd9b869caffe5fbf5da7361e3d7aedd39e39962b5bfffe0351088ba2f5d94505002a9d6af2ca1b3daf20aaa964ce1b9
-  data.tar.gz: 169673ae09a1044a932304b3c99dca2fb408ebdc4d948ec276258d7bba902d824b9c2da9fcca93018b6190458f68bd61a5747401de241dd7f263423ee465ec86
+  metadata.gz: 58174c64a1aca594a547ea78410d3b7d33e6db18db29090cecce0ec2780847f22e211190b7b45525636285ef00e3e8f752e6e19f6bbb0567e1e6265286ccc5fc
+  data.tar.gz: e1cd8eea88573982edc157a525df35394884f3b3373e9f5b33920fccd88a178895782271a26709e3d3dad5cf1e9a37788b7eceed12f3bd58976485126b004016

data/lib/growviral/warehouse.rb

@@ -4,7 +4,8 @@ require 'json'
 
 require 'warehouse/client'
 require 'warehouse/configuration'
-require 'warehouse/registers_followers'
+require 'warehouse/records_follow'
+require 'warehouse/records_unfollow'
 require 'warehouse/fetches_follow_history'
 require 'warehouse/checks_follow_history'
 require 'warehouse/fetches_users'

data/lib/warehouse/client.rb

@@ -8,7 +8,11 @@ module GrowViral
       end
 
       def register_follow!(uid, following_uid)
-        RegistersFollows.register(uid, following_uid, config: config)
+        RecordsFollow.register(uid, following_uid, config: config)
+      end
+
+      def register_unfollow!(uid, following_uid)
+        RecordsUnfollow.register(uid, following_uid, config: config)
       end
 
       def follow_history(uid)

data/lib/warehouse/fetches_users.rb

@@ -9,10 +9,12 @@ module GrowViral
         extend Forwardable
         def_delegators :@data, :each, :size
 
-        attr_reader :missing_ids
-        def initialize(data, missing_ids)
+        attr_reader :missing_ids, :invalid_ids, :protected_ids
+        def initialize(data, options = {})
           @data = data
-          @missing_ids = missing_ids
+          @missing_ids = options[:missing_ids] || []
+          @invalid_ids = options[:invalid_ids] || []
+          @protected_ids = options[:protected_ids] || []
         end
       end
 
@@ -34,7 +36,14 @@ module GrowViral
       def fetch
         response = Net::HTTP.get_response(uri)
         data = JSON.parse(response.body)
-        Users.new(data['data'], data['missing_ids'])
+        Users.new(
+          data['twitter_users'], 
+          {
+            missing_ids: data['missing_ids'],
+            invalid_ids: data['invalid_ids'],
+            protected_ids: data['protected_ids']
+          }
+        )
       end
 
       def uri

data/lib/warehouse/{registers_followers.rb → records_follow.rb}

@@ -1,6 +1,6 @@
 module GrowViral
   module Warehouse
-    class RegistersFollows
+    class RecordsFollow
       def self.register(*args)
         new(*args).register
       end
@@ -21,13 +21,18 @@ module GrowViral
       end
 
       def uri
-        @uri ||= URI.parse("#{config.host}/follows")
+        @uri ||= URI.parse("#{config.host}/interactions/mark_followed")
+      end
+
+      def timestamp
+        @timestamp ||= Time.now.to_i
       end
 
       def form_data
         {
           "uid" => uid,
-          "following_uid" => following_uid
+          "follow_uid" => following_uid,
+          "timestamp" => timestamp
         }
       end
     end

data/lib/warehouse/records_unfollow.rb

@@ -0,0 +1,40 @@
+module GrowViral
+  module Warehouse
+    class RecordsUnfollow
+      def self.register(*args)
+        new(*args).register
+      end
+
+      attr_reader :uid, :following_uid, :config
+      def initialize(uid, following_uid, deps)
+        raise HandleNotUidError unless uid.is_a? Numeric
+        raise HandleNotUidError unless following_uid.is_a? Numeric
+
+        @uid = uid
+        @following_uid = following_uid
+        @config = deps[:config]
+      end
+
+      def register
+        response = Net::HTTP.post_form(uri, form_data)
+        JSON.parse(response.body)
+      end
+
+      def uri
+        @uri ||= URI.parse("#{config.host}/interactions/mark_unfollowed")
+      end
+
+      def timestamp
+        @timestamp ||= Time.now.to_i
+      end
+
+      def form_data
+        {
+          "uid" => uid,
+          "follow_uid" => following_uid,
+          "timestamp" => timestamp
+        }
+      end
+    end
+  end
+end

data/lib/warehouse/version.rb

@@ -1,5 +1,5 @@
 module GrowViral
   module Warehouse
-    VERSION = "0.0.6"
+    VERSION = "0.0.7"
   end
 end

data/test/fetches_users_test.rb

@@ -1,22 +1,41 @@
 require_relative 'test_helper'
 
 module GrowViral::Warehouse
-  class FetchesFollowHistoryTest < Minitest::Test
+  class FetchesUsersTest < Minitest::Test
     def test_users_is_enumerable
       data = mock
       data.expects(:each)
       data.expects(:size)
-      users = FetchesUsers::Users.new(data, nil)
+      users = FetchesUsers::Users.new(data)
       users.each {}
       users.size
     end
 
     def test_users_returns_missing_ids
       missing_ids = mock
-      users = FetchesUsers::Users.new(nil, missing_ids)
+      users = FetchesUsers::Users.new(nil, missing_ids: missing_ids)
       assert_equal missing_ids, users.missing_ids
     end
 
+    def test_users_returns_invalid_ids
+      invalid_ids = mock
+      users = FetchesUsers::Users.new(nil, invalid_ids: invalid_ids)
+      assert_equal invalid_ids, users.invalid_ids
+    end
+
+    def test_users_returns_protected_ids
+      protected_ids = mock
+      users = FetchesUsers::Users.new(nil, protected_ids: protected_ids)
+      assert_equal protected_ids, users.protected_ids
+    end
+
+    def test_users_returns_arrays_as_defaults
+      users = FetchesUsers::Users.new(nil)
+      assert_equal [], users.missing_ids
+      assert_equal [], users.invalid_ids
+      assert_equal [], users.protected_ids
+    end
+
     def test_raises_for_a_handle
       client = GrowViral::Warehouse::Client.new(:development)
       assert_raises(GrowViral::HandleNotUidError) { client.users(["string"]) }
@@ -24,9 +43,11 @@ module GrowViral::Warehouse
 
     def test_returns_output_correctly
       uid1 = 328383
-      uid2 = 328383
+      uid2 = 39994
+      uid3 = 4757392
+      uid4 = 99122
 
-      output = { 'data' => [{'id' => uid1}], missing_ids: [uid2] }
+      output = { 'twitter_users' => [{'id' => uid1}], 'missing_ids' => [uid2], 'invalid_ids' => [uid3], 'protected_ids' => [uid4] }
       uri = URI.parse("http://localhost:3001/tweeps/#{uid1},#{uid2}")
       Net::HTTP.expects(:get_response).with(uri).returns(mock(body: output.to_json))
 
@@ -34,13 +55,15 @@ module GrowViral::Warehouse
       users = client.users([uid1, uid2])
       assert_equal [{'id' => uid1}], users.each {|u|u}
       assert_equal [uid2], users.missing_ids
+      assert_equal [uid3], users.invalid_ids
+      assert_equal [uid4], users.protected_ids
     end
 
     def test_posts_account_id_if_given
       uid = 328383
       account_uid = 848493
 
-      output = { 'data' => [], missing_ids: [] }
+      output = { }
       uri = URI.parse("http://localhost:3001/tweeps/#{uid}?for_account=#{account_uid}")
       Net::HTTP.expects(:get_response).with(uri).returns(mock(body: output.to_json))
 

data/test/preloads_users_test.rb

@@ -1,7 +1,7 @@
 require_relative 'test_helper'
 
 module GrowViral::Warehouse
-  class FetchesFollowHistoryTest < Minitest::Test
+  class PreloadsUsersTest < Minitest::Test
 
     def test_returns_output_correctly
       uid1 = 328383

data/test/{registers_followers_test.rb → records_follow_test.rb}

@@ -1,6 +1,6 @@
 require_relative 'test_helper'
 
-class RegistersFollowersTest < Minitest::Test
+class RecordsFollowWTest < Minitest::Test
   def test_raises_for_a_handle
     client = GrowViral::Warehouse::Client.new(:development)
     assert_raises(GrowViral::HandleNotUidError) { client.register_follow!("string", 2312422) }
@@ -14,18 +14,18 @@ class RegistersFollowersTest < Minitest::Test
   def test_response_is_correct
     uid = 23432423
     following_uid = 342423
+    now = 42
 
     input = {
       "uid" => uid,
-      "following_uid" => following_uid,
+      "follow_uid" => following_uid,
+      "timestamp" => now
     }
 
-    output = {
-      "uid" => uid,
-      "following_uid" => following_uid,
-    }
+    output = { "status" => "accepted" }
 
-    uri = URI.parse("http://localhost:3001/follows")
+    uri = URI.parse("http://localhost:3001/interactions/mark_followed")
+    Time.expects(:now).returns(mock(to_i: 42))
     Net::HTTP.expects(:post_form).with(uri, input).returns(mock(body: output.to_json))
 
     client = GrowViral::Warehouse::Client.new(:development)
@@ -34,5 +34,3 @@ class RegistersFollowersTest < Minitest::Test
     assert_equal output, actual
   end
 end
-
-

data/test/records_unfollow_test.rb

@@ -0,0 +1,36 @@
+require_relative 'test_helper'
+
+class RecordsUnFollowWTest < Minitest::Test
+  def test_raises_for_a_handle
+    client = GrowViral::Warehouse::Client.new(:development)
+    assert_raises(GrowViral::HandleNotUidError) { client.register_unfollow!("string", 2312422) }
+  end
+
+  def test_raises_for_a_following_handle
+    client = GrowViral::Warehouse::Client.new(:development)
+    assert_raises(GrowViral::HandleNotUidError) { client.register_unfollow!(972822, "string") }
+  end
+
+  def test_response_is_correct
+    uid = 23432423
+    following_uid = 342423
+    now = 42
+
+    input = {
+      "uid" => uid,
+      "follow_uid" => following_uid,
+      "timestamp" => now
+    }
+
+    output = { "status" => "accepted" }
+
+    uri = URI.parse("http://localhost:3001/interactions/mark_unfollowed")
+    Time.expects(:now).returns(mock(to_i: 42))
+    Net::HTTP.expects(:post_form).with(uri, input).returns(mock(body: output.to_json))
+
+    client = GrowViral::Warehouse::Client.new(:development)
+    actual = client.register_unfollow!(uid, following_uid)
+
+    assert_equal output, actual
+  end
+end

metadata

@@ -1,69 +1,69 @@
 --- !ruby/object:Gem::Specification
 name: growviral-warehouse
 version: !ruby/object:Gem::Version
-  version: 0.0.6
+  version: 0.0.7
 platform: ruby
 authors:
 - Jeremy Walker
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-01-08 00:00:00.000000000 Z
+date: 2015-01-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '1.7'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '1.7'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '10.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '10.0'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: mocha
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 description: ''
@@ -73,8 +73,8 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".gitignore"
-- ".travis.yml"
+- .gitignore
+- .travis.yml
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -87,13 +87,15 @@ files:
 - lib/warehouse/fetches_follow_history.rb
 - lib/warehouse/fetches_users.rb
 - lib/warehouse/preloads_users.rb
-- lib/warehouse/registers_followers.rb
+- lib/warehouse/records_follow.rb
+- lib/warehouse/records_unfollow.rb
 - lib/warehouse/version.rb
 - test/checks_follow_history_test.rb
 - test/fetches_follow_history_test.rb
 - test/fetches_users_test.rb
 - test/preloads_users_test.rb
-- test/registers_followers_test.rb
+- test/records_follow_test.rb
+- test/records_unfollow_test.rb
 - test/test_helper.rb
 homepage: ''
 licenses:
@@ -105,17 +107,17 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.1.9
 signing_key: 
 specification_version: 4
 summary: A wrapper for the GrowViral warehouse.
@@ -124,6 +126,6 @@ test_files:
 - test/fetches_follow_history_test.rb
 - test/fetches_users_test.rb
 - test/preloads_users_test.rb
-- test/registers_followers_test.rb
+- test/records_follow_test.rb
+- test/records_unfollow_test.rb
 - test/test_helper.rb
-has_rdoc: