test_track_rails_client 2.0.0 → 3.0.0

data/vendor/gems/her/lib/her/model/orm.rb

@@ -1,3 +1,4 @@
+# coding: utf-8
 module Her
   module Model
     # This module adds ORM-like capabilities to the model
@@ -35,36 +36,44 @@ module Her
       #   @user.save
       #   # Called via POST "/users"
       def save
-        callback = new? ? :create : :update
-        method = self.class.method_for(callback)
-
-        run_callbacks callback do
-          run_callbacks :save do
-            params = to_params
-            self.class.request(to_params.merge(:_method => method, :_path => request_path, :_headers => request_headers, :_options => request_options)) do |parsed_data, response|
-              assign_attributes(self.class.parse(parsed_data[:data])) if parsed_data[:data].any?
-              @metadata = parsed_data[:metadata]
-              @response_errors = parsed_data[:errors]
-              self.assign_attributes(status_code: response.status)
-
-              return false if !response.success? || @response_errors.any?
-              if self.changed_attributes.present?
-                @previously_changed = self.changed_attributes.clone
-                self.changed_attributes.clear
+        if valid?
+          callback = new? ? :create : :update
+          method = self.class.method_for(callback)
+
+          run_callbacks callback do
+            run_callbacks :save do
+              params = to_params
+              self.class.request(to_params.merge(:_method => method, :_path => request_path, :_headers => request_headers, :_options => request_options)) do |parsed_data, response|
+                assign_attributes(self.class.parse(parsed_data[:data])) if parsed_data[:data].any?
+                @metadata = parsed_data[:metadata]
+                @response_errors = parsed_data[:errors]
+                self.assign_attributes(status_code: response.status)
+
+                return false if !response.success? || @response_errors.any?
+                if self.changed_attributes.present?
+                  @previously_changed = self.changed_attributes.clone
+                  self.changed_attributes.clear
+                end
               end
             end
           end
+          true
+        else
+          false
         end
-
-        true
       end
 
       # Similar to save(), except that ResourceInvalid is raised if the save fails
       def save!
-        if !self.save
-          raise Her::Errors::ResponseError.for(status_code).new("Remote validation of #{self.class.name} failed with a #{status_code}: #{full_error_messages}")
+        if self.save
+          true
+        elsif response_errors.present?
+          raise Her::Errors::ResponseError.for(status_code), "Remote validation of #{self.class.name} failed with a #{status_code}: #{full_error_messages}"
+        elsif errors.present?
+          raise Her::Errors::RecordInvalid, "Local validation of #{self.class.name} failed: #{errors.full_messages}"
+        else
+          raise 'unreachable'
         end
-        true
       end
 
       # Destroy a resource
@@ -73,24 +82,24 @@ module Her
       #   @user = User.find(1)
       #   @user.destroy
       #   # Called via DELETE "/users/1"
-      def destroy
+      def destroy(params = {})
         method = self.class.method_for(:destroy)
         run_callbacks :destroy do
-          self.class.request(:_method => method, :_path => request_path, :_headers => request_headers, :_options => request_options) do |parsed_data, response|
+          self.class.request(params.merge(:_method => method, :_path => request_path, :_headers => request_headers, :_options => request_options)) do |parsed_data, response|
             raise Her::Errors::ResponseError.for(response.status).new("Request against #{self.class} returned a #{response.status}") if response.status >= 400
             assign_attributes(self.class.parse(parsed_data[:data])) if parsed_data[:data].any?
             @metadata = parsed_data[:metadata]
             @response_errors = parsed_data[:errors]
-            @destroyed = true
+            @destroyed = response.success?
           end
         end
         self
       end
 
       def full_error_messages
-          self.response_errors.map do |field, messages|
-            "#{field} #{messages.join(', ')}"
-          end.join('. ')
+        self.response_errors.map do |field, messages|
+          "#{field} #{messages.join(', ')}"
+        end.join('. ')
       end
 
       module ClassMethods
@@ -141,7 +150,7 @@ module Her
         end
 
         # Delegate the following methods to `scoped`
