dce_lti 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0943e97f6d2613ad998906f27e159dcc604171d8
-  data.tar.gz: b7b5e43868381971bb4977435a029306d5352db1
+  metadata.gz: 93969389a33a1c7eabee7af327fd8f3b38df303e
+  data.tar.gz: 9a042ba6f70ac543103a9c66087829f6db3cf672
 SHA512:
-  metadata.gz: f811c1fd2ddebd80675ee7f57dae0207363e8ec4e3c3d979e4496ad60dc403f9abb265025ff3f18d2e4e9049aca11e1a71c4afcc714991b46025ffff1c1a72d3
-  data.tar.gz: de233030855e214740d63d92b742802156d9821685bb92aeadc0ff3d5c36cea3592d818d86dafbf7cb91ff364b56402d32242b9bc730db9644063d240e1cb6be
+  metadata.gz: e71bb0b7f2f05bc3f360f194857efa660dbf37a1e06fd14ad2b48198317ed64a0877fd696f6aa7151137a6074f027919c1ee0dcc204539aef550ce0632ecdf94
+  data.tar.gz: e43d735ebe797fde5a6f78b0f5e81c8390bedba4df62c667fb44632e2cd732dd2ad638adc4ef5c6d5ebd58c66ded71bc48d155086f37e9f1a21813ed305f574b

data/README.md

@@ -128,7 +128,8 @@ These values come from the LTI values posted by the consumer.
 By default, a successful launch will redirect to your application's
 `root_path`. This is configured via `redirect_after_successful_auth` which is
 evaluated in engine controller context. This method should have access to
-`current_user`, rails route helpers and other controller-specific context.
+`current_user`, rails route helpers and other controller-specific context
+through the controller instance passed to it.
 
 ### Invalid LTI Sessions
 

data/app/concerns/dce_lti/session_helpers.rb

@@ -28,7 +28,7 @@ module DceLti
     end
 
     def redirect_after_successful_auth
-      Engine.config.redirect_after_successful_auth.call
+      Engine.config.redirect_after_successful_auth.call(self)
     end
 
     def tool_provider

data/lib/dce_lti/engine.rb

@@ -17,7 +17,7 @@ launch_presentation_return_url
       config.provider_title = (ENV['LTI_PROVIDER_TITLE'] || 'DCE LTI Provider')
       config.provider_description = (ENV['LTI_PROVIDER_DESCRIPTION'] || 'A description of this')
 
-      config.redirect_after_successful_auth = -> do
+      config.redirect_after_successful_auth = -> (controller) do
         Rails.application.routes.url_helpers.root_path
       end
       config.tool_config_extensions = ->(*) {}

data/lib/dce_lti/version.rb

@@ -1,3 +1,3 @@
 module DceLti
-  VERSION = "0.3.0"
+  VERSION = "0.4.0"
 end

data/spec/controllers/dce_lti/sessions_controller_spec.rb

@@ -124,7 +124,11 @@ module DceLti
       context 'valid LTI requests' do
         it 'redirects to "redirect_after_successful_auth" url when it is a proc' do
           url = '/sessions/create'
-          after_auth_url = ->{ url }
+          allow(controller).to receive(:canary_method)
+          after_auth_url = ->(controller) do
+            controller.canary_method
+            url
+          end
 
           with_overridden_lti_config_of(lti_config.merge(redirect_after_successful_auth: after_auth_url)) do
             tool_provider = stub_successful_tool_provider
@@ -132,6 +136,7 @@ module DceLti
             post_to_create_with_params
 
             expect(request).to redirect_to(url)
+            expect(controller).to have_received(:canary_method)
           end
         end
 

data/spec/dummy/config/initializers/dce_lti_config.rb

@@ -5,7 +5,7 @@ DceLti::Engine.setup do |lti|
   #
   # lti.provider_title = (ENV['LTI_PROVIDER_TITLE'] || 'DCE LTI Provider')
   # lti.provider_description = (ENV['LTI_PROVIDER_DESCRIPTION'] || 'A description of this')
-  # lti.redirect_after_successful_auth = ->{ Rails.application.routes.url_helpers.root_path }
+  # lti.redirect_after_successful_auth = ->(controller) { Rails.application.routes.url_helpers.root_path }
 
   lti.consumer_secret = (ENV['LTI_CONSUMER_SECRET'] || 'consumer_secret')
   lti.consumer_key = (ENV['LTI_CONSUMER_KEY'] || 'consumer_key')

