pupilfirst_xapi 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 99a6dddb084209a85ef5d3f6ad55cf489fb50cf21b0a1f1c1e01a392079a13e9
-  data.tar.gz: bff106abc8d633185bda3b156203ad901ae0beff5fdb544051014a462405fd67
+  metadata.gz: 6435e3040cef3863e99115b8a474bf193c05484c7c9048701425827b27353188
+  data.tar.gz: 412e2b47cc25daa790d214d111d8a7bf4645050dd2294c110859129b695bbee1
 SHA512:
-  metadata.gz: b41c20f2967044b42d4d3f11c5307a5970e680f8313d200a91e98e948f0e7ae8053793c976d091372d78eb3a9ede6da505d0253265cd588c51707d355fe26ad5
-  data.tar.gz: 8a526960635e5065792c221140153834f5eb3f01ab5ef66b221369b034e18510f094e972637702052a2a7a7bfe506fe1762838cf39d3f98a558b7878abf1d2f0
+  metadata.gz: 2c44119ab5421640425e20e55629d7281e68d284f72c9ccceaa5ee66c1e50f35776aad758a8f5a7f39d5b72215ba5f00185ad64385f23617cecc70d0aea66476
+  data.tar.gz: 776ae2649f8d242ad26a32b798373186df902855079a75668d50a3e9c6af25536abfc9e827e2ca5af0a4e429fbfb5309edcb7ac5910bbce624da641350b6d950

data/lib/pupilfirst_xapi.rb

@@ -2,6 +2,7 @@ require "action_controller/railtie"
 require "active_job/railtie"
 require "growthtribe_xapi"
 require "pupilfirst_xapi/version"
+require "pupilfirst_xapi/lrs"
 require "pupilfirst_xapi/outbox"
 require "pupilfirst_xapi/actors"
 require "pupilfirst_xapi/objects"
@@ -11,6 +12,7 @@ require "pupilfirst_xapi/statements"
 module PupilfirstXapi
   mattr_accessor :uri_for
   mattr_accessor :repository
+  mattr_accessor :lrs
 
   Statements.subscribe do |event_type|
     ActiveSupport::Notifications.subscribe("#{event_type}.pupilfirst") do |_name, _start, finish, _id, payload|

data/lib/pupilfirst_xapi/lrs.rb

@@ -0,0 +1,24 @@
+require 'xapi'
+
+module PupilfirstXapi
+  class Lrs
+    def initialize(
+      end_point: ENV['LRS_ENDPOINT'],
+      key: ENV['LRS_KEY'],
+      secret: ENV['LRS_SECRET']
+    )
+      @lrs = end_point && key && secret &&
+        Xapi.create_remote_lrs(
+          end_point: end_point,
+          user_name: key,
+          password: secret
+        )
+    end
+
+    def call(statement)
+      return unless statement && @lrs
+
+      Xapi.post_statement(remote_lrs: @lrs, statement: statement)
+    end
+  end
+end

data/lib/pupilfirst_xapi/outbox.rb

@@ -1,5 +1,4 @@
 require "active_job"
-require 'xapi'
 
 module PupilfirstXapi
   class Outbox
@@ -14,19 +13,11 @@ module PupilfirstXapi
 
       def outbox
         Outbox.new(
-          lrs: remote_lrs,
+          lrs: PupilfirstXapi.lrs || PupilfirstXapi::Lrs.new,
           repository: PupilfirstXapi.repository,
           uri_for: PupilfirstXapi.uri_for
         )
       end
-
-      def remote_lrs
-        Xapi.create_remote_lrs(
-          end_point: ENV['LRS_ENDPOINT'],
-          user_name: ENV['LRS_KEY'],
-          password: ENV['LRS_SECRET']
-        )
-      end
     end
 
     class << self
@@ -42,8 +33,7 @@ module PupilfirstXapi
     end
 
     def call(**payload)
-      statement = statement_for(**payload)
-      Xapi.post_statement(remote_lrs: @lrs, statement: statement) if statement
+      @lrs.call(statement_for(**payload))
     end
 
     private

data/lib/pupilfirst_xapi/version.rb

@@ -1,3 +1,3 @@
 module PupilfirstXapi
-  VERSION = '0.1.0'
+  VERSION = '0.2.0'
 end

data/spec/spec_helper.rb

@@ -50,7 +50,9 @@ class FakeLrs
   end
   attr_reader :statements
 
-  def save_statement(statement)
+  def call(statement)
+    return unless statement
+
     @statements << statement
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pupilfirst_xapi
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - GrowthTribe
@@ -98,6 +98,7 @@ files:
 - Rakefile
 - lib/pupilfirst_xapi.rb
 - lib/pupilfirst_xapi/actors.rb
+- lib/pupilfirst_xapi/lrs.rb
 - lib/pupilfirst_xapi/objects.rb
 - lib/pupilfirst_xapi/objects/builder.rb
 - lib/pupilfirst_xapi/objects/course.rb