looker-sdk 0.0.7 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 34b933e51a973a90071e6232bf2146b380243f4f
-  data.tar.gz: 2bd895405ba590ac8c1c5e9a34841c90270b3c51
+SHA256:
+  metadata.gz: 03c0187c3befd7923934d6e0d6fdc2cf376cec20c3befab062e831266e191863
+  data.tar.gz: '082e731d16af22fa6ccdc3853c7451ff641b016c52ee25d76b6f52d1af6072bd'
 SHA512:
-  metadata.gz: e5998b8fa8b0daa43a9e55b8b65d0025b39d030b7593b08daad280c31a99ba7acfbf1fd36c73234be1ccc529ac1d4945db7efdb7860bd02b70db59c7e8086aa1
-  data.tar.gz: 0e3f84e946e5f6dfa09413188971736ba96a49dbd85c63adcb0328cbdb8f28f2f5acba1d5715b8b1fb7abddb02add35730323ea8afbf97084c22403354f2c264
+  metadata.gz: ed80dfc335ba667ee501a734f86d13788552e1903208bc36e8407354ea3aa51fd5bbc6fde6c0e58a728469781c95c975de555e246d75c82cb83484e07b8ba727
+  data.tar.gz: 21edf0dc58ebcd17102b9ec7c88006fdda9b229b46ee463d06c8a454ced09d06154a5e6fe8720a3de8dcbf2b819e16947cedb38d91e86aab2cdd60e36e492380

data/Gemfile

@@ -11,7 +11,7 @@ end
 
 group :test do
   # gem 'json', '~> 1.7', :platforms => [:jruby] look TODO needed?
-  gem 'minitest',         '5.3.5'
+  gem 'minitest',         '5.9.1'
   gem 'mocha',            '1.1.0'
   gem 'rack',             '1.6.4'
   gem 'rack-test',        '0.6.2'

data/Makefile

@@ -0,0 +1,81 @@
+#############################################################################################
+# The MIT License (MIT)
+#
+# Copyright (c) 2014 Zee Spencer
+# Copyright (c) 2020 Google LLC
+#
+# Permission is hereby granted, free of charge, to any person obtaining a copy
+# of this software and associated documentation files (the "Software"), to deal
+# in the Software without restriction, including without limitation the rights
+# to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+# copies of the Software, and to permit persons to whom the Software is
+# furnished to do so, subject to the following conditions:
+#
+# The above copyright notice and this permission notice shall be included in all
+# copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+# AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+# SOFTWARE.
+#############################################################################################
+
+# Allows running (and re-running) of tests against several ruby versions,
+# assuming you use rbenv instead of rvm.
+
+# Uses pattern rules (task-$:) and automatic variables ($*).
+# Pattern rules: http://ftp.gnu.org/old-gnu/Manuals/make-3.79.1/html_chapter/make_10.html#SEC98
+# Automatic variables: http://ftp.gnu.org/old-gnu/Manuals/make-3.79.1/html_chapter/make_10.html#SEC101
+
+# Rbenv-friendly version identifiers for supported Rubys
+25_version = 2.5.7
+jruby_92160_version = jruby-9.2.16.0
+
+# The ruby version for use in a given rule.
+# Requires a matched pattern rule and a supported ruby version.
+#
+# Given a pattern rule defined as "install-ruby-%"
+# When the rule is ran as "install-ruby-193"
+# Then the inner addsuffix call evaluates to "193_version"
+# And given_ruby_version becomes "1.9.3-p551"
+given_ruby_version = $($(addsuffix _version, $*))
+
+# Instruct rbenv on which Ruby version to use when running a command.
+# Requires a pattern rule and a supported ruby version.
+#
+# Given a pattern rule defined as "test-%"
+# When the rule is ran as "test-187"
+# Then with_given_ruby becomes "RBENV_VERSION=1.8.7-p375"
+with_given_ruby = RBENV_VERSION=$(given_ruby_version)
+
+# Runs tests for all supported ruby versions.
+test: test-25 test-jruby_92160
+
+# Runs tests against a specific ruby version
+test-%:
+	rm -f Gemfile.lock
+	$(with_given_ruby) bundle install --quiet
+	$(with_given_ruby) rake
+
+# Installs all ruby versions and their gems
+install: install-25 install-jruby_92160
+
+# Install a particular ruby version
+install-ruby-%:
+	rm -f Gemfile.lock
+	rbenv install -s $(given_ruby_version)
+
+# Install gems into a specific ruby version
+install-gems-%:
+	rm -f Gemfile.lock
+	$(with_given_ruby) gem update --system
+	$(with_given_ruby) gem install bundler
+	$(with_given_ruby) bundle install
+
+# Installs a specific ruby version and it's gems
+# At the bottom so it doesn't match install-gems and install-ruby tasks.
+install-%:
+	make install-ruby-$* install-gems-$*

