[JS TESTS] Replace - symbol in module name with _

js/js.tests/test/org/jetbrains/kotlin/js/test/BasicBoxTest.kt

@@ -1265,7 +1265,7 @@ abstract class BasicBoxTest(
         const val TEST_MODULE = "JS_TESTS"
         const val DEFAULT_MODULE = "main"
         private const val TEST_FUNCTION = "box"
-        private const val OLD_MODULE_SUFFIX = "-old"
+        private const val OLD_MODULE_SUFFIX = "_old"
 
         const val KOTLIN_TEST_INTERNAL = "\$kotlin_test_internal\$"
         private val engineForMinifier = createScriptEngine()

js/js.translator/testData/box/crossModuleRef/inlineJsModuleNonIdentifier.kt

@@ -1,16 +1,16 @@
 // EXPECTED_REACHABLE_NODES: 1316
-// MODULE: lib-1
-// FILE: lib-1.js
-define("lib-1", [], function() {
+// MODULE: lib_1
+// FILE: lib_1.js
+define("lib_1", [], function() {
     return function() {
         return "OK";
     }
 })
 
-// MODULE: lib2(lib-1)
+// MODULE: lib2(lib_1)
 // FILE: lib2.kt
 // MODULE_KIND: AMD
-@JsModule("lib-1")
+@JsModule("lib_1")
 external fun foo(): String
 
 // MODULE: lib3(lib2)

js/js.translator/testData/box/crossModuleRefIR/inlineJsModuleNonIdentifier.kt

@@ -1,17 +1,17 @@
 // SPLIT_PER_MODULE
 // EXPECTED_REACHABLE_NODES: 1316
-// MODULE: lib-1
-// FILE: lib-1.js
-define("lib-1", [], function() {
+// MODULE: lib_1
+// FILE: lib_1.js
+define("lib_1", [], function() {
     return function() {
         return "OK";
     }
 })
 
-// MODULE: lib2(lib-1)
+// MODULE: lib2(lib_1)
 // FILE: lib2.kt
 // MODULE_KIND: AMD
-@JsModule("lib-1")
+@JsModule("lib_1")
 external fun foo(): String
 
 // MODULE: lib3(lib2)

js/js.translator/testData/box/esModules/export/overriddenExternalMethodWithSameStableNameMethod.kt

@@ -6,7 +6,7 @@
 // TODO: Fix tests on Windows
 // DONT_TARGET_EXACT_BACKEND: JS_IR
 
-// MODULE: overriden-external-method-with-same-stable-name-method
+// MODULE: overriden_external_method_with_same_stable_name_method
 // FILE: lib.kt
 external abstract class Foo {
     abstract fun o(): String

js/js.translator/testData/box/export/exportAllFile.kt

@@ -3,7 +3,7 @@
 // RUN_PLAIN_BOX_FUNCTION
 // INFER_MAIN_MODULE
 
-// MODULE: export-all-file
+// MODULE: export_all_file
 // FILE: lib.kt
 @file:JsExport
 
@@ -19,5 +19,5 @@ class B : A() {
 
 // FILE: test.js
 function box() {
-    return new this["export-all-file"].B().foo("K");
+    return new this["export_all_file"].B().foo("K");
 }

js/js.translator/testData/box/export/exportNestedClass.kt

@@ -4,7 +4,7 @@
 // INFER_MAIN_MODULE
 // SKIP_DCE_DRIVEN
 
-// MODULE: export-nested-class
+// MODULE: export_nested_class
 // FILE: lib.kt
 
 abstract class A {
@@ -26,5 +26,5 @@ class B {
 
 // FILE: test.js
 function box() {
-    return new this["export-nested-class"].B.Foo().bar("K");
+    return new this["export_nested_class"].B.Foo().bar("K");
 }

js/js.translator/testData/box/export/nonIndetifierModuleName.kt

@@ -4,7 +4,7 @@
 // INFER_MAIN_MODULE
 // SKIP_NODE_JS
 
-// MODULE: non-identifier-module-name
+// MODULE: non_identifier_module_name
 // FILE: lib.kt
 @JsName("foo")
 @JsExport
@@ -12,5 +12,5 @@ public fun foo(k: String): String = "O$k"
 
 // FILE: test.js
 function box() {
-    return this["non-identifier-module-name"].foo("K");
+    return this["non_identifier_module_name"].foo("K");
 }

js/js.translator/testData/box/export/overriddenChainNonExportIntermediate.kt

@@ -3,7 +3,7 @@
 // RUN_PLAIN_BOX_FUNCTION
 // INFER_MAIN_MODULE
 
-// MODULE: overriden-chain-non-export-intermediate
+// MODULE: overriden_chain_non_export_intermediate
 // FILE: lib.kt
 @JsExport
 abstract class A {
@@ -29,7 +29,7 @@ class C : B() {
 // FILE: test.js
 
 function box() {
-    return test(new this["overriden-chain-non-export-intermediate"].C());
+    return test(new this["overriden_chain_non_export_intermediate"].C());
 }
 
 function test(c) {

js/js.translator/testData/box/export/overriddenExternalMethodWithSameNameMethod.kt

@@ -3,7 +3,7 @@
 // RUN_PLAIN_BOX_FUNCTION
 // INFER_MAIN_MODULE
 
-// MODULE: overriden-external-method-with-same-name-method
+// MODULE: overriden_external_method_with_same_name_method
 // FILE: lib.kt
 external abstract class Foo {
     abstract fun o(): String
@@ -31,7 +31,7 @@ Foo.prototype.k = function() {
 }
 
 function box() {
-    return test(new this["overriden-external-method-with-same-name-method"].Baz());
+    return test(new this["overriden_external_method_with_same_name_method"].Baz());
 }
 
 function test(foo) {

js/js.translator/testData/box/export/overriddenExternalMethodWithSameStableNameMethod.kt

@@ -3,7 +3,7 @@
 // RUN_PLAIN_BOX_FUNCTION
 // INFER_MAIN_MODULE
 
-// MODULE: overriden-external-method-with-same-stable-name-method
+// MODULE: overriden_external_method_with_same_stable_name_method
 // FILE: lib.kt
 external abstract class Foo {
     abstract fun o(): String
@@ -32,7 +32,7 @@ Foo.prototype.k = function() {
 }
 
 function box() {
-    return test(new this["overriden-external-method-with-same-stable-name-method"].Baz());
+    return test(new this["overriden_external_method_with_same_stable_name_method"].Baz());
 }
 
 function test(foo) {

js/js.translator/testData/box/multiModule/privateNameClash.kt

@@ -12,7 +12,7 @@ open class A {
     fun foo() = x
 }
 
-// MODULE: lib-old
+// MODULE: lib_old
 // FILE: lib.kt
 package lib
 
@@ -22,7 +22,7 @@ open class A {
 
 inline fun check() = true
 
-// MODULE: main(lib-old)
+// MODULE: main(lib_old)
 // FILE: main.kt
 package main
 

js/js.translator/testData/box/multiModuleWrappers/amd/moduleWithNonIdentifierName.kt

@@ -1,10 +1,10 @@
 // EXPECTED_REACHABLE_NODES: 1284
-// MODULE: module-1
+// MODULE: module_1
 // FILE: bar.kt
 // MODULE_KIND: AMD
 fun bar() = "bar"
 
-// MODULE: main(module-1)
+// MODULE: main(module_1)
 // FILE: box.kt
 // MODULE_KIND: AMD
 fun box(): String {

js/js.translator/testData/box/multiModuleWrappers/common_js/inlineFromModuleWithNonIdentifierName.kt

@@ -1,5 +1,5 @@
 // EXPECTED_REACHABLE_NODES: 1284
-// MODULE: #my-libr@ry
+// MODULE: #my_libr@ry
 // FILE: bar.kt
 // MODULE_KIND: COMMON_JS
 // PROPERTY_NOT_WRITTEN_TO: baz
@@ -11,7 +11,7 @@ inline fun foo() = "foo"
 inline fun baz() = "baz"
 inline fun <reified T> boo() = "boo"
 
-// MODULE: main(#my-libr@ry)
+// MODULE: main(#my_libr@ry)
 // FILE: box.kt
 // MODULE_KIND: COMMON_JS
 // CHECK_CONTAINS_NO_CALLS: box except=assertEquals;assertEquals$default

js/js.translator/testData/box/multiModuleWrappers/common_js/moduleWithNonIdentifierName.kt

@@ -1,10 +1,10 @@
 // EXPECTED_REACHABLE_NODES: 1285
-// MODULE: module-1
+// MODULE: module_1
 // FILE: bar.kt
 // MODULE_KIND: COMMON_JS
 fun bar() = "bar"
 
-// MODULE: main(module-1)
+// MODULE: main(module_1)
 // FILE: box.kt
 // MODULE_KIND: COMMON_JS
 fun box(): String {

js/js.translator/testData/box/multiModuleWrappers/plain/inlineFromModuleWithNonIdentifierName.kt

@@ -1,5 +1,5 @@
 // EXPECTED_REACHABLE_NODES: 1282
-// MODULE: #my-libr@ry
+// MODULE: #my_libr@ry
 // FILE: bar.kt
 // MODULE_KIND: PLAIN
 // PROPERTY_NOT_WRITTEN_TO: baz
@@ -11,7 +11,7 @@ inline fun foo() = "foo"
 inline fun baz() = "baz"
 inline fun <reified T> boo() = "boo"
 
-// MODULE: main(#my-libr@ry)
+// MODULE: main(#my_libr@ry)
 // FILE: box.kt
 // MODULE_KIND: PLAIN
 // CHECK_CONTAINS_NO_CALLS: box except=assertEquals TARGET_BACKENDS=JS

js/js.translator/testData/box/multiModuleWrappers/plain/moduleWithNonIdentifierName.kt

@@ -1,10 +1,10 @@
 // EXPECTED_REACHABLE_NODES: 1283
-// MODULE: module-1
+// MODULE: module_1
 // FILE: bar.kt
 // MODULE_KIND: PLAIN
 fun bar() = "bar"
 
-// MODULE: main(module-1)
+// MODULE: main(module_1)
 // FILE: box.kt
 // MODULE_KIND: PLAIN
 fun box(): String {

js/js.translator/testData/box/multiModuleWrappers/umd/moduleWithNonIdentifierName.kt

@@ -1,10 +1,10 @@
 // EXPECTED_REACHABLE_NODES: 1286
-// MODULE: module-1
+// MODULE: module_1
 // FILE: bar.kt
 // MODULE_KIND: UMD
 fun bar() = "bar"
 
-// MODULE: main(module-1)
+// MODULE: main(module_1)
 // FILE: box.kt
 // MODULE_KIND: UMD
 fun box(): String {