More specs for query options.

Author: woodie

lib/tenkit/client.rb

@@ -53,11 +53,7 @@ def weather_alert(id, language: 'en')
     def get(url, query: nil)
       headers = { Authorization: "Bearer #{token}" }
       params = { headers: headers }
-
-      if !query.nil?
-        params[:query] = query
-      end
-
+      params[:query] = query unless query.nil?
       self.class.get(url, params)
     end
 

lib/tenkit/utils.rb

@@ -5,5 +5,11 @@ def self.snake(str)
 
       str.gsub(/(.)([A-Z])/, '\1_\2').sub(/_UR_L$/, "_URL").downcase
     end
+
+    def self.camel(str)
+      return str.camelize(:lower) if str.respond_to? :camelize
+
+      str.gsub(/_(.)/) {|e| $1.upcase}
+    end
   end
 end

spec/tenkit/tenkit/utils_spec.rb

@@ -1,15 +1,16 @@
 require_relative "../spec_helper"
 RSpec.describe Tenkit::Utils do
+  let(:camel) { "someSpecialCaseString" }
+  let(:snake) { "some_special_case_string" }
+
   subject { Tenkit::Utils }
 
   describe ".snake" do
-    let(:simple) { "someCamelCaseString" }
-    let(:correct) { "some_camel_case_string" }
     let(:complex) { "someAWSCredentials" }
     let(:mangled) { "some_aw_scredentials" }
 
     it "converts to snake case" do
-      expect(subject.snake(simple)).to eq correct
+      expect(subject.snake(camel)).to eq snake
     end
 
     context "when passed an acronym" do
@@ -26,10 +27,28 @@
       end
     end
   end
+
+  describe ".camel" do
+    it "converts to camel case" do
+      expect(subject.camel(snake)).to eq camel
+    end
+
+    context "when a proper camelize package is available" do
+      let(:patched_string) { PatchedString.new snake }
+
+      it "uses string camelize method" do
+        expect(subject.camel(patched_string)).to eq :correct
+      end
+    end
+  end
 end
 
 class PatchedString < String
   def underscore
     :correct
   end
+
+  def camelize(sym)
+    :correct
+  end
 end

spec/tenkit/tenkit/weather_spec.rb

@@ -4,48 +4,59 @@
   let(:lat) { 37.32 }
   let(:lon) { 122.03 }
   let(:path) { "/weather/en/#{lat}/#{lon}" }
-  let(:data_sets) { [Tenkit::Utils.snake(data_set).to_sym] }
-  let(:query) { {query: {dataSets: data_set}} }
-
+  let(:params) { {dataSets: data_set} }
   let(:client) { Tenkit::Client.new }
-  let(:api_response) { double("WeatherResponse", body: json) }
   let(:json) { File.read("test/fixtures/#{data_set}.json") }
+  let(:resp) { double("WeatherResponse", body: json) }
+  let(:data_sets) { [Tenkit::Utils.snake(data_set).to_sym] }
 
-  before { allow(client).to receive(:get).and_return(api_response) }
+  before { expect(client).to receive(:get).with(path, {query: params}).and_return(resp) }
 
   describe "currentWeather" do
     let(:data_set) { "currentWeather" }
 
-    subject { client.weather(lat, lon, data_sets: data_sets).weather.current_weather }
+    context "with options" do
+      let(:fmt) { "%FT%TZ" }
+      let(:now) { Time.now }
+      let(:options) do
+        {country_code: "US",
+         current_as_of: now.strftime(fmt),
+         daily_end: (now + 12 * 3600).strftime(fmt),
+         daily_start: (now - 12 * 3600).strftime(fmt),
+         data_sets: data_sets,
+         hourly_end: (now + 6 * 3600).strftime(fmt),
+         hourly_start: (now - 6 * 3600).strftime(fmt),
+         timezone: "PST"}
+      end
+      let(:params) do
+        options.map { |k, v| [Tenkit::Utils.camel(k.to_s).to_sym, (k == :data_sets) ? data_set : v] }.to_h
+      end
 
-    it "returns response from correct data_sets" do
-      subject
-      expect(client).to have_received(:get).with(path, query)
-    end
+      subject { client.weather(lat, lon, **options).weather.current_weather }
 
-    it "includes expected metadata" do
-      expect(subject.name).to eq "CurrentWeather"
-      expect(subject.metadata.attribution_url).to start_with 'https://'
-      expect(subject.metadata.latitude).to be 37.32
-      expect(subject.metadata.longitude).to be 122.03
-    end
+      it "includes expected metadata" do
+        expect(subject.name).to eq "CurrentWeather"
+        expect(subject.metadata.attribution_url).to start_with "https://"
+        expect(subject.metadata.latitude).to be 37.32
+        expect(subject.metadata.longitude).to be 122.03
+      end
 
-    it "returns correct object types" do
-      expect(subject).to be_a Tenkit::CurrentWeather
-      expect(subject.metadata).to be_a Tenkit::Metadata
-    end
+      it "returns correct object types" do
+        expect(subject).to be_a Tenkit::CurrentWeather
+        expect(subject.metadata).to be_a Tenkit::Metadata
+      end
 
-    it "returns current weather data" do
-      expect(subject.temperature).to be(-5.68)
-      expect(subject.temperature_apparent).to be(-6.88)
+      it "returns current weather data" do
+        expect(subject.temperature).to be(-5.68)
+        expect(subject.temperature_apparent).to be(-6.88)
+      end
     end
 
     context "without options" do
       subject { client.weather(lat, lon).weather.current_weather }
 
-      it "returns response from default currentWeather data set" do
+      it "returns response" do
         expect(subject.name).to eq "CurrentWeather"
-        expect(client).to have_received(:get).with(path, query)
       end
     end
   end
@@ -56,9 +67,8 @@
 
     subject { client.weather(lat, lon, data_sets: data_sets).weather.forecast_daily }
 
-    it "returns 10 days of data from correct data sets" do
+    it "returns 10 days of data" do
       expect(subject.days.size).to be 10
-      expect(client).to have_received(:get).with(path, query)
     end
 
     it "returns correct object types" do
@@ -69,13 +79,9 @@
       expect(first_day.rest_of_day_forecast).to be_a Tenkit::RestOfDayForecast
     end
 
-    it "excludes learn_more_url node" do
-      expect(subject.respond_to? :learn_more_url).to be false
-    end
-
     it "includes expected metadata" do
       expect(subject.name).to eq "DailyForecast"
-      expect(subject.metadata.attribution_url).to start_with 'https://'
+      expect(subject.metadata.attribution_url).to start_with "https://"
       expect(subject.metadata.latitude).to be 37.32
       expect(subject.metadata.longitude).to be 122.03
     end
@@ -101,14 +107,13 @@
 
     subject { client.weather(lat, lon, data_sets: data_sets).weather.forecast_hourly }
 
-    it "returns 25 hours of data from correct data sets" do
+    it "returns 25 hours of data" do
       expect(subject.hours.size).to be 25
-      expect(client).to have_received(:get).with(path, query)
     end
 
     it "includes expected metadata" do
       expect(subject.name).to eq "HourlyForecast"
-      expect(subject.metadata.attribution_url).to start_with 'https://'
+      expect(subject.metadata.attribution_url).to start_with "https://"
       expect(subject.metadata.latitude).to be 37.32
       expect(subject.metadata.longitude).to be 122.03
     end