heap 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b7f4270c4f06512b348491c08a91e7e44c293337
-  data.tar.gz: d7f95a6e3e4dbb399ee899dca956690eec30aef6
+  metadata.gz: d13ef266e73536e3f334d975408393051f3240f4
+  data.tar.gz: fd288e976a5ac65d67495377491c2dfc493c658d
 SHA512:
-  metadata.gz: d873b49995396816be5c824b3dc47848a69ee614fd3228b9c3e62208e24a65e884303bd31c72e648597207289ef8b5e614e98bb0f9ca6f7591421c18ef3e0584
-  data.tar.gz: 8b27c441c8c46d0e2c15260006818dab9eed08f83f79567fd4564232e2fb73c33ff23aa918f575d11080f6fa71d7bcc194349b6b1938d49598ff27cdaa555911
+  metadata.gz: a9b4129ae78fcd15b1a75bd045464671b79ebb9856924f088ff2926e1f69eb228bff0d5e9c529fb782aac39fb55c1dac34b3c23ce27a381222894d8396944b1e
+  data.tar.gz: 2eeee128759cd9438df09755827576264e7766e4bbd8308394aee20280b45bc886bec4b2861411bb79c908aed4206249e32953fb7e17366c9ede2d2f25505d7f

data/.travis.yml

@@ -14,3 +14,4 @@ matrix:
       gemfile: Gemfile
     - rvm: 2.3.0
       gemfile: Gemfile
+      env: COVERAGE=1

data/Gemfile

@@ -7,9 +7,11 @@ gem 'faraday_middleware', '>= 0.8.0'
 # Add dependencies to develop your gem here.
 # Include everything needed to run rake, tests, features, etc.
 group :development do
-  gem 'minitest', '>= 0'
-  gem 'yard', '>= 0.8.7.6'
-  gem 'rdoc', '>= 3.12'
   gem 'bundler', '>= 1.0'
+  gem 'coveralls', '>= 0.8.10'
   gem 'jeweler', '>= 2.0.1'
+  gem 'minitest', '>= 5.8.4'
+  gem 'rdoc', '>= 3.12'
+  gem 'simplecov', '>= 0.11.2'
+  gem 'yard', '>= 0.8.7.6'
 end

data/Gemfile187

@@ -7,10 +7,10 @@ gem 'faraday_middleware', '~> 0.8.0'
 # Add dependencies to develop your gem here.
 # Include everything needed to run rake, tests, features, etc.
 group :development do
-  gem 'minitest', '>= 0'
-  gem 'yard', '>= 0.8.7.6'
-  gem 'rdoc', '>= 3.12'
   gem 'bundler', '>= 1.0'
+  gem 'minitest', '>= 5.8.4'
+  gem 'rdoc', '>= 4.2.2'
+  gem 'yard', '>= 0.8.7.6'
 
   # NOTE: The latest versions of these dependencies do not support Ruby 1.8.7.
   gem 'jeweler', '~> 1.8.8'

data/README.md

