Merge pull request #27 from codacy/fix-tests

fix: Run all tests

Author: machadoit

.codacy/codacy.yaml

@@ -1,4 +1,4 @@
 runtimes:
     - [email protected]
 tools:
-    - eslint@8.57.0
+    - eslint@9.3.0

.github/workflows/go.yml

@@ -1,6 +1,6 @@
 on:
   push:
-    branches: [main]
+    branches: ['*']
 
 permissions:
   contents: write
@@ -13,11 +13,17 @@ jobs:
         uses: actions/checkout@v4
       - name: Set up Go
         uses: actions/setup-go@v4
+      - name: Install dependencies from .codacy/codacy.yaml
+        run: |
+          go build ./cli-v2.go
+          ./cli-v2 install
       - name: "Run tests"
         run: |
-          go test
+          go test ./...
 
   release:
+    needs: test
+    if: github.ref == format('refs/heads/{0}', github.event.repository.default_branch) && github.event_name == 'push'
     runs-on: ubuntu-latest
     steps:
       - name: Checkout

CLAUDE.md

@@ -0,0 +1,24 @@
+# CLAUDE.md - Development Guidelines for codacy-cli-v2
+
+## Build & Test Commands
+- Build: `go build ./cli-v2.go`
+- Run all tests: `go test ./...`
+- Run specific test: `go test -run TestName ./package/path`
+- Example: `go test -run TestGetTools ./tools`
+- Format code: `go fmt ./...`
+- Lint: `golint ./...` (if installed)
+
+## Code Style Guidelines
+- **Imports**: Standard lib first, external packages second, internal last
+- **Naming**: PascalCase for exported (public), camelCase for unexported (private)
+- **Error handling**: Return errors as last value, check with `if err != nil`
+- **Testing**: Use testify/assert package for assertions
+- **Package organization**: Keep related functionality in dedicated packages
+- **Documentation**: Document all exported functions, types, and packages
+- **Commit messages**: Start with verb, be concise and descriptive
+
+## Project Structure
+- `cmd/`: CLI command implementations
+- `config/`: Configuration handling
+- `tools/`: Tool-specific implementations
+- `utils/`: Utility functions

tools/eslintConfigCreator.go

@@ -33,10 +33,6 @@ func CreateEslintConfig(configuration ToolConfiguration) string {
 		rule = strings.ReplaceAll(rule, "_", "/")
 		rule = strings.ReplaceAll(rule, tempstring, "_")
 
-		/*if strings.Contains(rule, "/") {
-			continue
-		}*/
-
 		parametersString := ""
 
 		for _, parameter := range patternConfiguration.ParamenterConfigurations {

tools/eslintConfigCreator_test.go

@@ -116,7 +116,7 @@ func TestCreateEslintConfigUnnamedAndNamedParam(t *testing.T) {
 ];`)
 }
 
-func TestCreateEslintConfigSkipPlugins(t *testing.T) {
+func TestCreateEslintConfigSupportPlugins(t *testing.T) {
 	testConfig(t,
 		ToolConfiguration{
 			PatternsConfiguration: []PatternConfiguration{
@@ -128,6 +128,7 @@ func TestCreateEslintConfigSkipPlugins(t *testing.T) {
 		`export default [
     {
         rules: {
+          "plugin/consistent-return": "error",
         }
     }
 ];`)