mumuki-laboratory 7.7.2 → 7.7.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3c7dbf715c210c593e2a509411ba1f39bfa2c061c9b4e45ffd132634abcd6060
-  data.tar.gz: decfa9f0385134c07cdb7e0fe933c29dcf1c3864741840a7a25573c1c8141c3f
+  metadata.gz: fa8619681047d1ea5b8230abc24e7b4b38c63f722d355a3facd956d725b0b3dd
+  data.tar.gz: bdefac6e50c9a9cb6950222bc822f8460e095f47a3602ae7ad86b4c3c41ea634
 SHA512:
-  metadata.gz: 11813d4b113221ebbc7012a4fa86c78b69cf070f979706d54fe70afa4b3543193f623f8837665c3879d081fca10fad3b3e1dbba66491e6eca338362f268acdb7
-  data.tar.gz: 472fc73e4c480f8f69797ad5639485ffc4baa1887c28861c4aa59d3b03719ac3df5ebae5e356dc00e8d06a343183793b1d9c89b813c09a35377537a1ad54794e
+  metadata.gz: 4a91405f88b73103163b212bbc08b6a00da9f1d109e13630d4713094cf32736621e38de66d007ab739e86bca4f7782a8081faabf50425e4e45dc49c8022ced0d
+  data.tar.gz: 9c7a7398eb308154dc585d8d4d3401e27ced3322314bffbbf5a016ef951aeeeb3c109860afb4457bd93f6b5f509b0f36d7b321baa4c4ae26c445e7c3b3275840

data/README.md