-        [:all, :where, :create, :build, :find, :find_by, :first, :first_or_create, :first_or_initialize].each do |method|
+        [:all, :where, :create, :create!, :build, :find, :find_by, :first, :first_or_create, :first_or_initialize].each do |method|
           class_eval <<-RUBY, __FILE__, __LINE__ + 1
             def #{method}(*params)
               scoped.send(#{method.to_sym.inspect}, *params)
@@ -172,7 +181,10 @@ module Her
         def destroy_existing(id, params = {}, headers = {}, options = {})
           request(params.merge(:_method => method_for(:destroy), :_headers => headers, :_options => options, :_path => build_request_path(params.merge(primary_key => id)))) do |parsed_data, response|
             raise Her::Errors::ResponseError.for(response.status).new("Request against #{name} returned a #{response.status}") if response.status >= 400
-            new(parse(parsed_data[:data]).merge(:_destroyed => true))
+            data = parse(parsed_data[:data])
+            metadata = parsed_data[:metadata]
+            response_errors = parsed_data[:errors]
+            new(data.merge(:_destroyed => response.success?, :metadata => metadata, :response_errors => response_errors))
           end
         end
 

data/vendor/gems/her/lib/her/model/parse.rb

@@ -58,6 +58,7 @@ module Her
         def embeded_params(attributes)
           associations[:has_many].select { |a| attributes.include?(a[:data_key])}.compact.inject({}) do |hash, association|
             params = attributes[association[:data_key]].map(&:to_params)
+            next hash if params.empty?
             if association[:class_name].constantize.include_root_in_json?
               root = association[:class_name].constantize.root_element
               hash[association[:data_key]] = params.map { |n| n[root] }

data/vendor/gems/her/lib/her/model/relation.rb

@@ -204,13 +204,20 @@ module Her
       #   @user = User.where(:email => "tobias@bluth.com").create(:fullname => "Tobias Fünke")
       #   # Called via POST "/users/1" with `&email=tobias@bluth.com&fullname=Tobias+Fünke`
       def create(attributes = {})
-        attributes ||= {}
-        resource = @parent.new(@params.merge(attributes))
-        resource.request_headers = request_headers
-        resource.request_options = request_options
-        resource.save
+        _build(attributes).tap(&:save)
+      end
 
-        resource
+      # Create a resource and return it, raises if there are validation or response errors
+      #
+      # @example
+      #   @user = User.create!(:fullname => "Tobias Fünke")
+      #   # Called via POST "/users/1" with `&fullname=Tobias+Fünke`
+      #
+      # @example
+      #   @user = User.where(:email => "tobias@bluth.com").create!(:fullname => "Tobias Fünke")
+      #   # Called via POST "/users/1" with `&email=tobias@bluth.com&fullname=Tobias+Fünke`
+      def create!(attributes = {})
+        _build(attributes).tap(&:save!)
       end
 
       # Fetch a resource and create it if it's not found
@@ -246,6 +253,14 @@ module Her
       def clear_fetch_cache!
         instance_variable_set(:@_fetch, nil)
       end
+
+      # @private
+      def _build(attributes)
+        resource = @parent.new(@params.merge(attributes))
+        resource.request_headers = request_headers
+        resource.request_options = request_options
+        resource
+      end
     end
   end
 end

data/vendor/gems/her/lib/her/version.rb

@@ -1,3 +1,3 @@
 module Her
-  VERSION = "0.8.1"
+  VERSION = "0.8.6"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: test_track_rails_client
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 3.0.0
 platform: ruby
 authors:
 - Ryan O'Neill
@@ -13,18 +13,15 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-10-12 00:00:00.000000000 Z
+date: 2018-01-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rails
+  name: airbrake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '4.1'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '6.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -32,31 +29,36 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '4.1'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '6.0'
 - !ruby/object:Gem::Dependency
-  name: faraday
+  name: delayed-plugins-airbrake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.8'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.8'
-    - - "<"
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: delayed_job
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '4.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.0'
 - !ruby/object:Gem::Dependency
-  name: faraday_middleware
+  name: delayed_job_active_record
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -70,74 +72,83 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: public_suffix
+  name: faraday
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.0.0
-    - - "<="
+        version: '0.8'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.0.0
-    - - "<="
+        version: '0.8'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: mixpanel-ruby
+  name: faraday_middleware
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.4'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.4'
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: delayed_job
+  name: mixpanel-ruby
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '1.4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '1.4'
 - !ruby/object:Gem::Dependency
-  name: delayed_job_active_record
+  name: public_suffix
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.0.0
+    - - "<="
+      - !ruby/object:Gem::Version
+        version: 3.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.0.0
+    - - "<="
+      - !ruby/object:Gem::Version
+        version: 3.0.0
 - !ruby/object:Gem::Dependency
-  name: airbrake
+  name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '4.1'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '6.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -145,20 +156,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '4.1'
-- !ruby/object:Gem::Dependency
-  name: delayed-plugins-airbrake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: request_store
   requirement: !ruby/object:Gem::Requirement
@@ -215,20 +215,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: rubocop
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0.36'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0.36'
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
@@ -275,7 +261,6 @@ files:
 - app/models/test_track/analytics/safe_wrapper.rb
 - app/models/test_track/assignment.rb
 - app/models/test_track/config_updater.rb
-- app/models/test_track/create_alias_job.rb
 - app/models/test_track/fake/split_detail.rb
 - app/models/test_track/fake/split_registry.rb
 - app/models/test_track/fake/visitor.rb
@@ -323,16 +308,7 @@ files:
 - vendor/gems/fakeable_her/lib/fakeable_her.rb
 - vendor/gems/fakeable_her/lib/fakeable_her/model.rb
 - vendor/gems/fakeable_her/lib/fakeable_her/version.rb
-- vendor/gems/her/CONTRIBUTING.md
-- vendor/gems/her/Gemfile
 - vendor/gems/her/LICENSE
-- vendor/gems/her/README.md
-- vendor/gems/her/Rakefile
-- vendor/gems/her/UPGRADE.md
-- vendor/gems/her/gemfiles/Gemfile.activemodel-3.2.x
-- vendor/gems/her/gemfiles/Gemfile.activemodel-4.0
-- vendor/gems/her/gemfiles/Gemfile.activemodel-4.1
-- vendor/gems/her/gemfiles/Gemfile.activemodel-4.2
 - vendor/gems/her/her.gemspec
 - vendor/gems/her/lib/her.rb
 - vendor/gems/her/lib/her/api.rb
@@ -365,69 +341,6 @@ files:
 - vendor/gems/her/lib/her/model/paths.rb
 - vendor/gems/her/lib/her/model/relation.rb
 - vendor/gems/her/lib/her/version.rb
-- vendor/gems/her/spec/api_spec.rb
-- vendor/gems/her/spec/collection_spec.rb
-- vendor/gems/her/spec/error_collection_spec.rb
-- vendor/gems/her/spec/json_api/model_spec.rb
-- vendor/gems/her/spec/middleware/accept_json_spec.rb
-- vendor/gems/her/spec/middleware/first_level_parse_json_spec.rb
-- vendor/gems/her/spec/middleware/json_api_parser_spec.rb
-- vendor/gems/her/spec/middleware/second_level_parse_json_spec.rb
-- vendor/gems/her/spec/model/associations/association_proxy_spec.rb
-- vendor/gems/her/spec/model/associations_spec.rb
-- vendor/gems/her/spec/model/attributes_spec.rb
-- vendor/gems/her/spec/model/callbacks_spec.rb
-- vendor/gems/her/spec/model/dirty_spec.rb
-- vendor/gems/her/spec/model/http_spec.rb
-- vendor/gems/her/spec/model/introspection_spec.rb
-- vendor/gems/her/spec/model/nested_attributes_spec.rb
-- vendor/gems/her/spec/model/orm_spec.rb
-- vendor/gems/her/spec/model/parse_spec.rb
-- vendor/gems/her/spec/model/paths_spec.rb
-- vendor/gems/her/spec/model/relation_spec.rb
-- vendor/gems/her/spec/model/validations_spec.rb
-- vendor/gems/her/spec/model_spec.rb
-- vendor/gems/her/spec/spec_helper.rb
-- vendor/gems/her/spec/support/extensions/array.rb
-- vendor/gems/her/spec/support/extensions/hash.rb
-- vendor/gems/her/spec/support/macros/her_macros.rb
-- vendor/gems/her/spec/support/macros/model_macros.rb
-- vendor/gems/her/spec/support/macros/request_macros.rb
-- vendor/gems/publicsuffix-ruby/CHANGELOG.md
-- vendor/gems/publicsuffix-ruby/Gemfile
-- vendor/gems/publicsuffix-ruby/LICENSE.txt
-- vendor/gems/publicsuffix-ruby/README.md
-- vendor/gems/publicsuffix-ruby/Rakefile
-- vendor/gems/publicsuffix-ruby/lib/definitions.txt
-- vendor/gems/publicsuffix-ruby/lib/public_suffix.rb
-- vendor/gems/publicsuffix-ruby/lib/public_suffix/domain.rb
-- vendor/gems/publicsuffix-ruby/lib/public_suffix/errors.rb
-- vendor/gems/publicsuffix-ruby/lib/public_suffix/list.rb
-- vendor/gems/publicsuffix-ruby/lib/public_suffix/rule.rb
-- vendor/gems/publicsuffix-ruby/lib/public_suffix/version.rb
-- vendor/gems/publicsuffix-ruby/public_suffix.gemspec
-- vendor/gems/publicsuffix-ruby/test/acceptance_test.rb
-- vendor/gems/publicsuffix-ruby/test/test_helper.rb
-- vendor/gems/publicsuffix-ruby/test/unit/domain_test.rb
-- vendor/gems/publicsuffix-ruby/test/unit/errors_test.rb
-- vendor/gems/publicsuffix-ruby/test/unit/list_test.rb
-- vendor/gems/publicsuffix-ruby/test/unit/public_suffix_test.rb
-- vendor/gems/publicsuffix-ruby/test/unit/rule_test.rb
-- vendor/gems/ruby_spec_helpers/lib/ruby_spec_helpers.rb
-- vendor/gems/ruby_spec_helpers/lib/ruby_spec_helpers/capybara_configuration.rb
-- vendor/gems/ruby_spec_helpers/lib/ruby_spec_helpers/matchers.rb
-- vendor/gems/ruby_spec_helpers/lib/ruby_spec_helpers/rspec_configuration.rb
-- vendor/gems/ruby_spec_helpers/lib/ruby_spec_helpers/rubocop.rb
-- vendor/gems/ruby_spec_helpers/lib/ruby_spec_helpers/rubocop/cop/betterment/html_safe.rb
-- vendor/gems/ruby_spec_helpers/lib/ruby_spec_helpers/rubocop/cop/betterment/raw.rb
-- vendor/gems/ruby_spec_helpers/lib/ruby_spec_helpers/rubocop/cop/betterment/safe_concat.rb
-- vendor/gems/ruby_spec_helpers/lib/ruby_spec_helpers/shared_examples.rb
-- vendor/gems/ruby_spec_helpers/lib/ruby_spec_helpers/shared_examples/betterment_application_examples.rb
-- vendor/gems/ruby_spec_helpers/lib/ruby_spec_helpers/site_prism_configuration.rb
-- vendor/gems/ruby_spec_helpers/lib/ruby_spec_helpers/site_prism_dropdown.rb
-- vendor/gems/ruby_spec_helpers/lib/ruby_spec_helpers/version.rb
-- vendor/gems/ruby_spec_helpers/lib/ruby_spec_helpers/webmock_configuration.rb
-- vendor/gems/ruby_spec_helpers/ruby_spec_helpers.gemspec
 homepage: https://github.com/Betterment
 licenses:
 - MIT
@@ -440,7 +353,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.0.0
+      version: 2.1.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -448,7 +361,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.13
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: Rails client for TestTrack

data/app/models/test_track/create_alias_job.rb

@@ -1,18 +0,0 @@
-class TestTrack::CreateAliasJob
-  attr_reader :existing_id, :alias_id
-
-  def initialize(opts)
-    @existing_id = opts.delete(:existing_id)
-    @alias_id = opts.delete(:alias_id)
-
-    %w(existing_id alias_id).each do |param_name|
-      raise "#{param_name} must be present" unless send(param_name).present?
-    end
-    raise "unknown opts: #{opts.keys.to_sentence}" if opts.present?
-  end
-
-  def perform
-    return unless TestTrack.enabled?
-    TestTrack.analytics.alias(alias_id, existing_id)
-  end
-end

data/vendor/gems/her/CONTRIBUTING.md

@@ -1,26 +0,0 @@
-# How to contribute
-
-_(This file is heavily based on [factory\_girl\_rails](https://github.com/thoughtbot/factory_girl_rails/blob/master/CONTRIBUTING.md)’s Contribution Guide)_
-
-We love pull requests. Here’s a quick guide:
-
-* Fork the repository.
-* Run `rake spec` (to make sure you start with a clean slate).
-* Implement your feature or fix.
-* Add examples that describe it (in the `spec` directory). Only refactoring and documentation changes require no new tests. If you are adding functionality or fixing a bug, we need examples!
-* Make sure `rake spec` passes after your modifications.
-* Commit (bonus points for doing it in a `feature-*` branch).
-* Push to your fork and send your pull request!
-
-If we have not replied to your pull request in three or four days, do not hesitate to post another comment in it — yes, we can be lazy sometimes.
-
-## Syntax Guide
-
-Do not hesitate to submit patches that fix syntax issues. Some may have slipped under our nose.
-
-* Two spaces, no tabs (but you already knew that, right?).
-* No trailing whitespace. Blank lines should not have any space. There are few things we **hate** more than trailing whitespace. Seriously.
-* `MyClass.my_method(my_arg)` not `my_method( my_arg )` or `my_method my_arg`.
-* `[:foo, :bar]` and not `[ :foo, :bar ]`, `{ :foo => :bar }` and not `{:foo => :bar}`
-* `a = b` and not `a=b`.
-* Follow the conventions you see used in the source already.

data/vendor/gems/her/Gemfile

@@ -1,10 +0,0 @@
-source "https://rubygems.org"
-gemspec
-
-if RbConfig::CONFIG['RUBY_PROGRAM_VERSION'] && RbConfig::CONFIG['RUBY_PROGRAM_VERSION'] >= '1.9.3'
-  gem 'activemodel', '>= 3.2.0'
-  gem 'activesupport', '>= 3.2.0'
-else
-  gem 'activemodel', '~> 3.2.0'
-  gem 'activesupport', '~> 3.2.0'
-end