mumuki-laboratory 7.3.1 → 7.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2f2119060da9d8e6e5c83715a763f3f78d6433c7da2ed7de4f63ad0d36b9a85e
-  data.tar.gz: 73a5e031b87d2aeafd20bb9bf38d679badf56ea6bac35415c6397219841873b1
+  metadata.gz: 29f66dce581024a0293143b796f0eb335d70e95caa9ea46c502722795cbba45b
+  data.tar.gz: b469e0fcd7e4641b5f20f2efe445317d80427da353e1410c9f4a05a98b71c4de
 SHA512:
-  metadata.gz: c81b960760d56143c6b393418eac611d08c45cc31f827998c53833ea852dfbef23f082b138c203475cefedb4bf74e67f7d0f0079e1588606730416159ab5a038
-  data.tar.gz: 6dd3f6673aed2e5d4119171e5f361727bc4a6518fe409ea6d49db8dba53ace606f0578dbde35e9d85396dc5095eee7749cf38c5132a593fd77b13be94b52a9d7
+  metadata.gz: 283c6fda05975f588f00256b7e3f064e0dfb91f22e2b6268141c2c71f23ca0a61096f7f881e2669807c4f6adcfa2d78ebe30e1f690df8bd2227cb17d91eece72
+  data.tar.gz: ab0b73b864f01d3ca18249ecbf7dabd2fa2fb8d14b4caeb1d2f868e2ee3588d0893d27fb501b7c62dccbbab0409603b26c7060840e6bcc51aff1238e232400ae

data/README.md

@@ -178,6 +178,12 @@ which are granted to be safe and stable.
 
 * `mumuki.bridge.Laboratory`
   * `.runTests`
+* `mumuki.editor`
+  * `formatContent`
+  * `reset`
+  * `toggleFullscreen`
+* `mumuki.elipsis`
+  * `replaceHtml`
 * `mumuki.kids`
   * `registerBlocksAreaScaler`
   * `registerStateScaler`
@@ -185,6 +191,9 @@ which are granted to be safe and stable.
   * `scaleBlocksArea`
   * `scaleState`
   * `showResult`
+* `mumuki.renderers`
+  * `SpeechBubbleRenderer`
+  * `renderSpeechBubbleResultItem`
 * `mumuki.locale`
 * `mumuki.MultipleScenarios`
   * `scenarios`
@@ -200,22 +209,29 @@ which are granted to be safe and stable.
 
 ### Bridge Response Format
 
-```json
+```javascript
 {
-  "status": "failed",
-  "guide_finished_by_solution": false,
-  "class_for_progress_list_item":"progress-list-item text-center danger active",
-  "html":"...",
-  "title_html":"...",
-  "button_html":"...",
-  "expectations_html":"...",
-  "remaining_attempts_html":null,
-  "test_results":[
-      {
-        "title":null,
-        "status":"failed",
-        "result":"..."
-      }
+  "status": "passed|passed_with_warnings|failed",
+  "guide_finished_by_solution": "boolean",
+  "class_for_progress_list_item": "string",
+  "html": "string",
+  "remaining_attempts_html": "string" ,
+  "title_html": "string",                         // kids-only
+  "button_html": "string",                        // kids-only
+  "expectations": [                               // kids-only
+    {
+      "status": "passed|failed",
+      "explanation": "string"
+    }
+  ],
+  "tips": [ "string" ],                           // kids-only
+  "test_results": [                               // kids-only
+    {
+      "title": "string",
+      "status": "passed|failed",
+      "result": "string",
+      "summary": "string"
+    }
   ]
 }
 ```
@@ -463,7 +479,7 @@ POST /organization/:id/courses/
 
 ```json
 {
-   "name":"....",
+   "name": "....",
 }
 ```
 
@@ -539,7 +555,7 @@ Sample response body:
 ```json
 {
   "organizations": [
-    { "name": "academy", "contact_email": "a@a.com", "locale": "es-AR", "login_methods": ["facebook"], "books": ["libro"], "public": true, "logo_url":"http://..." },
+    { "name": "academy", "contact_email": "a@a.com", "locale": "es-AR", "login_methods": ["facebook"], "books": ["libro"], "public": true, "logo_url": "http://..." },
     { "name": "alcal", "contact_email": "b@b.com", "locale": "en-US", "login_methods": ["facebook", "github"], "books": ["book"], "public": false }
   ]
 }
@@ -555,7 +571,7 @@ get /organizations/:name
 Sample response body:
 
 ```json
-{ "name": "academy", "contact_email": "a@a.com", "locale": "es-AR", "login_methods": ["facebook"], "books": ["libro"], "public": true, "logo_url":"http://..." }
+{ "name": "academy", "contact_email": "a@a.com", "locale": "es-AR", "login_methods": ["facebook"], "books": ["libro"], "public": true, "logo_url": "http://..." }
 ```
 **Minimal permission**: `janitor` of the organization.
 

data/app/assets/javascripts/mumuki_laboratory/application/codemirror.js

@@ -23,6 +23,14 @@ var mumuki = mumuki || {};
     })
   }
 
