dumbwaiter 0.3.5 → 0.3.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 266f730b1f4a463399570a2a28da34b682d4fbe6
-  data.tar.gz: 405d175444fe7305d34c1fd129e46177e8be2f0f
+  metadata.gz: ecdc3bda7d10fbfb6e265a6bb7014bc6ff93f186
+  data.tar.gz: c2c3f6d0985a8e80088bcbbea20f3fc0f4ecaead
 SHA512:
-  metadata.gz: d85594c47019afaf3f888e8fc10e8deda6342a1ea1f38866c71447d09972b9af71cf635a1c3b2ca4ae659f00f5b4bb36c0e48c70222dc91a31973d891d22d228
-  data.tar.gz: 59884387325c6e6695d59594d02fa5afef33a28e8ce43351efd0d86abaac75df69e088faa4ce62a37a1c09d8dad9def721068688cadd8d915a56e6e6cc69f97d
+  metadata.gz: ea521ffdae488a9f1d2b8c8a3d1266ec7979f0ee143f8bccdd915dce9255d4fd5035b13af97d703d2c1c5c0a4cf5e30ebeaf58df9a1ac39217e0ce24ac724c8b
+  data.tar.gz: ad34b6685b65fcc70826277fc63ecc723addcaa1f90ee1ff7a4fefa49bcf47e784fefe483a4e4590b4d037c04f6475c616f0b7331e2b08595566e7b1aaf2bec3

data/lib/dumbwaiter/deployment.rb

@@ -34,7 +34,7 @@ class Dumbwaiter::Deployment
   end
 
   def to_log
-    "#{created_at} - #{command_name} - #{status} - #{git_ref}"
+    "#{created_at} - #{user_name} - #{command_name} - #{status} - #{git_ref}"
   end
 
   protected

data/lib/dumbwaiter/user_profile.rb

@@ -1,9 +1,12 @@
 class Dumbwaiter::UserProfile
   attr_reader :opsworks_user_profile, :opsworks
 
-  def self.find(iam_user_arn, opsworks = Aws::OpsWorks.new(region: "us-east-1"))
-    opsworks.describe_user_profiles(iam_user_arns: [iam_user_arn]).user_profiles.first
+  def self.cache
+    @cache ||= {}
+  end
 
+  def self.find(iam_user_arn, opsworks = Aws::OpsWorks.new(region: "us-east-1"))
+    cache[iam_user_arn] ||= opsworks.describe_user_profiles(iam_user_arns: [iam_user_arn]).user_profiles.first
   end
 
   def initialize(opsworks_user_profile, opsworks = Aws::OpsWorks.new(region: "us-east-1"))

data/lib/dumbwaiter/version.rb

@@ -1,3 +1,3 @@
 module Dumbwaiter
-  VERSION = "0.3.5"
+  VERSION = "0.3.6"
 end

data/spec/lib/dumbwaiter/deployment_spec.rb

@@ -25,7 +25,7 @@ describe Dumbwaiter::Deployment do
   its(:status) { should == "badical" }
   its(:git_ref) { should == "eh-buddy" }
   its(:user_name) { should == "goose" }
-  its(:to_log) { should == "#{DateTime.parse("last Tuesday")} - deplode - badical - eh-buddy" }
+  its(:to_log) { should == "#{DateTime.parse("last Tuesday")} - goose - deplode - badical - eh-buddy" }
 
   context "when custom_json is nil" do
     let(:fake_deployment) do

data/spec/lib/dumbwaiter/user_profile_spec.rb

@@ -16,5 +16,11 @@ describe Dumbwaiter::UserProfile do
       fake_opsworks.should_receive(:describe_user_profiles).with(iam_user_arns: ["schwarz"])
       Dumbwaiter::UserProfile.find("schwarz", fake_opsworks).should == fake_user_profile
     end
+
+    it "caches duplicate requests for the same arn" do
+      fake_opsworks.should_receive(:describe_user_profiles).once
+      Dumbwaiter::UserProfile.find("schwarz", fake_opsworks)
+      Dumbwaiter::UserProfile.find("schwarz", fake_opsworks)
+    end
   end
 end

data/spec/spec_helper.rb

@@ -21,4 +21,5 @@ RSpec.configure do |config|
   # the seed, which is printed after each run.
   #     --seed 1234
   config.order = "random"
+  config.after { Dumbwaiter::UserProfile.cache.clear }
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dumbwaiter
 version: !ruby/object:Gem::Version
-  version: 0.3.5
+  version: 0.3.6
 platform: ruby
 authors:
 - Doc Ritezel