fastly 1.1.1 → 1.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7185290c285aff8cefd7d9285f43f3e245051e2d
-  data.tar.gz: 70e9c38148ce099176ae05fc712a0b046b542e47
+  metadata.gz: 1ef96a7f613dfa4a86cf2312c52bce16235bf7cf
+  data.tar.gz: 4d5b36b94d9d90f263f3f831dc72cfb333d88813
 SHA512:
-  metadata.gz: 75e37814b6bc25c1317086e90ad0743298d6680597bb78f1dcc13497baa698a950b560f7a67b3b5d062797cc71712b269554c371685a626e523eccf27cf6b6c9
-  data.tar.gz: c01bcc6cca3b593c849a2bce956f3fe90741fead91100cadd8d2011baab80a78e32f8d092cddf27d75cf8ea4576e0dbadd65eaed7ecae06165baef490599ab1c
+  metadata.gz: 128bb682ee2ed4be291c9eb7d0bbf420e88d4c2f33c139dd606696bc80aefd74638040a3d77b034cefa3f8ba0e6d4f003be5e5f627762b88f4f4f5ffd8e6038b
+  data.tar.gz: a3bfafd5d8517237b3f0a452d2bb4397d3f57a6685c7cad38a57b582730e898d758ee49d5106b7c76431937b986f7f9a16adb1d30e085f76f4897713e6cb7a16

data/Gemfile

@@ -6,4 +6,5 @@ group :development, :test do
   gem 'rake', '~> 10.3.2'
   gem 'rdoc', '~> 4.1.1'
   gem 'minitest', '~> 5.3.4'
+  gem 'pry'
 end

data/{Changes → HISTORY.md}

@@ -1,3 +1,10 @@
+# 2014-06-12 v1.1.2
+* Replace `String#underscore` with `Fastly::Util.class_to_path` method.
+* Add first true unit test
+* Add `test_helper.rb`
+* Add `pry` as dependency
+* Add console Rake task
+
 2013-11-26 v1.02
 
 Fix rdoc dependency and quorum spelling (Kristoffer Renholm)

data/Rakefile

@@ -2,6 +2,14 @@ require 'bundler/gem_tasks'
 require 'rake/testtask'
 require 'rdoc/task'
 
+desc 'Run library from within a Pry console'
+task :console do
+  require 'pry'
+  require 'fastly'
+  ARGV.clear
+  Pry.start
+end
+
 RDoc::Task.new do |rdoc|
   rdoc.rdoc_dir = 'doc'
   rdoc.main = 'README.md'

data/lib/fastly.rb

@@ -5,7 +5,7 @@
 # A client library for interacting with the Fastly web acceleration service
 class Fastly
   require 'fastly/gem_version'
-  require 'fastly/string'
+  require 'fastly/util'
   require 'fastly/fetcher'
   require 'fastly/client'
 
@@ -155,7 +155,7 @@ class Fastly
   end
 
   [User, Customer, Backend, CacheSetting, Condition, Director, Domain, Header, Healthcheck, Gzip, Match, Origin, RequestSetting, ResponseObject, Service, S3Logging, Syslog, VCL, Version].each do |klass|   
-    type = klass.to_s.split("::")[-1].underscore
+    type = Util.class_to_path(klass)
     # unless the class doesn't have a list path or it already exists
     unless klass.list_path.nil? || klass.respond_to?("list_#{type}s".to_sym)
         self.send :define_method, "list_#{type}s".to_sym do |*args|

data/lib/fastly/base.rb

@@ -1,4 +1,3 @@
-
 class Fastly
   # Base class for all Fastly objects
   class Base # :nodoc: all
@@ -35,7 +34,7 @@ class Fastly
      end
 
      def self.path
-       self.to_s.split("::")[-1].underscore
+       Util.class_to_path(self)
      end
      
      def self.get_path(id)

data/lib/fastly/cache_setting.rb

@@ -41,7 +41,7 @@ class Fastly
     private
       # needs an 's' at the end of the url
       def self.path
-        self.to_s.split("::")[-1].underscore + "s"
+        Util.class_to_path(self, true)
       end
   end
 end

data/lib/fastly/gem_version.rb

@@ -1,4 +1,4 @@
 class Fastly
   # The current version of the library
-  VERSION = "1.1.1"
+  VERSION = "1.1.2"
 end

data/lib/fastly/request_setting.rb

@@ -84,7 +84,7 @@ class Fastly
 
     private
      def self.path
-       self.to_s.split("::")[-1].underscore + "s"
+       Util.class_to_path(self, true)
      end
   end
 end

data/lib/fastly/util.rb

@@ -0,0 +1,9 @@
+class Fastly
+  module Util
+    def self.class_to_path(klass, append_s = false)
+      klass_string = klass.to_s.split('::')[-1]
+      klass_string = klass_string.gsub(/([^A-Z])([A-Z]+)/, '\1_\2').downcase
+      append_s ? "#{klass_string}s" : klass_string
+    end
+  end
+end

data/test/fastly/util_test.rb

@@ -0,0 +1,20 @@
+require 'test_helper'
+
+class Fastly
+  class FooBar; end
+  class FooBarBaz; end
+  describe Util do
+    describe '.class_to_path' do
+      let(:klass)        { Fastly::FooBar }
+      let(:klass_with_s) { Fastly::FooBarBaz }
+
+      it 'should convert a class name to an underscored path' do
+        assert_equal 'foo_bar', Util.class_to_path(klass)
+      end
+
+      it 'should append an s if second argument is true' do
+        assert_equal 'foo_bar_bazs', Util.class_to_path(klass_with_s, true)
+      end
+    end
+  end
+end

data/test/test_helper.rb

@@ -0,0 +1,2 @@
+require 'minitest/autorun'
+require 'fastly'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fastly
 version: !ruby/object:Gem::Version
-  version: 1.1.1
+  version: 1.1.2
 platform: ruby
 authors:
 - Fastly Inc
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-05 00:00:00.000000000 Z
+date: 2014-06-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: curb
@@ -48,8 +48,8 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".travis.yml"
-- Changes
 - Gemfile
+- HISTORY.md
 - LICENSE
 - README.md
 - Rakefile
@@ -78,17 +78,19 @@ files:
 - lib/fastly/s3_logging.rb
 - lib/fastly/service.rb
 - lib/fastly/settings.rb
-- lib/fastly/string.rb
 - lib/fastly/syslog.rb
 - lib/fastly/user.rb
+- lib/fastly/util.rb
 - lib/fastly/vcl.rb
 - lib/fastly/version.rb
 - test/admin_test.rb
 - test/api_key_test.rb
 - test/common.rb
+- test/fastly/util_test.rb
 - test/full_login_test.rb
 - test/helper.rb
 - test/stats_test.rb
+- test/test_helper.rb
 homepage: http://github.com/fastly/fastly-ruby
 licenses:
 - MIT
@@ -117,6 +119,8 @@ test_files:
 - test/admin_test.rb
 - test/api_key_test.rb
 - test/common.rb
+- test/fastly/util_test.rb
 - test/full_login_test.rb
 - test/helper.rb
 - test/stats_test.rb
+- test/test_helper.rb

data/lib/fastly/string.rb

@@ -1,7 +0,0 @@
-# add an underscore method to String, so we can easily convert CamelCase to
-# camel_case for CacheSetting and RequestSetting objects.
-class String
-  def underscore
-    gsub(/([^A-Z])([A-Z]+)/, '\1_\2').downcase
-  end
-end