+  function formatContent() {
+    mumuki.page.editors.each(function (_, editor) {
+      editor.setSelection({line: 0, ch: 0}, {line: editor.lineCount()})
+      editor.indentSelection("smart")
+      editor.setSelection({line: 0})
+    });
+  }
+
   function toggleFullscreen() {
     $('body').toggleClass('fullscreen');
     $('.editor-resize .fa-stack-1x').toggleClass('fa-expand').toggleClass('fa-compress');
@@ -61,7 +69,9 @@ var mumuki = mumuki || {};
   }
 
   mumuki.editor = mumuki.editor || {};
+  mumuki.editor.reset = resetEditor;
   mumuki.editor.toggleFullscreen = toggleFullscreen;
+  mumuki.editor.formatContent = formatContent;
   mumuki.editor.indentWithSpaces = indentWithSpaces;
   mumuki.editor.syncContent = syncContent;
 
@@ -75,12 +85,19 @@ var mumuki = mumuki || {};
     updateCodeMirrorLanguage();
     onSelectUpdateCodeMirror();
 
-    $('.editor-reset').click(function () {
-      resetEditor();
+    $('.editor-reset').click(function (event) {
+      event.stopPropagation();
+      const selection = confirm(this.getAttribute('data-confirm'));
+      if(selection) resetEditor();
     });
+
     $('.editor-resize').click(function () {
       toggleFullscreen();
     });
+
+    $('.editor-format').click(function (){
+      formatContent();
+    });
   });
 
 }(mumuki));

data/app/assets/javascripts/mumuki_laboratory/application/elipsis.js

@@ -0,0 +1,31 @@
+// This module defines three client-side-only special sections, and allows them to be replaced
+// by adding the `mu-elipsis` class, or by calling `mumuki.elipsis()`:
+//
+// * <elipsis-for-student@ ...code... @elipsis-for-student>                     : replaces code with an elipsis
+// * <hidden-for-student@ ...code... @hidden-for-student>                       : completely hides code
+// * <description-for-student[...text...]@ ...code... @description-for-student> : replaces code a message sourrounded by elipsis
+//
+//
+// This module assumes the strings are already markdown-like escaped code strings, not plain code
+(function (mumuki) {
+
+  function elipsis(code) {
+    return code
+      .replace(/&lt;elipsis-for-student@[\s\S]*?@elipsis-for-student&gt;/g, ' ... ')
+      .replace(/&lt;hidden-for-student@[\s\S]*?@hidden-for-student&gt;/g, '')
+      .replace(/&lt;description-for-student\[([^\]]*)\]@[\s\S]*?@description-for-student&gt;/g, ' ... $1 ... ');
+  }
+
+  mumuki.elipsis = elipsis;
+  mumuki.elipsis.replaceHtml = () => {
+    let $elipsis = $('.mu-elipsis');
+    $elipsis.each((it, e) =>  {
+      let $e = $(e);
+      $e.html(mumuki.elipsis($e.html()));
+    })
+  };
+
+  mumuki.load(() => {
+    mumuki.elipsis.replaceHtml();
+  });
+})(mumuki);

data/app/assets/javascripts/mumuki_laboratory/application/interval.js

@@ -0,0 +1,17 @@
+var mumuki = mumuki || {};
+
+(function (mumuki) {
+  // When using Turbolinks, intervals loaded inside <body> aren't destroyed on page changes
+  // Use this function instead if you want the behaviour of a regular setInterval
+  mumuki.setInterval = function (intervalFunction, milliseconds) {
+    const interval = setInterval.apply(this, [intervalFunction, milliseconds]);
+
+    // Using one to avoid calling clearInterval on every page chance.
+    $(document).one('turbolinks:before-cache turbolinks:before-render', function () {
+      clearInterval(interval);
+    });
+
+    return interval;
+  }.bind(this);
+
+}(mumuki));

data/app/assets/javascripts/mumuki_laboratory/application/kids.js