data/spec/dummy/log/development.log

@@ -2928,3 +2928,58 @@ Completed 200 OK in 3ms (Views: 0.4ms | ActiveRecord: 0.6ms)
    (0.2ms)  SELECT version FROM "schema_migrations"
    (1.0ms)  INSERT INTO "schema_migrations" (version) VALUES ('20141008172001')
    (1.1ms)  INSERT INTO "schema_migrations" (version) VALUES ('20141003180140')
+   (111.0ms)  DROP DATABASE IF EXISTS "dce_lti_dummy_test"
+   (243.2ms)  CREATE DATABASE "dce_lti_dummy_test" ENCODING = 'utf8'
+  SQL (0.4ms)  CREATE EXTENSION IF NOT EXISTS "plpgsql"
+   (16.4ms)  CREATE TABLE "dce_lti_nonces" ("id" serial primary key, "nonce" character varying(255), "created_at" timestamp, "updated_at" timestamp) 
+   (7.6ms)  CREATE UNIQUE INDEX  "index_dce_lti_nonces_on_nonce" ON "dce_lti_nonces" USING btree ("nonce")
+   (13.0ms)  CREATE TABLE "dce_lti_users" ("id" serial primary key, "lti_user_id" character varying(255), "lis_person_contact_email_primary" character varying(255), "lis_person_name_family" character varying(255), "lis_person_name_full" character varying(255), "lis_person_name_given" character varying(255), "lis_person_sourcedid" character varying(255), "user_image" character varying(255), "roles" character varying(255)[] DEFAULT '{}', "created_at" timestamp, "updated_at" timestamp) 
+   (1.8ms)  CREATE TABLE "schema_migrations" ("version" character varying(255) NOT NULL) 
+   (3.8ms)  CREATE UNIQUE INDEX  "unique_schema_migrations" ON "schema_migrations"  ("version")
+   (0.4ms)  SELECT version FROM "schema_migrations"
+   (1.3ms)  INSERT INTO "schema_migrations" (version) VALUES ('20141008172001')
+   (1.2ms)  INSERT INTO "schema_migrations" (version) VALUES ('20141003180140')
+   (109.1ms)  DROP DATABASE IF EXISTS "dce_lti_dummy_test"
+   (238.0ms)  CREATE DATABASE "dce_lti_dummy_test" ENCODING = 'utf8'
+  SQL (0.3ms)  CREATE EXTENSION IF NOT EXISTS "plpgsql"
+   (10.0ms)  CREATE TABLE "dce_lti_nonces" ("id" serial primary key, "nonce" character varying(255), "created_at" timestamp, "updated_at" timestamp) 
+   (4.1ms)  CREATE UNIQUE INDEX  "index_dce_lti_nonces_on_nonce" ON "dce_lti_nonces" USING btree ("nonce")
+   (9.3ms)  CREATE TABLE "dce_lti_users" ("id" serial primary key, "lti_user_id" character varying(255), "lis_person_contact_email_primary" character varying(255), "lis_person_name_family" character varying(255), "lis_person_name_full" character varying(255), "lis_person_name_given" character varying(255), "lis_person_sourcedid" character varying(255), "user_image" character varying(255), "roles" character varying(255)[] DEFAULT '{}', "created_at" timestamp, "updated_at" timestamp) 
+   (1.7ms)  CREATE TABLE "schema_migrations" ("version" character varying(255) NOT NULL) 
+   (3.9ms)  CREATE UNIQUE INDEX  "unique_schema_migrations" ON "schema_migrations"  ("version")
+   (0.4ms)  SELECT version FROM "schema_migrations"
+   (2.5ms)  INSERT INTO "schema_migrations" (version) VALUES ('20141008172001')
+   (2.5ms)  INSERT INTO "schema_migrations" (version) VALUES ('20141003180140')
+   (105.9ms)  DROP DATABASE IF EXISTS "dce_lti_dummy_test"
+   (228.5ms)  CREATE DATABASE "dce_lti_dummy_test" ENCODING = 'utf8'
+  SQL (0.2ms)  CREATE EXTENSION IF NOT EXISTS "plpgsql"
+   (8.0ms)  CREATE TABLE "dce_lti_nonces" ("id" serial primary key, "nonce" character varying(255), "created_at" timestamp, "updated_at" timestamp) 
+   (7.2ms)  CREATE UNIQUE INDEX  "index_dce_lti_nonces_on_nonce" ON "dce_lti_nonces" USING btree ("nonce")
+   (13.3ms)  CREATE TABLE "dce_lti_users" ("id" serial primary key, "lti_user_id" character varying(255), "lis_person_contact_email_primary" character varying(255), "lis_person_name_family" character varying(255), "lis_person_name_full" character varying(255), "lis_person_name_given" character varying(255), "lis_person_sourcedid" character varying(255), "user_image" character varying(255), "roles" character varying(255)[] DEFAULT '{}', "created_at" timestamp, "updated_at" timestamp) 
+   (2.8ms)  CREATE TABLE "schema_migrations" ("version" character varying(255) NOT NULL) 
+   (7.6ms)  CREATE UNIQUE INDEX  "unique_schema_migrations" ON "schema_migrations"  ("version")
+   (0.3ms)  SELECT version FROM "schema_migrations"
+   (2.5ms)  INSERT INTO "schema_migrations" (version) VALUES ('20141008172001')
+   (2.5ms)  INSERT INTO "schema_migrations" (version) VALUES ('20141003180140')
+   (104.7ms)  DROP DATABASE IF EXISTS "dce_lti_dummy_test"
+   (235.8ms)  CREATE DATABASE "dce_lti_dummy_test" ENCODING = 'utf8'
+  SQL (0.1ms)  CREATE EXTENSION IF NOT EXISTS "plpgsql"
+   (9.8ms)  CREATE TABLE "dce_lti_nonces" ("id" serial primary key, "nonce" character varying(255), "created_at" timestamp, "updated_at" timestamp) 
+   (7.3ms)  CREATE UNIQUE INDEX  "index_dce_lti_nonces_on_nonce" ON "dce_lti_nonces" USING btree ("nonce")
+   (7.0ms)  CREATE TABLE "dce_lti_users" ("id" serial primary key, "lti_user_id" character varying(255), "lis_person_contact_email_primary" character varying(255), "lis_person_name_family" character varying(255), "lis_person_name_full" character varying(255), "lis_person_name_given" character varying(255), "lis_person_sourcedid" character varying(255), "user_image" character varying(255), "roles" character varying(255)[] DEFAULT '{}', "created_at" timestamp, "updated_at" timestamp) 
+   (1.5ms)  CREATE TABLE "schema_migrations" ("version" character varying(255) NOT NULL) 
+   (3.4ms)  CREATE UNIQUE INDEX  "unique_schema_migrations" ON "schema_migrations"  ("version")
+   (0.2ms)  SELECT version FROM "schema_migrations"
+   (1.0ms)  INSERT INTO "schema_migrations" (version) VALUES ('20141008172001')
+   (1.0ms)  INSERT INTO "schema_migrations" (version) VALUES ('20141003180140')
+   (106.0ms)  DROP DATABASE IF EXISTS "dce_lti_dummy_test"
+   (234.0ms)  CREATE DATABASE "dce_lti_dummy_test" ENCODING = 'utf8'
+  SQL (0.2ms)  CREATE EXTENSION IF NOT EXISTS "plpgsql"
+   (15.9ms)  CREATE TABLE "dce_lti_nonces" ("id" serial primary key, "nonce" character varying(255), "created_at" timestamp, "updated_at" timestamp) 
+   (7.0ms)  CREATE UNIQUE INDEX  "index_dce_lti_nonces_on_nonce" ON "dce_lti_nonces" USING btree ("nonce")
+   (13.1ms)  CREATE TABLE "dce_lti_users" ("id" serial primary key, "lti_user_id" character varying(255), "lis_person_contact_email_primary" character varying(255), "lis_person_name_family" character varying(255), "lis_person_name_full" character varying(255), "lis_person_name_given" character varying(255), "lis_person_sourcedid" character varying(255), "user_image" character varying(255), "roles" character varying(255)[] DEFAULT '{}', "created_at" timestamp, "updated_at" timestamp) 
+   (2.6ms)  CREATE TABLE "schema_migrations" ("version" character varying(255) NOT NULL) 
+   (7.1ms)  CREATE UNIQUE INDEX  "unique_schema_migrations" ON "schema_migrations"  ("version")
+   (0.2ms)  SELECT version FROM "schema_migrations"
+   (1.2ms)  INSERT INTO "schema_migrations" (version) VALUES ('20141008172001')
+   (1.1ms)  INSERT INTO "schema_migrations" (version) VALUES ('20141003180140')