ey_services_api 0.3.11 → 0.3.12

data/EYIntegratedGemfile

@@ -10,10 +10,14 @@ group :test, :development do
   #private Gem deps of fake_awsm
   gem 'ey_sso', :git => "git@github.com:engineyard/ey_sso.git"
   gem 'ey_services_api_internal', :git => "git@github.com:engineyard/ey_services_api_internal.git"
+  # gem 'ey_services_api_internal', :path => "../ey_services_api_internal"
 
   #"reals"
   gem 'tresfiestas', :git => "git@github.com:engineyard/tresfiestas.git"
+  # gem 'tresfiestas', :path => "../tresfiestas"
   gem 'fake_awsm', :git => "git@github.com:engineyard/fake_awsm.git"
+  # gem 'fake_awsm', :path => "../fake_awsm"
 
   gem 'guard-rspec'
+  gem 'pry'
 end

data/Gemfile

@@ -8,4 +8,5 @@ group :test, :development do
 
   gem 'rake'
   gem 'guard-rspec'
+  gem 'pry'
 end

data/lib/ey_services_api/service_account_response.rb

@@ -1,13 +1,14 @@
 module EY
   module ServicesAPI
-    class ServiceAccountResponse < APIStruct.new(:configuration_required, :configuration_url, :message, :provisioned_services_url, :url)
+    class ServiceAccountResponse < APIStruct.new(:configuration_required, :configuration_url, :message, :provisioned_services_url, :users_url, :url)
       def to_hash
         {
           :service_account            => {
             :url                      => self.url,
             :configuration_required   => self.configuration_required,
             :configuration_url        => self.configuration_url,
-            :provisioned_services_url => self.provisioned_services_url
+            :provisioned_services_url => self.provisioned_services_url,
+            :users_url                => self.users_url,
           },
           :message                    => self.message && self.message.to_hash
         }

data/lib/ey_services_api/version.rb

@@ -1,5 +1,5 @@
 module EY
   module ServicesAPI
-    VERSION = "0.3.11"
+    VERSION = "0.3.12"
   end
 end

data/spec/invoice_spec.rb

@@ -1,6 +1,5 @@
 #because there may be multiple 'spec_helper' in load path when running from external test helper
 require File.expand_path('../spec_helper.rb', __FILE__)
-require 'sinatra'
 
 describe EY::ServicesAPI::Invoice do
   before do

data/spec/message_spec.rb

@@ -1,6 +1,5 @@
 #because there may be multiple 'spec_helper' in load path when running from external test helper
 require File.expand_path('../spec_helper.rb', __FILE__)
-require 'sinatra'
 
 #TODO: support a generic message class too?
 describe EY::ServicesAPI::Message do

data/spec/provisioned_service_creation_spec.rb

@@ -1,6 +1,5 @@
 #because there may be multiple 'spec_helper' in load path when running from external test helper
 require File.expand_path('../spec_helper.rb', __FILE__)
-require 'sinatra'
 
 describe EY::ServicesAPI::ProvisionedServiceCreation do
 

data/spec/service_account_creation_spec.rb

@@ -1,6 +1,5 @@
 #because there may be multiple 'spec_helper' in load path when running from external test helper
 require File.expand_path('../spec_helper.rb', __FILE__)
-require 'sinatra'
 
 describe EY::ServicesAPI::ServiceAccountCreation do
 

data/spec/users_spec.rb

