coalescing_panda 2.0.0 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8ec1b5c9bfc79b28575ac9681953bc1589aaec58
-  data.tar.gz: 9cae9d48dbb29ed1ccaeab79498b293caee9653a
+  metadata.gz: e1fe754bf45b22224fa5d6800b9e70fd6aa3b262
+  data.tar.gz: e9fefae84355b40fa9f737901a691d8a3967021c
 SHA512:
-  metadata.gz: 35b2bde2067e5ce0e3c2eb863caa36c047ec88c6892b4a9590a9affbd34fc4199433759096f55c03d84b47710b8ad435f69f1ffb62e583a3a81d68edb3981d13
-  data.tar.gz: 855909206eebdcc7625cc2a0e21fce3966f6015b4bd441433cd7e59716489c75226982a44ee6e60fd408da655e6f68934e29995cea13ae4cf39201b5fc05dc75
+  metadata.gz: 8225718e3d84b9b558810dd44e689cb782398b38c6e7a3cb9716a2ada0ac74472eb9d3c9481da2c6ff54521000ffc0d5537eb61ae9e4a701fe6f9a7b97a6b0c0
+  data.tar.gz: 5dfc9a3752d0324076328907b059fe95a2af957e90bbf665808bdfac339c6e3535b8355de4b1ec352adf14e0cacd991206a01ed517d28dd3d0cb1a152dfc51f9

data/app/controllers/coalescing_panda/lti_controller.rb

@@ -13,6 +13,7 @@ module CoalescingPanda
       tc = IMS::LTI::ToolConfig.new(:title => lti_options[:title], :launch_url => ("#{host}#{lti_options[:launch_route]}") || 'ABC')
       tc.set_ext_param(platform, :domain, request.host)
       tc.set_ext_param(platform, :privacy_level, 'public')
+      tc.set_custom_param(:custom_canvas_role, '$Canvas.membership.roles')
       if lti_options.has_key?(:custom_fields)
         tc.set_ext_param(platform, :custom_fields, lti_options[:custom_fields])
         lti_options[:custom_fields].each do |k, v|

data/app/models/coalescing_panda/assignment.rb

@@ -0,0 +1,8 @@
+module CoalescingPanda
+  class Assignment < ActiveRecord::Base
+    belongs_to :course, foreign_key: :coalescing_panda_course_id, class_name: 'CoalescingPanda::Course'
+    has_many :submissions, foreign_key: :coalescing_panda_assignment_id, class_name: 'CoalescingPanda::Submission'
+
+    delegate :account, to: :course
+  end
+end

data/app/models/coalescing_panda/canvas_api_auth.rb

@@ -2,8 +2,6 @@ module CoalescingPanda
   class CanvasApiAuth < ActiveRecord::Base
     validates :user_id, :api_domain, presence: true
     validates :user_id, uniqueness: {scope: :api_domain}
-
-    attr_accessible :user_id, :api_domain
   end
 
 end

data/app/models/coalescing_panda/course.rb

@@ -0,0 +1,11 @@
+module CoalescingPanda
+  class Course < ActiveRecord::Base
+    belongs_to :account, foreign_key: :coalescing_panda_lti_account_id, class_name: 'CoalescingPanda::LtiAccount'
+    belongs_to :term, foreign_key: :coalescing_panda_term_id, class_name: 'CoalescingPanda::Term'
+    has_many :sections, foreign_key: :coalescing_panda_course_id, class_name: 'CoalescingPanda::Section'
+    has_many :enrollments, through: :sections
+    has_many :assignments, foreign_key: :coalescing_panda_course_id, class_name: 'CoalescingPanda::Assignment'
+    has_many :submissions, through: :assignments
+    has_many :users, through: :sections, source: :users, class_name: 'CoalescingPanda::User'
+  end
+end

data/app/models/coalescing_panda/enrollment.rb

@@ -0,0 +1,8 @@
+module CoalescingPanda
+  class Enrollment < ActiveRecord::Base
+    belongs_to :user, foreign_key: :coalescing_panda_user_id, class_name: 'CoalescingPanda::User'
+    belongs_to :section, foreign_key: :coalescing_panda_section_id, class_name: 'CoalescingPanda::Section'
+
+    delegate :account, to: :section
+  end
+end