data/Rakefile

@@ -22,7 +22,6 @@
 # THE SOFTWARE.
 ############################################################################################
 
-require 'bundler'
 require "bundler/gem_tasks"
 
 require "rake/testtask"
@@ -32,30 +31,4 @@ Rake::TestTask.new do |t|
   t.verbose = true
 end
 
-namespace :test do
-  desc "Run tests against all supported Rubies"
-  task :all do
-    supported_rubies = %w(ruby-2.0 ruby-2.1 ruby-2.3.1 jruby-1.7.19 jruby-9.1.5.0)
-    failing_rubies = []
-
-    supported_rubies.each do |ruby|
-      cmd = "rvm install #{ruby} && rvm #{ruby} exec gem install bundler && rvm #{ruby} exec bundle install && rvm #{ruby} exec bundle exec rake"
-      system cmd
-      if $? != 0
-        failing_rubies << ruby
-      end
-    end
-
-    failing_rubies.each do |ruby|
-      puts "FAIL: #{ruby}.  Problem with the tests on #{ruby}."
-    end
-
-    if failing_rubies
-      exit 1
-    else
-      exit 0
-    end
-  end
-end
-
 task :default => :test

data/authentication.md

@@ -22,7 +22,7 @@ Note that API 3 tokens should be created for 'regular' Looker users and *not* vi
 
 
 ### Ensure that the API is accessible
-Looker versions 3.4 (and beyond) expose the 3.0 API via a port different from the port used by the web app.
+Looker versions 3.4 (and beyond) expose the API via a port different from the port used by the web app.
 The default port is 19999. It may be necessary to have the Ops team managing the looker instance ensure that this
 port is made accessible network-wise to client software running on non-local hosts.
 
@@ -78,14 +78,14 @@ Content-Length: 99
 {"access_token":"4QDkCyCtZzYgj4C2p2cj3csJH7zqS5RzKs2kTnG4","token_type":"Bearer","expires_in":3600}
 
 # Use an access_token (the token can be used over and over for API calls until it expires)
-> curl -i -H "Authorization: token 4QDkCyCtZzYgj4C2p2cj3csJH7zqS5RzKs2kTnG4"  https://localhost:19999/api/3.0/user
+> curl -i -H "Authorization: token 4QDkCyCtZzYgj4C2p2cj3csJH7zqS5RzKs2kTnG4"  https://localhost:19999/api/4.0/user
 HTTP/1.1 200 OK
 Content-Type: application/json;charset=utf-8
 Vary: Accept-Encoding
 X-Content-Type-Options: nosniff
 Content-Length: 502
 
-{"id":14,"first_name":"Plain","last_name":"User","email":"dude+1@looker.com","models_dir":null,"is_disabled":false,"look_access":[14],"avatar_url":"https://www.gravatar.com/avatar/b7f792a6180a36a4058f36875584bc45?s=156&d=mm","credentials_email":{"email":"dude+1@looker.com","url":"https://localhost:19999/api/3.0/users/14/credentials_email","user_url":"https://localhost:19999/api/3.0/users/14","password_reset_url":"https://localhost:19999/api/3.0"},"url":"https://localhost:19999/api/3.0/users/14"}
+{"id":14,"first_name":"Plain","last_name":"User","email":"dude+1@looker.com","models_dir":null,"is_disabled":false,"look_access":[14],"avatar_url":"https://www.gravatar.com/avatar/b7f792a6180a36a4058f36875584bc45?s=156&d=mm","credentials_email":{"email":"dude+1@looker.com","url":"https://localhost:19999/api/4.0/users/14/credentials_email","user_url":"https://localhost:19999/api/4.0/users/14","password_reset_url":"https://localhost:19999/api/4.0"},"url":"https://localhost:19999/api/4.0/users/14"}
 
 # Logout to revoke an access_token
 > curl -i -X DELETE -H "Authorization: token 4QDkCyCtZzYgj4C2p2cj3csJH7zqS5RzKs2kTnG4"  https://localhost:19999/logout

