mumuki-laboratory 8.3.0 → 9.0.0

data/spec/features/notifications_flow_spec.rb

@@ -0,0 +1,46 @@
+require 'spec_helper'
+
+feature 'Notifications Flow', organization_workspace: :test do
+  let!(:chapter) { create(:chapter) }
+  let(:organization) { Organization.current }
+  let(:user) { create(:user) }
+
+  before { reindex_current_organization! }
+  before { set_current_user!(user) }
+
+  def notifications_bell
+    find '.badge-notifications'
+  end
+
+  def find_notification_number(number)
+    find('#notificationsDropdown').click
+    find "#notificationsPanel li:nth-child(#{number}) a"
+  end
+
+  context 'user with notifications' do
+    let(:exam_registration) { create(:exam_registration, description: 'Mid term exam 2020') }
+    let(:exam_authorization_request) { create(:exam_authorization_request, exam_registration: exam_registration, user: user) }
+
+    let!(:notifications) do
+      [exam_registration, exam_authorization_request].map { |target| create(:notification, user: user, target: target ) }
+    end
+
+    before { visit '/' }
+
+    scenario 'displays count on navigation bar' do
+      expect(notifications_bell).to have_text('2')
+    end
+
+    scenario 'navigates to target on notification click' do
+      find_notification_number(2).click
+      expect(page).to have_text 'Registration to Mid term exam 2020'
+      expect(page).to have_text 'Choose date and time to attend to the exam'
+    end
+
+    scenario 'removes notification after target is processed' do
+      # Notification for exam authorization request is considered read after user click
+      find_notification_number(1).click
+      expect(notifications_bell).to have_text('1')
+    end
+  end
+end

data/spec/features/profile_flow_spec.rb

@@ -95,34 +95,31 @@ feature 'Profile Flow', organization_workspace: :test do
           visit "/user/edit"
           fill_in('user_first_name', with: 'first_name')
 
-          # Match :first is used because there are two buttons: mobile and desktop.
           click_on(button_options)
-          expect(page).to have_text('Your data was updated successfuly')
-          expect(page).to have_text('Profile')
+          expect(page).to have_text('Your data was updated successfully')
+          expect(page).to have_text('My profile')
         end
       end
     end
 
     context 'with no messages' do
-
-      scenario 'visit messages tab' do
-        visit "/user#messages"
+      scenario 'visit messages' do
+        visit "/user/messages"
 
         expect(page).to have_text('It seems you don\'t have any messages yet!')
       end
     end
 
     context 'with messages' do
-      scenario 'visit messages tab' do
+      scenario 'visit messages' do
         Organization.find_by_name('test').switch!
         problem.submit_solution! user, {content: 'something'}
         Message.import_from_resource_h! message
-        visit "/user#messages"
+        visit "/user/messages"
 
         expect(page).to_not have_text('It seems you don\'t have any messages yet!')
         expect(page).to have_text(problem.name)
       end
     end
   end
-
 end

data/spec/features/terms_flow_spec.rb

@@ -153,5 +153,35 @@ feature 'Terms Flow', organization_workspace: :test do
     end
   end
 
+  context 'with incognito mode' do
+    before { test_organization.update! incognito_mode_enabled: true }
+
+    describe 'visit user terms path' do
+      let(:terms_path) { '/user/terms' }
+      let(:expected_terms) { general_terms_scopes }
+
+      it_behaves_like 'has expected terms'
+    end
+
+    scenario 'visit any other path' do
+      visit '/'
+
+      expect(page).to have_text('Start Practicing')
+    end
+
+    context 'visit forum' do
+      let(:terms_path) { '/discussions/terms' }
+
+      context 'with enabled forum' do
+        before { test_organization.update! forum_enabled: true }
+
+        scenario 'visit forum' do
+          visit '/discussions/terms'
+          expect(page).to have_text('You may have mistyped the address or the page may have moved')
+        end
+      end
+    end
+  end
+
 end
 

data/spec/javascripts/bridge-spec.js

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

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

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

data/spec/javascripts/editors-spec.js