@@ -169,28 +169,17 @@ mumuki.load(function () {
       $bubble.find('.mu-kids-character-speech-bubble-failed').hide();
       $bubble.find('.mu-kids-discussion-link').remove();
       Object.keys(mumuki.kids.resultAction).forEach($bubble.removeClass.bind($bubble));
-      mumuki.kids._getOverlay().hide();
+      mumuki.kids._getOverlay().hide()
     },
 
     _showMessageOnCharacterBubble: function (data) {
-      var $bubble = mumuki.kids._getCharacterBubble();
-      $bubble.find('.mu-kids-character-speech-bubble-tabs').hide();
-      $bubble.find('.mu-kids-character-speech-bubble-normal').hide();
-      $bubble.find('.mu-kids-character-speech-bubble-failed').show().html(data.title_html);
-      $bubble.addClass(data.status);
-      if (data.status === 'passed_with_warnings') {
-        $bubble.find('.mu-kids-character-speech-bubble-failed').append(data.expectations_html);
-      }
-      if (this._shouldDisplayDiscussionsLink(data.status)) {
-        $bubble.append(discussionsLinkHtml);
-      }
+      const renderer = new mumuki.renderers.SpeechBubbleRenderer(mumuki.kids._getCharacterBubble());
+      renderer.setDiscussionsLinkHtml(discussionsLinkHtml);
+      renderer.setResponseData(data);
+      renderer.render();
       mumuki.kids._getOverlay().show();
     },
 
-    _shouldDisplayDiscussionsLink: function (status) {
-      return ['failed', 'passed_with_warnings'].some(it => it === status);
-    },
-
     _showOnSuccessPopup: function (data) {
       mumuki.kids._updateSubmissionResult(data.html);
       mumuki.presenterCharacter.playAnimation('success_l', mumuki.kids._getCharacterImage());
@@ -313,7 +302,7 @@ mumuki.load(function () {
     });
 
     if (paragraphCount > 1) {
-      nextSpeechBlinking = setInterval(() => $nextSpeech.fadeTo('slow', 0.1).fadeTo('slow', 1.0), 1000);
+      nextSpeechBlinking = mumuki.setInterval(() => $nextSpeech.fadeTo('slow', 0.1).fadeTo('slow', 1.0), 1000);
     }
 
     $nextSpeech.click(showNextParagraph);

data/app/assets/javascripts/mumuki_laboratory/application/messages.js

@@ -45,7 +45,7 @@ mumuki.load(function () {
       });
     },
     setMessagesInterval: function () {
-      setInterval(Chat.getMessages, 60000);
+      mumuki.setInterval(Chat.getMessages, 60000);
     },
     submitMessagesForm: function (url, readUrl, errorUrl) {
       var $container = $('.mu-view-messages');

data/app/assets/javascripts/mumuki_laboratory/application/speech-bubble-renderer.js

@@ -0,0 +1,99 @@
+((mumuki)=> {
+
+  function renderSpeechBubbleResultItem(item) {
+    return `
+        <div class="results-item">
+          <ul class="results-list">
+            <li>${item}</li>
+          </ul>
+        </div>`;
+  }
+
+  // A mumuki.renderers.SpeechBubbleRenderer allows to configure a kids speech bubble
+  // with the data of a test response from bridge
+  class SpeechBubbleRenderer {
+    constructor($bubble) {
+      this.$bubble = $bubble;
+      this.$failedArea = $bubble.find('.mu-kids-character-speech-bubble-failed');
+    }
+
+    _chooseResultItem() {
+      if (this._responseStatus() !== 'passed' && this._hasTips()) {
+        this._appendFirstTip()
+      } else if (this._responseStatus() === 'failed') {
+        this._appendFirstFailedTestResultSummary();
+      } else if (this._responseStatus() === 'passed_with_warnings') {
+        this._appendFirstExpectationResult();
+      }
+    }
+
+    _appendFirstFailedTestResultSummary() {
+      const failedTestResult = this._failedTestResults()[0]
+      if (failedTestResult && failedTestResult.summary) {
+        this._appendResultItem(mumuki.renderers.renderSpeechBubbleResultItem(failedTestResult.summary));
+      }
+    }
+
+    _appendFirstExpectationResult() {
+      this._appendResultItem(mumuki.renderers.renderSpeechBubbleResultItem(this.responseData.expectations[0].explanation));
+    }
+
+    _appendFirstTip() {
+      this._appendResultItem(mumuki.renderers.renderSpeechBubbleResultItem(this.responseData.tips[0]));
+    }
+
+    _appendResultItem(item) {
+      this._addClass('with-result-item');
+      this.$failedArea.append(item);
+    }
+
+    _addClass(klass) {
+      this.$bubble.addClass(klass);
+    }
+
+    _appendDiscussionsLinkHtml() {
+      if (this._shouldDisplayDiscussionsLink()) {
+        this.$bubble.append(this.discussionsLinkHtml);
+      }
+    }
+
+    _shouldDisplayDiscussionsLink() {
+      return ['failed', 'passed_with_warnings'].some(it => it === this._responseStatus());
+    }
+
+    _responseStatus() {
+      return this.responseData.status;
+    }
+
+    _hasTips() {
+      return this.responseData.tips && this.responseData.tips.length
+    }
+
+    _failedTestResults() {
+      return this.responseData.test_results.filter((it) => it.status === 'failed');
+    }
+
+    setDiscussionsLinkHtml(discussionsLinkHtml) {
+      this.discussionsLinkHtml = discussionsLinkHtml;
+    }
+
+    setResponseData(responseData) {
+      this.responseData = responseData;
+    }
+
+    // Entry point of the renderer. It just updates the UI, not returning anything
+    // Configure the renderer with `setDiscussionsLinkHtml` and `setResponseData` first.
+    render() {
+      this.$bubble.find('.mu-kids-character-speech-bubble-tabs').hide();
+      this.$bubble.find('.mu-kids-character-speech-bubble-normal').hide();
+      this.$failedArea.show().html(this.responseData.title_html);
+      this._addClass(this._responseStatus());
+      this._chooseResultItem();
+      this._appendDiscussionsLinkHtml();
+    }
+  }
+
+  mumuki.renderers = mumuki.renderers || {};
+  mumuki.renderers.SpeechBubbleRenderer = SpeechBubbleRenderer;
+  mumuki.renderers.renderSpeechBubbleResultItem = renderSpeechBubbleResultItem;
+})(mumuki)

data/app/assets/javascripts/mumuki_laboratory/application/timer.js

@@ -8,7 +8,7 @@ var mumuki = mumuki || {};
     var duration = moment.duration(diffTime, 'milliseconds');
     var intervalDuration = 1000;
 
-    var interval = setInterval(function () {
+    var interval = mumuki.setInterval(function () {
       duration = moment.utc(duration - intervalDuration);
       if(duration.valueOf() <= 0) {
         clearInterval(interval);

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

@@ -13,25 +13,6 @@ $multifile-inactive-tab-border-color: #ECF0F1;
   max-height: none;
 }
 
-.submission-reset {
-  padding: 5px;
-  cursor: pointer;
-}
-
-.editor-resize {
-  position: absolute;
-  padding: 5px;
-  cursor: pointer;
-  z-index: 10;
-  top: 0;
-  right: 0;
-  font-size: 0.5em;
-  &.multiple-files {
-    top: 43px;
-  }
-}
-
-
 body:not(.fullscreen) {
   .CodeMirror-wrap,
   .CodeMirror-scroll {
@@ -69,10 +50,6 @@ body.fullscreen {
       overflow-y: scroll;
       overflow-x: auto;
     }
-    .submission-reset,
-    .editor-resize {
-      margin-right: 15px;
-    }
   }
 }
 

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

@@ -10,6 +10,9 @@
 ul.results-list {
   list-style-type: none;
   padding-left: 10px;
+  li:first-letter {
+    text-transform: capitalize
+  }
 }
 
 .results-list pre {
@@ -27,3 +30,11 @@ ul.results-list {
     color: white;
   }
 }
+
+.mu-single-test-result {
+  .error-text, /* backward compatibility with gobstones runner */
+  .mu-test-result-header {
+    display: block;
+    line-height: 25px;
+  }
+}

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

@@ -154,7 +154,7 @@ $kids-speech-tabs-width: 40px;
         }
       }
 
-      &.passed_with_warnings {
+      &.with-result-item {
         padding-top: 5px;
         padding-bottom: 5px;
 

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

@@ -5,6 +5,25 @@
 .mu-overlapped {
   z-index: 10;
   position: absolute;
-  bottom: 0;
+  top: 0;
   right: 0;
-}
+  display: flex;
+  flex-direction: column;
+  a {
+    cursor: pointer;
+    padding: 5px;
+  }
+  &.multiple-files {
+    top: 43px;
+  }
+}
+
+.editor-resize {
+  font-size: 0.5em;
+}
+
+body.fullscreen {
+  .mu-overlapped a {
+    margin-right: 15px;
+  }
+}