data/app/models/coalescing_panda/lti_account.rb

@@ -2,24 +2,28 @@ module CoalescingPanda
   class LtiAccount < ActiveRecord::Base
     validates :name, :key, uniqueness: true
     validates :name, :key, :secret, presence: true
-    has_many :coalescing_panda_lti_nonces,
-             :foreign_key => :coalescing_panda_lti_account_id,
-             :class_name => 'CoalescingPanda::LtiNonce'
+    has_many :lti_nonces, foreign_key: :coalescing_panda_lti_account_id, class_name: 'CoalescingPanda::LtiNonce'
+    has_many :terms, foreign_key: :coalescing_panda_lti_account_id, class_name: 'CoalescingPanda::Term'
+    has_many :courses, foreign_key: :coalescing_panda_lti_account_id, class_name: 'CoalescingPanda::Course'
+    has_many :users, foreign_key: :coalescing_panda_lti_account_id, class_name: 'CoalescingPanda::User'
+    has_many :sections, through: :courses
+    has_many :enrollments, through: :sections
+    has_many :assignments, through: :courses
+    has_many :submissions, through: :assignments
 
-    attr_accessible :name, :key, :secret, :oauth2_client_id, :oauth2_client_key, :settings
     serialize :settings
 
     def validate_nonce(nonce, timestamp)
       cleanup_nonce
       if timestamp > 15.minutes.ago
-        coalescing_panda_lti_nonces.create(nonce: nonce, timestamp: timestamp).persisted?
+        lti_nonces.create(nonce: nonce, timestamp: timestamp).persisted?
       end
     end
 
     private
 
     def cleanup_nonce
-      coalescing_panda_lti_nonces.where('timestamp > ?', 15.minutes.ago).delete_all
+      lti_nonces.where('timestamp > ?', 15.minutes.ago).delete_all
     end
 
   end

data/app/models/coalescing_panda/lti_nonce.rb

@@ -3,8 +3,6 @@ module CoalescingPanda
     validates :coalescing_panda_lti_account, :nonce, :timestamp, :presence => true
     validates :nonce, uniqueness: {scope: :coalescing_panda_lti_account}
     belongs_to :coalescing_panda_lti_account, :class_name => 'CoalescingPanda::LtiAccount'
-
-    attr_accessible :nonce, :timestamp
   end
 
   def cleanup

data/app/models/coalescing_panda/section.rb

@@ -0,0 +1,9 @@
+module CoalescingPanda
+  class Section < ActiveRecord::Base
+    belongs_to :course, foreign_key: :coalescing_panda_course_id, class_name: 'CoalescingPanda::Course'
+    has_many :enrollments, foreign_key: :coalescing_panda_section_id, class_name: 'CoalescingPanda::Enrollment'
+    has_many :users, through: :enrollments, class_name: 'CoalescingPanda::User'
+
+    delegate :account, to: :course
+  end
+end

data/app/models/coalescing_panda/session.rb

@@ -1,6 +1,5 @@
 module CoalescingPanda
   class Session < ActiveRecord::Base
-    attr_accessible :token, :data
     serialize :data, Hash
 
     def self.create_from_session(session)

data/app/models/coalescing_panda/submission.rb

@@ -0,0 +1,8 @@
+module CoalescingPanda
+  class Submission < ActiveRecord::Base
+    belongs_to :user, foreign_key: :coalescing_panda_user_id, class_name: 'CoalescingPanda::User'
+    belongs_to :assignment, foreign_key: :coalescing_panda_assignment_id, class_name: 'CoalescingPanda::Assignment'
+
+    delegate :account, to: :assignment
+  end
+end

data/app/models/coalescing_panda/term.rb

@@ -0,0 +1,6 @@
+module CoalescingPanda
+  class Term < ActiveRecord::Base
+    belongs_to :account, foreign_key: :coalescing_panda_lti_account_id, class_name: 'CoalescingPanda::LtiAccount'
+    has_many :courses, foreign_key: :coalescing_panda_term_id, class_name: 'CoalescingPanda::Course'
+  end
+end