@@ -1,11 +1,10 @@
 describe('editors', () => {
-
   beforeEach(() => {
-    mumuki.CustomEditor.clearSources()
-  })
+    mumuki.CustomEditor.clearSources();
+  });
 
   it('has initially no sources', () => {
-    expect(mumuki.CustomEditor.hasSources).toBe(false)
+    expect(mumuki.CustomEditor.hasSources).toBe(false);
   });
 
   it('can add a custom source', () => {
@@ -26,7 +25,7 @@ describe('editors', () => {
       <div class="field form-group editor-code">
         <textarea class="form-control editor" name="solution[content]" id="solution_content">the standard solution</textarea>
       </div>
-    </form>`)
+    </form>`);
 
     mumuki.editors.addCustomSource({
       getContent() {
@@ -43,11 +42,10 @@ describe('editors', () => {
       <div class="field form-group editor-code">
         <textarea class="form-control editor" name="solution[content]" id="solution_content">the solution</textarea>
       </div>
-    </form>`)
+    </form>`);
     expect(mumuki.editors.getSubmission()).toEqual({"solution[content]":"the solution"});
   });
 
-
   it('reads the form if no sources and exercise is multifile', () => {
     $('body').html(`
     <form role="form" class="new_solution">
@@ -63,7 +61,7 @@ describe('editors', () => {
           name="solution[content[receta.css]]"
           id="solution_content[receta.css]">some css</textarea>
       </div>
-    </form>`)
+    </form>`);
     expect(mumuki.editors.getSubmission()).toEqual({
       "solution[content[index.html]]": "some html",
       "solution[content[receta.css]]": "some css"
@@ -74,4 +72,4 @@ describe('editors', () => {
     $('body').html(``);
     expect(mumuki.editors.getSubmission()).toEqual({});
   });
-})
+});

data/spec/javascripts/elipsis-spec.js

@@ -1,7 +1,7 @@
 describe('elipsis', () => {
   it('does nothing when no elipsis', () => {
     expect(mumuki.elipsis('hello')).toEqual('hello');
-  })
+  });
 
   it('replaces student elipsis', () => {
     expect(mumuki.elipsis(`function longitud(unString) {
@@ -10,7 +10,7 @@ describe('elipsis', () => {
       /*@elipsis-for-student>*/
     }`)).toEqual(`function longitud(unString) {
       /* ... */
-    }`)
+    }`);
   });
 
   it('replaces student hidden', () => {
@@ -20,6 +20,6 @@ describe('elipsis', () => {
       /*@hidden-for-student>*/
     }`)).toEqual(`function longitud(unString) {
       /**/
-    }`)
+    }`);
   });
-})
+});

data/spec/javascripts/events-spec.js

@@ -6,18 +6,18 @@ describe('events', () => {
   it('is not called when it is not fired', () => {
     mumuki.events.on('foo', (e) => {
       fail(`should not be called, but got ${JSON.stringify(e)}`);
-    })
-  })
+    });
+  });
 
   it('is not called when it is fired but not enabled', () => {
     let fired = false;
     mumuki.events.on('foo', (e) => {
       fail(`should not be called, but got ${JSON.stringify(e)}`);
       fired = true;
-    })
+    });
     mumuki.events.fire('foo', 42);
     expect(fired).toBe(false);
-  })
+  });
 
   it('is called when it is fired and enabled', () => {
     let fired = false;
@@ -25,9 +25,9 @@ describe('events', () => {
     mumuki.events.on('foo', (event) => {
       expect(event).toEqual(42);
       fired = true;
-    })
+    });
 
     mumuki.events.fire('foo', 42);
     expect(fired).toBe(true);
-  })
-})
+  });
+});

data/spec/javascripts/exercise-spec.js

@@ -1,10 +1,9 @@
 describe('exercise', () => {
-
   it('current exercise information is available when present', () => {
     $('body').html(`
     <input type="hidden" name="mu-exercise-id" id="mu-exercise-id" value="3361" />
     <input type="hidden" name="mu-exercise-layout" id="mu-exercise-layout" value="input_right" />
-    <input type="hidden" name="mu-exercise-settings" id="mu-exercise-settings" value="{}" />`)
+    <input type="hidden" name="mu-exercise-settings" id="mu-exercise-settings" value="{}" />`);
 
     mumuki.exercise.load();
 
@@ -12,13 +11,13 @@ describe('exercise', () => {
     expect(mumuki.exercise.layout).toBe('input_right');
     expect(mumuki.exercise.settings).toEqual({});
     expect(mumuki.exercise.current).not.toBe(null);
-  })
+  });
 
   it('current exercise information is available when present and settings are not empty', () => {
     $('body').html(`
     <input type="hidden" name="mu-exercise-id" id="mu-exercise-id" value="3361" />
     <input type="hidden" name="mu-exercise-layout" id="mu-exercise-layout" value="input_right" />
-    <input type="hidden" name="mu-exercise-settings" id="mu-exercise-settings" value="{&quot;game_framework&quot;:true}" />`)
+    <input type="hidden" name="mu-exercise-settings" id="mu-exercise-settings" value="{&quot;game_framework&quot;:true}" />`);
 
     mumuki.exercise.load();
 
@@ -26,10 +25,10 @@ describe('exercise', () => {
     expect(mumuki.exercise.layout).toBe('input_right');
     expect(mumuki.exercise.settings.game_framework).toBe(true);
     expect(mumuki.exercise.current).not.toBe(null);
-  })
+  });
 
   it('current exercise information is available when not present', () => {
-    $('body').html(``)
+    $('body').html(``);
 
     mumuki.exercise.load();
 
@@ -37,5 +36,5 @@ describe('exercise', () => {
     expect(mumuki.exercise.layout).toBe(null);
     expect(mumuki.exercise.settings).toBe(null);
     expect(mumuki.exercise.current).toBe(null);
-  })
-})
+  });
+});

