mumuki-cpp-runner 1.2.0 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: dc5b24afc85d7b43741270c7c2c5f7c2910f03c4
-  data.tar.gz: 505fe18a3a1f5979ffacb3a276939a6936dbd35b
+  metadata.gz: 9c481b077c1b72bf64d1b1121a1227d7b07d21c0
+  data.tar.gz: f8c626d9ec01ead7616c1196152978da08b1d816
 SHA512:
-  metadata.gz: 1aaca1717287a0147b97821724e120c32b57f7884bd1ea09c776033385e4d87b881255b63d924a4d8bbee9aa830f7cfbcb701bf6baef776aa57b3f54d49bb02d
-  data.tar.gz: a635187fe6dbee7a5ef0e992b2063525cdf08e000012a673d5d98d60f3fb008f3c4ab110a8c49044a1e1471ef2864b80d73d41930b3e683236b0a2d3c0606e00
+  metadata.gz: b25b4fd4e73a974e48fd5c2011ba219a329bced7661433a4671d79c578f6d56a88a8b44e643ad756b6afcd7b5cc7ab1e8794c5aa12b22ac80b8654cd8509b7af
+  data.tar.gz: 4206a939d4f2a453258538de886da2618613eb4e1f7fecddff296ecc433192cf22d52922dd05fa1b623d467fa993bb357b383caf613d358748b39e4386fc7e8f

data/lib/cpp_runner.rb

@@ -1,5 +1,7 @@
 require 'mumukit'
 
+I18n.load_translations_path File.join(__dir__, 'locales', '*.yml')
+
 Mumukit.runner_name = 'cpp'
 Mumukit.configure do |config|
   config.docker_image = 'mumuki/mumuki-cppunit-worker'
@@ -7,4 +9,5 @@ end
 
 require_relative './test_hook'
 require_relative './version_hook'
-require_relative './metadata_hook'
+require_relative './metadata_hook'
+require_relative './feedback_hook'

data/lib/feedback_hook.rb