@@ -1,10 +1,13 @@
 # Heap Server-Side API Client for Ruby
 
 [![Build Status](https://travis-ci.org/heap/heap-ruby.svg?branch=master)](https://travis-ci.org/heap/heap-ruby)
+[![Coverage Status](https://coveralls.io/repos/github/heap/heap-ruby/badge.svg?branch=master)](https://coveralls.io/github/heap/heap-ruby?branch=master)
+[![Dependency Status](https://gemnasium.com/heap/heap-ruby.svg)](https://gemnasium.com/heap/heap-ruby)
 [![Yard Docs](http://img.shields.io/badge/yard-docs-blue.svg)](http://rubydoc.info/github/heap/heap-ruby/master/frames)
 [![Gem Version](https://badge.fury.io/rb/heap.svg)](https://badge.fury.io/rb/heap)
 
-This is a Ruby client for the [Heap](https://heapanalytics.com/)
+This is a [Ruby](https://www.ruby-lang.org/) client for the
+[Heap](https://heapanalytics.com/)
 [server-side API](https://heapanalytics.com/docs/server-side).
 
 
@@ -79,14 +82,14 @@ end
 The properties are optional.
 
 ```ruby
-Heap.track 'user-handle', 'event-name', property: 'value'
+Heap.track 'event-name', 'user-identity', property: 'value'
 ```
 
 [Add properties](https://heapanalytics.com/docs/server-side#identify) to a
 user.
 
 ```ruby
-Heap.add_user_properties 'user-handle', plan: 'premium1'
+Heap.add_user_properties 'user-identity', plan: 'premium1'
 ```
 
 If the global API client instance stored in `Heap` is not a good fit for your
@@ -94,7 +97,7 @@ application's architecture, create your own client instances.
 
 ```ruby
 heap_client = Heap.new app_id: 'YOUR_APP_ID'
-heap_client.track 'user-handle', 'event-name', property: 'value'
+heap_client.track 'user-identity', 'event-name', property: 'value'
 ```
 
 

data/Rakefile

@@ -32,6 +32,12 @@ Rake::TestTask.new(:test) do |test|
   test.verbose = true
 end
 
+desc 'Code coverage detail'
+task :simplecov do
+  ENV['COVERAGE'] = 'true'
+  Rake::Task['test'].execute
+end
+
 task :default => :test
 
 require 'yard'

data/VERSION

@@ -1 +1 @@
-1.0.0
+1.0.1

data/heap.gemspec

@@ -2,16 +2,16 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: heap 1.0.0 ruby lib
+# stub: heap 1.0.1 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "heap"
-  s.version = "1.0.0"
+  s.version = "1.0.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib"]
   s.authors = ["Victor Costan"]
-  s.date = "2016-02-29"
+  s.date = "2016-03-08"
   s.description = "Implements Heap's server-side API"
   s.email = "victor@heapanalytics.com"
   s.extra_rdoc_files = [
@@ -49,28 +49,34 @@ Gem::Specification.new do |s|
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
       s.add_runtime_dependency(%q<faraday>, [">= 0.8.11"])
       s.add_runtime_dependency(%q<faraday_middleware>, [">= 0.8.0"])
-      s.add_development_dependency(%q<minitest>, [">= 0"])
-      s.add_development_dependency(%q<yard>, [">= 0.8.7.6"])
-      s.add_development_dependency(%q<rdoc>, [">= 3.12"])
       s.add_development_dependency(%q<bundler>, [">= 1.0"])
+      s.add_development_dependency(%q<coveralls>, [">= 0.8.10"])
       s.add_development_dependency(%q<jeweler>, [">= 2.0.1"])
+      s.add_development_dependency(%q<minitest>, [">= 5.8.4"])
+      s.add_development_dependency(%q<rdoc>, [">= 3.12"])
+      s.add_development_dependency(%q<simplecov>, [">= 0.11.2"])
+      s.add_development_dependency(%q<yard>, [">= 0.8.7.6"])
     else
       s.add_dependency(%q<faraday>, [">= 0.8.11"])
       s.add_dependency(%q<faraday_middleware>, [">= 0.8.0"])
-      s.add_dependency(%q<minitest>, [">= 0"])
-      s.add_dependency(%q<yard>, [">= 0.8.7.6"])
-      s.add_dependency(%q<rdoc>, [">= 3.12"])
       s.add_dependency(%q<bundler>, [">= 1.0"])
+      s.add_dependency(%q<coveralls>, [">= 0.8.10"])
       s.add_dependency(%q<jeweler>, [">= 2.0.1"])
+      s.add_dependency(%q<minitest>, [">= 5.8.4"])
+      s.add_dependency(%q<rdoc>, [">= 3.12"])
+      s.add_dependency(%q<simplecov>, [">= 0.11.2"])
+      s.add_dependency(%q<yard>, [">= 0.8.7.6"])
     end
   else
     s.add_dependency(%q<faraday>, [">= 0.8.11"])
     s.add_dependency(%q<faraday_middleware>, [">= 0.8.0"])
-    s.add_dependency(%q<minitest>, [">= 0"])
-    s.add_dependency(%q<yard>, [">= 0.8.7.6"])
-    s.add_dependency(%q<rdoc>, [">= 3.12"])
     s.add_dependency(%q<bundler>, [">= 1.0"])
+    s.add_dependency(%q<coveralls>, [">= 0.8.10"])
     s.add_dependency(%q<jeweler>, [">= 2.0.1"])
+    s.add_dependency(%q<minitest>, [">= 5.8.4"])
+    s.add_dependency(%q<rdoc>, [">= 3.12"])
+    s.add_dependency(%q<simplecov>, [">= 0.11.2"])
+    s.add_dependency(%q<yard>, [">= 0.8.7.6"])
   end
 end
 

data/lib/heap/client.rb

@@ -29,6 +29,7 @@ class HeapAPI::Client
   #
   # @param [Hash<Symbol, Object>] options initial values for attributes
   # @option options [String] app_id the Heap application ID from
+  #   https://heapanalytics.com/app/install
   # @option options [Hash<Symbol, Object>] js_options default heap.js advanced
   #   options
   # @option options [Boolean] stubbed if true, all the requests to the Heap API
@@ -66,12 +67,12 @@ class HeapAPI::Client
   def faraday_adapter_args=(new_args)
     raise RuntimeError, 'Faraday connection already initialized' if @connection
     unless new_args.instance_of? Array
-      raise ArgumentErrorm, "Arguments must be an array"
+      raise ArgumentError, "Arguments must be an Array"
     end
     @faraday_adapter_args = new_args
   end
 
-  # Assign custom properties to an existing user.
+  # Assigns custom properties to an existing user.
   #
   # @param [String] identity an e-mail, handle, or Heap-generated user ID
   # @param [Hash<String, String|Number>] properties key-value properties
@@ -81,6 +82,7 @@ class HeapAPI::Client
   # @see https://heapanalytics.com/docs/server-side#identify
   def add_user_properties(identity, properties)
     ensure_valid_app_id!
+    ensure_valid_identity! identity
     ensure_valid_properties! properties
 
     body = {
@@ -109,6 +111,7 @@ class HeapAPI::Client
 
     event_name = event.to_s
     ensure_valid_event_name! event_name
+    ensure_valid_identity! identity
 
     body = {
       :app_id => @app_id,

data/lib/heap/validations.rb

@@ -23,6 +23,26 @@ class HeapAPI::Client
   end
   private :ensure_valid_event_name!
 
+  # Validates identity, making sure it is a valid string or integer.
+  #
+  # @param [String|Integer] identity
+  # @raise ArgumentError if identity is of an invalid type or too long.
+  # @return [HeapAPI::Client] self
+  def ensure_valid_identity!(identity)
+    identity = identity.to_s if identity.kind_of?(Integer)
+
+    if identity.kind_of?(String) || identity.kind_of?(Symbol)
+      if identity.to_s.length > 255
+        raise ArgumentError, "Identity field too long; " +
+            "#{identity.to_s.length} is above the 255-character limit"
+      end
+    else
+      raise ArgumentError,
+        "Unsupported type for identity value #{identity.inspect}"
+    end
+  end
+  private :ensure_valid_identity!
+
   # Validates a bag of properties sent to a Heap server-side API.
   #
   # @param [Hash<String, String|Number>] properties key-value property bag;

data/test/client_add_user_properties_test.rb

@@ -16,17 +16,57 @@ class ClientAddUserPropertiesTest < MiniTest::Test
   def test_add_user_properties_without_app_id
     @heap.app_id = nil
     exception = assert_raises RuntimeError do
-      @heap.add_user_properties 'test-handle', 'key' => 'value'
+      @heap.add_user_properties 'test-identity', 'key' => 'value'
     end
     assert_equal RuntimeError, exception.class
     assert_equal 'Heap app_id not set', exception.message
   end
 
+  def test_add_user_properties_with_invalid_property_object
+    exception = assert_raises ArgumentError do
+      @heap.add_user_properties 'test-identity', false
+    end
+    assert_equal ArgumentError, exception.class
+    assert_equal 'Properties object does not implement #each',
+        exception.message
+  end
+
+  def test_add_user_properties_with_long_string_identity
+    long_identity = 'A' * 256
+
+    exception = assert_raises ArgumentError do
+      @heap.add_user_properties long_identity, 'key' => 'value'
+    end
+    assert_equal ArgumentError, exception.class
+    assert_equal "Identity field too long; " +
+        '256 is above the 255-character limit', exception.message
+  end
+
+  def test_add_user_properties_with_long_symbol_identity
+    long_identity = ('A' * 256).to_sym
+
+    exception = assert_raises ArgumentError do
+      @heap.add_user_properties long_identity, 'key' => 'value'
+    end
+    assert_equal ArgumentError, exception.class
+    assert_equal "Identity field too long; " +
+        '256 is above the 255-character limit', exception.message
+  end
+
+  def test_add_user_properties_with_array_identity
+    exception = assert_raises ArgumentError do
+      @heap.add_user_properties([], 'key' => 'value')
+    end
+    assert_equal ArgumentError, exception.class
+    assert_equal 'Unsupported type for identity value []', exception.message
+  end
+
+
   def test_add_user_properties_with_long_property_name
     long_name = 'A' * 1025
 
     exception = assert_raises ArgumentError do
-      @heap.add_user_properties 'test-handle', long_name => 'value'
+      @heap.add_user_properties 'test-identity', long_name => 'value'
     end
     assert_equal ArgumentError, exception.class
     assert_equal "Property name #{long_name} too long; " +
@@ -36,7 +76,8 @@ class ClientAddUserPropertiesTest < MiniTest::Test
   def test_add_user_properties_with_long_string_property_value
     long_value = 'A' * 1025
     exception = assert_raises ArgumentError do
-      @heap.add_user_properties 'test-handle', 'long_value_name' => long_value
+      @heap.add_user_properties 'test-identity',
+          'long_value_name' => long_value
     end
     assert_equal ArgumentError, exception.class
     assert_equal "Property long_value_name value \"#{long_value}\" too " +
@@ -46,7 +87,8 @@ class ClientAddUserPropertiesTest < MiniTest::Test
   def test_add_user_properties_with_long_symbol_property_value
     long_value = ('A' * 1025).to_sym
     exception = assert_raises ArgumentError do
-      @heap.add_user_properties 'test-handle', 'long_value_name' => long_value
+      @heap.add_user_properties 'test-identity',
+          'long_value_name' => long_value
     end
     assert_equal ArgumentError, exception.class
     assert_equal "Property long_value_name value :#{long_value} too long; " +
@@ -55,19 +97,18 @@ class ClientAddUserPropertiesTest < MiniTest::Test
 
   def test_add_user_properties_with_array_property_value
     exception = assert_raises ArgumentError do
-      @heap.add_user_properties 'test-handle', 'array_value_name' => []
+      @heap.add_user_properties 'test-identity', 'array_value_name' => []
     end
     assert_equal ArgumentError, exception.class
     assert_equal 'Unsupported type for property array_value_name value []',
         exception.message
   end
 
-
   def test_add_user_properties
     @stubs.post '/api/identify' do |env|
       golden_body = {
         'app_id' => 'test-app-id',
-        'identity' => 'test-handle',
+        'identity' => 'test-identity',
         'properties' => { 'foo' => 'bar', 'heap' => 'hurray' }
       }
       assert_equal 'application/json', env[:request_headers]['Content-Type']
@@ -77,17 +118,34 @@ class ClientAddUserPropertiesTest < MiniTest::Test
       [200, { 'Content-Type' => 'text/plain; encoding=utf8' }, '']
     end
 
-    assert_equal @heap, @heap.add_user_properties('test-handle',
+    assert_equal @heap, @heap.add_user_properties('test-identity',
         'foo' => 'bar', :heap => :hurray)
   end
 
+  def test_add_user_properties_with_integer_identity
+    @stubs.post '/api/identify' do |env|
+      golden_body = {
+        'app_id' => 'test-app-id',
+        'identity' => '123456789',
+        'properties' => { 'foo' => 'bar' }
+      }
+      assert_equal 'application/json', env[:request_headers]['Content-Type']
+      assert_equal @heap.user_agent, env[:request_headers]['User-Agent']
+      assert_equal golden_body, JSON.parse(env[:body])
+
+      [200, { 'Content-Type' => 'text/plain; encoding=utf8' }, '']
+    end
+
+    assert_equal @heap, @heap.add_user_properties(123456789, 'foo' => 'bar')
+  end
+
   def test_add_user_properties_error
     @stubs.post '/api/identify' do |env|
       [400, { 'Content-Type' => 'text/plain; encoding=utf8' }, 'Bad request']
     end
 
     exception = assert_raises HeapAPI::Error do
-      @heap.add_user_properties 'test-handle', 'foo' => 'bar'
+      @heap.add_user_properties 'test-identity', 'foo' => 'bar'
     end
 
     assert_equal HeapAPI::ApiError, exception.class
@@ -101,7 +159,7 @@ class ClientAddUserPropertiesTest < MiniTest::Test
     @heap.faraday_adapter = :net_http
     @heap.faraday_adapter_args = []
 
-    assert_equal @heap, @heap.add_user_properties('test-handle',
+    assert_equal @heap, @heap.add_user_properties('test-identity',
         'language/ruby' => 1, 'heap/heap-ruby' => 1)
   end
 
@@ -110,7 +168,7 @@ class ClientAddUserPropertiesTest < MiniTest::Test
     @heap.faraday_adapter_args = []
 
     assert_raises HeapAPI::Error do
-      @heap.add_user_properties 'test-handle',
+      @heap.add_user_properties 'test-identity',
           'language/ruby' => 1, 'heap/heap-ruby' => 1
     end
   end
@@ -118,7 +176,7 @@ class ClientAddUserPropertiesTest < MiniTest::Test
   def test_add_user_properties_with_stubbed_connection
     @heap.stubbed = true
 
-    assert_equal @heap, @heap.add_user_properties('test-handle',
+    assert_equal @heap, @heap.add_user_properties('test-identity',
         'foo' => 'bar', :heap => :hurray)
   end
 end

data/test/client_test.rb

@@ -36,18 +36,26 @@ class ClientTest < MiniTest::Test
     assert_equal [stubs], @heap.faraday_adapter_args
   end
 
+  def test_faraday_adapter_args_setter_with_invalid_value
+    exception = assert_raises ArgumentError do
+      @heap.faraday_adapter_args = false
+    end
+    assert_equal ArgumentError, exception.class
+    assert_equal 'Arguments must be an Array', exception.message
+  end
+
   def test_stubbed_setter
     @heap.app_id = 'test-app-id'
     @heap.faraday_adapter = :net_http
 
     @heap.stubbed = true
-    assert_equal @heap, @heap.track('test_stubbed_setter', 'test-handle',
+    assert_equal @heap, @heap.track('test_stubbed_setter', 'test-identity',
         'language' => 'ruby', 'project' => 'heap/heap-ruby')
     stubbed_connection = @heap.connection
 
     @heap.stubbed = false
     assert_raises HeapAPI::Error do
-      @heap.track 'test_stubbed_setter', 'test-handle',
+      @heap.track 'test_stubbed_setter', 'test-identity',
           'language' => 'ruby', 'project' => 'heap/heap-ruby'
     end
     live_connection = @heap.connection

data/test/client_track_test.rb

@@ -17,7 +17,7 @@ class ClientTrackTest < MiniTest::Test
   def test_track_without_app_id
     @heap.app_id = nil
     exception = assert_raises RuntimeError do
-      @heap.track 'test_track_without_app_id', 'test-handle'
+      @heap.track 'test_track_without_app_id', 'test-identity'
     end
     assert_equal RuntimeError, exception.class
     assert_equal 'Heap app_id not set', exception.message
@@ -25,7 +25,7 @@ class ClientTrackTest < MiniTest::Test
 
   def test_track_without_event_name
     exception = assert_raises ArgumentError do
-      @heap.track '', 'test-handle'
+      @heap.track '', 'test-identity'
     end
     assert_equal ArgumentError, exception.class
     assert_equal 'Missing or empty event name', exception.message
@@ -33,16 +33,55 @@ class ClientTrackTest < MiniTest::Test
 
   def test_track_with_long_event_name
     exception = assert_raises ArgumentError do
-      @heap.track 'A' * 1025, 'test-handle'
+      @heap.track 'A' * 1025, 'test-identity'
     end
     assert_equal ArgumentError, exception.class
     assert_equal 'Event name too long', exception.message
   end
 
+  def test_track_with_long_string_identity
+    long_identity = 'A' * 256
+
+    exception = assert_raises ArgumentError do
+      @heap.track 'some-event', long_identity
+    end
+    assert_equal ArgumentError, exception.class
+    assert_equal "Identity field too long; " +
+        '256 is above the 255-character limit', exception.message
+  end
+
+  def test_track_with_long_symbol_identity
+    long_identity = ('A' * 256).to_sym
+
+    exception = assert_raises ArgumentError do
+      @heap.track 'some-event', long_identity
+    end
+    assert_equal ArgumentError, exception.class
+    assert_equal "Identity field too long; " +
+        '256 is above the 255-character limit', exception.message
+  end
+
+  def test_track_with_array_identity
+    exception = assert_raises ArgumentError do
+      @heap.track 'test_track_with_array_property_value', []
+    end
+    assert_equal ArgumentError, exception.class
+    assert_equal 'Unsupported type for identity value []', exception.message
+  end
+
+  def test_track_with_invalid_property_object
+    exception = assert_raises ArgumentError do
+      @heap.track 'test_track_with_long_property_name', 'test-identity', false
+    end
+    assert_equal ArgumentError, exception.class
+    assert_equal 'Properties object does not implement #each',
+        exception.message
+  end
+
   def test_track_with_long_property_name
     long_name = 'A' * 1025
     exception = assert_raises ArgumentError do
-      @heap.track 'test_track_with_long_property_name', 'test-handle',
+      @heap.track 'test_track_with_long_property_name', 'test-identity',
           long_name => 'value'
     end
     assert_equal ArgumentError, exception.class
@@ -53,8 +92,8 @@ class ClientTrackTest < MiniTest::Test
   def test_track_with_long_string_property_value
     long_value = 'A' * 1025
     exception = assert_raises ArgumentError do
-      @heap.track 'test_track_with_long_string_property_value', 'test-handle',
-          'long_value_name' => long_value
+      @heap.track 'test_track_with_long_string_property_value',
+          'test-identity', 'long_value_name' => long_value
     end
     assert_equal ArgumentError, exception.class
     assert_equal "Property long_value_name value \"#{long_value}\" too " +
@@ -64,19 +103,46 @@ class ClientTrackTest < MiniTest::Test
   def test_track_with_long_symbol_property_value
     long_value = ('A' * 1025).to_sym
     exception = assert_raises ArgumentError do
-      @heap.track 'test_track_with_long_symbol_property_value', 'test-handle',
-          'long_value_name' => long_value
+      @heap.track 'test_track_with_long_symbol_property_value',
+          'test-identity', 'long_value_name' => long_value
     end
     assert_equal ArgumentError, exception.class
     assert_equal "Property long_value_name value :#{long_value} too long; " +
         '1025 is above the 1024-character limit', exception.message
   end
 
+  def test_track_with_array_property_value
+    exception = assert_raises ArgumentError do
+      @heap.track 'test_track_with_array_property_value', 'test-identity',
+          'array_value_name' => []
+    end
+    assert_equal ArgumentError, exception.class
+    assert_equal 'Unsupported type for property array_value_name value []',
+        exception.message
+  end
+
   def test_track
     @stubs.post '/api/track' do |env|
       golden_body = {
         'app_id' => 'test-app-id',
-        'identity' => 'test-handle',
+        'identity' => 'test-identity',
+        'event' => 'test_track',
+      }
+      assert_equal 'application/json', env[:request_headers]['Content-Type']
+      assert_equal @heap.user_agent, env[:request_headers]['User-Agent']
+      assert_equal golden_body, JSON.parse(env[:body])
+
+      [200, { 'Content-Type' => 'text/plain; encoding=utf8' }, '']
+    end
+
+    assert_equal @heap, @heap.track('test_track', 'test-identity')
+  end
+
+  def test_track_with_integer_identity
+    @stubs.post '/api/track' do |env|
+      golden_body = {
+        'app_id' => 'test-app-id',
+        'identity' => '123456789',
         'event' => 'test_track',
       }
       assert_equal 'application/json', env[:request_headers]['Content-Type']
@@ -86,14 +152,14 @@ class ClientTrackTest < MiniTest::Test
       [200, { 'Content-Type' => 'text/plain; encoding=utf8' }, '']
     end
 
-    assert_equal @heap, @heap.track('test_track', 'test-handle')
+    assert_equal @heap, @heap.track('test_track', 123456789)
   end
 
   def test_track_with_properties
     @stubs.post '/api/track' do |env|
       golden_body = {
         'app_id' => 'test-app-id',
-        'identity' => 'test-handle',
+        'identity' => 'test-identity',
         'event' => 'test_track_with_properties',
         'properties' => { 'foo' => 'bar', 'heap' => 'hurray' }
       }
@@ -105,7 +171,7 @@ class ClientTrackTest < MiniTest::Test
     end
 
     assert_equal @heap, @heap.track('test_track_with_properties',
-        'test-handle','foo' => 'bar', :heap => :hurray)
+        'test-identity','foo' => 'bar', :heap => :hurray)
   end
 
   def test_track_error
@@ -114,7 +180,7 @@ class ClientTrackTest < MiniTest::Test
     end
 
     exception = assert_raises HeapAPI::Error do
-      @heap.track('test-handle', 'test_track')
+      @heap.track('test-identity', 'test_track')
     end
     assert_equal HeapAPI::ApiError, exception.class
     assert_equal 'Heap API server error: 400 Bad request', exception.message
@@ -127,7 +193,7 @@ class ClientTrackTest < MiniTest::Test
     @heap.faraday_adapter = :net_http
     @heap.faraday_adapter_args = []
 
-    assert_equal @heap, @heap.track('test_track_integration', 'test-handle',
+    assert_equal @heap, @heap.track('test_track_integration', 'test-identity',
         'language' => 'ruby', 'project' => 'heap/heap-ruby')
   end
 
@@ -136,7 +202,7 @@ class ClientTrackTest < MiniTest::Test
     @heap.faraday_adapter_args = []
 
     assert_raises HeapAPI::Error do
-      @heap.track('test_track_integration', 'test-handle',
+      @heap.track('test_track_integration', 'test-identity',
           'language' => 'ruby', 'project' => 'heap/heap-ruby')
     end
   end
@@ -145,7 +211,7 @@ class ClientTrackTest < MiniTest::Test
   def test_track_with_stubbed_connection
     @heap.stubbed = true
 
-    assert_equal @heap, @heap.add_user_properties('test-handle',
+    assert_equal @heap, @heap.add_user_properties('test-identity',
         'foo' => 'bar', :heap => :hurray)
   end
 end

data/test/helper.rb

@@ -1,3 +1,29 @@
+if RUBY_VERSION >= '1.9'  # simplecov only works on MRI 1.9+
+  require 'simplecov'
+  require 'coveralls'
+
+  module SimpleCov::Configuration
+    def clean_filters
+      @filters = []
+    end
+  end
+
+  SimpleCov.configure do
+    clean_filters
+    load_adapter 'test_frameworks'
+  end
+
+  SimpleCov.formatters = [
+    SimpleCov::Formatter::HTMLFormatter,
+    Coveralls::SimpleCov::Formatter,
+  ]
+
+  ENV["COVERAGE"] && SimpleCov.start do
+    add_filter "/.rvm/"
+    add_filter "/.rbenv/"
+  end
+end
+
 require 'rubygems'
 require 'bundler'
 begin

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: heap
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Victor Costan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-29 00:00:00.000000000 Z
+date: 2016-03-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -39,33 +39,61 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 0.8.0
 - !ruby/object:Gem::Dependency
-  name: minitest
+  name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: yard
+  name: coveralls
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.7.6
+        version: 0.8.10
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.7.6
+        version: 0.8.10
+- !ruby/object:Gem::Dependency
+  name: jeweler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.0.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.0.1
+- !ruby/object:Gem::Dependency
+  name: minitest
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 5.8.4
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 5.8.4
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
@@ -81,33 +109,33 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '3.12'
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: 0.11.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: 0.11.2
 - !ruby/object:Gem::Dependency
-  name: jeweler
+  name: yard
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.0.1
+        version: 0.8.7.6
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.0.1
+        version: 0.8.7.6
 description: Implements Heap's server-side API
 email: victor@heapanalytics.com
 executables: []