Feature incognito mode

app/models/concerns/navigation/siblings_navigation.rb

@@ -1,11 +1,11 @@
 module SiblingsNavigation
 
   def next_for(user)
-    pending_siblings_for(user).select { |it| it.number > number }.sort_by(&:number).first
+    user.pending_siblings_at(self).select { |it| it.number > number }.sort_by(&:number).first
   end
 
   def restart(user)
-    pending_siblings_for(user).sort_by(&:number).first
+    user.pending_siblings_at(self).sort_by(&:number).first
   end
 
   def siblings

app/models/concerns/submittable/submittable.rb

@@ -5,7 +5,7 @@ def submit!(user, submission)
 
   def find_assignment_and_submit!(user, submission)
     assignment = assignment_for user
-    results = submission.run! assignment, evaluation_class.new
+    results = user.run_submission! submission, assignment, evaluation_class.new
     [assignment, results]
   end
 end

app/models/concerns/with_assignments.rb

@@ -28,6 +28,6 @@ def status_for(user)
   end
 
   def assignment_for(user, organization=Organization.current)
-    find_assignment_for(user, organization) || user.assignments.build(exercise: self, organization: organization)
+    find_assignment_for(user, organization) || user.build_assignment(self, organization)
   end
 end

app/models/concerns/with_progress.rb

@@ -1,6 +1,6 @@
 module WithProgress
   def progress_for(user, organization)
-    Indicator.find_or_initialize_by(user: user, organization: organization, content: self)
+    user.progress_at(self, organization)
   end
 
   def completion_percentage_for(user, organization=Organization.current)

app/models/guide.rb

@@ -42,7 +42,7 @@ def exercises_count
   end
 
   def next_exercise(user)
-    pending_exercises(user).order('public.exercises.number asc').first
+    user.next_exercise_at(self)
   end
 
   # TODO: Make use of pending_siblings logic
@@ -52,7 +52,8 @@ def pending_exercises(user)
                 on assignments.exercise_id = exercises.id
                 and assignments.submitter_id = #{user.id}
                 and assignments.submission_status = #{Mumuki::Domain::Status::Submission::Passed.to_i}").
-        where('assignments.id is null')
+        where('assignments.id is null').
+        order('public.exercises.number asc')
   end
 
   def first_exercise

app/models/user.rb

@@ -189,6 +189,30 @@ def name_initials
     name.split.map(&:first).map(&:capitalize).join(' ')
   end
 
+  def progress_at(content, organization)
+    Indicator.find_or_initialize_by(user: self, organization: organization, content: content)
+  end
+
+  def build_assignment(exercise, organization)
+    assignments.build(exercise: exercise, organization: organization)
+  end
+
+  def pending_siblings_at(content)
+    content.pending_siblings_for(self)
+  end
+
+  def next_exercise_at(guide)
+    guide.pending_exercises(self).first
+  end
+
+  def run_submission!(submission, assignment, evaluation)
+    submission.run! assignment, evaluation
+  end
+
+  def incognito?
+    false
+  end
+
   private
 
   def set_uid!

db/migrate/20200804191643_add_incognito_mode_enabled_to_organization.rb

@@ -0,0 +1,5 @@
+class AddIncognitoModeEnabledToOrganization < ActiveRecord::Migration[5.1]
+  def change
+    add_column :organizations, :incognito_mode_enabled, :boolean
+  end
+end

lib/mumuki/domain.rb

@@ -26,6 +26,7 @@ module Domain
 end
 
 require_relative './domain/area'
+require_relative './domain/incognito'
 require_relative './domain/evaluation'
 require_relative './domain/submission'
 require_relative './domain/status'

lib/mumuki/domain/incognito.rb