@@ -0,0 +1,104 @@
+class CppFeedbackHook < Mumukit::Hook
+  def run!(request, results)
+    content = request.content
+    test_results = results.test_results[0]
+
+    CppExplainer.new.explain(content, test_results) if test_results.is_a? String
+  end
+
+  class CppExplainer < Mumukit::Explainer
+
+    def near_regex()
+      '.*[\t \n]* *(.*)\n[ \t]+\^'
+    end
+
+    def error()
+      '[eE]rror:'
+    end
+
+    def explain_has_no_member_named(_, result)
+      (/#{error} '(.*)' has no member named '(.*)'#{near_regex}/.match result).try do |it|
+        {type: it[1], target: it[2], near: it[3]}
+      end
+    end
+
+    def explain_was_not_declared_in_this_scope(_, result)
+      (/#{error} '(.*)' was not declared in this scope#{near_regex}/.match result).try do |it|
+        {target: it[1], near: it[2]}
+      end
+    end
+
+    def explain_does_not_a_type(_, result)
+      (/#{error} '(.*)' does not name a type#{near_regex}/.match result).try do |it|
+        {type: it[1], near: it[2]}
+      end
+    end
+
+    def explain_has_incomplete_type_and_cannot_be_defined(_, result)
+      (/#{error} aggregate '(.*)' has incomplete type and cannot be defined#{near_regex}/.match result).try do |it|
+        {type: it[1], near: it[2]}
+      end
+    end
+
+    def explain_invalid_suffix(_, result)
+      (/#{error} invalid suffix "(.*)" on integer constant#{near_regex}/.match result).try do |it|
+        {suffix: it[1], near: it[2]}
+      end
+    end
+
+    def explain_expected_comma_before_token(_, result)
+      (/#{error} expected ',' or '.{3}' before '\.' token#{near_regex}/.match result).try do |it|
+        {near: it[1]}
+      end
+    end
+
+    def explain_expected_initializer_before_token(_, result)
+      (/#{error} expected initializer before '.' token#{near_regex}/.match result).try do |it|
+        {near: it[1]}
+      end
+    end
+
+    def explain_too_many_arguments_to_function(_, result)
+      (/#{error} too many arguments to function '(.*)'#{near_regex}/.match result).try do |it|
+        {target: it[1], near: it[2]}
+      end
+    end
+
+    def explain_too_few_arguments_to_function(_, result)
+      (/#{error} too few arguments to function '(.*)'#{near_regex}/.match result).try do |it|
+        {target: it[1], near: it[2]}
+      end
+    end
+
+    def explain_call_of_overloaded_is_ambiguous(_, result)
+      (/#{error} call of overloaded '(.*)' is ambiguous#{near_regex}/.match result).try do |it|
+        {target: it[1], near: it[2]}
+      end
+    end
+
+    def explain_invalid_conversion_from_to(_, result)
+      (/#{error} invalid conversion from '(.*)' to '(.*)'#{near_regex}/.match result).try do |it|
+        {expected: it[1], actual: it[2], near: it[3]}
+      end
+    end
+
+    def explain_expected_after_struct_definition(_, result)
+      (/#{error} expected ';' after struct definition#{near_regex}/.match result).try do |it|
+        {near: it[1]}
+      end
+    end
+
+    def explain_expected_initializer_before(_, result)
+      (/#{error} expected initializer before '(.*)'#{near_regex}/.match result).try do |it|
+        {near: it[2]}
+      end
+    end
+
+    def explain_expected_unqualified_id(_, result)
+      (/#{error} expected unqualified-id before '(.*)' token#{near_regex}/.match result).try do |it|
+        {token: it[1], near: it[2]}
+      end
+    end
+
+  end
+end

data/lib/locales/en.yml

@@ -0,0 +1 @@
+en:

data/lib/locales/es.yml

@@ -0,0 +1,15 @@
+en:
+  has_no_member_named: 'Parece que no existe el campo, atributo o método `%{target}` en `%{type}`. Revisá en esta parte `%{near}` que exista o el nombre esté bien escrito.'
+  was_not_declared_in_this_scope: 'El identificador `%{target}` no existe. Revisá en esta parte `%{near}` si está bien escrito o si estás usando la variable correcta.'
+  does_not_a_type: 'No hay un tipo de dato con el nombre `%{type}`. Revisá en esta parte `%{near}` si está bien escrito.'
+  has_incomplete_type_and_cannot_be_defined: 'Parece que no definiste el tipo `%{type}`. Revisá en esta parte `%{near}` que el nombre esté bien escrito.'
+  invalid_suffix: 'Parece que intentaste definir un identificador `%{suffix}` no válido. Revisá en esta parte `%{near}` si lo empezaste con un número. Recordá que los nombres de los identificadores deben comenzar con una **letra**, **$** o **_** (guión bajo).'
+  expected_comma_before_token: 'El nombre de uno de los parámetros está mal. Revisá en esta parte `%{near}` que no tenga puntos en el nombre.'
+  expected_initializer_before_token: 'El nombre de una de las variables está mal. Revisá en esta parte `%{near}` que no tenga puntos en el nombre.'
+  too_many_arguments_to_function: 'Parece que invocaste a la función `%{target}` con más argumentos de los que lleva. Revisá en esta parte `%{near}` la llamada a la función.'
+  too_few_arguments_to_function: 'Parece que invocaste a la función `%{target}` con menos argumentos de los que lleva. Revisá en esta parte `%{near}` la llamada a la función.'
+  call_of_overloaded_is_ambiguous: 'Uno o más argumentos están mal al invocar a `%{target}`. Revisá en esta parte `%{near}` con qué argumentos la estas llamando.'
+  invalid_conversion_from_to: 'Se esperaba un valor de tipo `%{expected}` pero se recibió uno de `%{actual}` en esta línea `%{near}`.'
+  expected_after_struct_definition: 'Falta el `;` después de definir el registro.'
+  expected_initializer_before: 'Probablemente falte un `;` después de declarar una variable en la línea anterior a `%{near}`.'
+  expected_unqualified_id: 'El identificador que se encuentra entes que `%{token}` no es válido. Revisá en esta parte `%{near}`.'

data/lib/metadata_hook.rb

@@ -2,7 +2,7 @@ class CppMetadataHook < Mumukit::Hook
   def metadata
     {language: {
         name: 'c++',
-        icon: {type: 'devicon', name: 'c'},
+        icon: {type: 'devicon', name: 'cpp'},
         extension: 'cpp',
         ace_mode: 'c_cpp'
     },

data/lib/test_hook.rb

@@ -1,6 +1,7 @@
 class CppTestHook < Mumukit::Templates::FileHook
   mashup
   isolated true
+  structured true
 
   def tempfile_extension
     '.cpp'
@@ -13,10 +14,16 @@ class CppTestHook < Mumukit::Templates::FileHook
   def post_process_file(file, result, status)
     if result.include? '!!TEST FINISHED WITH COMPILATION ERROR!!'
       [result, :errored]
-    elsif result.include? '!!!FAILURES!!!'
-      [result, :failed]
     else
-      [result, :passed]
+      super
+    end
+  end
+
+  def to_structured_result(result)
+    if result.include? '!!!FAILURES!!!'
+      transform(result)
+    else
+      [['All tests passed', :passed]]
     end
   end
 
@@ -35,4 +42,19 @@ int main( int argc, char **argv)
 }
 EOF
   end
+
+  private
+
+  def transform(result)
+    result
+        .split(/\d{0,9}\) test: MumukiTest::/)
+        .drop(1)
+        .map do |it|
+      captures = it.split("\n")
+      title = captures.first.split(' (F)').first
+      result = [captures[1], captures[2], captures[3], captures[4], captures[5]].compact.join(' ').strip
+      [title, :failed, result]
+    end
+  end
+
 end

data/lib/version_hook.rb

@@ -1,3 +1,3 @@
 module CppVersionHook
-  VERSION = '1.2.0'
+  VERSION = '1.3.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mumuki-cpp-runner
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.3.0
 platform: ruby
 authors:
 - Franco Leonardo Bulgarelli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-15 00:00:00.000000000 Z
+date: 2017-07-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mumukit
@@ -102,6 +102,9 @@ extensions: []
 extra_rdoc_files: []
 files:
 - lib/cpp_runner.rb
+- lib/feedback_hook.rb
+- lib/locales/en.yml
+- lib/locales/es.yml
 - lib/metadata_hook.rb
 - lib/test_hook.rb
 - lib/version_hook.rb
@@ -125,7 +128,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 2.6.11
 signing_key: 
 specification_version: 4
 summary: Cpp Runner for Mumuki