From d977e9307080978538a0727cc86c02229fcd643c Mon Sep 17 00:00:00 2001
From: sulyi <sulyi.gbox@gmail.com>
Date: Sun, 3 Jun 2018 23:23:41 +0200
Subject: [PATCH] [jsinterp] Fixing test skip messages

---
 test/js2tests/debug.py           | 4 ++--
 test/js2tests/do_loop.py         | 2 +-
 test/js2tests/for_empty.py       | 2 +-
 test/js2tests/for_in.py          | 2 +-
 test/js2tests/for_loop.py        | 2 +-
 test/js2tests/func_expr.py       | 2 +-
 test/js2tests/label.py           | 4 ++--
 test/js2tests/object_literal.py  | 2 +-
 test/js2tests/precedence.py      | 2 +-
 test/js2tests/stringprototype.py | 4 ++--
 test/js2tests/switch.py          | 2 +-
 test/js2tests/try_statement.py   | 4 ++--
 test/js2tests/unary.py           | 2 +-
 test/js2tests/unshift.py         | 6 +++---
 test/js2tests/while_loop.py      | 2 +-
 test/js2tests/with_statement.py  | 4 ++--
 16 files changed, 23 insertions(+), 23 deletions(-)

diff --git a/test/js2tests/debug.py b/test/js2tests/debug.py
index f233176aa..9bdbdab7e 100644
--- a/test/js2tests/debug.py
+++ b/test/js2tests/debug.py
@@ -4,8 +4,8 @@ from youtube_dl.jsinterp2.jsgrammar import Token
 
 skip = {
     'jsinterp': 'Debugger statement is not supported',
-    'interpret': 'Interpreting debugger statement not yet implemented',
-    'parse': 'Test not yet implemented: missing code and ast'
+    'interpret': 'Interpreting debugger statement is not yet implemented',
+    'parse': 'Test is not yet implemented: missing code and ast'
 }
 
 tests = [
diff --git a/test/js2tests/do_loop.py b/test/js2tests/do_loop.py
index c2d4650b4..98bdf144a 100644
--- a/test/js2tests/do_loop.py
+++ b/test/js2tests/do_loop.py
@@ -5,7 +5,7 @@ from youtube_dl.jsinterp2.tstream import _ASSIGN_OPERATORS, _UNARY_OPERATORS, _R
 
 skip = {
     'jsinterp': 'Do loop is not supported',
-    'interpret': 'Interpreting do loop not yet implemented'
+    'interpret': 'Interpreting do loop is not yet implemented'
 }
 
 tests = [
diff --git a/test/js2tests/for_empty.py b/test/js2tests/for_empty.py
index 4ef5ba1ee..8085eb8e0 100644
--- a/test/js2tests/for_empty.py
+++ b/test/js2tests/for_empty.py
@@ -5,7 +5,7 @@ from youtube_dl.jsinterp2.tstream import _ASSIGN_OPERATORS, _UNARY_OPERATORS, _R
 
 skip = {
     'jsinterp': 'For loop is not supported',
-    'interpret': 'Interpreting for empty loop not yet implemented'
+    'interpret': 'Interpreting for empty loop is not yet implemented'
 }
 
 tests = [
diff --git a/test/js2tests/for_in.py b/test/js2tests/for_in.py
index 12ad7f672..b19424ae4 100644
--- a/test/js2tests/for_in.py
+++ b/test/js2tests/for_in.py
@@ -5,7 +5,7 @@ from youtube_dl.jsinterp2.tstream import _ASSIGN_OPERATORS
 
 skip = {
     'jsinterp': 'For in loop is not supported',
-    'interpret': 'Interpreting for in loop not yet implemented'
+    'interpret': 'Interpreting for in loop is not yet implemented'
 }
 
 tests = [
diff --git a/test/js2tests/for_loop.py b/test/js2tests/for_loop.py
index 6b4d2a876..64f834593 100644
--- a/test/js2tests/for_loop.py
+++ b/test/js2tests/for_loop.py
@@ -5,7 +5,7 @@ from youtube_dl.jsinterp2.tstream import _ASSIGN_OPERATORS, _UNARY_OPERATORS, _R
 
 skip = {
     'jsinterp': 'For loop is not supported',
-    'interpret': 'Interpreting for loop not yet implemented'
+    'interpret': 'Interpreting for loop is not yet implemented'
 }
 
 tests = [
diff --git a/test/js2tests/func_expr.py b/test/js2tests/func_expr.py
index 2071fdac6..4873500e0 100644
--- a/test/js2tests/func_expr.py
+++ b/test/js2tests/func_expr.py
@@ -5,7 +5,7 @@ from youtube_dl.jsinterp2.tstream import _ASSIGN_OPERATORS
 
 skip = {
     'jsinterp': 'not supported',
-    'interpret': 'Interpreting function expression not yet implemented'
+    'interpret': 'Interpreting function expression is not yet implemented'
 }
 
 tests = [
diff --git a/test/js2tests/label.py b/test/js2tests/label.py
index 45eac8bd7..ed33c4d13 100644
--- a/test/js2tests/label.py
+++ b/test/js2tests/label.py
@@ -4,8 +4,8 @@ from youtube_dl.jsinterp2.jsgrammar import Token
 
 skip = {
     'jsinterp': 'Label statement is not supported',
-    'interpret': 'Interpreting label not yet implemented',
-    'parse': 'Test not yet implemented: missing code and ast'
+    'interpret': 'Interpreting label is not yet implemented',
+    'parse': 'Test is not yet implemented: missing code and ast'
 }
 
 tests = [
diff --git a/test/js2tests/object_literal.py b/test/js2tests/object_literal.py
index b486591ef..32b896a7f 100644
--- a/test/js2tests/object_literal.py
+++ b/test/js2tests/object_literal.py
@@ -3,7 +3,7 @@ from __future__ import unicode_literals
 from youtube_dl.jsinterp2.jsgrammar import Token
 from youtube_dl.jsinterp2.tstream import _ASSIGN_OPERATORS, _OPERATORS
 
-skip = {'interpret': 'Interpreting object literals not yet implemented'}
+skip = {'interpret': 'Interpreting object literals is not yet implemented'}
 
 tests = [
     {
diff --git a/test/js2tests/precedence.py b/test/js2tests/precedence.py
index e80142597..51845a646 100644
--- a/test/js2tests/precedence.py
+++ b/test/js2tests/precedence.py
@@ -3,7 +3,7 @@ from __future__ import unicode_literals
 from youtube_dl.jsinterp2.jsgrammar import Token
 from youtube_dl.jsinterp2.tstream import _ASSIGN_OPERATORS, _OPERATORS
 
-skip = {'interpret': 'Interpreting built-in fields not yet implemented'}
+skip = {'interpret': 'Interpreting built-in fields are not yet implemented'}
 
 tests = [
     {
diff --git a/test/js2tests/stringprototype.py b/test/js2tests/stringprototype.py
index 756be2563..e7d75b35b 100644
--- a/test/js2tests/stringprototype.py
+++ b/test/js2tests/stringprototype.py
@@ -2,12 +2,12 @@ from __future__ import unicode_literals
 
 skip = {
     'jsinterp': 'String literals are not supported',
-    'parse': 'Ast not yet implemented'
+    'interpret': 'Builtins are not yet implemented',
+    'parse': 'Test is not yet implemented: missing ast'
 }
 
 tests = [
     {
-        'exclude': ('jsinterp2',),
         'code': 'function f() {return "hello".split(""); }',
         'globals': {},
         'asserts': [{'value': ['h', 'e', 'l', 'l', 'o'], 'call': ('f',)}],
diff --git a/test/js2tests/switch.py b/test/js2tests/switch.py
index d858e255f..29547ec05 100644
--- a/test/js2tests/switch.py
+++ b/test/js2tests/switch.py
@@ -5,7 +5,7 @@ from youtube_dl.jsinterp2.tstream import _ASSIGN_OPERATORS, _UNARY_OPERATORS
 
 skip = {
     'jsinterp': 'Switch statement is not supported',
-    'interpret': 'Interpreting switch statement not yet implemented'
+    'interpret': 'Interpreting switch statement is not yet implemented'
 }
 
 tests = [
diff --git a/test/js2tests/try_statement.py b/test/js2tests/try_statement.py
index eb7882517..82f2a5d34 100644
--- a/test/js2tests/try_statement.py
+++ b/test/js2tests/try_statement.py
@@ -4,8 +4,8 @@ from youtube_dl.jsinterp2.jsgrammar import Token
 
 skip = {
     'jsinterp': 'Try statement is not supported',
-    'interpret': 'Interpreting try statement not yet implemented',
-    'parse': 'Test not yet implemented: missing code and ast'
+    'interpret': 'Interpreting try statement is not yet implemented',
+    'parse': 'Test is not yet implemented: missing code and ast'
 }
 
 tests = [
diff --git a/test/js2tests/unary.py b/test/js2tests/unary.py
index 6d2372fdd..b82130f89 100644
--- a/test/js2tests/unary.py
+++ b/test/js2tests/unary.py
@@ -2,7 +2,7 @@ from __future__ import unicode_literals
 
 skip = {
     'jsinterp': 'Unary operations are not supported',
-    'parse': True
+    'parse': 'Test is not yet implemented: missing ast'
 }
 
 tests = [
diff --git a/test/js2tests/unshift.py b/test/js2tests/unshift.py
index b2a4cd34e..04d4a161a 100644
--- a/test/js2tests/unshift.py
+++ b/test/js2tests/unshift.py
@@ -1,9 +1,9 @@
 from __future__ import unicode_literals
 
 skip = {
-    'jsinterp': 'Test not implemented',
-    'parse': 'Test not implemented',
-    'interpert': 'Test not implemented'
+    'jsinterp': 'Test is not implemented',
+    'interpert': 'Test is not implemented',
+    'parse': 'Test is not implemented'
 }
 
 tests = [
diff --git a/test/js2tests/while_loop.py b/test/js2tests/while_loop.py
index f215cd673..edb358451 100644
--- a/test/js2tests/while_loop.py
+++ b/test/js2tests/while_loop.py
@@ -5,7 +5,7 @@ from youtube_dl.jsinterp2.tstream import _ASSIGN_OPERATORS, _UNARY_OPERATORS, _R
 
 skip = {
     'jsinterp': 'While loop is not supported',
-    'interpret': 'Interpreting while loop not yet implemented'
+    'interpret': 'Interpreting while loop is not yet implemented'
 }
 
 tests = [
diff --git a/test/js2tests/with_statement.py b/test/js2tests/with_statement.py
index e0869c9d2..7369a3c90 100644
--- a/test/js2tests/with_statement.py
+++ b/test/js2tests/with_statement.py
@@ -4,8 +4,8 @@ from youtube_dl.jsinterp2.jsgrammar import Token
 
 skip = {
     'jsinterp': 'With statement is not supported',
-    'interpret': 'Interpreting with statement not yet implemented',
-    'parse': 'Test not yet implemented: missing code and ast'
+    'interpret': 'Interpreting with statement is not yet implemented',
+    'parse': 'Test is not yet implemented: missing code and ast'
 }
 
 tests = [