@@ -0,0 +1,108 @@
+module Mumuki::Domain
+  class IncognitoClass
+
+    def incognito?
+      true
+    end
+
+    # ============
+    # Permissions
+    # ============
+
+    def ensure_enabled!
+    end
+
+    def has_student_granted_organizations?
+      false
+    end
+
+    def teacher_here?
+      false
+    end
+
+    def teacher_of?(*)
+      false
+    end
+
+    def profile_completed?
+      true
+    end
+
+    def writer?
+      false
+    end
+
+    def moderator_here?
+      false
+    end
+
+    # ========
+    # Visiting
+    # ========
+
+    def visit!(*)
+    end
+
+    # ========
+    # Progress
+    # ========
+
+    def next_exercise_at(guide)
+      guide.exercises.first
+    end
+
+    # def completed_containers_with_lookahead(*)
+    #   raise 'Unsupported operation. Userless mode and progressive display modes are incompatible'
+    # end
+
+    def progress_at(content, organization)
+      Indicator.new content: content, organization: organization
+    end
+
+    def build_assignment(exercise, organization)
+      Assignment.new exercise: exercise, organization: organization, submitter: self
+    end
+
+    def pending_siblings_at(content)
+      []
+    end
+
+    # ============
+    # ActiveRecord
+    # ============
+
+    def id
+      '<incognito>'
+    end
+
+    def is_a?(other)
+      other.is_a?(Class) && other.name == 'User' || super
+    end
+
+    def _read_attribute(key)
+      return id if key == 'id'
+      raise "unknown attribute #{key}"
+    end
+
+    def self.primary_key
+      'id'
+    end
+
+    # ==========
+    # Evaluation
+    # ==========
+
+    def interpolations
+      []
+    end
+
+    def run_submission!(submission, assignment, evaluation)
+      results = submission.dry_run! assignment, evaluation
+      assignment.assign_attributes results
+      results
+    end
+
+  end
+
+  Incognito = IncognitoClass.new
+end

lib/mumuki/domain/submission/base.rb

@@ -21,12 +21,16 @@ def self.mapping_attributes
 
   def run!(assignment, evaluation)
     save_submission! assignment
-    results = evaluation.evaluate! assignment, self
+    results = dry_run! assignment, evaluation
     save_results! results, assignment
     notify_results! results, assignment
     results
   end
 
+  def dry_run!(assignment, evaluation)
+    evaluation.evaluate! assignment, self
+  end
+
   def with_client_result(result)
     self.client_result = result if result.present?
     self

spec/dummy/db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20200731081757) do
+ActiveRecord::Schema.define(version: 20200804191643) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -309,6 +309,7 @@
     t.text "theme", default: "{}", null: false
     t.text "profile", default: "{}", null: false
     t.integer "progressive_display_lookahead"
+    t.boolean "incognito_mode_enabled"
     t.index ["book_id"], name: "index_organizations_on_book_id"
     t.index ["name"], name: "index_organizations_on_name", unique: true
   end

spec/models/incognito_spec.rb

@@ -0,0 +1,71 @@
+require 'spec_helper'
+
+describe Mumuki::Domain::Incognito do
+  let(:user) { Mumuki::Domain::Incognito }
+  let(:organization) { create(:organization) }
+
+  it 'can mock an AR relation' do
+    Assignment.new submitter: user
+  end
+
+  describe 'ensure_enabled!' do
+    it { expect { user.ensure_enabled! }.to_not raise_error }
+  end
+
+  describe 'assignment fooling' do
+    let(:assignment) { user.build_assignment(exercise, organization) }
+
+    context 'when exercise has no default content' do
+      let(:exercise) { create(:exercise) }
+
+      it { expect(assignment).to be_an Assignment }
+      it { expect(assignment.submitter).to be user }
+      it { expect(assignment.current_content).to eq '' }
+    end
+
+    context 'when exercise has default content' do
+      let(:exercise) { create(:exercise, default_content: '...') }
+
+      it { expect(assignment.current_content).to eq '...' }
+    end
+  end
+
+  describe 'guide fooling' do
+    let(:guide) { create(:guide, exercises: [ create(:exercise), create(:exercise) ]) }
+
+    describe 'completion_percentage_for', organization_workspace: :test do
+      it { expect(guide.completion_percentage_for(user)).to eq 0 }
+    end
+
+    describe 'next_exercise', organization_workspace: :test do
+      # TODO this looks weird. Why do we need a non-polymorphic
+      # next_exercise? Aren't indicators not enought?
+      it { expect(guide.next_exercise(user)).to eq guide.exercises.first }
+    end
+  end
+
+  describe 'exercise fooling' do
+    let(:problem) { create(:problem) }
+
+    describe 'next_for' do
+      it { expect(problem.next_for(user)).to be nil }
+    end
+
+    describe 'try_submit_solution!', organization_workspace: :test do
+      let!(:chapter) do
+        create(:chapter, name: 'Functional Programming', lessons: [
+          create(:lesson, exercises: [problem])
+        ])
+      end
+      let(:bridge_response) { {result: '0 failures', status: :passed} }
+
+      before { reindex_current_organization! }
+      before { expect_any_instance_of(Language).to receive(:run_tests!).and_return(bridge_response) }
+
+      let!(:assignment) { problem.try_submit_solution!(user) }
+
+      it { expect(assignment).to_not be nil }
+      it { expect(assignment.status).to be_like :passed }
+    end
+  end
+end