data/app/models/coalescing_panda/user.rb

@@ -0,0 +1,9 @@
+module CoalescingPanda
+  class User < ActiveRecord::Base
+    belongs_to :account, foreign_key: :coalescing_panda_lti_account_id, class_name: 'CoalescingPanda::LtiAccount'
+    has_many :enrollments, foreign_key: :coalescing_panda_user_id, class_name: 'CoalescingPanda::Enrollment'
+    has_many :submissions, foreign_key: :coalescing_panda_user_id, class_name: 'CoalescingPanda::Submission'
+    has_many :sections, through: :enrollments
+    has_many :courses, through: :sections
+  end
+end

data/app/models/coalescing_panda/workers/course_miner.rb

@@ -0,0 +1,91 @@
+class CoalescingPanda::Workers::CourseMiner
+  SUPPORTED_MODELS = [:sections, :users, :enrollments, :assignments, :submissions] #ORDER MATTERS!!
+
+  attr_accessor :api_client, :options, :account, :course, :canvas_account_id
+
+  def initialize(course, options = [])
+    @course = course
+    @account = course.account
+    @canvas_account_id = course.account.canvas_account_id
+    @api_client = Bearcat::Client.new(prefix: account.settings[:base_url], token: account.settings[:account_admin_api_token])
+    @options = options
+  end
+
+  def start
+    SUPPORTED_MODELS.each do |model_key|
+      canvas_model_data(api_method(model_key.to_sym), model_key) if options.include?(model_key)
+    end
+  end
+
+  def api_method(key)
+    case key
+    when :users
+      :list_course_users
+    when :sections
+      :course_sections
+    when :enrollments
+      :course_enrollments
+    when :assignments
+      :assignments
+    when :submissions
+      :get_course_submission
+    else
+      raise "API METHOD DOESN'T EXIST"
+    end
+  end
+
+  def canvas_model_data(method, model_key)
+    if model_key == :submissions
+      collection = []
+      course.assignments.each do |assignment|
+        api_client.get_course_submissions(course.canvas_course_id, assignment.canvas_assignment_id).all_pages!.each do |submissions|
+          collection << submissions
+        end
+      end
+      create_records(collection, model_key)
+    else
+      collection = api_client.send(method, course.canvas_course_id).all_pages!
+      create_records(collection, model_key)
+    end
+  end
+
+  def create_records(collection, model_key)
+    model = "CoalescingPanda::#{model_key.to_s.singularize.titleize}".constantize
+    collection.each do |values|
+      canvas_id_key = "canvas_#{model_key.to_s.singularize}_id"
+      values[canvas_id_key] = values["id"]
+      record = course.send(model_key).where("#{canvas_id_key} = ?", values['id'].to_s).first_or_initialize
+      record.coalescing_panda_lti_account_id = account.id if record.respond_to?(:coalescing_panda_lti_account_id)
+      record.assign_attributes(standard_attributes(record, values))
+      record.sis_id = sis_id(model_key, values) if record.respond_to?(:sis_id)
+      create_associations(record, model_key, values)
+      record.save
+    end
+  end
+
+  def standard_attributes(record, attributes)
+    new_attributes = attributes.dup
+    new_attributes.delete('id')
+    new_attributes.delete_if { |key, value| !record.attributes.include?(key) }
+  end
+
+  def sis_id(model_key, values)
+    return values['sis_source_id'] if values.has_key?('sis_source_id')
+    return values['sis_section_id'] if model_key == :sections
+  end
+
+  def create_associations(record, model_key, values)
+    case model_key
+    when :users
+      record.account = account
+    when :enrollments
+      record.user = account.users.where(canvas_user_id: values['user_id'].to_s).first
+      record.section = course.sections.where(canvas_section_id: values['course_section_id'].to_s).first
+    when :assignments
+      record.course = course
+    when :submissions
+      record.user = account.users.where(canvas_user_id: values['user_id'].to_s).first
+      record.assignment = course.assignments.where(canvas_assignment_id: values['assignment_id'].to_s).first
+    end
+  end
+end

data/db/migrate/20131118211442_create_coalescing_panda_lti_accounts.rb

