From a52ab7329b4c1d6b1fa75ed91b6d7b646f26af22 Mon Sep 17 00:00:00 2001 From: Lars Hvam Date: Sun, 18 Jul 2021 13:17:24 +0200 Subject: [PATCH] update deps + additional unit testing via open-abap (#4871) --- .github/workflows/unit.yml | 20 +++++++++++++++++++ abap_transpile.json | 40 +++++++++++++++++++++++++++++++++++--- abaplint.json | 2 ++ package.json | 8 ++++---- 4 files changed, 63 insertions(+), 7 deletions(-) create mode 100644 .github/workflows/unit.yml diff --git a/.github/workflows/unit.yml b/.github/workflows/unit.yml new file mode 100644 index 000000000..95511436d --- /dev/null +++ b/.github/workflows/unit.yml @@ -0,0 +1,20 @@ +name: unit + +on: [push] + +jobs: + unit: + + runs-on: ubuntu-latest + + steps: + - uses: actions/checkout@v2 + with: + fetch-depth: 2 + - uses: actions/setup-node@v2 + with: + node-version: '14' + - name: npm run unit + run: | + npm install + npm run unit \ No newline at end of file diff --git a/abap_transpile.json b/abap_transpile.json index dac376719..39b0f3a02 100644 --- a/abap_transpile.json +++ b/abap_transpile.json @@ -13,7 +13,13 @@ "zcl_abapgit_path", "zcl_abapgit_progress", "zcl_abapgit_string_map", + "zcl_abapgit_gui_utils", + "zcl_abapgit_gui_html_processor", + "zcl_abapgit_gui_css_processor", + "zif_abapgit_gui_html_processor", "zcl_abapgit_syntax_*", + "zcl_abapgit_html_parts", + "zcl_abapgit_gui_asset_manager", "zif_abapgit_ajson*", "zcl_abapgit_gui_functions", "zcl_abapgit_ajson*", @@ -35,6 +41,8 @@ "zif_abapgit_cts_api", "zif_abapgit_definitions", "zif_abapgit_dot_abapgit", + "zif_abapgit_apack_definitions", + "zcl_abapgit_dot_abapgit", "zif_abapgit_environment", "zif_abapgit_exit", "zcl_abapgit_gui_buttons", @@ -126,8 +134,7 @@ {"object": "ZCL_ABAPGIT_AJSON", "class": "ltcl_serializer_test", "method": "simple_indented", "note": "fix 'array_index' first"}, {"object": "ZCL_ABAPGIT_AJSON", "class": "ltcl_serializer_test", "method": "escape", "note": "ASSERT failed, ??"}, - {"object": "ZCL_ABAPGIT_AJSON", "class": "ltcl_reader_test", "method": "value_number"}, - {"object": "ZCL_ABAPGIT_AJSON", "class": "ltcl_reader_test", "method": "members"}, + {"object": "ZCL_ABAPGIT_AJSON", "class": "ltcl_reader_test", "method": "members", "note": "sorted internal tables/arrays are required"}, {"object": "ZCL_ABAPGIT_AJSON", "class": "ltcl_reader_test", "method": "slice"}, {"object": "ZCL_ABAPGIT_AJSON", "class": "ltcl_reader_test", "method": "array_to_string_table"}, {"object": "ZCL_ABAPGIT_AJSON", "class": "ltcl_reader_test", "method": "get_timestamp"}, @@ -183,7 +190,34 @@ {"object": "ZCL_ABAPGIT_AJSON_UTILITIES", "class": "ltcl_json_utils", "method": "json_diff"}, {"object": "ZCL_ABAPGIT_AJSON_UTILITIES", "class": "ltcl_json_utils", "method": "json_diff_types"}, - {"object": "ZCL_ABAPGIT_AJSON_UTILITIES", "class": "ltcl_json_utils", "method": "json_sort"} + {"object": "ZCL_ABAPGIT_AJSON_UTILITIES", "class": "ltcl_json_utils", "method": "json_sort"}, + + {"object": "ZCL_ABAPGIT_GUI_UTILS", "class": "ltcl_gui_utils", "method": "is_renderable"}, + {"object": "ZCL_ABAPGIT_GUI_UTILS", "class": "ltcl_gui_utils", "method": "is_event_handler"}, + + {"object": "ZCL_ABAPGIT_GUI_ASSET_MANAGER", "class": "ltcl_abapgit_gui_asset_manager", "method": "get_inline_asset"}, + {"object": "ZCL_ABAPGIT_GUI_ASSET_MANAGER", "class": "ltcl_abapgit_gui_asset_manager", "method": "get_text_asset"}, + {"object": "ZCL_ABAPGIT_GUI_ASSET_MANAGER", "class": "ltcl_abapgit_gui_asset_manager", "method": "get_mime_asset"}, + {"object": "ZCL_ABAPGIT_GUI_ASSET_MANAGER", "class": "ltcl_abapgit_gui_asset_manager", "method": "get_base64_asset"}, + {"object": "ZCL_ABAPGIT_GUI_ASSET_MANAGER", "class": "ltcl_abapgit_gui_asset_manager", "method": "get_all"}, + + {"object": "ZCL_ABAPGIT_GUI_CSS_PROCESSOR", "class": "ltcl_single_file", "method": "test_file_exists", "note": "something wrong with the inheritance?"}, + {"object": "ZCL_ABAPGIT_GUI_CSS_PROCESSOR", "class": "ltcl_single_file", "method": "test_file_does_not_exist"}, + {"object": "ZCL_ABAPGIT_GUI_CSS_PROCESSOR", "class": "ltcl_single_file", "method": "test_empty_file"}, + {"object": "ZCL_ABAPGIT_GUI_CSS_PROCESSOR", "class": "ltcl_single_file", "method": "test_no_variables"}, + {"object": "ZCL_ABAPGIT_GUI_CSS_PROCESSOR", "class": "ltcl_single_file", "method": "test_simple_variables"}, + {"object": "ZCL_ABAPGIT_GUI_CSS_PROCESSOR", "class": "ltcl_single_file", "method": "test_complex_variables"}, + {"object": "ZCL_ABAPGIT_GUI_CSS_PROCESSOR", "class": "ltcl_single_file", "method": "test_overwrite"}, + {"object": "ZCL_ABAPGIT_GUI_CSS_PROCESSOR", "class": "ltcl_multiple_files", "method": "test_overwrite"}, + + {"object": "ZCL_ABAPGIT_GUI_HTML_PROCESSOR", "class": "ltcl_html_processor_test", "method": "process_typical"}, + {"object": "ZCL_ABAPGIT_GUI_HTML_PROCESSOR", "class": "ltcl_html_processor_test", "method": "process_with_preserve"}, + {"object": "ZCL_ABAPGIT_GUI_HTML_PROCESSOR", "class": "ltcl_html_processor_test", "method": "process_no_css"}, + {"object": "ZCL_ABAPGIT_GUI_HTML_PROCESSOR", "class": "ltcl_html_processor_test", "method": "process_fails"}, + {"object": "ZCL_ABAPGIT_GUI_HTML_PROCESSOR", "class": "ltcl_html_processor_test", "method": "find_head_closing_tag"}, + + {"object": "ZCL_ABAPGIT_DOT_ABAPGIT", "class": "ltcl_dot_abapgit", "method": "ignore"}, + {"object": "ZCL_ABAPGIT_DOT_ABAPGIT", "class": "ltcl_dot_abapgit", "method": "identity"} ] } } \ No newline at end of file diff --git a/abaplint.json b/abaplint.json index 8fb61f718..ecaf7a267 100644 --- a/abaplint.json +++ b/abaplint.json @@ -69,6 +69,7 @@ "db_operation_in_loop": false, "select_performance": false, "max_one_method_parameter_per_line": true, + "unused_ddic": true, "prefer_raise_exception_new": true, "use_class_based_exceptions": true, "uncaught_exception": true, @@ -130,6 +131,7 @@ "prefix_is_current_class": true, "allowed_object_naming": true, "check_comments": false, + "cyclic_oo": false, "fully_type_constants": true, "keep_single_parameter_on_one_line": true, "prefer_returning_to_exporting": true, diff --git a/package.json b/package.json index 2d31a60e0..4d5e1c55c 100644 --- a/package.json +++ b/package.json @@ -17,10 +17,10 @@ "url": "git+https://github.com/abapGit/abapGit.git" }, "devDependencies": { - "@abaplint/cli": "^2.74.54", - "@abaplint/transpiler-cli": "^1.3.49", - "@abaplint/runtime": "^1.3.49", + "@abaplint/cli": "^2.76.3", + "@abaplint/transpiler-cli": "^1.4.0", + "@abaplint/runtime": "^1.4.0", "abapmerge": "^0.14.3", - "eslint": "^7.29.0" + "eslint": "^7.31.0" } }