data/lib/looker-sdk/authentication.rb

@@ -51,7 +51,7 @@ module LookerSDK
 
       set_access_token_from_params(nil)
       without_authentication do
-        encoded_auth = Faraday::Utils.build_query application_credentials
+        encoded_auth = Faraday::Utils.build_query(application_credentials)
         post("#{URI.parse(api_endpoint).path}/login", encoded_auth, header: {:'Content-Type' => 'application/x-www-form-urlencoded'})
         raise "login failure #{last_response.status}" unless last_response.status == 200
         set_access_token_from_params(last_response.data)

data/lib/looker-sdk/client/dynamic.rb

@@ -53,20 +53,17 @@ module LookerSDK
         @swagger ||= without_authentication { try_load_swagger }
 
         unless @swagger
-          # capture the bits we may need later, avoiding potential buffer reuse in last_response between requests
-          response_wo_auth_status = last_response&.status
-          response_wo_auth_data = last_response&.data
-
           # try again, this time with authentication
           @swagger = try_load_swagger
         end
 
         # in unit tests, @swagger may be nil and last_response nil because no HTTP request was made
-        if @swagger.nil? && (response_wo_auth_status || last_response)
-          msg = "Load of swagger.json failed. "
-          msg << "Without authentication HTTP response status: (#{response_wo_auth_status}) data: #{response_wo_auth_data}. " if response_wo_auth_status
-          msg << "WITH authentication HTTP response status: (#{last_response.status}) data: #{last_response.data}" if last_response
-          looker_warn(msg)
+        if @swagger.nil?
+          if @last_error
+            raise @last_error
+          else
+            raise "Load of swagger.json failed."
+          end
         end
 
         @swagger
@@ -75,12 +72,16 @@ module LookerSDK
       def operations
         return @@sharable_operations[api_endpoint] if shared_swagger && @@sharable_operations[api_endpoint]
 
+        if !@swagger && @lazy_swagger
+          load_swagger
+        end
+
         return nil unless @swagger
         @operations ||= Hash[
           @swagger[:paths].map do |path_name, path_info|
             path_info.map do |method, route_info|
               route = @swagger[:basePath].to_s + path_name.to_s
-              [route_info[:operationId], {:route => route, :method => method, :info => route_info}]
+              [route_info[:operationId].to_sym, {:route => route, :method => method, :info => route_info}]
             end
           end.reduce(:+)
         ].freeze
@@ -113,7 +114,7 @@ module LookerSDK
       private
 
       def find_entry(method_name)
-        operations && operations[method_name.to_s] if dynamic
+        operations && operations[method_name.to_sym] if dynamic
       end
 
       def invoke_remote(entry, method_name, *args, &block)
@@ -128,19 +129,25 @@ module LookerSDK
           raise ArgumentError.new("wrong number of arguments (#{params_passed} for #{params_required}) in call to '#{method_name}'. See '#{method_link(entry)}'")
         end
 
-        # substitute the actual params into the route template
-        params.each {|param| route.sub!("{#{param[:name]}}", args.shift.to_s) }
+        # substitute the actual params into the route template, encoding if needed
+        params.each do |param|
+          value = args.shift.to_s
+          if value == CGI.unescape(value)
+            value = CGI.escape(value)
+          end
+          route.sub!("{#{param[:name]}}", value)
+        end
 
         a = args.length > 0 ? args[0] : {}
         b = args.length > 1 ? args[1] : {}
 
         method = entry[:method].to_sym
         case method
-        when :get     then get(route, a, &block)
-        when :post    then post(route, a, merge_content_type_if_body(a, b), &block)
-        when :put     then put(route, a, merge_content_type_if_body(a, b), &block)
-        when :patch   then patch(route, a, merge_content_type_if_body(a, b), &block)
-        when :delete  then delete(route, a) ; @raw_responses ? last_response : delete_succeeded?
+        when :get     then get(route, a, true, &block)
+        when :post    then post(route, a, merge_content_type_if_body(a, b), true, &block)
+        when :put     then put(route, a, merge_content_type_if_body(a, b), true, &block)
+        when :patch   then patch(route, a, merge_content_type_if_body(a, b), true, &block)
+        when :delete  then delete(route, a, true) ; @raw_responses ? last_response : delete_succeeded?
         else raise "unsupported method '#{method}' in call to '#{method_name}'. See '#{method_link(entry)}'"
         end
       end

data/lib/looker-sdk/client.rb

@@ -58,7 +58,9 @@ module LookerSDK
       @original_options = options.dup
 
       load_credentials_from_netrc unless application_credentials?
-      load_swagger
+      if !@lazy_swagger
+        load_swagger
+      end
       self.dynamic = true
     end
 
@@ -89,11 +91,12 @@ module LookerSDK
     #
     # @param url [String] The path, relative to {#api_endpoint}
     # @param options [Hash] Query and header params for request
+    # @param encoded [Boolean] true: url already encoded, false: url needs encoding
     # @param &block [Block] Block to be called with |response, chunk| for each chunk of the body from
     #   the server. The block must return true to continue, or false to abort streaming.
     # @return [Sawyer::Resource]
-    def get(url, options = {}, &block)
-      request :get, url, nil, parse_query_and_convenience_headers(options), &block
+    def get(url, options = {}, encoded=false, &block)
+      request :get, url, nil, parse_query_and_convenience_headers(options), encoded, &block
     end
 
     # Make a HTTP POST request
@@ -101,11 +104,12 @@ module LookerSDK
     # @param url [String] The path, relative to {#api_endpoint}
     # @param data [String|Array|Hash] Body and optionally header params for request
     # @param options [Hash] Optional header params for request
+    # @param encoded [Boolean] true: url already encoded, false: url needs encoding
     # @param &block [Block] Block to be called with |response, chunk| for each chunk of the body from
     #   the server. The block must return true to continue, or false to abort streaming.
     # @return [Sawyer::Resource]
-    def post(url, data = {}, options = {}, &block)
-      request :post, url, data, parse_query_and_convenience_headers(options), &block
+    def post(url, data = {}, options = {}, encoded=false, &block)
+      request :post, url, data, parse_query_and_convenience_headers(options), encoded, &block
     end
 
     # Make a HTTP PUT request
@@ -113,11 +117,12 @@ module LookerSDK
     # @param url [String] The path, relative to {#api_endpoint}
     # @param data [String|Array|Hash] Body and optionally header params for request
     # @param options [Hash] Optional header params for request
+    # @param encoded [Boolean] true: url already encoded, false: url needs encoding
     # @param &block [Block] Block to be called with |response, chunk| for each chunk of the body from
     #   the server. The block must return true to continue, or false to abort streaming.
     # @return [Sawyer::Resource]
-    def put(url, data = {}, options = {}, &block)
-      request :put, url, data, parse_query_and_convenience_headers(options), &block
+    def put(url, data = {}, options = {}, encoded=false, &block)
+      request :put, url, data, parse_query_and_convenience_headers(options), encoded, &block
     end
 
     # Make a HTTP PATCH request
@@ -125,29 +130,32 @@ module LookerSDK
     # @param url [String] The path, relative to {#api_endpoint}
     # @param data [String|Array|Hash] Body and optionally header params for request
     # @param options [Hash] Optional header params for request
+    # @param encoded [Boolean] true: url already encoded, false: url needs encoding
     # @param &block [Block] Block to be called with |response, chunk| for each chunk of the body from
     #   the server. The block must return true to continue, or false to abort streaming.
     # @return [Sawyer::Resource]
-    def patch(url, data = {}, options = {}, &block)
-      request :patch, url, data, parse_query_and_convenience_headers(options), &block
+    def patch(url, data = {}, options = {}, encoded=false, &block)
+      request :patch, url, data, parse_query_and_convenience_headers(options), encoded, &block
     end
 
     # Make a HTTP DELETE request
     #
     # @param url [String] The path, relative to {#api_endpoint}
     # @param options [Hash] Query and header params for request
+    # @param encoded [Boolean] true: url already encoded, false: url needs encoding
     # @return [Sawyer::Resource]
-    def delete(url, options = {}, &block)
-      request :delete, url, nil, parse_query_and_convenience_headers(options)
+    def delete(url, options = {}, encoded=false, &block)
+      request :delete, url, nil, parse_query_and_convenience_headers(options), encoded, &block
     end
 
     # Make a HTTP HEAD request
     #
     # @param url [String] The path, relative to {#api_endpoint}
     # @param options [Hash] Query and header params for request
+    # @param encoded [Boolean] true: url already encoded, false: url needs encoding
     # @return [Sawyer::Resource]
-    def head(url, options = {}, &block)
-      request :head, url, nil, parse_query_and_convenience_headers(options)
+    def head(url, options = {}, encoded=false, &block)
+      request :head, url, nil, parse_query_and_convenience_headers(options), encoded
     end
 
     # Make one or more HTTP GET requests, optionally fetching
@@ -296,12 +304,16 @@ module LookerSDK
       @agent = nil
     end
 
-    def request(method, path, data, options, &block)
+    def request(method, path, data, options, encoded, &block)
       ensure_logged_in
       begin
+        path = path.to_s
+        if !encoded
+          path = URI::Parser.new.escape(path)
+        end
         @last_response = @last_error = nil
         return stream_request(method, path, data, options, &block) if block_given?
-        @last_response = response = agent.call(method, URI::Parser.new.escape(path.to_s), data, options)
+        @last_response = response = agent.call(method, path, data, options)
         @raw_responses ? response : response.data
       rescue StandardError => e
         @last_error = e

data/lib/looker-sdk/configurable.rb

@@ -66,7 +66,8 @@ module LookerSDK
     attr_accessor :access_token, :auto_paginate, :client_id,
                   :client_secret, :default_media_type, :connection_options,
                   :middleware, :netrc, :netrc_file,
-                  :per_page, :proxy, :user_agent, :faraday, :swagger, :shared_swagger, :raw_responses
+                  :per_page, :proxy, :user_agent, :faraday, :swagger, :shared_swagger, :raw_responses,
+                  :lazy_swagger
     attr_writer :web_endpoint, :api_endpoint
 
     class << self
@@ -92,7 +93,8 @@ module LookerSDK
           :shared_swagger,
           :swagger,
           :raw_responses,
-          :web_endpoint
+          :web_endpoint,
+          :lazy_swagger,
         ]
       end
     end

data/lib/looker-sdk/default.rb

@@ -125,6 +125,12 @@ module LookerSDK
         false
       end
 
+      # Default behavior for loading swagger during initialization or at first call
+      # @return [Boolean]
+      def lazy_swagger
+        false
+      end
+
       def raw_responses
         false
       end

data/lib/looker-sdk/response/raise_error.rb

@@ -30,9 +30,7 @@ module LookerSDK
   module Response
 
     # HTTP status codes returned by the API
-    class RaiseError < Faraday::Response::Middleware
-
-      private
+    class RaiseError < Faraday::Middleware
 
       def on_complete(response)
         if error = LookerSDK::Error.from_response(response)

data/lib/looker-sdk/version.rb

@@ -26,6 +26,6 @@ module LookerSDK
 
   # Current version
   # @return [String]
-  VERSION = "0.0.7"
+  VERSION = "0.1.0"
 
 end

data/lib/looker-sdk.rb

@@ -43,7 +43,6 @@ require 'faraday/rack_builder'
 require 'faraday/request'
 require 'faraday/request/authorization'
 require 'faraday/response'
-require 'faraday/upload_io'
 require 'faraday/utils'
 
 #require 'rack'

data/looker-sdk.gemspec

@@ -15,7 +15,7 @@ Gem::Specification.new do |s|
       'the Looker data analytics engine to fetch data or render visualizations defined in your Looker data models. '+
       'For more information, see https://looker.com.'
   s.license     = 'MIT'
-  s.required_ruby_version = '>= 2.0'
+  s.required_ruby_version = '>= 2.5'
   s.requirements = 'Looker version 4.0 or later'  # informational
 
   s.files         = `git ls-files`.split("\n")
@@ -24,5 +24,5 @@ Gem::Specification.new do |s|
   s.require_paths = %w(lib)
   s.add_dependency 'jruby-openssl' if s.platform == :jruby
   s.add_dependency 'sawyer', '~> 0.8'
-  s.add_dependency 'faraday', ['>= 0.9.0', '< 1.0']
+  s.add_dependency 'faraday', ['>= 1.0', '< 2.0']
 end