@@ -7,7 +7,6 @@ class CreateCoalescingPandaLtiAccounts < ActiveRecord::Migration
       t.string :oauth2_client_id
       t.string :oauth2_client_key
 
-
       t.timestamps
     end
   end

data/db/migrate/20141119225319_create_coalescing_panda_terms.rb

@@ -0,0 +1,19 @@
+class CreateCoalescingPandaTerms < ActiveRecord::Migration
+  def change
+    create_table :coalescing_panda_terms do |t|
+      t.belongs_to :coalescing_panda_lti_account
+      t.string :name
+      t.string :code
+      t.string :sis_id
+      t.string :canvas_term_id
+      t.datetime :start_at
+      t.datetime :end_at
+      t.string :workflow_state
+
+      t.timestamps
+    end
+
+    add_index :coalescing_panda_terms, :canvas_term_id
+    add_index :coalescing_panda_terms, :sis_id
+  end
+end

data/db/migrate/20141119225721_create_coalescing_panda_courses.rb

@@ -0,0 +1,22 @@
+class CreateCoalescingPandaCourses < ActiveRecord::Migration
+  def change
+    create_table :coalescing_panda_courses do |t|
+      t.belongs_to :coalescing_panda_lti_account
+      t.belongs_to :coalescing_panda_term
+      t.string :name
+      t.string :canvas_course_id
+      t.string :sis_id
+      t.datetime :start_at
+      t.datetime :conclude_at
+      t.string :workflow_state
+      t.string :course_code
+
+      t.timestamps
+    end
+
+    add_index :coalescing_panda_courses, :coalescing_panda_lti_account_id, name: :index_courses_account
+    add_index :coalescing_panda_courses, :coalescing_panda_term_id, name: :index_courses_term
+    add_index :coalescing_panda_courses, :canvas_course_id
+    add_index :coalescing_panda_courses, :sis_id
+  end
+end

data/db/migrate/20141120151432_create_coalescing_panda_sections.rb

@@ -0,0 +1,19 @@
+class CreateCoalescingPandaSections < ActiveRecord::Migration
+  def change
+    create_table :coalescing_panda_sections do |t|
+      t.belongs_to :coalescing_panda_course
+      t.string :name
+      t.string :canvas_section_id
+      t.string :sis_id
+      t.string :workflow_state
+      t.datetime :start_at
+      t.datetime :end_at
+
+      t.timestamps
+    end
+
+    add_index :coalescing_panda_sections, :coalescing_panda_course_id
+    add_index :coalescing_panda_sections, :canvas_section_id
+    add_index :coalescing_panda_sections, :sis_id
+  end
+end

data/db/migrate/20141120151940_create_coalescing_panda_assignments.rb

@@ -0,0 +1,22 @@
+class CreateCoalescingPandaAssignments < ActiveRecord::Migration
+  def change
+    create_table :coalescing_panda_assignments do |t|
+      t.belongs_to :coalescing_panda_course
+      t.string :name
+      t.string :description
+      t.string :canvas_assignment_id
+      t.string :sis_id
+      t.string :workflow_state
+      t.float :points_possible
+      t.datetime :due_at
+      t.datetime :unlock_at
+      t.datetime :lock_at
+
+      t.timestamps
+    end
+
+    add_index :coalescing_panda_assignments, :coalescing_panda_course_id, name: :index_assignments_course
+    add_index :coalescing_panda_assignments, :canvas_assignment_id
+    add_index :coalescing_panda_assignments, :sis_id
+  end
+end

data/db/migrate/20141120152458_create_coalescing_panda_users.rb

@@ -0,0 +1,19 @@
+class CreateCoalescingPandaUsers < ActiveRecord::Migration
+  def change
+    create_table :coalescing_panda_users do |t|
+      t.belongs_to :coalescing_panda_lti_account
+      t.string :name
+      t.string :email
+      t.string :roles
+      t.string :workflow_state
+      t.string :sis_id
+      t.string :canvas_user_id
+
+      t.timestamps
+    end
+
+    add_index :coalescing_panda_users, :coalescing_panda_lti_account_id, name: :index_users_account
+    add_index :coalescing_panda_users, :canvas_user_id
+    add_index :coalescing_panda_users, :sis_id
+  end
+end