data/spec/javascripts/global-spec.js

@@ -1,6 +1,6 @@
 describe("global loading", () => {
   it("produces no global loading errors", () => {
     const error = window['__globalLoadingError__'];
-    expect(!error).toBe(true, `Expected no global loading errors but got ${error && error.message}`)
-  })
-})
+    expect(!error).toBe(true, `Expected no global loading errors but got ${error && error.message}`);
+  });
+});

data/spec/javascripts/i18n-spec.js

@@ -1,29 +1,30 @@
 describe('I18n', () => {
-
   describe('t / translate', () => {
-
     it('accept english translations', () => {
       mumuki.locale = 'en';
       expect(mumuki.I18n.translate('passed')).toBe('Everything is in order! Your solution passed all our tests!');
-    })
+    });
+
     it('accept spanish translations', () => {
       mumuki.locale = 'es';
       expect(mumuki.I18n.translate('passed_with_warnings')).toBe('Tu solución funcionó, pero hay cosas que mejorar');
-    })
+    });
+
     it('accept chilean translations', () => {
       mumuki.locale = 'es-CL';
       expect(mumuki.I18n.translate('failed')).toBe('Tu solución no pasó las pruebas');
-    })
+    });
+
     it('accept portuguese translations', () => {
       mumuki.locale = 'pt';
       expect(mumuki.I18n.translate('errored')).toBe('Opa! Sua solução não pode ser executada');
-    })
+    });
+
     it('fails when translation missing', () => {
       mumuki.locale = 'en';
       expect(mumuki.I18n.translate('foo')).toBe('Translation missing: en, `foo`');
-    })
-
-  })
+    });
+  });
 
   describe('register', () => {
     beforeEach(() => {
@@ -33,21 +34,23 @@ describe('I18n', () => {
           greet: (data) => `Hi ${data.name}`,
           errored: "D'Oh!"
         }
-      })
-    })
+      });
+    });
+
     it('overrides existing translation key', () => {
       expect(mumuki.I18n.translate('errored')).toBe("D'Oh!");
-    })
+    });
+
     it('add missing translation key', () => {
       expect(mumuki.I18n.translate('greet', {name: 'Jane'})).toBe('Hi Jane');
-    })
+    });
+
     it('keep non overriding translation key', () => {
       expect(mumuki.I18n.translate('passed')).toBe('Everything is in order! Your solution passed all our tests!');
-    })
-  })
+    });
+  });
 
   describe('with some prefix', () => {
-
     fixture.set(`
         <div class="mu-kindergarten" data-i18n-prefix="testing">
             <button class="mu-kids-button">Click me<button>
@@ -61,19 +64,19 @@ describe('I18n', () => {
         en: {
           testing_failed: 'Ops! Execution failed',
         }
-      })
+      });
     });
 
     it('Using prefix with existing translation key', () => {
       expect(mumuki.I18n.t('passed')).toBe('Everything is in order! Your solution passed all our tests!');
     });
+
     it('Use prefix with none existing translation key but default key exists', () => {
       expect(mumuki.I18n.t('failed')).toBe('Ops! Execution failed');
     });
+
     it('No key found', () => {
       expect(mumuki.I18n.t('foo')).toBe('Translation missing: en, `foo`');
-    })
-
+    });
   });
-
 });

data/spec/javascripts/kids-button-spec.js

@@ -1,5 +1,4 @@
 describe('KidsButton', () => {
-
   let button;
 
   fixture.set(`
@@ -12,25 +11,24 @@ describe('KidsButton', () => {
   beforeEach(() => {
     mumuki.kids = new mumuki.Kids();
     button = new mumuki.submission.KidsSubmitButton($('.mu-kids-button'));
-  })
+  });
 
   it('can create button', () => {
     expect(button).not.toBe(null);
-  })
+  });
 
   it('overlay is hidden by default', () => {
     expect(mumuki.kids.$overlay.css('display')).toBe('none');
-  })
+  });
 
   it('call showOverlay on wait', () => {
     button.wait();
     expect(mumuki.kids.$overlay.css('display')).toBe('block');
-  })
+  });
 
   it('call hideOverlay on continue', () => {
     button.wait();
     button.continue();
     expect(mumuki.kids.$overlay.css('display')).toBe('none');
-  })
-
-})
+  });
+});