mumuki-javascript-runner 1.3.3 → 1.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ec3828b95e23b39b3306bcb2a507cde9acbf3d98d444e9dd5bb95cf3c648a915
-  data.tar.gz: b286fa2b2764f084c4f1763cfabcace5e4a975403039d40d89ed20adf4872264
+  metadata.gz: 20226f79e13d2ab4258f79b40b3fedc4c0cbc95533663170fb1c3c7fdbad9f8f
+  data.tar.gz: 9b52ec14fe5153e27cea40130b2a143a6ba9c73c91bb87c5974908ea27404b89
 SHA512:
-  metadata.gz: d717df05b3e3dd99882730c368c13a753264d63bd6c8ded6b60eb13f90feef16a301b5ac240b9ac5bc67bb7b812f3222fbdb680b243da3bbc9c397f711b56e3a
-  data.tar.gz: e27e945d2adbe2720a3204a87f9a70ebc180eb26b2ae61bebb0ed9103dd81af91308529835ee4bd9e82baebe8e8de57d0ae1c56d209427a4ee7caceb285e43af
+  metadata.gz: de05d16296a3b89dd7291d623d2fe6f3cead5fa8b577374a5d5e4d3e42626a0794dfb1d4da1a17db2f7878d511841e73802870c20220c7278fb05ee8197cfb43
+  data.tar.gz: 5e37e9518248fc5ae9faef8f5337da68cacdfa8287fcb480f3ab911975aa4330ebd30079f2ab189571478e84579ff94a701316018fbe3c8a0dd38b62172ea27c

data/lib/feedback_hook.rb

@@ -0,0 +1,25 @@
+class JavascriptFeedbackHook < Mumukit::Hook
+  def run!(request, results)
+    content = request.content
+    test_results = test_failure_messages results
+
+    JavascriptExplainer.new.explain(content, test_results)
+  end
+
+  def test_failure_messages(results)
+    structured_test_results = results.test_results[0]
+    if structured_test_results.is_a? Array
+      structured_test_results.select { |it| it[1].failed? }.map { |it| it[2] }.join("\n")
+    else
+      ''
+    end
+  end
+
+  class JavascriptExplainer < Mumukit::Explainer
+    def explain_is_not_defined(_, test_results)
+      (/([a-zA-Z0-9_]+) is not defined/.match test_results).try do |it|
+        {identifier: it[1]}
+      end
+    end
+  end
+end

data/lib/javascript_runner.rb

@@ -1,10 +1,13 @@
 require 'mumukit'
 
+I18n.load_translations_path File.join(__dir__, 'locales', '*.yml')
+
 Mumukit.runner_name = 'javascript'
 Mumukit.configure do |config|
   config.docker_image = 'mumuki/mumuki-mocha-worker'
   config.structured = true
   config.stateful = true
+  config.content_type = 'markdown'
 end
 
 require_relative './version'
@@ -12,4 +15,6 @@ require_relative './metadata_hook'
 require_relative './validation_hook'
 require_relative './test_hook'
 require_relative './expectations_hook'
+require_relative './feedback_hook'
 require_relative './query_hook'
+require_relative './try_hook'

data/lib/locales/en.yml

@@ -0,0 +1,2 @@
+en:
+  is_not_defined: "it looks like you are using `%{identifier}` but it is not defined. Perhaps you have misspelled it or you need to define it"

data/lib/locales/es.yml

@@ -0,0 +1,2 @@
+es:
+  is_not_defined: "parece que estás usando `%{identifier}` pero no está definido. ¿Puede que hayas escrito mal su nombre o tengas que definirlo?"

data/lib/locales/pt.yml

@@ -0,0 +1,2 @@
+pt:
+  is_not_defined: "Parece que você usa %{identifier} mas não está indefinido. Você digitou o nome errado ou precisa defini-lo"

data/lib/query_hook.rb

@@ -1,18 +1,27 @@
 class JavascriptQueryHook < Mumukit::Templates::FileHook
+  with_error_patterns
   isolated true
 
+  VAR_REGEXP = /^ *(?:var|let) +([a-zA-Z_$][a-zA-Z_$0-9]*)/
+  VAR_ASSIGN_REGEXP = /^ *(?:var|let) +([a-zA-Z_$][a-zA-Z_$0-9]*) *=/
+  CONST_ASSIGN_REGEXP = /^ *const +([a-zA-Z_$][a-zA-Z_$0-9]*) *=/
+
   def tempfile_extension
     '.js'
   end
 
-
   def compile_file_content(r)
+    "#{compile_file_header(r)}\n#{compile_query(r.query)}"
+  end
+
+  def compile_file_header(r)
 <<javascript
 'use strict';
 
 function mumukiConsolePrettyPrint(e) {
     if (e instanceof Function) return '<function>';
-    return JSON.stringify(e);
+    const json = JSON.stringify(e);
+    return json && json.replace(/"(\\w+)"\s*:/g, '$1:');
  }
 
 #{r.extra}
