mumuki-laboratory 7.6.2 → 7.7.4

data/spec/features/exercise_flow_spec.rb

@@ -17,7 +17,7 @@ feature 'Exercise Flow', organization_workspace: :test do
   let(:playground_2) { build(:playground, name: 'Succ6', description: 'Description of Succ4', layout: :input_right, extra: 'x = 4') }
   let!(:reading) { build(:reading, name: 'Reading about Succ', description: 'Lets understand succ history') }
   let!(:exercise_not_in_path) { create :exercise }
-  let(:kids_problem) { build(:problem, name: 'Kids prob', description: 'Description of kids prob', layout: :input_kids, hint: 'lele', language: gobstones) }
+  let(:kids_problem) { build(:problem, name: 'Kids prob', description: 'Description of kids prob', layout: :input_primary, hint: 'lele', language: gobstones) }
 
   let!(:chapter) {
     create(:chapter, name: 'Functional Programming', lessons: [

data/spec/features/menu_bar_spec.rb

@@ -1,7 +1,8 @@
 require 'spec_helper'
 
 feature 'menu bar' do
-  let(:chapter) { create(:chapter, lessons: [create(:lesson)]) }
+  let(:lesson) { create(:lesson, exercises: create_list(:exercise, 3))}
+  let(:chapter) { create(:chapter, lessons: [lesson]) }
   let(:book) { create(:book, chapters: [chapter], name: 'private', slug: 'mumuki/mumuki-the-private-book') }
   let(:private_organization) { create(:organization, name: 'private', book: book) }
 
@@ -32,6 +33,8 @@ feature 'menu bar' do
         expect(page).not_to have_text('Profile')
         expect(page).not_to have_text('Classroom')
         expect(page).not_to have_text('Bibliotheca')
+        expect(page).not_to have_text('Solve other\'s doubts')
+        expect(page).not_to have_text('My doubts')
       end
     end
   end
@@ -52,15 +55,91 @@ feature 'menu bar' do
       expect(page).to have_text('Profile')
       expect(page).not_to have_text('Classroom')
       expect(page).not_to have_text('Bibliotheca')
+      expect(page).not_to have_text('Solve other\'s doubts')
+      expect(page).not_to have_text('My doubts')
     end
 
-    scenario 'student should only see profile' do
-      set_current_user! visitor
+    context 'student with no discussions should' do
+      scenario 'only see profile if forum is not enabled' do
+        set_current_user! student
 
-      visit '/'
-      expect(page).to have_text('Profile')
-      expect(page).not_to have_text('Classroom')
-      expect(page).not_to have_text('Bibliotheca')
+        visit '/'
+        expect(page).to have_text('Profile')
+        expect(page).not_to have_text('Classroom')
+        expect(page).not_to have_text('Bibliotheca')
+        expect(page).not_to have_text('Solve other\'s doubts')
+        expect(page).not_to have_text('My doubts')
+      end
+
+      scenario 'see profile and solve_other_doubts links if forum is enabled' do
+        set_current_user! student
+        private_organization.update! forum_enabled: true
+
+        visit '/'
+        expect(page).to have_text('Profile')
+        expect(page).not_to have_text('Classroom')
+        expect(page).not_to have_text('Bibliotheca')
+        expect(page).to have_text('Solve other\'s doubts')
+        expect(page).not_to have_text('My doubts')
+      end
+    end
+
+    context 'student with discussions should' do
+      let(:discussion) { create(:discussion, item: lesson.exercises.last, initiator: student)}
+
+      scenario 'only see profile if forum is not enabled' do
+        set_current_user! student
+        student.subscribe_to! discussion
+
+        visit '/'
+        expect(page).to have_text('Profile')
+        expect(page).not_to have_text('Classroom')
+        expect(page).not_to have_text('Bibliotheca')
+        expect(page).not_to have_text('Solve other\'s doubts')
+        expect(page).not_to have_text('My doubts')
+      end
+
+      scenario 'see all discussions links if forum is enabled' do
+        set_current_user! student
+        private_organization.update! forum_enabled: true
+        student.subscribe_to! discussion
+
+        visit '/'
+        expect(page).to have_text('Profile')
+        expect(page).not_to have_text('Classroom')
+        expect(page).not_to have_text('Bibliotheca')
+        expect(page).to have_text('Solve other\'s doubts')
+        expect(page).to have_text('My doubts')
+      end
+
+      scenario 'only see profile if forum is enabled in a forum_only_for_trusted organization' do
+        set_current_user! student
+        student.subscribe_to! discussion
+        private_organization.update! forum_enabled: true
+        private_organization.update! forum_only_for_trusted: true
+
+        visit '/'
+        expect(page).to have_text('Profile')
+        expect(page).not_to have_text('Classroom')
+        expect(page).not_to have_text('Bibliotheca')
+        expect(page).not_to have_text('Solve other\'s doubts')
+        expect(page).not_to have_text('My doubts')
+      end
+
+      scenario 'see all discussions links if forum is enabled in a forum_only_for_trusted organization but it is trusted' do
+        student.update! trusted_for_forum: true
+        set_current_user! student
+        student.subscribe_to! discussion
+        private_organization.update! forum_enabled: true
+        private_organization.update! forum_only_for_trusted: true
+
+        visit '/'
+        expect(page).to have_text('Profile')
+        expect(page).not_to have_text('Classroom')
+        expect(page).not_to have_text('Bibliotheca')
+        expect(page).to have_text('Solve other\'s doubts')
+        expect(page).to have_text('My doubts')
+      end
     end
 
     scenario 'teacher should see profile and classroom' do
@@ -71,6 +150,8 @@ feature 'menu bar' do
       expect(page).to have_text('Profile')
       expect(page).to have_text('Classroom')
       expect(page).not_to have_text('Bibliotheca')
+      expect(page).not_to have_text('Solve other\'s doubts')
+      expect(page).not_to have_text('My doubts')
     end
 
     scenario 'writer should see profile and bibliotheca' do

data/spec/helpers/breadcrumbs_helper_spec.rb

@@ -91,7 +91,7 @@ describe BreadcrumbsHelper, organization_workspace: :test do
         expect(breadcrumb).to include('my lesson')
         expect(breadcrumb).to include('my exercise')
         expect(breadcrumb).to include('discussions')
-        expect(breadcrumb).to include(discussion.title)
+        expect(breadcrumb).to include(discussion.item.navigable_name)
         expect(breadcrumb).to be_html_safe
       end
     end

data/spec/javascripts/bridge-spec.js

@@ -0,0 +1,5 @@
+describe('bridge', () => {
+  it('can create bridge', () => {
+    expect(new mumuki.bridge.Laboratory()).not.toBe(null);
+  })
+})

data/spec/javascripts/csrf-token-spec.js

@@ -0,0 +1,7 @@
+describe('csrf token', () => {
+  it('can create token', () => {
+    expect(new mumuki.CsrfToken()).not.toBe(null);
+  })
+})
+
+

data/spec/javascripts/elipsis-spec.js

@@ -0,0 +1,25 @@
+describe('elipsis', () => {
+  it('does nothing when no elipsis', () => {
+    expect(mumuki.elipsis('hello')).toEqual('hello');
+  })
+
+  it('replaces student elipsis', () => {
+    expect(mumuki.elipsis(`function longitud(unString) {
+      /*<elipsis-for-student@*/
+      return unString.length;
+      /*@elipsis-for-student>*/
+    }`)).toEqual(`function longitud(unString) {
+      /* ... */
+    }`)
+  });
+
+  it('replaces student hidden', () => {
+    expect(mumuki.elipsis(`function longitud(unString) {
+      /*<hidden-for-student@*/
+      return unString.length;
+      /*@hidden-for-student>*/
+    }`)).toEqual(`function longitud(unString) {
+      /**/
+    }`)
+  });
+})

data/spec/javascripts/results-renderers-spec.js

@@ -0,0 +1,17 @@
+describe('results renderers', () => {
+  it('can compute class for status', () => {
+    expect(mumuki.renderers.classForStatus('passed')).toEqual('success');
+  })
+
+  it('can compute icon for status', () => {
+    expect(mumuki.renderers.iconForStatus('pending')).toEqual('fa-circle');
+  })
+
+  it('can compute progress list item for status', () => {
+    expect(mumuki.renderers.progressListItemClassForStatus('passed_with_warnings')).toEqual('progress-list-item text-center warning ');
+  })
+
+  it('can compute progress list item for status when active', () => {
+    expect(mumuki.renderers.progressListItemClassForStatus('failed', true)).toEqual('progress-list-item text-center danger active');
+  })
+})

data/spec/javascripts/spec-helper.js

@@ -0,0 +1,30 @@
+// Teaspoon includes some support files, but you can use anything from your own support path too.
+// require support/jasmine-jquery-1.7.0
+// require support/jasmine-jquery-2.0.0
+// require support/jasmine-jquery-2.1.0
+// require support/sinon
+//
+// PhantomJS (Teaspoons default driver) doesn't have support for Function.prototype.bind, which has caused confusion.
+// Use this polyfill to avoid the confusion.
+//
+// You can require your own javascript files here. By default this will include everything in application, however you
+// may get better load performance if you require the specific files that are being used in the spec that tests them.
+//=require mumuki_laboratory/application
+//
+// Deferring execution
+// If you're using CommonJS, RequireJS or some other asynchronous library you can defer execution. Call
+// Teaspoon.execute() after everything has been loaded. Simple example of a timeout:
+//
+// Teaspoon.defer = true
+// setTimeout(Teaspoon.execute, 1000)
+//
+// Matching files
+// By default Teaspoon will look for files that match _spec.{js,js.coffee,.coffee}. Add a filename_spec.js file in your
+// spec path and it'll be included in the default suite automatically. If you want to customize suites, check out the
+// configuration in teaspoon_env.rb
+//
+// Manifest
+// If you'd rather require your spec files manually (to control order for instance) you can disable the suite matcher in
+// the configuration and use this file as a manifest.
+//
+// For more information: http://github.com/modeset/teaspoon

data/spec/javascripts/speech-bubble-renderer-spec.js

@@ -0,0 +1,11 @@
+describe('results renderers', () => {
+  it('can render result item', () => {
+    expect(mumuki.renderers.renderSpeechBubbleResultItem('fix that')).toEqual(`
+        <div class="results-item">
+          <ul class="results-list">
+            <li>fix that</li>
+          </ul>
+        </div>`);
+  })
+
+})

data/spec/javascripts/timeout-spec.js

@@ -0,0 +1,5 @@
+describe('setTimeout', () => {
+  it('can create setTimeout', () => {
+    expect(mumuki.setTimeout).not.toBe(null);
+  })
+})

data/spec/javascripts/timer-spec.js

@@ -0,0 +1,5 @@
+describe('timer', () => {
+  it('can create timer', () => {
+    expect(mumuki.startTimer).not.toBe(null);
+  })
+})

data/spec/teaspoon_env.rb

@@ -0,0 +1,187 @@
+unless defined?(Rails)
+  ENV["RAILS_ROOT"] = File.expand_path("../dummy", __FILE__)
+  require File.expand_path("../dummy/config/environment", __FILE__)
+end
+
+
+Teaspoon.configure do |config|
+  # Determines where the Teaspoon routes will be mounted. Changing this to "/jasmine" would allow you to browse to
+  # `http://localhost:3000/jasmine` to run your tests.
+  config.mount_at = "/teaspoon"
+
+  # Specifies the root where Teaspoon will look for files. If you're testing an engine using a dummy application it can
+  # be useful to set this to your engines root (e.g. `Teaspoon::Engine.root`).
+  # Note: Defaults to `Rails.root` if nil.
+  config.root = Mumuki::Laboratory::Engine.root
+
+  # Paths that will be appended to the Rails assets paths
+  # Note: Relative to `config.root`.
+  config.asset_paths = ["spec/javascripts", "spec/javascripts/stylesheets"]
+
+  # Fixtures are rendered through a controller, which allows using HAML, RABL/JBuilder, etc. Files in these paths will
+  # be rendered as fixtures.
+  config.fixture_paths = ["spec/javascripts/fixtures"]
+
+  # SUITES
+  #
+  # You can modify the default suite configuration and create new suites here. Suites are isolated from one another.
+  #
+  # When defining a suite you can provide a name and a block. If the name is left blank, :default is assumed. You can
+  # omit various directives and the ones defined in the default suite will be used.
+  #
+  # To run a specific suite
+  # - in the browser: http://localhost/teaspoon/[suite_name]
+  # - with the rake task: rake teaspoon suite=[suite_name]
+  # - with the cli: teaspoon --suite=[suite_name]
+  config.suite do |suite|
+    # Specify the framework you would like to use. This allows you to select versions, and will do some basic setup for
+    # you -- which you can override with the directives below. This should be specified first, as it can override other
+    # directives.
+    # Note: If no version is specified, the latest is assumed.
+    #
+    # Versions: 1.3.1, 2.0.3, 2.1.3, 2.2.0, 2.2.1, 2.3.4
+    suite.use_framework :jasmine, "2.3.4"
+
+    # Specify a file matcher as a regular expression and all matching files will be loaded when the suite is run. These
+    # files need to be within an asset path. You can add asset paths using the `config.asset_paths`.
+    suite.matcher = "{spec/javascripts,app/assets}/**/*-spec.{js,js.coffee,coffee}"
+
+    # Load additional JS files, but requiring them in your spec helper is the preferred way to do this.
+    #suite.javascripts = []
+
+    # You can include your own stylesheets if you want to change how Teaspoon looks.
+    # Note: Spec related CSS can and should be loaded using fixtures.
+    #suite.stylesheets = ["teaspoon"]
+
+    # This suites spec helper, which can require additional support files. This file is loaded before any of your test
+    # files are loaded.
+    suite.helper = "spec-helper"
+
+    # Partial to be rendered in the head tag of the runner. You can use the provided ones or define your own by creating
+    # a `_boot.html.erb` in your fixtures path, and adjust the config to `"/boot"` for instance.
+    #
+    # Available: boot, boot_require_js
+    suite.boot_partial = "boot"
+
+    # Partial to be rendered in the body tag of the runner. You can define your own to create a custom body structure.
+    suite.body_partial = "body"
+
+    # Hooks allow you to use `Teaspoon.hook("fixtures")` before, after, or during your spec run. This will make a
+    # synchronous Ajax request to the server that will call all of the blocks you've defined for that hook name.
+    #suite.hook :fixtures, &proc{}
+
+    # Determine whether specs loaded into the test harness should be embedded as individual script tags or concatenated
+    # into a single file. Similar to Rails' asset `debug: true` and `config.assets.debug = true` options. By default,
+    # Teaspoon expands all assets to provide more valuable stack traces that reference individual source files.
+    #suite.expand_assets = true
+
+    # Non-.js file extensions Teaspoon should consider JavaScript files
+    #suite.js_extensions = [/(\.js)?.coffee/, /(\.js)?.es6/, ".es6.js"]
+  end
+
+  # Example suite. Since we're just filtering to files already within the root test/javascripts, these files will also
+  # be run in the default suite -- but can be focused into a more specific suite.
+  #config.suite :targeted do |suite|
+  #  suite.matcher = "spec/javascripts/targeted/*_spec.{js,js.coffee,coffee}"
+  #end
+
+  # CONSOLE RUNNER SPECIFIC
+  #
+  # These configuration directives are applicable only when running via the rake task or command line interface. These
+  # directives can be overridden using the command line interface arguments or with ENV variables when using the rake
+  # task.
+  #
+  # Command Line Interface:
+  # teaspoon --driver=phantomjs --server-port=31337 --fail-fast=true --format=junit --suite=my_suite /spec/file_spec.js
+  #
+  # Rake:
+  # teaspoon DRIVER=phantomjs SERVER_PORT=31337 FAIL_FAST=true FORMATTERS=junit suite=my_suite
+
+  # Specify which headless driver to use. Supports PhantomJS, Selenium Webdriver and BrowserStack Webdriver.
+  #
+  # Available: :phantomjs, :selenium, :browserstack, :capybara_webkit
+  # PhantomJS: https://github.com/modeset/teaspoon/wiki/Using-PhantomJS
+  # Selenium Webdriver: https://github.com/modeset/teaspoon/wiki/Using-Selenium-WebDriver
+  # BrowserStack Webdriver: https://github.com/modeset/teaspoon/wiki/Using-BrowserStack-WebDriver
+  # Capybara Webkit: https://github.com/modeset/teaspoon/wiki/Using-Capybara-Webkit
+  #  config.driver = :capybara_webkit
+  config.driver = :selenium
+  config.driver_options = {client_driver: :firefox}
+
+
+  # Specify the timeout for the driver. Specs are expected to complete within this time frame or the run will be
+  # considered a failure. This is to avoid issues that can arise where tests stall.
+  #config.driver_timeout = 180
+
+  # Specify a server to use with Rack (e.g. thin, mongrel). If nil is provided Rack::Server is used.
+  #config.server = nil
+
+  # Specify a host to run on a specific host, otherwise Teaspoon will use 127.0.0.1.
+  #config.server_host = nil
+
+  # Specify a port to run on a specific port, otherwise Teaspoon will use a random available port.
+  #config.server_port = nil
+
+  # Timeout for starting the server in seconds. If your server is slow to start you may have to bump this, or you may
+  # want to lower this if you know it shouldn't take long to start.
+  #config.server_timeout = 20
+
+  # Force Teaspoon to fail immediately after a failing suite. Can be useful to make Teaspoon fail early if you have
+  # several suites, but in environments like CI this may not be desirable.
+  #config.fail_fast = true
+
+  # Specify the formatters to use when outputting the results.
+  # Note: Output files can be specified by using `"junit>/path/to/output.xml"`.
+  #
+  # Available: :dot, :clean, :documentation, :json, :junit, :pride, :rspec_html, :snowday, :swayze_or_oprah, :tap, :tap_y, :teamcity
+  config.formatters = [:documentation]
+
+  # Specify if you want color output from the formatters.
+  #config.color = true
+
+  # Teaspoon pipes all console[log/debug/error] to $stdout. This is useful to catch places where you've forgotten to
+  # remove them, but in verbose applications this may not be desirable.
+  #config.suppress_log = false
+
+  # COVERAGE REPORTS / THRESHOLD ASSERTIONS
+  #
+  # Coverage reports requires Istanbul (https://github.com/gotwarlost/istanbul) to add instrumentation to your code and
+  # display coverage statistics.
+  #
+  # Coverage configurations are similar to suites. You can define several, and use different ones under different
+  # conditions.
+  #
+  # To run with a specific coverage configuration
+  # - with the rake task: rake teaspoon USE_COVERAGE=[coverage_name]
+  # - with the cli: teaspoon --coverage=[coverage_name]
+
+  # Specify that you always want a coverage configuration to be used. Otherwise, specify that you want coverage
+  # on the CLI.
+  # Set this to "true" or the name of your coverage config.
+  #config.use_coverage = nil
+
+  # You can have multiple coverage configs by passing a name to config.coverage.
+  # e.g. config.coverage :ci do |coverage|
+  # The default coverage config name is :default.
+  config.coverage do |coverage|
+    # Which coverage reports Istanbul should generate. Correlates directly to what Istanbul supports.
+    #
+    # Available: text-summary, text, html, lcov, lcovonly, cobertura, teamcity
+    #coverage.reports = ["text-summary", "html"]
+
+    # The path that the coverage should be written to - when there's an artifact to write to disk.
+    # Note: Relative to `config.root`.
+    #coverage.output_path = "coverage"
+
+    # Assets to be ignored when generating coverage reports. Accepts an array of filenames or regular expressions. The
+    # default excludes assets from vendor, gems and support libraries.
+    #coverage.ignore = [%r{/lib/ruby/gems/}, %r{/vendor/assets/}, %r{/support/}, %r{/(.+)_helper.}]
+
+    # Various thresholds requirements can be defined, and those thresholds will be checked at the end of a run. If any
+    # aren't met the run will fail with a message. Thresholds can be defined as a percentage (0-100), or nil.
+    #coverage.statements = nil
+    #coverage.functions = nil
+    #coverage.branches = nil
+    #coverage.lines = nil
+  end
+end