multiforecast-client 0.80.0.1 → 0.80.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0f3ff190a2dcae55cf761b76617bfa61215eecfc
-  data.tar.gz: 18b80b8eda5635555fb26464c27ece3036b1a7f2
+  metadata.gz: e05fb5d0476e998fd94384fcada44dad95b49c90
+  data.tar.gz: e2c8e233d42d092ebcc31e4fe417c912607202fa
 SHA512:
-  metadata.gz: 0c903900bcff9705f2143e3ba2cd0d7e86d4662fe7c4a246ffefa8a7baae8198d0c8865cdbd162e2e7e27e7030dfcefda8c39d3b86456b8a007299dc2525d9db
-  data.tar.gz: 39eee4322bb3e203c366d1477f9c4d4f9c17f5b69ded6cc11f4cb93cae1e575fdb5f6160cff8dcde3ae452157140343bae7ad5486a5d46a940557c979ad74681
+  metadata.gz: 6dec979c161addb240c65b1b687d0c4c7d9b2adb1b7a4f3daf33342b3832de4cff128dac61297b456c699d3b568efd0b61028b7b60362a66de134cb35c5583b2
+  data.tar.gz: c02f71a74c95572b54bfd1a73a002ba5a7efd10daa98e6af0226e0fe6671bfc71e6808a502289983010ca10cc82efd9b632bd6a800d985bee30f91a6d317bcca

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+# 0.80.0.2 (2014/03/04)
+
+Fixes
+
+* Fix `short_metrics=` option is not working
+
 # 0.80.0.1 (2014/03/04)
 
 Fixes

data/VERSION

@@ -1 +1 @@
-0.80.0.1
+0.80.0.2

data/lib/multiforecast/client.rb

@@ -19,7 +19,7 @@ module MultiForecast
       mapping = opts['mapping'] || { '' => 'http://localhost:5125' }
       # remove heading / of path
       mapping.each {|key, val| @mapping[lstrip(key, '/')] = val }
-      @short_metrics = opts['short_metrics'] || true
+      @short_metrics = opts['short_metrics'].nil? || opts['short_metrics'] # default true
 
       @clients = {}
       @base_uris = {}
@@ -31,6 +31,7 @@ module MultiForecast
         @clients[path] = GrowthForecast::Client.new(base_uri)
         @base_uris[path] = out_uri || base_uri
       end
+      self.debug_dev = opts['debug_dev'] if opts['debug_dev']
     end
 
     # set the `debug_dev` attribute of HTTPClient

data/spec/multiforecast/client_spec.rb

@@ -20,9 +20,33 @@ describe MultiForecast::Client do
     end
   end
 
-  context "#debug_dev" do
-    subject { MultiForecast::Client.new('mapping' => {'app1/' => 'http://localhost:5125'}) }
-    it { expect { subject.debug_dev = STDOUT }.not_to raise_error }
+  context "options" do
+    context "debug_dev" do
+      context 'default' do
+        subject { MultiForecast::Client.new('mapping' => {'app1/' => 'http://localhost:5125'}) }
+        it { expect(subject.debug_dev).to be_nil }
+      end
+
+      context 'STDOUT' do
+        subject { MultiForecast::Client.new('mapping' => {'app1/' => 'http://localhost:5125'}, 'debug_dev' => STDOUT) }
+        it { expect(subject.debug_dev).to eql(STDOUT) }
+      end
+    end
+
+    context "short_metrics" do
+      context 'default' do
+        subject { MultiForecast::Client.new('mapping' => {'app1/' => 'http://localhost:5125'}) }
+        it { expect(subject.short_metrics).to eql(true) }
+      end
+      context 'false' do
+        subject { MultiForecast::Client.new('mapping' => {'app1/' => 'http://localhost:5125'}, 'short_metrics' => false) }
+        it { expect(subject.short_metrics).to eql(false) }
+      end
+      context 'true' do
+        subject { MultiForecast::Client.new('mapping' => {'app1/' => 'http://localhost:5125'}, 'short_metrics' => true) }
+        it { expect(subject.short_metrics).to eql(true) }
+      end
+    end
   end
 
   context "#list_graph" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: multiforecast-client
 version: !ruby/object:Gem::Version
-  version: 0.80.0.1
+  version: 0.80.0.2
 platform: ruby
 authors:
 - Naotoshi Seo