@@ -20,25 +29,81 @@ function mumukiConsolePrettyPrint(e) {
 #{r.content}
 
 #{compile_cookie(r.cookie)}
-
-#{compile_query(r.query)}
 javascript
   end
 
-  def compile_query(query)
-    if query.start_with?('var ') || query.start_with?('let ')
-      "#{query}\nconsole.log('=> undefined')"
+  def compile_query(query, output_prefix = "=> ", output_var = "__mumuki_query_result__")
+    if ['var', 'let', 'const'].any? { |type| query.start_with? "#{type} " }
+      "#{query}\nconsole.log('#{output_prefix}undefined')"
     else
-      "var __mumuki_query_result__ = #{query};\nconsole.log('=> ' + mumukiConsolePrettyPrint(__mumuki_query_result__))"
+      "var #{output_var} = #{query};\nconsole.log('#{output_prefix}' + mumukiConsolePrettyPrint(#{output_var}))"
     end
   end
 
   def compile_cookie(cookie)
     return if cookie.blank?
-    cookie.map { |query| "try { #{query} } catch (e) {}" }.join("\n")
+
+    compile_statements(cookie).join "\n"
   end
 
   def command_line(filename)
     "node #{filename}"
   end
+
+  private
+
+  def compile_statements(cookie)
+    reject_duplicated_statements wrap_statements(cookie)
+  end
+
+  def wrap_statements(cookie)
+    cookie.map do |query|
+      case query
+      when CONST_ASSIGN_REGEXP
+        declaration_with_assignment 'const', CONST_ASSIGN_REGEXP, $1, query
+      when VAR_ASSIGN_REGEXP
+        declaration_with_assignment 'var', VAR_ASSIGN_REGEXP, $1, query
+      when VAR_REGEXP
+        "var #{$1}"
+      else
+        "try { #{query} } catch (e) {}"
+      end
+    end
+  end
+
+  def reject_duplicated_statements(sentences)
+    sentences.select.with_index do |line, index|
+      next line if !line.match(VAR_ASSIGN_REGEXP) && !line.match(VAR_REGEXP) && !line.match(CONST_ASSIGN_REGEXP)
+      name = $1
+      sentences.slice(0, index).none? { |previous_line| declaration? previous_line, name }
+    end
+  end
+
+  def declaration_with_assignment(type, type_pattern, name, expression)
+    "#{type} #{name} = (function() { try { return #{expression.gsub(type_pattern, '')} } catch(e) { return undefined } })()"
+  end
+
+  def declaration?(line, name)
+    declaration_of_type?(VAR_REGEXP, line, name) ||
+      declaration_of_type?(VAR_ASSIGN_REGEXP, line, name) ||
+      declaration_of_type?(CONST_ASSIGN_REGEXP, line, name)
+  end
+
+  def declaration_of_type?(type_pattern, line, name)
+    line.match(type_pattern) && $1 == name
+  end
+
+  def error_patterns
+    [
+      Mumukit::ErrorPattern::Errored.new(error_regexp)
+    ]
+  end
+
+  def error_types
+    '(Reference|Syntax|Type)Error'
+  end
+
+  def error_regexp
+    /(?=\X*#{error_types})(solution.*\n|var __mumuki_query_result__ = )|#{error_types}.*\n\K\X*/
+  end
 end

data/lib/try_hook.rb

@@ -0,0 +1,48 @@
+class JavascriptTryHook < Mumukit::Templates::TryHook
+  isolated true
+  attr_reader :query_hook
+
+  def initialize(config = nil)
+    super config
+    @query_hook = JavascriptQueryHook.new
+  end
+
+  def compile_file_content(r)
+    <<js
+#{@query_hook.compile_file_header(r)}
+console.log("#{query_separator}");
+#{@query_hook.compile_query(r.query, '')}
+console.log("#{goal_separator}");
+#{@query_hook.compile_query(r.goal.with_indifferent_access[:query] || 'null', '', '__mumuki_goal_query_result__')}
+js
+  end
+
+  delegate :tempfile_extension, to: :query_hook
+  delegate :command_line, to: :query_hook
+
+  def query_separator
+    '!!!MUMUKI-QUERY-START!!!'
+  end
+
+  def goal_separator
+    '!!!MUMUKI-GOAL-START!!!'
+  end
+
+  def to_structured_results(_file, result, status)
+    /#{query_separator}
+?(.*)
+#{goal_separator}
+?(.*)
+/m =~ result
+
+    {
+        query: to_query_result($1, status),
+        goal: $2,
+        status: status
+    }
+  end
+
+  def to_query_result(result, status)
+    { result: result, status: status }
+  end
+end

data/lib/validation_hook.rb

@@ -1,9 +1,9 @@
 class JavascriptValidationHook < Mumukit::Hook
   def validate!(request)
     matches = request_matches?(request) do |it|
-      it =~ /\W*(require|process|os|fs|eval|cluster|v8|vm|tty|tls|root|global|crypto|stream|events)\W*/
+      it =~ /(^|[^_[[:alnum:]]]+)(require|process|os|fs|eval|cluster|v8|vm|tty|tls|root|global|crypto|stream|events)([^_[[:alnum:]]]+|$)/
     end
-    raise Mumukit::RequestValidationError, "You can not use #{$1} here" if matches
+    raise Mumukit::RequestValidationError, "You can not use #{$2} here" if matches
   end
 
   private

data/lib/version.rb

@@ -1,3 +1,3 @@
 module JavascriptVersionHook
-  VERSION = '1.3.3'
+  VERSION = '1.6.2'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mumuki-javascript-runner
 version: !ruby/object:Gem::Version
-  version: 1.3.3
+  version: 1.6.2
 platform: ruby
 authors:
 - Franco Leonardo Bulgarelli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-05-27 00:00:00.000000000 Z
+date: 2020-12-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mumukit
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.20'
+        version: '2.30'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.20'
+        version: '2.30'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -102,10 +102,15 @@ extensions: []
 extra_rdoc_files: []
 files:
 - lib/expectations_hook.rb
+- lib/feedback_hook.rb
 - lib/javascript_runner.rb
+- lib/locales/en.yml
+- lib/locales/es.yml
+- lib/locales/pt.yml
 - lib/metadata_hook.rb
 - lib/query_hook.rb
 - lib/test_hook.rb
+- lib/try_hook.rb
 - lib/validation_hook.rb
 - lib/version.rb
 homepage: http://github.com/mumuki/mumuki-javascript-server
@@ -127,8 +132,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.7
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Javascript Runner for Mumuki