Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test Reporter doesn't take into account my updated .yml file #504

Open
williamatpaper opened this issue Dec 20, 2022 · 1 comment
Open

Comments

@williamatpaper
Copy link

The first 6 lines of the exclude_pattern section are not getting picked up when I run before-build and after-build. No complex multi test running here.

version: "2"
# docs https://docs.codeclimate.com/docs/advanced-configuration
checks:
  argument-count:
    enabled: true
    config:
      threshold: 4
  complex-logic:
    enabled: true
    config:
      threshold: 4
  file-lines:
    enabled: true
    config:
      threshold: 250
  method-complexity:
    enabled: true
    config:
      threshold: 5
  method-count:
    enabled: true
    config:
      threshold: 20
  method-lines:
    enabled: true
    config:
      threshold: 25
  nested-control-flow:
    enabled: true
    config:
      threshold: 4
  return-statements:
    enabled: true
    config:
      threshold: 4
  similar-code:
    enabled: true
    config:
      threshold: #language-specific defaults. overrides affect all languages.
  identical-code:
    enabled: true
    config:
      threshold: #language-specific defaults. overrides affect all languages.
exclude_patterns:
  - "app/Providers/BroadcastServiceProvider.php" #not technically application code
  - "app/Exceptions/Handler.php"
  - "app/Http/Controllers/Controller.php"
  - "app/Http/Middleware/TrimStrings.php"
  - "app/Infrastructure/JWT/JWTDecoderInterface.php"
  - "app/Services/LaunchDarkly.php" #this test has to be mocked, and so doesn't get counted for code coverage https://launchdarkly.com/blog/automated-testing-in-launchdarkly/
  - "config/"
  - "docker/"
  - "infra/"
  - "node_modules/"
  - "public/"
  - "dist/"
  - "tests/"
  - "src/assets/"
  - "**/*.spec.js"
  - "**/*.md"
  - "**/*.yml"
  - "**/*.yaml"
@camillof
Copy link
Contributor

Hey there, from what I understand, .codeclimate.yml configuration doesn't affect the test-reporter functionality, It depends solely on your test coverage tool report.

Have you experienced changes on the test-reporter based on that configuration?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants