canvas_oauth_engine 2.1.2 → 2.1.3

data/spec/lib/canvas_oauth/canvas_api_spec.rb

@@ -271,6 +271,13 @@ describe CanvasOauth::CanvasApi do
           expect(canvas).to receive(:authenticated_get).exactly(3).times.and_return(created, running, aborted)
           canvas.get_report(1, :provisioning_csv, params)
         end
+
+        it "uses the default UTF 8 parser it its get call" do
+          allow(canvas).to receive(:authenticated_get).and_return(complete)
+          expect(canvas).to receive(:authenticated_post).with("/api/v1/accounts/1/reports/provisioning_csv", { body: params })
+          expect(CanvasOauth::CanvasApi).to receive(:get).with(file['url'], hash_including(parser: CanvasOauth::DefaultUTF8Parser)).and_return(response)
+          canvas.get_report(1, :provisioning_csv, params)
+        end
       end
     end
   end

data/spec/lib/canvas_oauth/default_utf8_parser_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+describe CanvasOauth::DefaultUTF8Parser do
+  describe "#parse" do
+    it "forces the body passed in to UTF-8 if it is ASCII_8BIT" do
+      body = String.new("I am some text", encoding: Encoding::ASCII_8BIT)
+
+      parser = CanvasOauth::DefaultUTF8Parser.new(body, "text/plain")
+      expect(body).to receive(:force_encoding).with("UTF-8")
+      parser.parse
+    end
+
+    it "does not force the body passed in to UTF-8 if not ASCII_8BIT" do
+      body = String.new("I am some text", encoding: Encoding::US_ASCII)
+
+      parser = CanvasOauth::DefaultUTF8Parser.new(body, "text/plain")
+      expect(body).not_to receive(:force_encoding)
+      parser.parse
+    end
+  end
+end

metadata

@@ -1,16 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: canvas_oauth_engine
 version: !ruby/object:Gem::Version
-  version: 2.1.2
+  version: 2.1.3
 platform: ruby
 authors:
 - Dave Donahue
 - Adam Anderson
 - Simon Williams
+- Keith Garner
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-11-22 00:00:00.000000000 Z
+date: 2020-02-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -38,14 +39,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.9.0
+        version: 0.17.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.9.0
+        version: 0.17.0
 - !ruby/object:Gem::Dependency
   name: link_header
   requirement: !ruby/object:Gem::Requirement
@@ -186,6 +187,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 3.0.6
+- !ruby/object:Gem::Dependency
+  name: sprockets
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: rb-fsevent
   requirement: !ruby/object:Gem::Requirement
@@ -216,8 +231,8 @@ dependencies:
         version: '0'
 description: 
 email:
-- adam.anderson@12spokes.com
 - simon@instructure.com
+- kgarner@instructure.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -237,6 +252,7 @@ files:
 - lib/canvas_oauth/canvas_application.rb
 - lib/canvas_oauth/canvas_config.rb
 - lib/canvas_oauth/config.rb
+- lib/canvas_oauth/default_utf8_parser.rb
 - lib/canvas_oauth/engine.rb
 - lib/canvas_oauth/version.rb
 - lib/tasks/canvas_oauth_tasks.rake
@@ -262,9 +278,11 @@ files:
 - spec/dummy/config/initializers/wrap_parameters.rb
 - spec/dummy/config/locales/en.yml
 - spec/dummy/config/routes.rb
+- spec/dummy/db/development.sqlite3
 - spec/dummy/db/migrate/20160711200737_create_canvas_oauth_authorizations.canvas_oauth.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/db/test.sqlite3
+- spec/dummy/log/development.log
 - spec/dummy/log/test.log
 - spec/dummy/public/404.html
 - spec/dummy/public/422.html
@@ -274,6 +292,7 @@ files:
 - spec/dummy/script/rails
 - spec/lib/canvas_oauth/canvas_api_extensions_spec.rb
 - spec/lib/canvas_oauth/canvas_api_spec.rb
+- spec/lib/canvas_oauth/default_utf8_parser_spec.rb
 - spec/models/canvas_oauth/authorization_spec.rb
 - spec/spec_helper.rb
 homepage: ''
@@ -331,9 +350,12 @@ test_files:
 - spec/dummy/db/schema.rb
 - spec/dummy/db/test.sqlite3
 - spec/dummy/db/migrate/20160711200737_create_canvas_oauth_authorizations.canvas_oauth.rb
+- spec/dummy/db/development.sqlite3
 - spec/dummy/log/test.log
+- spec/dummy/log/development.log
 - spec/dummy/README.rdoc
 - spec/models/canvas_oauth/authorization_spec.rb
 - spec/lib/canvas_oauth/canvas_api_extensions_spec.rb
+- spec/lib/canvas_oauth/default_utf8_parser_spec.rb
 - spec/lib/canvas_oauth/canvas_api_spec.rb
 - spec/controllers/canvas_oauth/canvas_controller_spec.rb