@@ -135,6 +135,15 @@ rails s
 bundle exec rspec
 ```
 
+## Running JS tests
+
+> You need first to download [geckodriver](https://github.com/mozilla/geckodriver/releases/download/v0.27.0/geckodriver-v0.27.0-linux64.tar.gz), uncrompress
+> it and add it to your path
+
+```bash
+bundle exec rake teaspoon
+```
+
 ## JavaScript API Docs
 
 In order to be customized by runners, Laboratory exposes the following selectors and methods

data/Rakefile

@@ -27,6 +27,9 @@ require 'rspec/core/rake_task'
 desc "Run all specs in spec directory (excluding plugin specs)"
 RSpec::Core::RakeTask.new(:spec => 'app:db:test:prepare')
 
+desc "Run the javascript specs"
+task :teaspoon => "app:teaspoon"
+
 task default: :spec
 
 

data/app/assets/stylesheets/mumuki_laboratory/application/modules/_discussion.scss

@@ -327,9 +327,9 @@ summary.discussion-summary {
         float: right;
         a {
           margin-left: 5px;
+          cursor: pointer;
         }
         .discussion-message-approved {
-          cursor: pointer;
           text-decoration: none;
           &.selected {
             i {
@@ -338,7 +338,6 @@ summary.discussion-summary {
           }
         }
         .discussion-message-not-actually-a-question {
-          cursor: pointer;
           text-decoration: none;
           i {
             position: relative;

data/app/controllers/concerns/with_user_discussion_validation.rb

@@ -0,0 +1,14 @@
+module WithUserDiscussionValidation
+  extend ActiveSupport::Concern
+
+  included do
+    # discussions are not enabled for all organizations nor all users
+    before_action :validate_user_can_discuss!
+  end
+
+  private
+
+  def validate_user_can_discuss!
+    raise Mumuki::Domain::NotFoundError unless current_user&.can_discuss_here?
+  end
+end

data/app/controllers/discussions_controller.rb

@@ -1,11 +1,6 @@
 class DiscussionsController < ApplicationController
   include Mumuki::Laboratory::Controllers::Content
-
-  # discussions are not enabled for all organitions nor all users
-  # there is no need to validate forum existance when creating; next validation is stronger
-  before_action :validate_forum_enabled!, except: :create
-  before_action :validate_can_create_discusssion!, only: :create
-
+  include WithUserDiscussionValidation
   # users are not allowed to access discussions during exams
   before_action :validate_not_in_exam!
 
@@ -76,14 +71,6 @@ class DiscussionsController < ApplicationController
     @filter_params ||= params.permit(Discussion.permitted_query_params)
   end
 
-  def validate_forum_enabled!
-    raise Mumuki::Domain::NotFoundError unless Organization.current.forum_enabled?
-  end
-
-  def validate_can_create_discusssion!
-    raise Mumuki::Domain::NotFoundError unless Organization.current.can_create_discussions?(current_user)
-  end
-
   def validate_not_in_exam!
     raise Mumuki::Domain::BlockedForumError if current_user&.currently_in_exam?
   end

data/app/controllers/discussions_messages_controller.rb

@@ -1,5 +1,6 @@
 class DiscussionsMessagesController < AjaxController
   include WithAuthorization
+  include WithUserDiscussionValidation
 
   before_action :set_discussion!, only: [:create, :destroy]
   before_action :authorize_user!, only: [:destroy]

data/app/helpers/discussions_helper.rb

@@ -23,8 +23,9 @@ module DiscussionsHelper
     fixed_fa_icon 'comment', text: text
   end
 
-  def discussions_link(item, path, html_options = nil, organization = Organization.current)
-    link_to item, path, html_options if organization.forum_enabled?
+  def discussions_link(item, path, html_options = nil)
+    return unless current_user&.can_discuss_here?
+    link_to item, path, html_options
   end
 
   def item_discussion_path(discussion, params = {})
@@ -91,8 +92,6 @@ module DiscussionsHelper
   end
 
   def new_discussion_link(teaser_text, link_text)
-    return '' unless Organization.current.can_create_discussions?(current_user)
-
     %Q{
       <h4>
         <span>#{t(teaser_text)}</span>
@@ -113,8 +112,10 @@ module DiscussionsHelper
     Mumuki::Domain::Status::Discussion::STATUSES
   end
 
+  #TODO: this one uses a long method chain in order to take advantage of eager load
+  # Delegate it once again when polymorphic association is removed
   def discussions_languages(discussions)
-    @languages ||= discussions.map { |it| it.language.name }.uniq
+    @languages ||= discussions.map { |it| it.exercise.language.name }.uniq
   end
 
   def discussion_status_filter_link(status, discussions)
@@ -183,4 +184,7 @@ module DiscussionsHelper
     discussion_filter_params.except(:page)
   end
 
+  def should_show_approved_for?(user, message)
+    !user&.moderator_here? && message.approved? && !message.from_moderator?
+  end
 end

data/app/views/discussions/_message.html.erb

@@ -7,21 +7,28 @@
           <%= t(:time_since, time: time_ago_in_words(message.created_at)) %>
         </span>
         <% if user.moderator_here? %>
-          <%= fa_icon(:star, {'data-toggle': 'tooltip', title: (t :moderator), class: 'moderator-star'}) %>
+          <%= fa_icon(:star, 'data-toggle': 'tooltip', title: (t :moderator), class: 'moderator-star') %>
         <% end %>
-        <% if message.authorized? current_user %>
-          <span class="actions">
-            <% if current_user.moderator_here? %>
+        <span class="actions">
+          <% if message.authorized? current_user %>
+            <% if current_user&.moderator_here? %>
               <a class="discussion-message-approved <%= 'selected' if message.approved? %>" onclick="mumuki.Forum.discussionMessageToggleApprove('<%= approve_discussion_message_url(@discussion, message) %>', $(this))">
                 <%= fa_icon(:check, class: 'fa-xs') %>
               </a>
-              <a class="discussion-message-not-actually-a-question <%= 'selected' if message.not_actually_a_question? %>" onclick="mumuki.Forum.discussionMessageToggleNotActuallyAQuestion('<%= question_discussion_message_url(@discussion, message) %>', $(this))">
-                <%= fa_icon('question-circle-o', class: 'fa-xs') %>
-              </a>
+              <% if message.from_initiator? %>
+                <a class="discussion-message-not-actually-a-question <%= 'selected' if message.not_actually_a_question? %>" onclick="mumuki.Forum.discussionMessageToggleNotActuallyAQuestion('<%= question_discussion_message_url(@discussion, message) %>', $(this))">
+                  <%= fa_icon('question-circle-o', class: 'fa-xs') %>
+                </a>
+              <% end %>
             <% end %>
             <%= link_to fa_icon('trash-o'), discussion_message_path(@discussion, message), method: :delete, data: { confirm: t(:are_you_sure, action: t(:destroy_message)) } %>
-          </span>
-        <% end %>
+          <% end %>
+          <% if should_show_approved_for?(current_user, message) %>
+            <span class="discussion-message-approved selected">
+              <%= fa_icon(:check, class: 'fa-xs', 'data-toggle': 'tooltip', title: (t :approved_message)) %>
+            </span>
+          <% end %>
+        </span>
       </div>
     </div>
     <div class="discussion-message-bubble-content">

data/app/views/discussions/show.html.erb

@@ -9,6 +9,8 @@
 </div>
 
 <% if @discussion.has_messages? || @discussion.commentable_by?(current_user) %>
+  <hr class="message-divider">
+
   <div>
     <h3>
       <%= t :messages %>
@@ -21,8 +23,6 @@
     </h3>
   </div>
 
-  <hr class="message-divider">
-
   <% if @discussion.has_messages? %>
     <div class="discussion-messages">
       <% if @discussion.description.present? %>
@@ -41,8 +41,10 @@
 
 <% end %>
 
-<div class="discussion-actions">
-  <% @discussion.reachable_statuses_for(current_user).each do |status| %>
-    <%= discussion_update_status_button(status) %>
-  <% end if current_user %>
-</div>
+<% if current_user&.moderator_here? %>
+  <div class="discussion-actions">
+    <% @discussion.reachable_statuses_for(current_user).each do |status| %>
+      <%= discussion_update_status_button(status) %>
+    <% end %>
+  </div>
+<% end %>

data/app/views/layouts/_copyright.html.erb

@@ -1,2 +1,2 @@
 &copy; Copyright 2015-<%= DateTime.now.year %>
-<a href="http://mumuki.org/" class="mu-org-link"><span class="da da-mumuki-circle"></span> Mumuki Project</a>
+<a href="http://mumuki.org/" class="mu-org-link"><span class="da da-mumuki-circle"></span> Mumuki</a>

data/app/views/layouts/_discussions.html.erb

@@ -42,13 +42,13 @@
   <% else %>
     <div class="discussions">
       <% @filtered_discussions.each do |discussion| %>
-        <%= link_to item_discussion_path(discussion) do %>
+        <%= link_to exercise_discussion_path(discussion.exercise.id, discussion) do %>
           <div class="discussion">
             <div class="discussion-row">
               <%= discussion_messages_icon(discussion) %>
               <% unless @debatable.respond_to? :language %>
                 <div class="discussion-language-icon">
-                  <%= language_icon(discussion.language) %>
+                  <%= language_icon(discussion.exercise.language) %>
                 </div>
               <% end %>
               <div>
@@ -56,8 +56,8 @@
                   <span class="discussion-status-icon">
                     <%= discussion_status_fa_icon(discussion) %>
                   </span>
-                  <span class="hidden-sm hidden-xs"><%= discussion.exercise.friendly %></span>
-                  <span class="hidden-md hidden-lg"><%= discussion.exercise.name %></span>
+                  <span class="hidden-sm hidden-xs"><%= discussion.exercise.guide.name %> -</span>
+                  <span><%= discussion.exercise.name %></span>
                   <% if discussion.last_moderator_access_visible_for?(current_user) %>
                     <div class="pull-right discussion-moderator-access" >
                       <%= profile_picture_for(discussion.last_moderator_access_by, 32) %>

data/app/views/user_mailer/1st_reminder.html.erb

@@ -313,7 +313,7 @@
 
                             <td valign="top" class="muMailTextContent" style="padding-top:0; padding-right:18px; padding-bottom:9px; padding-left:18px;">
 
-                              © Copyright 2015-<%= DateTime.now.year %>&nbsp;<a href="https://mumuki.org/home/">Mumuki Project</a><em>.</em><br>
+                              © Copyright 2015-<%= DateTime.now.year %>&nbsp;<a href="https://mumuki.org/home/">Mumuki</a><em>.</em><br>
                               <br>
                               <%= t :stop_emails? %><br>
                               <%= link_to t(:cancel_subscription), @organization.url_for("/user/unsubscribe?id=#{@unsubscribe_code}"), target: :_blank %>

data/app/views/user_mailer/1st_reminder.text.erb

@@ -7,7 +7,7 @@
 <%= t :keep_learning %>
 
 
-© Copyright 2015-<%= DateTime.now.year %> Mumuki Project.
+© Copyright 2015-<%= DateTime.now.year %> Mumuki.
 
 <%= t :stop_emails? %>
 <%= t :cancel_subscription %>

data/app/views/user_mailer/2nd_reminder.html.erb

@@ -313,7 +313,7 @@
 
                             <td valign="top" class="muMailTextContent" style="padding-top:0; padding-right:18px; padding-bottom:9px; padding-left:18px;">
 
-                              © Copyright 2015-<%= DateTime.now.year %>&nbsp;<a href="https://mumuki.org/home/">Mumuki Project</a><em>.</em><br>
+                              © Copyright 2015-<%= DateTime.now.year %>&nbsp;<a href="https://mumuki.org/home/">Mumuki</a><em>.</em><br>
                               <br>
                               <%= t :stop_emails? %><br>
                               <%= link_to t(:cancel_subscription), @organization.url_for("/user/unsubscribe?id=#{@unsubscribe_code}"), target: :_blank %>

data/app/views/user_mailer/2nd_reminder.text.erb

@@ -7,7 +7,7 @@
 <%= t :keep_learning %>
 
 
-© Copyright 2015-<%= DateTime.now.year %> Mumuki Project.
+© Copyright 2015-<%= DateTime.now.year %> Mumuki.
 
 <%= t :stop_emails? %>
 <%= t :cancel_subscription %>

data/app/views/user_mailer/3rd_reminder.html.erb

@@ -313,7 +313,7 @@
 
                             <td valign="top" class="muMailTextContent" style="padding-top:0; padding-right:18px; padding-bottom:9px; padding-left:18px;">
 
-                              © Copyright 2015-<%= DateTime.now.year %>&nbsp;<a href="https://mumuki.org/home/">Mumuki Project</a><em>.</em><br>
+                              © Copyright 2015-<%= DateTime.now.year %>&nbsp;<a href="https://mumuki.org/home/">Mumuki</a><em>.</em><br>
                               <br>
                               <%= t :stop_emails? %><br>
                               <%= link_to t(:cancel_subscription), @organization.url_for("/user/unsubscribe?id=#{@unsubscribe_code}"), target: :_blank %>

data/app/views/user_mailer/3rd_reminder.text.erb

@@ -7,7 +7,7 @@
 <%= t :keep_learning %>
 
 
-© Copyright 2015-<%= DateTime.now.year %> Mumuki Project.
+© Copyright 2015-<%= DateTime.now.year %> Mumuki.
 
 <%= t :stop_emails? %>
 <%= t :cancel_subscription %>

data/app/views/user_mailer/no_submissions_reminder.html.erb

@@ -313,7 +313,7 @@
 
                             <td valign="top" class="muMailTextContent" style="padding-top:0; padding-right:18px; padding-bottom:9px; padding-left:18px;">
 
-                              © Copyright 2015-<%= DateTime.now.year %>&nbsp;<a href="https://mumuki.org/home/">Mumuki Project</a><em>.</em><br>
+                              © Copyright 2015-<%= DateTime.now.year %>&nbsp;<a href="https://mumuki.org/home/">Mumuki</a><em>.</em><br>
                               <br>
                               <%= t :stop_emails? %><br>
                               <%= link_to t(:cancel_subscription), @organization.url_for("/user/unsubscribe?id=#{@unsubscribe_code}"), target: :_blank %>

data/app/views/user_mailer/no_submissions_reminder.text.erb

@@ -7,7 +7,7 @@
 <%= t :keep_learning %>
 
 
-© Copyright 2015-<%= DateTime.now.year %> Mumuki Project.
+© Copyright 2015-<%= DateTime.now.year %> Mumuki.
 
 <%= t :stop_emails? %>
 <%= t :cancel_subscription %>

data/lib/mumuki/laboratory/locales/en.yml

@@ -10,6 +10,7 @@ en:
   all: All
   appendix: "Appendix"
   appendix_teaser: Do you want to learn more? <a href="%{link}">Check this chapter's appendix</a>"
+  approved_message: Validated by mentor
   are_you_sure: 'Are you sure you want to %{action}?'
   ask_a_question: Ask a question!
   ask_community: Ask community for help
@@ -134,7 +135,7 @@ en:
   messages: Messages
   messages_error: Previous messages are unavailable. Please try again later.
   more_messages: More
-  moderator: Moderator
+  moderator: Mentor
   mumuki_catchphrase: Improve your programming skills
   mumuki_short_description: Mumuki is a simple, open and collaborative platform for sharing and solving programming exercises. It is aimed to help people with learning and teaching programing languages and paradigms
   my_doubts: My doubts
@@ -156,7 +157,7 @@ en:
   no_messages: It seems you don't have any messages yet!
   no_questions: It seems there isn't any question yet.
   no_submissions: It seems like you haven't tried to solve this exercise
-  no_useful_result: Não encontrou o que estava buscando?
+  no_useful_result: Didn't find what you were looking for?
   not_found_explanation: 'You may have mistyped the address or the page may have moved.'
   not_in_any_organizations: It seems you aren't in any organizations yet!
   notify_problem_with_exercise: Report a bug

data/lib/mumuki/laboratory/locales/es.yml

@@ -11,6 +11,7 @@ es:
   and: y
   appendix: "Apéndice"
   appendix_teaser: ¿Querés saber más? <a href="%{link}">Consultá el apéndice de este capítulo</a>
+  approved_message: Validado por mentor
   are_you_sure: '¿Estás seguro que querés %{action}?'
   ask_a_question: ¡Hacé una pregunta!
   ask_community: Preguntá a la comunidad
@@ -146,7 +147,7 @@ es:
   minute: minuto
   minutes: minutos
   more_messages: Ver mensajes anteriores
-  moderator: Moderador
+  moderator: Mentor
   mumuki_catchphrase: Aprendé a programar
   mumuki_short_description: Mumuki es la plataforma libre y gratuita para aprender a programar, de la práctica a la teoría y en tu idioma
   my_doubts: Mis consultas

data/lib/mumuki/laboratory/locales/pt.yml

@@ -11,6 +11,7 @@ pt:
   and: e
   appendix: Apêndice
   appendix_teaser: Você quer saber mais? <a href="%{link}"> Consulte o apêndice deste capítulo </a>
+  approved_message: Validado pelo mentor
   are_you_sure: 'Tem certeza de que deseja %{action}?'
   ask_a_question: Faça uma pergunta!
   ask_community: Pergunte à comunidade
@@ -142,7 +143,7 @@ pt:
   minute: minuto
   minutes: minutos
   more_messages: Ver as mensagens anteriores
-  moderator: Moderador
+  moderator: Mentor
   mumuki_catchphrase: Aprendi a programar
   mumuki_short_description: Mumuki é a plataforma gratuita e gratuita para aprender a programar, desde a prática até a teoria e no seu idioma
   my_doubts: Minhas duvidas
@@ -164,7 +165,7 @@ pt:
   no_messages: Parece que você ainda não tem uma mensagem!
   no_questions: Parece que ainda não há dúvida.
   no_submissions: Parece que você ainda não tentou resolver este exercício!
-  no_useful_result: Didn't find what you were looking for
+  no_useful_result: Não encontrou o que estava buscando?
   not_found: A página que você pesquisou não existe!
   not_found_explanation: Olhe se você escreveu o endereço corretamente. Mas...
   not_in_any_organizations: Parece que você ainda não está em nenhuma organização!

data/lib/mumuki/laboratory/version.rb

@@ -1,5 +1,5 @@
 module Mumuki
   module Laboratory
-    VERSION = '7.7.2'
+    VERSION = '7.7.3'
   end
 end

data/spec/controllers/discussions_messages_controller_spec.rb

@@ -0,0 +1,73 @@
+require 'spec_helper'
+
+describe DiscussionsMessagesController, organization_workspace: :test do
+  let(:student) { create(:user, permissions: {student: 'test/*'}) }
+  let(:moderator) { create(:user, permissions: {moderator: 'test/*', student: 'test/*'}) }
+
+  let(:exercise) { create(:exercise) }
+  let(:discussion) { create(:discussion, item: exercise, organization: Organization.current) }
+
+  before { Organization.current.update! forum_enabled: true }
+
+  describe 'post' do
+    describe 'for student' do
+      before { set_current_user! student }
+      before { allow_any_instance_of(DiscussionsMessagesController).to receive(:message_params).and_return content: 'Need help' }
+      before { post :create, params: {discussion_id: discussion.id} }
+
+      it { expect(response.status).to eq 302 }
+      it { expect(discussion.messages.size).to eq 1 }
+      it { expect(discussion.messages.last.content).to eq 'Need help' }
+    end
+
+    describe 'for moderator' do
+      before { set_current_user! moderator }
+      before { allow_any_instance_of(DiscussionsMessagesController).to receive(:message_params).and_return content: 'Do this!' }
+      before { post :create, params: {discussion_id: discussion.id} }
+
+      it { expect(response.status).to eq 302 }
+      it { expect(discussion.messages.size).to eq 1 }
+      it { expect(discussion.messages.last.content).to eq 'Do this!' }
+    end
+  end
+
+  describe 'approve' do
+    let(:message) { create(:message, discussion: discussion, sender: student.uid) }
+
+    describe 'for student' do
+      before { set_current_user! student }
+      before { post :approve, params: {id: message.id, discussion_id: discussion.id} }
+
+      it { expect(response.status).to eq 403 }
+      it { expect(message.reload.approved).to be false }
+    end
+
+    describe 'for student' do
+      before { set_current_user! moderator }
+      before { post :approve, params: {id: message.id, discussion_id: discussion.id} }
+
+      it { expect(response.status).to eq 200 }
+      it { expect(message.reload.approved).to be true }
+    end
+  end
+
+  describe 'question' do
+    let(:message) { create(:message, discussion: discussion, sender: student.uid) }
+
+    describe 'for student' do
+      before { set_current_user! student }
+      before { post :question, params: {id: message.id, discussion_id: discussion.id} }
+
+      it { expect(response.status).to eq 403 }
+      it { expect(message.reload.not_actually_a_question).to be false }
+    end
+
+    describe 'for student' do
+      before { set_current_user! moderator }
+      before { post :question, params: {id: message.id, discussion_id: discussion.id} }
+
+      it { expect(response.status).to eq 200 }
+      it { expect(message.reload.not_actually_a_question).to be true }
+    end
+  end
+end

data/spec/dummy/db/schema.rb

@@ -385,6 +385,7 @@ ActiveRecord::Schema.define(version: 20200731081757) do
     t.string "verified_last_name"
     t.bigint "avatar_id"
     t.datetime "disabled_at"
+    t.boolean "trusted_for_forum"
     t.index ["disabled_at"], name: "index_users_on_disabled_at"
     t.index ["last_organization_id"], name: "index_users_on_last_organization_id"
     t.index ["uid"], name: "index_users_on_uid", unique: true

data/spec/features/discussion_flow_spec.rb

@@ -0,0 +1,190 @@
+require 'spec_helper'
+
+feature 'Discussion Flow', organization_workspace: :test do
+
+  ## =============
+  ## Users
+  ## =============
+
+  let(:student) { create(:user, permissions: {student: 'test/*:other/*:empty/*'}) }
+  let(:another_student) { create(:user, permissions: {student: 'test/*:other/*:empty/*'}) }
+  let(:moderator) { create(:user, permissions: {moderator: 'test/*:other/*:empty/*', student: 'test/*:other/*:empty/*'}) }
+
+  ## =================
+  ## Organizations
+  ## =================
+
+  let(:test_organization) { Organization.locate! 'test' }
+  let(:other_organization) { create(:organization, name: 'other') }
+  let!(:empty_organization) { create(:organization, name: 'empty') }
+
+  ## ================================
+  ## Content for test_organization
+  ## ================================
+
+  let(:problem_1) { create(:problem) }
+  let(:problem_2) { create(:problem) }
+  let(:problem_3) { create(:problem) }
+
+  let!(:chapter) {
+    create(:chapter, lessons: [create(:lesson, exercises: [ problem_1, problem_2, problem_3])])
+  }
+
+  ## =====================================
+  ## Discussions for test_organization
+  ## =====================================
+
+  let(:problem_2_discussions) { create_list(:discussion, 5, initiator: another_student, item: problem_2, organization: test_organization) }
+  let(:problem_3_discussions) { create_list(:discussion, 5, initiator: student, item: problem_2, organization: test_organization) }
+  let!(:test_organization_discussions) { problem_2_discussions + problem_3_discussions }
+
+  ## ===================================
+  ## Content for other_organization
+  ## ===================================
+
+  let(:other_problem) { create(:problem) }
+  let!(:other_chapter) {
+    create(:chapter, book: other_organization.book, lessons: [create(:lesson, exercises: [other_problem])])
+  }
+
+  ## =====================================
+  ## Discussions for other_organization
+  ## =====================================
+
+  let(:other_problem_discussions) { create_list(:discussion, 5, initiator: student, item: problem_2, organization: other_organization) }
+  let!(:other_organization_discussions) { other_problem_discussions }
+
+  before { reindex_current_organization! }
+  before { reindex_organization! other_organization }
+  before { reindex_organization! empty_organization }
+
+  shared_examples 'no forum access' do
+    scenario 'has no forum access' do
+      visit current_path
+      expect(page).to have_text('You may have mistyped the address or the page may have moved')
+    end
+  end
+
+  context 'exercise discussions' do
+    let(:current_path) { "/exercises/#{problem_1.id}/discussions" }
+
+    context 'with no current user' do
+      it_behaves_like 'no forum access'
+    end
+
+    context 'with logged user' do
+      before { set_current_user! student }
+
+      context 'but no forum enabled' do
+        it_behaves_like 'no forum access'
+      end
+
+      context 'and forum enabled' do
+        before { Organization.current.update! forum_enabled: true }
+
+        scenario 'empty discussion list for problem 1' do
+          visit "/exercises/#{problem_1.id}/discussions"
+          expect(page).to have_text(problem_1.name)
+          expect(page).to have_text('It seems there isn\'t any question yet.')
+        end
+
+        scenario 'discussion list for problem 2' do
+          visit "/exercises/#{problem_2.id}/discussions"
+          expect(page).to have_text(problem_2.name)
+          problem_2_discussions.each do |discussion|
+            expect(page).to have_text(discussion.description)
+          end
+          expect(page).to have_text('Didn\'t find what you were looking for?')
+        end
+      end
+    end
+  end
+
+  context 'book discussions' do
+    let(:current_path) { '/discussions' }
+
+    context 'with no current user' do
+      it_behaves_like 'no forum access'
+    end
+
+    context 'with logged user' do
+      before { set_current_user! student }
+
+      context 'but no forum enabled' do
+        it_behaves_like 'no forum access'
+      end
+
+      context 'and forum enabled' do
+        before { Organization.current.update! forum_enabled: true }
+
+        context 'in empty organization' do
+          before { empty_organization.update! forum_enabled: true }
+          before { set_subdomain_host! 'empty' }
+          after { set_subdomain_host! 'test' }
+
+          scenario 'empty discussion list for book without discussions' do
+            visit current_path
+            expect(page).to have_text(empty_organization.book.name)
+            expect(page).to have_text('It seems there isn\'t any question yet.')
+          end
+
+        end
+
+        scenario 'discussion list for book with discussions' do
+          visit current_path
+          expect(page).to have_text(test_organization.book.name)
+          test_organization_discussions.each do |discussion|
+            expect(page).to have_text(discussion.description)
+          end
+          other_organization_discussions.each do |discussion|
+            expect(page).not_to have_text(discussion.description)
+          end
+          expect(page).not_to have_text('Didn\'t find what you were looking for?')
+          expect(page).not_to have_text('It seems there isn\'t any question yet.')
+        end
+      end
+    end
+  end
+
+  context 'exercise discussion' do
+    let(:current_path) { "/exercises/#{problem_2.id}/discussions/#{problem_2_discussions.first.id}" }
+    before { test_organization.switch! }
+
+    context 'with no current user' do
+      it_behaves_like 'no forum access'
+    end
+
+    context 'with logged student' do
+      before { set_current_user! student }
+
+      context 'but no forum enabled' do
+        it_behaves_like 'no forum access'
+      end
+
+      context 'and forum enabled' do
+        before { Organization.current.update! forum_enabled: true }
+
+        scenario 'newly created discussion' do
+          visit current_path
+          expect(page).to have_text(problem_2.name)
+          expect(page).to have_text('Open')
+          expect(page).to have_text('Messages')
+          expect(page).not_to have_xpath("//div[@class='discussion-actions']")
+        end
+
+        context 'for moderator' do
+          before { set_current_user! moderator }
+
+          scenario 'newly created discussion' do
+            visit current_path
+            expect(page).to have_text(problem_2.name)
+            expect(page).to have_text('Open')
+            expect(page).to have_text('Messages')
+            expect(page).to have_xpath("//div[@class='discussion-actions']")
+          end
+        end
+      end
+    end
+  end
+
+end

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/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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mumuki-laboratory
 version: !ruby/object:Gem::Version
-  version: 7.7.2
+  version: 7.7.3
 platform: ruby
 authors:
 - Franco Bulgarelli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-08-01 00:00:00.000000000 Z
+date: 2020-08-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 7.7.1
+        version: 7.7.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 7.7.1
+        version: 7.7.2
 - !ruby/object:Gem::Dependency
   name: mumukit-bridge
   requirement: !ruby/object:Gem::Requirement
@@ -424,6 +424,7 @@ files:
 - app/controllers/concerns/users_controller_template.rb
 - app/controllers/concerns/with_authorization.rb
 - app/controllers/concerns/with_errors_filter.rb
+- app/controllers/concerns/with_user_discussion_validation.rb
 - app/controllers/concerns/with_user_params.rb
 - app/controllers/discussions_controller.rb
 - app/controllers/discussions_messages_controller.rb
@@ -632,6 +633,7 @@ files:
 - spec/controllers/confirmations_controller_spec.rb
 - spec/controllers/courses_api_controller_spec.rb
 - spec/controllers/discussions_controller_spec.rb
+- spec/controllers/discussions_messages_controller_spec.rb
 - spec/controllers/exercise_solutions_controller_spec.rb
 - spec/controllers/guide_progress_controller_spec.rb
 - spec/controllers/invitations_controller_spec.rb
@@ -684,6 +686,7 @@ files:
 - spec/features/complements_flow_spec.rb
 - spec/features/disable_user_flow_spec.rb
 - spec/features/disabled_organization_flow_spec.rb
+- spec/features/discussion_flow_spec.rb
 - spec/features/dynamic_exam_spec.rb
 - spec/features/exams_flow_spec.rb
 - spec/features/exercise_flow_spec.rb
@@ -713,10 +716,19 @@ files:
 - spec/helpers/test_results_rendering_spec.rb
 - spec/helpers/with_choices_spec.rb
 - spec/helpers/with_navigation_spec.rb
+- spec/javascripts/bridge-spec.js
+- spec/javascripts/csrf-token-spec.js
+- spec/javascripts/elipsis-spec.js
+- spec/javascripts/results-renderers-spec.js
+- spec/javascripts/spec-helper.js
+- spec/javascripts/speech-bubble-renderer-spec.js
+- spec/javascripts/timeout-spec.js
+- spec/javascripts/timer-spec.js
 - spec/login_helper.rb
 - spec/mailers/previews/user_mailer_preview.rb
 - spec/mailers/user_mailer_spec.rb
 - spec/spec_helper.rb
+- spec/teaspoon_env.rb
 - vendor/assets/javascripts/analytics.js
 - vendor/assets/javascripts/codemirror-autorefresh.js
 - vendor/assets/javascripts/codemirror-modes/assembly_x86.js
@@ -784,6 +796,7 @@ test_files:
 - spec/mailers/previews/user_mailer_preview.rb
 - spec/mailers/user_mailer_spec.rb
 - spec/spec_helper.rb
+- spec/teaspoon_env.rb
 - spec/features/guide_reset_spec.rb
 - spec/features/complements_flow_spec.rb
 - spec/features/not_found_public_flow_spec.rb
@@ -798,6 +811,7 @@ test_files:
 - spec/features/home_private_flow_spec.rb
 - spec/features/lessons_flow_spec.rb
 - spec/features/dynamic_exam_spec.rb
+- spec/features/discussion_flow_spec.rb
 - spec/features/not_found_private_flow_spec.rb
 - spec/features/progressive_tips_spec.rb
 - spec/features/exams_flow_spec.rb
@@ -860,6 +874,14 @@ test_files:
 - spec/api_helper.rb
 - spec/capybara_helper.rb
 - spec/evaluation_helper.rb
+- spec/javascripts/bridge-spec.js
+- spec/javascripts/speech-bubble-renderer-spec.js
+- spec/javascripts/timeout-spec.js
+- spec/javascripts/results-renderers-spec.js
+- spec/javascripts/csrf-token-spec.js
+- spec/javascripts/elipsis-spec.js
+- spec/javascripts/timer-spec.js
+- spec/javascripts/spec-helper.js
 - spec/controllers/discussions_controller_spec.rb
 - spec/controllers/chapters_controller_spec.rb
 - spec/controllers/students_api_controller_spec.rb
@@ -869,6 +891,7 @@ test_files:
 - spec/controllers/invitations_controller_spec.rb
 - spec/controllers/users_api_controller_spec.rb
 - spec/controllers/users_controller_spec.rb
+- spec/controllers/discussions_messages_controller_spec.rb
 - spec/controllers/guide_progress_controller_spec.rb
 - spec/controllers/organizations_api_controller_spec.rb
 - spec/controllers/messages_controller_spec.rb