From c160d12fd39199d39d446e78fe6c4b563d140cbb Mon Sep 17 00:00:00 2001 From: Oleksandr Redko Date: Thu, 5 Dec 2024 14:01:47 +0200 Subject: [PATCH] test: remove useless comments (#1173) --- test/atomic_test.go | 1 - test/bool_literal_in_expr_test.go | 1 - test/call_to_gc_test.go | 1 - test/confusing_naming_test.go | 1 - test/constant_logical_expr_test.go | 1 - test/defer_test.go | 1 - test/early_return_test.go | 1 - test/empty_block_test.go | 1 - test/empty_lines_test.go | 1 - test/identical_branches_test.go | 1 - test/if_return_test.go | 1 - test/modifies_param_test.go | 1 - test/redefines_builtin_id_test.go | 1 - test/redundant_import_alias_test.go | 1 - test/string_of_int_test.go | 1 - test/struct_tag_test.go | 1 - test/superfluous_else_test.go | 1 - test/time_equal_test.go | 1 - test/time_naming_test.go | 1 - test/unnecessary_stmt_test.go | 1 - test/useless_break_test.go | 1 - 21 files changed, 21 deletions(-) diff --git a/test/atomic_test.go b/test/atomic_test.go index 1bf29bb..2da7bef 100644 --- a/test/atomic_test.go +++ b/test/atomic_test.go @@ -6,7 +6,6 @@ import ( "github.com/mgechev/revive/rule" ) -// Atomic rule. func TestAtomic(t *testing.T) { testRule(t, "atomic", &rule.AtomicRule{}) } diff --git a/test/bool_literal_in_expr_test.go b/test/bool_literal_in_expr_test.go index d67c6d2..501db69 100644 --- a/test/bool_literal_in_expr_test.go +++ b/test/bool_literal_in_expr_test.go @@ -6,7 +6,6 @@ import ( "github.com/mgechev/revive/rule" ) -// BoolLiteral rule. func TestBoolLiteral(t *testing.T) { testRule(t, "bool_literal_in_expr", &rule.BoolLiteralRule{}) } diff --git a/test/call_to_gc_test.go b/test/call_to_gc_test.go index 9c11ad4..5944a9d 100644 --- a/test/call_to_gc_test.go +++ b/test/call_to_gc_test.go @@ -6,7 +6,6 @@ import ( "github.com/mgechev/revive/rule" ) -// TestCallToGC test call-to-gc rule func TestCallToGC(t *testing.T) { testRule(t, "call_to_gc", &rule.CallToGCRule{}) } diff --git a/test/confusing_naming_test.go b/test/confusing_naming_test.go index 22f44fe..ce60d92 100644 --- a/test/confusing_naming_test.go +++ b/test/confusing_naming_test.go @@ -6,7 +6,6 @@ import ( "github.com/mgechev/revive/rule" ) -// TestConfusingNaming rule. func TestConfusingNaming(t *testing.T) { testRule(t, "confusing_naming1", &rule.ConfusingNamingRule{}) } diff --git a/test/constant_logical_expr_test.go b/test/constant_logical_expr_test.go index d3d6d90..ce53098 100644 --- a/test/constant_logical_expr_test.go +++ b/test/constant_logical_expr_test.go @@ -6,7 +6,6 @@ import ( "github.com/mgechev/revive/rule" ) -// ConstantLogicalExpr rule. func TestConstantLogicalExpr(t *testing.T) { testRule(t, "constant_logical_expr", &rule.ConstantLogicalExprRule{}) } diff --git a/test/defer_test.go b/test/defer_test.go index ce2904a..78ce52a 100644 --- a/test/defer_test.go +++ b/test/defer_test.go @@ -7,7 +7,6 @@ import ( "github.com/mgechev/revive/rule" ) -// Defer rule. func TestDefer(t *testing.T) { testRule(t, "defer", &rule.DeferRule{}) } diff --git a/test/early_return_test.go b/test/early_return_test.go index 493ee6f..cc4ef66 100644 --- a/test/early_return_test.go +++ b/test/early_return_test.go @@ -8,7 +8,6 @@ import ( "github.com/mgechev/revive/rule" ) -// 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}}) diff --git a/test/empty_block_test.go b/test/empty_block_test.go index bc496d4..c5b8c5b 100644 --- a/test/empty_block_test.go +++ b/test/empty_block_test.go @@ -6,7 +6,6 @@ import ( "github.com/mgechev/revive/rule" ) -// TestEmptyBlock rule. func TestEmptyBlock(t *testing.T) { testRule(t, "empty_block", &rule.EmptyBlockRule{}) } diff --git a/test/empty_lines_test.go b/test/empty_lines_test.go index e74d9f1..faad9b7 100644 --- a/test/empty_lines_test.go +++ b/test/empty_lines_test.go @@ -6,7 +6,6 @@ import ( "github.com/mgechev/revive/rule" ) -// TestEmptyLines rule. func TestEmptyLines(t *testing.T) { testRule(t, "empty_lines", &rule.EmptyLinesRule{}) } diff --git a/test/identical_branches_test.go b/test/identical_branches_test.go index a0fc161..341f981 100644 --- a/test/identical_branches_test.go +++ b/test/identical_branches_test.go @@ -6,7 +6,6 @@ import ( "github.com/mgechev/revive/rule" ) -// IdenticalBranches rule. func TestIdenticalBranches(t *testing.T) { testRule(t, "identical_branches", &rule.IdenticalBranchesRule{}) } diff --git a/test/if_return_test.go b/test/if_return_test.go index 2e7348b..ce23fa0 100644 --- a/test/if_return_test.go +++ b/test/if_return_test.go @@ -6,7 +6,6 @@ import ( "github.com/mgechev/revive/rule" ) -// TestIfReturn rule. func TestIfReturn(t *testing.T) { testRule(t, "if_return", &rule.IfReturnRule{}) } diff --git a/test/modifies_param_test.go b/test/modifies_param_test.go index d6ab7c9..2c1d99f 100644 --- a/test/modifies_param_test.go +++ b/test/modifies_param_test.go @@ -6,7 +6,6 @@ import ( "github.com/mgechev/revive/rule" ) -// TestModifiesParam rule. func TestModifiesParam(t *testing.T) { testRule(t, "modifies_param", &rule.ModifiesParamRule{}) } diff --git a/test/redefines_builtin_id_test.go b/test/redefines_builtin_id_test.go index 68bb94b..f8cdf69 100644 --- a/test/redefines_builtin_id_test.go +++ b/test/redefines_builtin_id_test.go @@ -6,7 +6,6 @@ import ( "github.com/mgechev/revive/rule" ) -// Tests RedefinesBuiltinID rule. func TestRedefinesBuiltinID(t *testing.T) { testRule(t, "redefines_builtin_id", &rule.RedefinesBuiltinIDRule{}) } diff --git a/test/redundant_import_alias_test.go b/test/redundant_import_alias_test.go index eeb3e2e..a91e5ca 100644 --- a/test/redundant_import_alias_test.go +++ b/test/redundant_import_alias_test.go @@ -5,7 +5,6 @@ import ( "testing" ) -// TestRedundantImportAlias rule. func TestRedundantImportAlias(t *testing.T) { testRule(t, "redundant_import_alias", &rule.RedundantImportAlias{}) } diff --git a/test/string_of_int_test.go b/test/string_of_int_test.go index c6e2f0c..f01148c 100644 --- a/test/string_of_int_test.go +++ b/test/string_of_int_test.go @@ -6,7 +6,6 @@ import ( "github.com/mgechev/revive/rule" ) -// String-of-int rule. func TestStringOfInt(t *testing.T) { testRule(t, "string_of_int", &rule.StringOfIntRule{}) } diff --git a/test/struct_tag_test.go b/test/struct_tag_test.go index 44381e3..c930b10 100644 --- a/test/struct_tag_test.go +++ b/test/struct_tag_test.go @@ -7,7 +7,6 @@ import ( "github.com/mgechev/revive/rule" ) -// TestStructTag tests struct-tag rule func TestStructTag(t *testing.T) { testRule(t, "struct_tag", &rule.StructTagRule{}) } diff --git a/test/superfluous_else_test.go b/test/superfluous_else_test.go index 88815c8..0ec40ac 100644 --- a/test/superfluous_else_test.go +++ b/test/superfluous_else_test.go @@ -8,7 +8,6 @@ import ( "github.com/mgechev/revive/rule" ) -// TestSuperfluousElse rule. func TestSuperfluousElse(t *testing.T) { testRule(t, "superfluous_else", &rule.SuperfluousElseRule{}) testRule(t, "superfluous_else_scope", &rule.SuperfluousElseRule{}, &lint.RuleConfig{Arguments: []any{ifelse.PreserveScope}}) diff --git a/test/time_equal_test.go b/test/time_equal_test.go index 8b2a98a..d574991 100644 --- a/test/time_equal_test.go +++ b/test/time_equal_test.go @@ -6,7 +6,6 @@ import ( "github.com/mgechev/revive/rule" ) -// TestTimeEqual rule. func TestTimeEqual(t *testing.T) { testRule(t, "time_equal", &rule.TimeEqualRule{}) } diff --git a/test/time_naming_test.go b/test/time_naming_test.go index be5fbf3..d2ea0d8 100644 --- a/test/time_naming_test.go +++ b/test/time_naming_test.go @@ -6,7 +6,6 @@ import ( "github.com/mgechev/revive/rule" ) -// TestTimeNamingRule rule. func TestTimeNaming(t *testing.T) { testRule(t, "time_naming", &rule.TimeNamingRule{}) } diff --git a/test/unnecessary_stmt_test.go b/test/unnecessary_stmt_test.go index 79c27ba..a77b5e8 100644 --- a/test/unnecessary_stmt_test.go +++ b/test/unnecessary_stmt_test.go @@ -6,7 +6,6 @@ import ( "github.com/mgechev/revive/rule" ) -// TestUnnecessaryStmt rule. func TestUnnecessaryStmt(t *testing.T) { testRule(t, "unnecessary_stmt", &rule.UnnecessaryStmtRule{}) } diff --git a/test/useless_break_test.go b/test/useless_break_test.go index 439bca5..4757dcc 100644 --- a/test/useless_break_test.go +++ b/test/useless_break_test.go @@ -6,7 +6,6 @@ import ( "github.com/mgechev/revive/rule" ) -// UselessBreak rule. func TestUselessBreak(t *testing.T) { testRule(t, "useless_break", &rule.UselessBreak{}) }