refactor: rename files to follow Go convention

mgechev · Nov 11, 2024 · be95bfa · be95bfa
1 parent c0d4d07
commit be95bfa
Show file tree

Hide file tree

Showing 299 changed files with 181 additions and 178 deletions.
diff --git a/revive.toml b/revive.toml
@@ -19,6 +19,9 @@ warningCode = 1
 [rule.error-strings]
 [rule.errorf]
 [rule.exported]
+[rule.filename-format]
+    # Override the default pattern to forbid .go files with uppercase letters and dashes.
+    arguments=["^[_a-z][_a-z0-9]*.go$"]
 [rule.increment-decrement]
 [rule.indent-error-flow]
 [rule.line-length-limit]

diff --git a/revivelib/core_test.go b/revivelib/core_test.go
@@ -16,7 +16,7 @@ func TestReviveLint(t *testing.T) {
 	revive := getMockRevive(t)
 
 	// ACT
-	failures, err := revive.Lint(revivelib.Include("../testdata/if-return.go"))
+	failures, err := revive.Lint(revivelib.Include("../testdata/if_return.go"))
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -40,7 +40,7 @@ func TestReviveFormat(t *testing.T) {
 	// ARRANGE
 	revive := getMockRevive(t)
 
-	failuresChan, err := revive.Lint(revivelib.Include("../testdata/if-return.go"))
+	failuresChan, err := revive.Lint(revivelib.Include("../testdata/if_return.go"))
 	if err != nil {
 		t.Fatal(err)
 	}

diff --git a/rule/add-constant.go → rule/add_constant.go b/rule/add-constant.go → rule/add_constant.go
diff --git a/rule/argument-limit.go → rule/argument_limit.go b/rule/argument-limit.go → rule/argument_limit.go
diff --git a/rule/banned-characters.go → rule/banned_characters.go b/rule/banned-characters.go → rule/banned_characters.go
diff --git a/rule/bare-return.go → rule/bare_return.go b/rule/bare-return.go → rule/bare_return.go
diff --git a/rule/blank-imports.go → rule/blank_imports.go b/rule/blank-imports.go → rule/blank_imports.go
diff --git a/rule/bool-literal-in-expr.go → rule/bool_literal_in_expr.go b/rule/bool-literal-in-expr.go → rule/bool_literal_in_expr.go
diff --git a/rule/call-to-gc.go → rule/call_to_gc.go b/rule/call-to-gc.go → rule/call_to_gc.go
diff --git a/rule/cognitive-complexity.go → rule/cognitive_complexity.go b/rule/cognitive-complexity.go → rule/cognitive_complexity.go
diff --git a/rule/comment-spacings.go → rule/comment_spacings.go b/rule/comment-spacings.go → rule/comment_spacings.go
diff --git a/rule/comments-density.go → rule/comments_density.go b/rule/comments-density.go → rule/comments_density.go
diff --git a/rule/confusing-naming.go → rule/confusing_naming.go b/rule/confusing-naming.go → rule/confusing_naming.go
diff --git a/rule/confusing-results.go → rule/confusing_results.go b/rule/confusing-results.go → rule/confusing_results.go
diff --git a/rule/constant-logical-expr.go → rule/constant_logical_expr.go b/rule/constant-logical-expr.go → rule/constant_logical_expr.go
diff --git a/rule/context-as-argument.go → rule/context_as_argument.go b/rule/context-as-argument.go → rule/context_as_argument.go
diff --git a/rule/context-keys-type.go → rule/context_keys_type.go b/rule/context-keys-type.go → rule/context_keys_type.go
diff --git a/rule/deep-exit.go → rule/deep_exit.go b/rule/deep-exit.go → rule/deep_exit.go
diff --git a/rule/dot-imports.go → rule/dot_imports.go b/rule/dot-imports.go → rule/dot_imports.go
diff --git a/rule/duplicated-imports.go → rule/duplicated_imports.go b/rule/duplicated-imports.go → rule/duplicated_imports.go
diff --git a/rule/early-return.go → rule/early_return.go b/rule/early-return.go → rule/early_return.go
diff --git a/rule/empty-block.go → rule/empty_block.go b/rule/empty-block.go → rule/empty_block.go
diff --git a/rule/empty-lines.go → rule/empty_lines.go b/rule/empty-lines.go → rule/empty_lines.go
diff --git a/rule/enforce-map-style.go → rule/enforce_map_style.go b/rule/enforce-map-style.go → rule/enforce_map_style.go
diff --git a/rule/enforce-repeated-arg-type-style.go → rule/enforce_repeated_arg_type_style.go b/rule/enforce-repeated-arg-type-style.go → rule/enforce_repeated_arg_type_style.go
diff --git a/rule/enforce-slice-style.go → rule/enforce_slice_style.go b/rule/enforce-slice-style.go → rule/enforce_slice_style.go
diff --git a/rule/error-naming.go → rule/error_naming.go b/rule/error-naming.go → rule/error_naming.go
diff --git a/rule/error-return.go → rule/error_return.go b/rule/error-return.go → rule/error_return.go
diff --git a/rule/error-strings.go → rule/error_strings.go b/rule/error-strings.go → rule/error_strings.go
diff --git a/rule/file-header.go → rule/file_header.go b/rule/file-header.go → rule/file_header.go
diff --git a/rule/file-length-limit.go → rule/file_length_limit.go b/rule/file-length-limit.go → rule/file_length_limit.go
diff --git a/rule/filename-format.go → rule/filename_format.go b/rule/filename-format.go → rule/filename_format.go
diff --git a/rule/flag-param.go → rule/flag_param.go b/rule/flag-param.go → rule/flag_param.go
diff --git a/rule/function-length.go → rule/function_length.go b/rule/function-length.go → rule/function_length.go
diff --git a/rule/function-result-limit.go → rule/function_result_limit.go b/rule/function-result-limit.go → rule/function_result_limit.go
diff --git a/rule/get-return.go → rule/get_return.go b/rule/get-return.go → rule/get_return.go
diff --git a/rule/identical-branches.go → rule/identical_branches.go b/rule/identical-branches.go → rule/identical_branches.go
diff --git a/rule/if-return.go → rule/if_return.go b/rule/if-return.go → rule/if_return.go
diff --git a/rule/import-alias-naming.go → rule/import_alias_naming.go b/rule/import-alias-naming.go → rule/import_alias_naming.go
diff --git a/rule/import-shadowing.go → rule/import_shadowing.go b/rule/import-shadowing.go → rule/import_shadowing.go
diff --git a/rule/imports-blocklist.go → rule/imports_blocklist.go b/rule/imports-blocklist.go → rule/imports_blocklist.go
diff --git a/rule/increment-decrement.go → rule/increment_decrement.go b/rule/increment-decrement.go → rule/increment_decrement.go
diff --git a/rule/indent-error-flow.go → rule/indent_error_flow.go b/rule/indent-error-flow.go → rule/indent_error_flow.go
diff --git a/rule/line-length-limit.go → rule/line_length_limit.go b/rule/line-length-limit.go → rule/line_length_limit.go
diff --git a/rule/max-control-nesting.go → rule/max_control_nesting.go b/rule/max-control-nesting.go → rule/max_control_nesting.go
diff --git a/rule/max-public-structs.go → rule/max_public_structs.go b/rule/max-public-structs.go → rule/max_public_structs.go
diff --git a/rule/modifies-param.go → rule/modifies_param.go b/rule/modifies-param.go → rule/modifies_param.go
diff --git a/rule/modifies-value-receiver.go → rule/modifies_value_receiver.go b/rule/modifies-value-receiver.go → rule/modifies_value_receiver.go
diff --git a/rule/nested-structs.go → rule/nested_structs.go b/rule/nested-structs.go → rule/nested_structs.go
diff --git a/rule/optimize-operands-order.go → rule/optimize_operands_order.go b/rule/optimize-operands-order.go → rule/optimize_operands_order.go
diff --git a/rule/package-comments.go → rule/package_comments.go b/rule/package-comments.go → rule/package_comments.go
diff --git a/rule/range-val-address.go → rule/range_val_address.go b/rule/range-val-address.go → rule/range_val_address.go
diff --git a/rule/range-val-in-closure.go → rule/range_val_in_closure.go b/rule/range-val-in-closure.go → rule/range_val_in_closure.go
diff --git a/rule/receiver-naming.go → rule/receiver_naming.go b/rule/receiver-naming.go → rule/receiver_naming.go
diff --git a/rule/redefines-builtin-id.go → rule/redefines_builtin_id.go b/rule/redefines-builtin-id.go → rule/redefines_builtin_id.go
diff --git a/rule/redundant-import-alias.go → rule/redundant_import_alias.go b/rule/redundant-import-alias.go → rule/redundant_import_alias.go
diff --git a/rule/string-format.go → rule/string_format.go b/rule/string-format.go → rule/string_format.go
diff --git a/rule/string-of-int.go → rule/string_of_int.go b/rule/string-of-int.go → rule/string_of_int.go
diff --git a/rule/struct-tag.go → rule/struct_tag.go b/rule/struct-tag.go → rule/struct_tag.go
diff --git a/rule/superfluous-else.go → rule/superfluous_else.go b/rule/superfluous-else.go → rule/superfluous_else.go
diff --git a/rule/time-equal.go → rule/time_equal.go b/rule/time-equal.go → rule/time_equal.go
diff --git a/rule/time-naming.go → rule/time_naming.go b/rule/time-naming.go → rule/time_naming.go
diff --git a/rule/unchecked-type-assertion.go → rule/unchecked_type_assertion.go b/rule/unchecked-type-assertion.go → rule/unchecked_type_assertion.go
diff --git a/rule/unconditional-recursion.go → rule/unconditional_recursion.go b/rule/unconditional-recursion.go → rule/unconditional_recursion.go
diff --git a/rule/unexported-naming.go → rule/unexported_naming.go b/rule/unexported-naming.go → rule/unexported_naming.go
diff --git a/rule/unexported-return.go → rule/unexported_return.go b/rule/unexported-return.go → rule/unexported_return.go
diff --git a/rule/unhandled-error.go → rule/unhandled_error.go b/rule/unhandled-error.go → rule/unhandled_error.go
diff --git a/rule/unnecessary-stmt.go → rule/unnecessary_stmt.go b/rule/unnecessary-stmt.go → rule/unnecessary_stmt.go
diff --git a/rule/unreachable-code.go → rule/unreachable_code.go b/rule/unreachable-code.go → rule/unreachable_code.go
diff --git a/rule/unused-param.go → rule/unused_param.go b/rule/unused-param.go → rule/unused_param.go
diff --git a/rule/unused-receiver.go → rule/unused_receiver.go b/rule/unused-receiver.go → rule/unused_receiver.go
diff --git a/rule/use-any.go → rule/use_any.go b/rule/use-any.go → rule/use_any.go
diff --git a/rule/useless-break.go → rule/useless_break.go b/rule/useless-break.go → rule/useless_break.go
diff --git a/rule/var-declarations.go → rule/var_declarations.go b/rule/var-declarations.go → rule/var_declarations.go
diff --git a/rule/var-naming.go → rule/var_naming.go b/rule/var-naming.go → rule/var_naming.go
diff --git a/rule/waitgroup-by-value.go → rule/waitgroup_by_value.go b/rule/waitgroup-by-value.go → rule/waitgroup_by_value.go
diff --git a/test/add-constant_test.go → test/add_constant_test.go b/test/add-constant_test.go → test/add_constant_test.go
@@ -16,7 +16,7 @@ func TestAddConstant(t *testing.T) {
 		"ignoreFuncs": "os\\.(CreateFile|WriteFile|Chmod|FindProcess),\\.Println,ignoredFunc,\\.Info",
 	}}
 
-	testRule(t, "add-constant", &rule.AddConstantRule{}, &lint.RuleConfig{
+	testRule(t, "add_constant", &rule.AddConstantRule{}, &lint.RuleConfig{
 		Arguments: args,
 	})
 }
diff --git a/test/argument-limit_test.go → test/argument_limit_test.go b/test/argument-limit_test.go → test/argument_limit_test.go
@@ -8,7 +8,7 @@ import (
 )
 
 func TestArgumentLimit(t *testing.T) {
-	testRule(t, "argument-limit", &rule.ArgumentsLimitRule{}, &lint.RuleConfig{
+	testRule(t, "argument_limit", &rule.ArgumentsLimitRule{}, &lint.RuleConfig{
 		Arguments: []any{int64(3)},
 	})
 }
diff --git a/test/banned-characters_test.go → test/banned_characters_test.go b/test/banned-characters_test.go → test/banned_characters_test.go
@@ -11,7 +11,7 @@ import (
 // One banned character is in the comment and should not be checked.
 // One banned character from the list is not present in the fixture file.
 func TestBannedCharacters(t *testing.T) {
-	testRule(t, "banned-characters", &rule.BannedCharsRule{}, &lint.RuleConfig{
+	testRule(t, "banned_characters", &rule.BannedCharsRule{}, &lint.RuleConfig{
 		Arguments: []any{"Ω", "Σ", "σ", "1"},
 	})
 }
diff --git a/test/bare-return_test.go → test/bare_return_test.go b/test/bare-return_test.go → test/bare_return_test.go
@@ -7,5 +7,5 @@ import (
 )
 
 func TestBareReturn(t *testing.T) {
-	testRule(t, "bare-return", &rule.BareReturnRule{})
+	testRule(t, "bare_return", &rule.BareReturnRule{})
 }
diff --git a/test/bool-literal-in-expr_test.go → test/bool_literal_in_expr_test.go b/test/bool-literal-in-expr_test.go → test/bool_literal_in_expr_test.go
@@ -8,5 +8,5 @@ import (
 
 // BoolLiteral rule.
 func TestBoolLiteral(t *testing.T) {
-	testRule(t, "bool-literal-in-expr", &rule.BoolLiteralRule{})
+	testRule(t, "bool_literal_in_expr", &rule.BoolLiteralRule{})
 }
diff --git a/test/call-to-gc_test.go → test/call_to_gc_test.go b/test/call-to-gc_test.go → test/call_to_gc_test.go
@@ -8,5 +8,5 @@ import (
 
 // TestCallToGC test call-to-gc rule
 func TestCallToGC(t *testing.T) {
-	testRule(t, "call-to-gc", &rule.CallToGCRule{})
+	testRule(t, "call_to_gc", &rule.CallToGCRule{})
 }
diff --git a/test/cognitive-complexity_test.go → test/cognitive_complexity_test.go b/test/cognitive-complexity_test.go → test/cognitive_complexity_test.go
@@ -8,7 +8,7 @@ import (
 )
 
 func TestCognitiveComplexity(t *testing.T) {
-	testRule(t, "cognitive-complexity", &rule.CognitiveComplexityRule{}, &lint.RuleConfig{
+	testRule(t, "cognitive_complexity", &rule.CognitiveComplexityRule{}, &lint.RuleConfig{
 		Arguments: []any{int64(0)},
 	})
 }
diff --git a/test/comment-spacings_test.go → test/comment_spacings_test.go b/test/comment-spacings_test.go → test/comment_spacings_test.go
@@ -8,7 +8,7 @@ import (
 )
 
 func TestCommentSpacings(t *testing.T) {
-	testRule(t, "comment-spacings", &rule.CommentSpacingsRule{}, &lint.RuleConfig{
+	testRule(t, "comment_spacings", &rule.CommentSpacingsRule{}, &lint.RuleConfig{
 		Arguments: []any{"myOwnDirective:", "+optional"}},
 	)
 }
diff --git a/test/comments-density_test.go → test/comments_density_test.go b/test/comments-density_test.go → test/comments_density_test.go
@@ -8,13 +8,13 @@ import (
 )
 
 func TestCommentsDensity(t *testing.T) {
-	testRule(t, "comments-density-1", &rule.CommentsDensityRule{}, &lint.RuleConfig{
+	testRule(t, "comments_density_1", &rule.CommentsDensityRule{}, &lint.RuleConfig{
 		Arguments: []any{int64(60)},
 	})
 
-	testRule(t, "comments-density-2", &rule.CommentsDensityRule{}, &lint.RuleConfig{
+	testRule(t, "comments_density_2", &rule.CommentsDensityRule{}, &lint.RuleConfig{
 		Arguments: []any{int64(90)},
 	})
 
-	testRule(t, "comments-density-3", &rule.CommentsDensityRule{}, &lint.RuleConfig{})
+	testRule(t, "comments_density_3", &rule.CommentsDensityRule{}, &lint.RuleConfig{})
 }
diff --git a/test/confusing-naming_test.go → test/confusing_naming_test.go b/test/confusing-naming_test.go → test/confusing_naming_test.go
@@ -8,5 +8,5 @@ import (
 
 // TestConfusingNaming rule.
 func TestConfusingNaming(t *testing.T) {
-	testRule(t, "confusing-naming1", &rule.ConfusingNamingRule{})
+	testRule(t, "confusing_naming1", &rule.ConfusingNamingRule{})
 }
diff --git a/test/confusing-results_test.go → test/confusing_results_test.go b/test/confusing-results_test.go → test/confusing_results_test.go
@@ -7,5 +7,5 @@ import (
 )
 
 func TestConfusingResults(t *testing.T) {
-	testRule(t, "confusing-results", &rule.ConfusingResultsRule{})
+	testRule(t, "confusing_results", &rule.ConfusingResultsRule{})
 }
diff --git a/test/constant-logical-expr_test.go → test/constant_logical_expr_test.go b/test/constant-logical-expr_test.go → test/constant_logical_expr_test.go
@@ -8,5 +8,5 @@ import (
 
 // ConstantLogicalExpr rule.
 func TestConstantLogicalExpr(t *testing.T) {
-	testRule(t, "constant-logical-expr", &rule.ConstantLogicalExprRule{})
+	testRule(t, "constant_logical_expr", &rule.ConstantLogicalExprRule{})
 }
diff --git a/test/context-as-argument_test.go → test/context_as_argument_test.go b/test/context-as-argument_test.go → test/context_as_argument_test.go
@@ -8,7 +8,7 @@ import (
 )
 
 func TestContextAsArgument(t *testing.T) {
-	testRule(t, "context-as-argument", &rule.ContextAsArgumentRule{}, &lint.RuleConfig{
+	testRule(t, "context_as_argument", &rule.ContextAsArgumentRule{}, &lint.RuleConfig{
 		Arguments: []any{
 			map[string]any{
 				"allowTypesBefore": "AllowedBeforeType,AllowedBeforeStruct,*AllowedBeforePtrStruct,*testing.T",

diff --git a/test/cyclomatic_test.go b/test/cyclomatic_test.go
@@ -11,7 +11,7 @@ func TestCyclomatic(t *testing.T) {
 	testRule(t, "cyclomatic", &rule.CyclomaticRule{}, &lint.RuleConfig{
 		Arguments: []any{int64(1)},
 	})
-	testRule(t, "cyclomatic-2", &rule.CyclomaticRule{}, &lint.RuleConfig{
+	testRule(t, "cyclomatic_2", &rule.CyclomaticRule{}, &lint.RuleConfig{
 		Arguments: []any{int64(3)},
 	})
 }
diff --git a/test/deep-exit_test.go → test/deep_exit_test.go b/test/deep-exit_test.go → test/deep_exit_test.go
@@ -7,6 +7,6 @@ import (
 )
 
 func TestDeepExit(t *testing.T) {
-	testRule(t, "deep-exit", &rule.DeepExitRule{})
-	testRule(t, "deep-exit_test", &rule.DeepExitRule{})
+	testRule(t, "deep_exit", &rule.DeepExitRule{})
+	testRule(t, "deep_exit_test", &rule.DeepExitRule{})
 }
diff --git a/test/defer_test.go b/test/defer_test.go
@@ -13,13 +13,13 @@ func TestDefer(t *testing.T) {
 }
 
 func TestDeferLoopDisabled(t *testing.T) {
-	testRule(t, "defer-loop-disabled", &rule.DeferRule{}, &lint.RuleConfig{
+	testRule(t, "defer_loop_disabled", &rule.DeferRule{}, &lint.RuleConfig{
 		Arguments: []any{[]any{"return", "recover", "call-chain", "method-call"}},
 	})
 }
 
 func TestDeferOthersDisabled(t *testing.T) {
-	testRule(t, "defer-only-loop-enabled", &rule.DeferRule{}, &lint.RuleConfig{
+	testRule(t, "defer_only_loop_enabled", &rule.DeferRule{}, &lint.RuleConfig{
 		Arguments: []any{[]any{"loop"}},
 	})
 }
diff --git a/test/disable-annotations_test.go → test/disable_annotations_test.go b/test/disable-annotations_test.go → test/disable_annotations_test.go
@@ -8,13 +8,13 @@ import (
 )
 
 func TestDisabledAnnotations(t *testing.T) {
-	testRule(t, "disable-annotations", &rule.ExportedRule{}, &lint.RuleConfig{})
+	testRule(t, "disable_annotations", &rule.ExportedRule{}, &lint.RuleConfig{})
 }
 
 func TestModifiedAnnotations(t *testing.T) {
-	testRule(t, "disable-annotations2", &rule.VarNamingRule{}, &lint.RuleConfig{})
+	testRule(t, "disable_annotations2", &rule.VarNamingRule{}, &lint.RuleConfig{})
 }
 
 func TestDisableNextLineAnnotations(t *testing.T) {
-	testRule(t, "disable-annotations3", &rule.VarNamingRule{}, &lint.RuleConfig{})
+	testRule(t, "disable_annotations3", &rule.VarNamingRule{}, &lint.RuleConfig{})
 }
diff --git a/test/dot_imports_test.go b/test/dot_imports_test.go
@@ -12,7 +12,7 @@ func TestDotImports(t *testing.T) {
 		"allowedPackages": []any{"errors", "context", "github.com/BurntSushi/toml"},
 	}}
 
-	testRule(t, "import-dot", &rule.DotImportsRule{}, &lint.RuleConfig{
+	testRule(t, "import_dot", &rule.DotImportsRule{}, &lint.RuleConfig{
 		Arguments: args,
 	})
 }
diff --git a/test/duplicated-import_test.go → test/duplicated_imports_test.go b/test/duplicated-import_test.go → test/duplicated_imports_test.go
@@ -7,5 +7,5 @@ import (
 )
 
 func TestDuplicatedImports(t *testing.T) {
-	testRule(t, "duplicated-imports", &rule.DuplicatedImportsRule{})
+	testRule(t, "duplicated_imports", &rule.DuplicatedImportsRule{})
 }
diff --git a/test/early-return_test.go → test/early_return_test.go b/test/early-return_test.go → test/early_return_test.go
@@ -10,6 +10,6 @@ import (
 
 // TestEarlyReturn tests early-return rule.
 func TestEarlyReturn(t *testing.T) {
-	testRule(t, "early-return", &rule.EarlyReturnRule{})
-	testRule(t, "early-return-scope", &rule.EarlyReturnRule{}, &lint.RuleConfig{Arguments: []any{ifelse.PreserveScope}})
+	testRule(t, "early_return", &rule.EarlyReturnRule{})
+	testRule(t, "early_return_scope", &rule.EarlyReturnRule{}, &lint.RuleConfig{Arguments: []any{ifelse.PreserveScope}})
 }
diff --git a/test/empty-block_test.go → test/empty_block_test.go b/test/empty-block_test.go → test/empty_block_test.go
@@ -8,5 +8,5 @@ import (
 
 // TestEmptyBlock rule.
 func TestEmptyBlock(t *testing.T) {
-	testRule(t, "empty-block", &rule.EmptyBlockRule{})
+	testRule(t, "empty_block", &rule.EmptyBlockRule{})
 }
diff --git a/test/empty-lines_test.go → test/empty_lines_test.go b/test/empty-lines_test.go → test/empty_lines_test.go
@@ -8,5 +8,5 @@ import (
 
 // TestEmptyLines rule.
 func TestEmptyLines(t *testing.T) {
-	testRule(t, "empty-lines", &rule.EmptyLinesRule{})
+	testRule(t, "empty_lines", &rule.EmptyLinesRule{})
 }
diff --git a/test/enforce-map-style_test.go → test/enforce_map_style_test.go b/test/enforce-map-style_test.go → test/enforce_map_style_test.go
@@ -8,17 +8,17 @@ import (
 )
 
 func TestEnforceMapStyle_any(t *testing.T) {
-	testRule(t, "enforce-map-style-any", &rule.EnforceMapStyleRule{})
+	testRule(t, "enforce_map_style_any", &rule.EnforceMapStyleRule{})
 }
 
 func TestEnforceMapStyle_make(t *testing.T) {
-	testRule(t, "enforce-map-style-make", &rule.EnforceMapStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_map_style_make", &rule.EnforceMapStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{"make"},
 	})
 }
 
 func TestEnforceMapStyle_literal(t *testing.T) {
-	testRule(t, "enforce-map-style-literal", &rule.EnforceMapStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_map_style_literal", &rule.EnforceMapStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{"literal"},
 	})
 }
diff --git a/test/enforce-repeated-arg-type-style_test.go → test/enforce_repeated_arg_type_style_test.go b/test/enforce-repeated-arg-type-style_test.go → test/enforce_repeated_arg_type_style_test.go
@@ -8,21 +8,21 @@ import (
 )
 
 func TestEnforceRepeatedArgTypeStyleShort(t *testing.T) {
-	testRule(t, "enforce-repeated-arg-type-style-short-args", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_repeated_arg_type_style_short_args", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{"short"},
 	})
-	testRule(t, "enforce-repeated-arg-type-style-short-return", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_repeated_arg_type_style_short_return", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{"short"},
 	})
 
-	testRule(t, "enforce-repeated-arg-type-style-short-args", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_repeated_arg_type_style_short_args", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{
 			map[string]any{
 				"funcArgStyle": `short`,
 			},
 		},
 	})
-	testRule(t, "enforce-repeated-arg-type-style-short-return", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_repeated_arg_type_style_short_return", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{
 			map[string]any{
 				"funcRetValStyle": `short`,
@@ -32,21 +32,21 @@ func TestEnforceRepeatedArgTypeStyleShort(t *testing.T) {
 }
 
 func TestEnforceRepeatedArgTypeStyleFull(t *testing.T) {
-	testRule(t, "enforce-repeated-arg-type-style-full-args", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_repeated_arg_type_style_full_args", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{"full"},
 	})
-	testRule(t, "enforce-repeated-arg-type-style-full-return", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_repeated_arg_type_style_full_return", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{"full"},
 	})
 
-	testRule(t, "enforce-repeated-arg-type-style-full-args", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_repeated_arg_type_style_full_args", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{
 			map[string]any{
 				"funcArgStyle": `full`,
 			},
 		},
 	})
-	testRule(t, "enforce-repeated-arg-type-style-full-return", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_repeated_arg_type_style_full_return", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{
 			map[string]any{
 				"funcRetValStyle": `full`,
@@ -56,22 +56,22 @@ func TestEnforceRepeatedArgTypeStyleFull(t *testing.T) {
 }
 
 func TestEnforceRepeatedArgTypeStyleMixed(t *testing.T) {
-	testRule(t, "enforce-repeated-arg-type-style-full-args", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_repeated_arg_type_style_full_args", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{
 			map[string]any{
 				"funcArgStyle": `full`,
 			},
 		},
 	})
-	testRule(t, "enforce-repeated-arg-type-style-full-args", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_repeated_arg_type_style_full_args", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{
 			map[string]any{
 				"funcArgStyle":    `full`,
 				"funcRetValStyle": `any`,
 			},
 		},
 	})
-	testRule(t, "enforce-repeated-arg-type-style-full-args", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_repeated_arg_type_style_full_args", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{
 			map[string]any{
 				"funcArgStyle":    `full`,
@@ -80,22 +80,22 @@ func TestEnforceRepeatedArgTypeStyleMixed(t *testing.T) {
 		},
 	})
 
-	testRule(t, "enforce-repeated-arg-type-style-full-return", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_repeated_arg_type_style_full_return", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{
 			map[string]any{
 				"funcRetValStyle": `full`,
 			},
 		},
 	})
-	testRule(t, "enforce-repeated-arg-type-style-full-return", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_repeated_arg_type_style_full_return", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{
 			map[string]any{
 				"funcArgStyle":    `any`,
 				"funcRetValStyle": `full`,
 			},
 		},
 	})
-	testRule(t, "enforce-repeated-arg-type-style-full-return", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_repeated_arg_type_style_full_return", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{
 			map[string]any{
 				"funcArgStyle":    `short`,
@@ -104,15 +104,15 @@ func TestEnforceRepeatedArgTypeStyleMixed(t *testing.T) {
 		},
 	})
 
-	testRule(t, "enforce-repeated-arg-type-style-mixed-full-short", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_repeated_arg_type_style_mixed_full_short", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{
 			map[string]any{
 				"funcArgStyle":    `full`,
 				"funcRetValStyle": `short`,
 			},
 		},
 	})
-	testRule(t, "enforce-repeated-arg-type-style-mixed-short-full", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
+	testRule(t, "enforce_repeated_arg_type_style_mixed_short_full", &rule.EnforceRepeatedArgTypeStyleRule{}, &lint.RuleConfig{
 		Arguments: []any{
 			map[string]any{
 				"funcArgStyle":    `short`,