mumuki-javascript-runner 1.3.2 → 1.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7a4a4e866fb702c176bcc075c039f147e773a9289251430106f6bdb3baf895e3
-  data.tar.gz: 30ed0dec052be2274099624d062ccaa087c27ea62ae586563fe54039cb072021
+  metadata.gz: 27f5bfb23b79be6ab30498db14509c411937ea95e009dcf2c3e08a47e262d2e9
+  data.tar.gz: af6d8ed5c9783ce18ac3eab40230530718b9cee0b18197201ffa85bcd273212a
 SHA512:
-  metadata.gz: 28b7216bd532615a59599a0e7bf6704532a212c18268f578f24669fd49ab950af578f59ae1ef7e34c889ca8a67f1303eab790f86cfadbbc28747ef1945328854
-  data.tar.gz: d9504c226a9a77ae7e373eb8a6a4dba0fe1c8245f7d8dbc740926e972e980ee399fe7d1732c850ce855326c2315b96cee061a61b0e393a95a46b2955f3c7face
+  metadata.gz: cf2f50685744e546a02740be2811d7bab9e1d76c7bc0574bffa3798bed65a87fba3b22871046f8b4dffb799630f38d85db2a4444c48a02b48227e7911a9c50bc
+  data.tar.gz: 1fe3a8346f15707cc100c6b494df926d43842f527d471b87c2cab1ef699c4796fbc05dff38af86d81caa5663da6119a026ead2c76f7bc77c512202dd8bd31ede

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,14 +1,20 @@
 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'
 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

@@ -0,0 +1,18 @@
+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+|$)/
+    end
+    raise Mumukit::RequestValidationError, "You can not use #{$2} here" if matches
+  end
+
+  private
+
+  def request_matches?(request, &block)
+    [
+        request.content,
+        request.extra,
+        request.query
+    ].compact.any?(&block)
+  end
+end

data/lib/version.rb

@@ -1,3 +1,3 @@
 module JavascriptVersionHook
-  VERSION = '1.3.2'
+  VERSION = '1.6.1'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mumuki-javascript-runner
 version: !ruby/object:Gem::Version
-  version: 1.3.2
+  version: 1.6.1
 platform: ruby
 authors:
 - Franco Leonardo Bulgarelli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-02-20 00:00:00.000000000 Z
+date: 2020-11-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mumukit
@@ -16,14 +16,14 @@ 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
@@ -102,10 +102,16 @@ 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
 licenses:
@@ -126,8 +132,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.0.8
 signing_key: 
 specification_version: 4
 summary: Javascript Runner for Mumuki