data/readme.md

@@ -23,10 +23,12 @@ $ rake install
 
 ### Development
 
+Customize test/fixtures/.netrc for tests to pass
+Comment out coverage configuration in test/helper.rb for debugging
 ```bash
 $ bundle install
 $ rake test # run the test suite
-$ rake test:all # run the test suite on all supported Rubies
+$ make install test # run the test suite on all supported Rubies
 ```
 
 ### Basic Usage
@@ -39,7 +41,7 @@ require 'looker-sdk'
 sdk = LookerSDK::Client.new(
   :client_id => "4CN7jzm7yrkcy2MC4CCG",
   :client_secret => "Js3rZZ7vHfbc2hBynSj7zqKh",
-  :api_endpoint => "https://mygreatcompany.looker.com:19999/api/3.0"
+  :api_endpoint => "https://mygreatcompany.looker.com:19999/api/4.0"
 )
 
 # If you don't want to provide explicit credentials: (trust me you don't)
@@ -53,7 +55,7 @@ sdk = LookerSDK::Client.new(
 sdk = LookerSDK::Client.new(
   :netrc      => true,
   :netrc_file => "~/.net_rc",
-  :api_endpoint => "https://mygreatcompany.looker.com:19999/api/3.0",
+  :api_endpoint => "https://mygreatcompany.looker.com:19999/api/4.0",
 
   # Set longer timeout to allow for long running queries. The default is 60 seconds and can be problematic.
   :connection_options => {:request => {:timeout => 60 * 60, :open_timeout => 30}},

data/test/helper.rb

@@ -57,6 +57,7 @@ fix_netrc_permissions(File.join(fixture_path, '.netrc'))
 def setup_sdk
   LookerSDK.reset!
   LookerSDK.configure do |c|
+    c.lazy_swagger = true
     c.connection_options = {:ssl => {:verify => false}}
     c.netrc = true
     c.netrc_file =  File.join(fixture_path, '.netrc')
@@ -64,7 +65,6 @@ def setup_sdk
 end
 
 def teardown_sdk
+  setup_sdk  # put back initial config
   LookerSDK.logout
 end
-
-

data/test/looker/swagger.json

@@ -5,7 +5,7 @@
     "title": "Looker API",
     "description": "This document is a representative sample (subset) of the Looker API that should only be used to unit test the Looker ruby sdk. To get the current, actual Looker API metadata visit /api/3.0/swagger.json on your Looker instance.",
     "contact": {
-      "name": "Looker Team <support@looker.com>"
+      "name": "Looker Team <https://help.looker.com>"
     },
     "license": {
       "name": "EULA",

data/test/looker/test_client.rb

@@ -30,8 +30,31 @@ describe LookerSDK::Client do
    setup_sdk
   end
 
-  after(:each) do
-   teardown_sdk
+  describe "lazy load swagger" do
+
+    it "lazy loads swagger" do
+      LookerSDK.reset!
+      client = LookerSDK::Client.new(
+        :lazy_swagger => true,
+        :netrc => true,
+        :netrc_file => File.join(fixture_path, '.netrc'),
+        :connection_options => {:ssl => {:verify => false}},
+      )
+      assert_nil client.swagger
+      client.me()
+      assert client.swagger
+    end
+
+    it "loads swagger initially" do
+      LookerSDK.reset!
+      client = LookerSDK::Client.new(
+        :lazy_swagger => false,
+        :netrc => true,
+        :netrc_file => File.join(fixture_path, '.netrc'),
+        :connection_options => {:ssl => {:verify => false}},
+      )
+      assert client.swagger
+    end
   end
 
   describe "module configuration" do
@@ -50,8 +73,9 @@ describe LookerSDK::Client do
     end
 
     it "inherits the module configuration" do
-      client = LookerSDK::Client.new
+      client = LookerSDK::Client.new(:lazy_swagger => true)
       LookerSDK::Configurable.keys.each do |key|
+        next if key == :lazy_swagger
         client.instance_variable_get(:"@#{key}").must_equal("Some #{key}")
       end
     end
@@ -63,7 +87,8 @@ describe LookerSDK::Client do
             :connection_options => {:ssl => {:verify => false}},
             :per_page => 40,
             :client_id    => "looker_client_id",
-            :client_secret => "client_secret2"
+            :client_secret => "client_secret2",
+            :lazy_swagger => true,
         }
       end
 
@@ -111,7 +136,11 @@ describe LookerSDK::Client do
       describe "with .netrc"  do
         it "can read .netrc files" do
           LookerSDK.reset!
-          client = LookerSDK::Client.new(:netrc => true, :netrc_file => File.join(fixture_path, '.netrc'))
+          client = LookerSDK::Client.new(
+            :lazy_swagger => true,
+            :netrc => true,
+            :netrc_file => File.join(fixture_path, '.netrc'),
+          )
           client.client_id.wont_be_nil
           client.client_secret.wont_be_nil
         end
@@ -122,6 +151,9 @@ describe LookerSDK::Client do
 
       before do
         LookerSDK.reset!
+        LookerSDK.configure do |c|
+          c.lazy_swagger = true
+        end
       end
 
       it "sets oauth token with .configure" do
@@ -232,6 +264,40 @@ describe LookerSDK::Client do
       end
     end
 
+    [
+        [:get, '/api/3.0/users/foo%2Fbar', false],
+        [:get, '/api/3.0/users/foo%252Fbar', true],
+        [:post, '/api/3.0/users/foo%2Fbar', false],
+        [:post, '/api/3.0/users/foo%252Fbar', true],
+        [:put, '/api/3.0/users/foo%2Fbar', false],
+        [:put, '/api/3.0/users/foo%252Fbar', true],
+        [:patch, '/api/3.0/users/foo%2Fbar', false],
+        [:patch, '/api/3.0/users/foo%252Fbar', true],
+        [:delete, '/api/3.0/users/foo%2Fbar', false],
+        [:delete, '/api/3.0/users/foo%252Fbar', true],
+        [:head, '/api/3.0/users/foo%2Fbar', false],
+        [:head, '/api/3.0/users/foo%252Fbar', true],
+    ].each do |method, path, encoded|
+      it "handles request path encoding" do
+        expected_path = '/api/3.0/users/foo%252Fbar'
+
+        resp = OpenStruct.new(:data => "hi", :status => 204)
+        mock = MiniTest::Mock.new.expect(:call, resp, [method, expected_path, nil, {}])
+        Sawyer::Agent.stubs(:new).returns(mock, mock)
+
+        sdk = LookerSDK::Client.new
+        if [:get, :delete, :head].include? method
+          args = [method, path, nil, encoded]
+        else
+          args = [method, path, nil, nil, encoded]
+        end
+        sdk.without_authentication do
+          value = sdk.public_send *args
+          assert_equal "hi", value
+        end
+        mock.verify
+      end
+    end
   end
 
   describe 'Sawyer date/time parsing patch' do

data/test/looker/test_dynamic_client.rb

@@ -36,6 +36,7 @@ class LookerDynamicClientTest < MiniTest::Spec
       conn.adapter :rack, engine
     end
 
+    LookerSDK.reset!
     LookerSDK::Client.new do |config|
       config.swagger = swagger
       config.access_token = access_token
@@ -92,41 +93,30 @@ class LookerDynamicClientTest < MiniTest::Spec
 
   describe "swagger" do
 
-    it "warns when swagger.json can't be loaded" do
-      sdk = sdk_client(nil, nil)
-
-      obj = OpenStruct.new({status: 200, data: {key: "my data"}})
-      sdk.stubs(:last_response).returns(obj)
-      sdk.stubs(:looker_warn).once
-
-      sdk.load_swagger
+    it "raises when swagger.json can't be loaded" do
+      mock = MiniTest::Mock.new.expect(:call, nil) {raise "no swagger for you"}
+      mock.expect(:call, nil) {raise "still no swagger for you"}
+      err = assert_raises(RuntimeError) { sdk_client(nil, mock) }
+      assert_equal "still no swagger for you", err.message
     end
 
-    it "no warning when swagger.json can be loaded with authentication" do
-      sdk = sdk_client(nil, nil)
-
-      # first call to try_load_swagger fails, returning nil
-      # second call succeeds
-      sdk.stubs(:try_load_swagger).returns(nil, {key: "my data"})
-      sdk.stubs(:looker_warn).never
-
-      sdk.load_swagger
+    it "loads swagger without authentication" do
+      resp = [200, {'Content-Type' => 'application/json'}, [default_swagger.to_json]]
+      mock = MiniTest::Mock.new.expect(:call, resp, [Hash])
+      sdk = sdk_client(nil, mock)
+      assert_equal default_swagger, sdk.swagger
     end
 
-    it "no warning when swagger.json can be loaded without authentication" do
-      sdk = sdk_client(nil, nil)
-
-      # first call to try_load_swagger fails, returning nil
-      # second call succeeds
-      sdk.stubs(:try_load_swagger).returns({key: "my data"}, nil)
-      sdk.stubs(:looker_warn).never
-
-      sdk.load_swagger
+    it "loads swagger with authentication" do
+      resp = [200, {'Content-Type' => 'application/json'}, [default_swagger.to_json]]
+      mock = MiniTest::Mock.new.expect(:call, nil) {raise "login first!"}
+      mock.expect(:call, resp, [Hash])
+      sdk = sdk_client(nil, mock)
+      assert_equal default_swagger, sdk.swagger
     end
 
     it "invalid method name" do
-      mock = MiniTest::Mock.new.expect(:call, response){|env| confirm_env(env, method, path, body, query, content_type)}
-      sdk = sdk_client(default_swagger, mock)
+      sdk = sdk_client(default_swagger, nil)
       assert_raises NoMethodError do
         sdk.this_method_name_doesnt_exist()
       end
@@ -136,18 +126,44 @@ class LookerDynamicClientTest < MiniTest::Spec
       end
     end
 
+    describe "operation maps" do
+      it "invoke by string operationId" do
+        verify(response, :get, '/api/3.0/user') do |sdk|
+          sdk.invoke('me')
+        end
+      end
+
+      it "invoke by symbol operationId" do
+        verify(response, :get, '/api/3.0/user') do |sdk|
+          sdk.invoke(:me)
+        end
+      end
+    end
+
     it "get no params" do
       verify(response, :get, '/api/3.0/user') do |sdk|
         sdk.me
       end
     end
 
-    it "get with parms" do
+    it "get with params" do
       verify(response, :get, '/api/3.0/users/25') do |sdk|
         sdk.user(25)
       end
     end
 
+    it "get with params that need encoding" do
+      verify(response, :get, '/api/3.0/users/foo%2Fbar') do |sdk|
+        sdk.user("foo/bar")
+      end
+    end
+
+    it "get with params already encoded" do
+        verify(response, :get, '/api/3.0/users/foo%2Fbar') do |sdk|
+        sdk.user("foo%2Fbar")
+      end
+    end
+
     it "get with query" do
       verify(response, :get, '/api/3.0/user', '', {bar:"foo"}) do |sdk|
         sdk.me({bar:'foo'})

metadata

@@ -1,49 +1,49 @@
 --- !ruby/object:Gem::Specification
 name: looker-sdk
 version: !ruby/object:Gem::Version
-  version: 0.0.7
+  version: 0.1.0
 platform: ruby
 authors:
 - Looker
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-12-12 00:00:00.000000000 Z
+date: 2021-08-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
+  name: sawyer
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.8'
-  name: sawyer
-  prerelease: false
   type: :runtime
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.8'
 - !ruby/object:Gem::Dependency
+  name: faraday
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.9.0
+        version: '1.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '1.0'
-  name: faraday
-  prerelease: false
+        version: '2.0'
   type: :runtime
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.9.0
+        version: '1.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.0'
 description: Use this SDK to access the Looker API. The Looker API provides functions
   to perform administrative tasks such as provisioning users, configuring database
   connections, and so on. It also enables you to leverage the Looker data analytics
@@ -60,6 +60,7 @@ files:
 - CODE_OF_CONDUCT.md
 - Gemfile
 - LICENSE.md
+- Makefile
 - Rakefile
 - authentication.md
 - examples/.netrc
@@ -110,7 +111,7 @@ homepage: https://github.com/looker/looker-sdk-ruby
 licenses:
 - MIT
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -118,7 +119,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.0'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -126,9 +127,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements:
 - Looker version 4.0 or later
-rubyforge_project:
-rubygems_version: 2.6.14.1
-signing_key:
+rubygems_version: 3.2.25
+signing_key: 
 specification_version: 4
 summary: Looker Ruby SDK
 test_files: []