Remove ES_NEXT macro (#4915)

- remove all '#JERRY_ESNEXT' macro
- remove 5.1 build profile, update test runner accordingly (Note: all builtins are turn on by default)
- move tests from tests/jerry/esnext into tests/jerry, concatenate files with same names
- add skiplist to some snapshot tests that were supported only in 5.1
- fix doxygen issues that were hidden before (bc. of es.next macro)

Co-authored-by: Martin Negyokru [email protected]
JerryScript-DCO-1.0-Signed-off-by: Adam Szilagyi [email protected]

Author: szilagyiadam

.github/workflows/gh-actions.yml

@@ -75,18 +75,11 @@ jobs:
       - run: sudo apt install gcc-multilib
       - run: $RUNNER --buildoption-test
 
-  Conformance_Tests_ES5_1:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v2
-      - run: $RUNNER --test262
-      - run: $RUNNER --test262 --build-debug
-
   Conformance_Tests_ESNext:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - run: $RUNNER --test262-esnext=update
+      - run: $RUNNER --test262 update
       - uses: actions/upload-artifact@v2
         if: success() || failure()
         with:
@@ -98,7 +91,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - run: $RUNNER --test262-esnext=update --build-debug --test262-test-list=built-ins,annexB,harness,intl402
+      - run: $RUNNER --test262 update --build-debug --test262-test-list=built-ins,annexB,harness,intl402
       - uses: actions/upload-artifact@v2
         if: success() || failure()
         with:
@@ -110,7 +103,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - run: $RUNNER --test262-esnext=update --build-debug --test262-test-list=language
+      - run: $RUNNER --test262 update --build-debug --test262-test-list=language
       - uses: actions/upload-artifact@v2
         if: success() || failure()
         with:

docs/02.API-REFERENCE.md

@@ -2863,13 +2863,6 @@ jerry_value_is_object (const jerry_value_t value)
 
 Returns whether the given `jerry_value_t` is a promise value.
 
-*Notes*:
-- This API depends on a build option (`JERRY_ESNEXT`) and can be checked
-  in runtime with the `JERRY_FEATURE_PROMISE` feature enum value,
-  see: [jerry_feature_enabled](#jerry_feature_enabled).
-- The es.next profile enables this by default.
-
-
 **Prototype**
 
 ```c
@@ -2884,8 +2877,6 @@ jerry_value_is_promise (const jerry_value_t value)
 
 *New in version 2.0*.
 
-*Changed in version [[NEXT_RELEASE]]*: Build option dependency changed from `JERRY_BUILTIN_PROMISE` to `JERRY_ESNEXT`.
-
 **Example**
 
 ```c
@@ -3003,12 +2994,6 @@ jerry_value_is_string (const jerry_value_t value)
 
 Returns whether the given `jerry_value_t` is a symbol value.
 
-*Notes*:
-- This API depends on a build option (`JERRY_BUILTIN_SYMBOL`) and can be checked
-  in runtime with the `JERRY_FEATURE_SYMBOL` feature enum value,
-  see: [jerry_feature_enabled](#jerry_feature_enabled).
-- The es.next profile enables this by default.
-
 **Prototype**
 
 ```c
@@ -5728,11 +5713,6 @@ The function returns the result of a Promise object.
 *Notes*:
 - Returned value must be freed with [jerry_value_free](#jerry_value_free) when it
   is no longer needed.
-- This API depends on a build option (`JERRY_ESNEXT`) and can be checked
-  in runtime with the `JERRY_FEATURE_PROMISE` feature enum value,
-  see: [jerry_feature_enabled](#jerry_feature_enabled).
-- The es.next profile enables this by default.
-
 
 **Prototype**
 
@@ -5750,8 +5730,6 @@ jerry_promise_result (const jerry_value_t promise);
 
 *New in version 2.2*.
 
-*Changed in version [[NEXT_RELEASE]]*: Build option dependency changed from `JERRY_BUILTIN_PROMISE` to `JERRY_ESNEXT`.
-
 **Example**
 
 [doctest]: # (test="compile")
@@ -5796,11 +5774,6 @@ example (void)
 *Notes*:
 - Returned value must be freed with [jerry_value_free](#jerry_value_free) when it
   is no longer needed.
-- This API depends on a build option (`JERRY_ESNEXT`) and can be checked
-  in runtime with the `JERRY_FEATURE_PROMISE` feature enum value,
-  see: [jerry_feature_enabled](#jerry_feature_enabled).
-- The es.next profile enables this by default.
-
 
 **Prototype**
 
@@ -5817,8 +5790,6 @@ jerry_promise_state (const jerry_value_t promise);
 
 *New in version 2.2*.
 
-*Changed in version [[NEXT_RELEASE]]*: Build option dependency changed from `JERRY_BUILTIN_PROMISE` to `JERRY_ESNEXT`.
-
 **Example**
 
 [doctest]: # (test="compile")
@@ -6107,11 +6078,6 @@ Reject the promise with an argument.
 *Note*:
 - Returned value must be freed with [jerry_value_free](#jerry_value_free) when it
   is no longer needed.
-- This API depends on a build option (`JERRY_ESNEXT`) and can be checked
-  in runtime with the `JERRY_FEATURE_PROMISE` feature enum value,
-  see: [jerry_feature_enabled](#jerry_feature_enabled).
-- The es.next profile enables this by default.
-
 
 **Prototype**
 
@@ -6128,8 +6094,6 @@ jerry_promise_reject (jerry_value_t promise, jerry_value_t argument);
 
 *New in version 2.0*.
 
-*Changed in version [[NEXT_RELEASE]]*: Build option dependency changed from `JERRY_BUILTIN_PROMISE` to `JERRY_ESNEXT`.
-
 **Example**
 
 ```c
@@ -6171,10 +6135,6 @@ Get the well-known symbol corresponding to the given [well-known symbol id](#jer
 *Notes*:
 - Returned value must be freed with [jerry_value_free](#jerry_value_free) when it
   is no longer needed.
-- This API depends on a build option (`JERRY_BUILTIN_SYMBOL`) and can be checked
-  in runtime with the `JERRY_FEATURE_SYMBOL` feature enum value,
-  see: [jerry_feature_enabled](#jerry_feature_enabled).
-- The es.next profile enables this by default.
 - If the symbol support is not enabled an undefined will be returned.
 
 **Prototype**
@@ -6231,11 +6191,6 @@ Returns with the `[[Description]]` internal property of a symbol value.
 *Notes*:
 - Returned value must be freed with [jerry_value_free](#jerry_value_free) when it
   is no longer needed.
-- This API depends on a build option (`JERRY_BUILTIN_SYMBOL`) and can be checked
-  in runtime with the `JERRY_FEATURE_SYMBOL` feature enum value,
-  see: [jerry_feature_enabled](#jerry_feature_enabled).
-- The es.next profile enables this by default.
-- If symbol support is not enabled an exception will be returned.
 
 **Prototype**
 
@@ -6294,11 +6249,6 @@ Based on ECMA 262 v6 19.4.3.2.1 this is in the form of `Symbol(<description>)`.
 *Notes*:
 - Returned value must be freed with [jerry_value_free](#jerry_value_free) when it
   is no longer needed.
-- This API depends on a build option (`JERRY_BUILTIN_SYMBOL`) and can be checked
-  in runtime with the `JERRY_FEATURE_SYMBOL` feature enum value,
-  see: [jerry_feature_enabled](#jerry_feature_enabled).
-- The es.next profile enables this by default.
-- If symbol support is not enabled an exception will be returned.
 
 **Prototype**
 

jerry-core/api/jerry-snapshot.c

@@ -50,9 +50,8 @@ snapshot_get_global_flags (bool has_regex, /**< regex literal is present */
 #if JERRY_BUILTIN_REGEXP
   flags |= (has_regex ? JERRY_SNAPSHOT_HAS_REGEX_LITERAL : 0);
 #endif /* JERRY_BUILTIN_REGEXP */
-#if JERRY_ESNEXT
+
   flags |= (has_class ? JERRY_SNAPSHOT_HAS_CLASS_LITERAL : 0);
-#endif /* JERRY_ESNEXT */
 
   return flags;
 } /* snapshot_get_global_flags */
@@ -68,9 +67,8 @@ snapshot_check_global_flags (uint32_t global_flags) /**< global flags */
 #if JERRY_BUILTIN_REGEXP
   global_flags &= (uint32_t) ~JERRY_SNAPSHOT_HAS_REGEX_LITERAL;
 #endif /* JERRY_BUILTIN_REGEXP */
-#if JERRY_ESNEXT
+
   global_flags &= (uint32_t) ~JERRY_SNAPSHOT_HAS_CLASS_LITERAL;
-#endif /* JERRY_ESNEXT */
 
   return global_flags == snapshot_get_global_flags (false, false);
 } /* snapshot_check_global_flags */
@@ -164,7 +162,6 @@ snapshot_add_compiled_code (const ecma_compiled_code_t *compiled_code_p, /**< co
   uint8_t *copied_code_start_p = snapshot_buffer_p + globals_p->snapshot_buffer_write_offset;
   ecma_compiled_code_t *copied_code_p = (ecma_compiled_code_t *) copied_code_start_p;
 
-#if JERRY_ESNEXT
   if (compiled_code_p->status_flags & CBC_CODE_FLAGS_HAS_TAGGED_LITERALS)
   {
     globals_p->snapshot_error =
@@ -176,7 +173,6 @@ snapshot_add_compiled_code (const ecma_compiled_code_t *compiled_code_p, /**< co
   {
     globals_p->class_found = true;
   }
-#endif /* JERRY_ESNEXT */
 
 #if JERRY_BUILTIN_REGEXP
   if (!CBC_IS_FUNCTION (compiled_code_p->status_flags))
@@ -629,7 +625,6 @@ snapshot_load_compiled_code (const uint8_t *base_addr_p, /**< base address of th
       extra_bytes += (uint32_t) (argument_end * sizeof (ecma_value_t));
     }
 
-#if JERRY_ESNEXT
     /* function name */
     if (CBC_FUNCTION_GET_TYPE (bytecode_p->status_flags) != CBC_FUNCTION_CONSTRUCTOR)
     {
@@ -641,7 +636,6 @@ snapshot_load_compiled_code (const uint8_t *base_addr_p, /**< base address of th
     {
       extra_bytes += (uint32_t) sizeof (ecma_value_t);
     }
-#endif /* JERRY_ESNEXT */
 
 #if JERRY_SOURCE_NAME
     /* source name */
@@ -1011,12 +1005,10 @@ jerry_exec_snapshot (const uint32_t *snapshot_p, /**< snapshot */
     JERRY_ASSERT (global_object_p == (ecma_object_t *) ecma_op_function_get_realm (bytecode_p));
 #endif /* JERRY_BUILTIN_REALMS */
 
-#if JERRY_ESNEXT
     if (bytecode_p->status_flags & CBC_CODE_FLAGS_LEXICAL_BLOCK_NEEDED)
     {
       ecma_create_global_lexical_block (global_object_p);
     }
-#endif /* JERRY_ESNEXT */
 
     ecma_object_t *lex_env_p = ecma_get_global_scope (global_object_p);
     ecma_object_t *func_obj_p = ecma_op_create_simple_function_object (lex_env_p, bytecode_p);