@@ -0,0 +1,116 @@
+#because there may be multiple 'spec_helper' in load path when running from external test helper
+require File.expand_path('../spec_helper.rb', __FILE__)
+
+require 'ey_services_fake/mocking_bird_service'
+class UserHavingService < EyServicesFake::MockingBirdService
+  def self.record_of_users
+    @record_of_users ||= {}
+  end
+
+  def self.implement_the_app(app)
+    super(app)
+    app.class_eval do
+      get "/api/1/users" do
+        content_type :json
+        parent.record_of_users.map do |user_id, user|
+          {
+            "user" => user,
+            "url" => "#{parent.base_url}api/1/users/#{user_id}"
+          }
+        end.to_json
+      end
+      put "/api/1/users/:ey_user_id" do |ey_user_id|
+        content_type :json
+        json_post_body = JSON.parse(request.body.read)
+        parent.record_of_users[ey_user_id].merge! json_post_body['user']
+        {}.to_json
+      end
+      delete "/api/1/users/:ey_user_id" do |ey_user_id|
+        content_type :json
+        parent.record_of_users.delete(ey_user_id)
+        {}.to_json
+      end
+    end
+  end
+
+  def self.account_sso_hook(params)
+    @record_of_users[params["ey_user_id"]] = {
+      'access_level' => params["access_level"],
+      'ey_user_id' => params["ey_user_id"],
+      'ey_user_email' => params["ey_user_email"],
+      'ey_user_name' => params["ey_user_name"],
+    }
+  end
+
+  def self.service_account_creation_params
+    super.merge(:users_url => "#{base_url}api/1/users")
+  end
+
+end
+
+describe "users" do
+  describe "with a service account" do
+    before do
+      UserHavingService.record_of_users.clear
+      @tresfiestas.actors[:service_provider] = UserHavingService.new
+      @service_account = @tresfiestas.service_account
+      @messages_url = @service_account[:messages_url]
+      @connection = EY::ServicesAPI.connection
+      @connection_to_partner_service = EY::ApiHMAC::BaseConnection.new
+      @connection_to_partner_service.backend = @tresfiestas.app_for(:service_provider)
+      @original_email = @tresfiestas.sso_user.email
+      @ey_user_id = @tresfiestas.sso_user.external_service_id.to_s
+    end
+
+    describe "if the service doesn't provide a users_url, when a user SSOs to the service" do
+      before do
+        UserHavingService.record_of_users.should be_empty
+        @connection.update_service_account(@service_account[:url], :users_url => nil)
+        response = @connection_to_partner_service.get(@tresfiestas.service_account_sso_url)
+        response.status.should eq 200
+        UserHavingService.record_of_users.size.should eq 1
+      end
+
+      it "doesn't send the email" do
+        UserHavingService.record_of_users[@ey_user_id]['ey_user_email'].should be_nil
+      end
+    end
+
+    describe "when a user SSOs to the service" do
+      before do
+        UserHavingService.record_of_users.should be_empty
+        response = @connection_to_partner_service.get(@tresfiestas.service_account_sso_url)
+        response.status.should eq 200
+        UserHavingService.record_of_users.size.should eq 1
+      end
+
+      it "saves the e-mail" do
+        UserHavingService.record_of_users[@ey_user_id]['ey_user_email'].should eq @original_email
+      end
+
+      describe "when a user's e-mail changes" do
+        before do
+          @new_email = "new_email@example.com"
+          @tresfiestas.trigger_mock_user_update(@new_email)
+        end
+
+        it "updates the saved value" do
+          @new_email.should_not eq @original_email
+          UserHavingService.record_of_users[@ey_user_id]['ey_user_email'].should eq @new_email
+        end
+      end
+
+      describe "when a user is removed" do
+        before do
+          @tresfiestas.trigger_mock_user_delete
+        end
+
+        it "updates the saved value" do
+          UserHavingService.record_of_users.size.should eq 0
+        end
+      end
+
+    end
+
+  end
+end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: ey_services_api
 version: !ruby/object:Gem::Version 
-  hash: 5
+  hash: 11
   prerelease: 
   segments: 
   - 0
   - 3
-  - 11
-  version: 0.3.11
+  - 12
+  version: 0.3.12
 platform: ruby
 authors: 
 - Jacob & Thorben & David & mkb & Josh & Others
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-05-03 00:00:00 Z
+date: 2012-05-08 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: rspec
@@ -101,6 +101,7 @@ files:
 - spec/service_account_creation_spec.rb
 - spec/service_spec.rb
 - spec/spec_helper.rb
+- spec/users_spec.rb
 homepage: https://github.com/engineyard/ey_services_api
 licenses: []
 
@@ -130,7 +131,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: ey_services_api
-rubygems_version: 1.8.19
+rubygems_version: 1.8.10
 signing_key: 
 specification_version: 3
 summary: API for Partner Services (talks to services.engineyard.com)
@@ -141,3 +142,4 @@ test_files:
 - spec/service_account_creation_spec.rb
 - spec/service_spec.rb
 - spec/spec_helper.rb
+- spec/users_spec.rb