data/db/migrate/20141120152546_create_coalescing_panda_submissions.rb

@@ -0,0 +1,19 @@
+class CreateCoalescingPandaSubmissions < ActiveRecord::Migration
+  def change
+    create_table :coalescing_panda_submissions do |t|
+      t.belongs_to :coalescing_panda_user
+      t.belongs_to :coalescing_panda_assignment
+      t.string :url
+      t.string :grade
+      t.string :score
+      t.datetime :submitted_at
+      t.string :workflow_state
+      t.string :canvas_submission_id
+
+      t.timestamps
+    end
+
+    add_index :coalescing_panda_submissions, [:coalescing_panda_user_id, :coalescing_panda_assignment_id], name: :index_submissions_user_and_assignment
+    add_index :coalescing_panda_submissions, :canvas_submission_id
+  end
+end

data/db/migrate/20141120153135_create_coalescing_panda_enrollments.rb

@@ -0,0 +1,19 @@
+class CreateCoalescingPandaEnrollments < ActiveRecord::Migration
+  def change
+    create_table :coalescing_panda_enrollments do |t|
+      t.belongs_to :coalescing_panda_user
+      t.belongs_to :coalescing_panda_section
+      t.string :workflow_state
+      t.string :sis_id
+      t.string :canvas_enrollment_id
+      t.datetime :start_at
+      t.datetime :end_at
+
+      t.timestamps
+    end
+
+    add_index :coalescing_panda_enrollments, [:coalescing_panda_user_id, :coalescing_panda_section_id], name: :index_enrollments_user_and_assignment
+    add_index :coalescing_panda_enrollments, :canvas_enrollment_id
+    add_index :coalescing_panda_enrollments, :sis_id
+  end
+end

data/db/migrate/20141120205729_add_canvas_account_id_to_lti_account.rb

@@ -0,0 +1,5 @@
+class AddCanvasAccountIdToLtiAccount < ActiveRecord::Migration
+  def change
+    add_column :coalescing_panda_lti_accounts, :canvas_account_id, :string
+  end
+end

data/lib/coalescing_panda/engine.rb

@@ -3,6 +3,11 @@ module CoalescingPanda
     config.autoload_once_paths += Dir["#{config.root}/lib/**/"]
     isolate_namespace CoalescingPanda
 
+    config.generators do |g|
+      g.test_framework :rspec
+      g.fixture_replacement :factory_girl, :dir => 'spec/factories'
+    end
+
     initializer :append_migrations do |app|
       unless app.root.to_s.match root.to_s
         config.paths["db/migrate"].expanded.each do |expanded_path|

data/lib/coalescing_panda/version.rb

@@ -1,3 +1,3 @@
 module CoalescingPanda
-  VERSION = '2.0.0'
+  VERSION = '3.0.0'
 end

data/spec/controllers/coalescing_panda/lti_controller_spec.rb

@@ -1,12 +1,11 @@
 require 'spec_helper'
 
-describe CoalescingPanda::LtiController do
+describe CoalescingPanda::LtiController, :type => :controller do
   routes { CoalescingPanda::Engine.routes }
 
   describe '#lti_config' do
 
     it 'generates lti xml config'do
-      controller.main_app.stub(:test_action_url) {'foo'}
       get(:lti_config)
       xml = Nokogiri::XML(response.body)
       xml.at_xpath('//blti:title').text.should == 'LTI Tool'
@@ -15,7 +14,6 @@ describe CoalescingPanda::LtiController do
     end
 
     it 'generates lti nav config' do
-      controller.main_app.stub(:test_action_url) {'foo'}
       CoalescingPanda.stage_navigation(:account, {
         url: 'test_action',
         text: 'My Title',
@@ -33,12 +31,9 @@ describe CoalescingPanda::LtiController do
 
   end
 
-
   it 'get the url, from the action string' do
-    controller.main_app.stub(:test_action_url) {'foo'}
     options = controller.send(:ext_params, {url:'test_action'})
     options[:url].should == 'foo'
   end
 
-
 end