refactor + add debounce (close #41)

.eslintrc.json

@@ -1,7 +1,7 @@
 {
   "env": {
     "browser": false,
-    "es2021": true
+    "es2022": true
   },
   "extends": ["standard-with-typescript", "prettier"],
   "overrides": [],
@@ -11,7 +11,8 @@
     "project": "tsconfig.json"
   },
   "rules": {
-    "@typescript-eslint/strict-boolean-expressions": "off"
+    "@typescript-eslint/strict-boolean-expressions": "off",
+    "@typescript-eslint/no-confusing-void-expression": "off"
   },
   "ignorePatterns": ["*.js", "**/node_modules/"]
 }

README.md

@@ -10,10 +10,14 @@ This extension adds a tree view to the test view container. It shows the coverag
 
 This extension contributes the following settings:
 
-* `koverage.coverageFileNames`: coverage file names to look for, default: ["lcov.info", "cov.xml", "coverage.xml","jacoco.xml"]
-* `koverage.coverageFilePaths`: coverage paths where coverage files are located, default: ["coverage"]
-* `koverage.lowCoverageThreshold`: Percentage under which, the coverage is considered too low (Renders as Error icon)
-* `koverage.sufficientCoverageThreshold`: Percentage above which, the coverage is considered sufficient (Renders as Success icon)
+- `koverage.coverageCommand`: Command to run to generate coverage. (Default: "")
+- `koverage.autoRefresh`: Whether to watch the coverage files and configurations. (Default: true)
+- `koverage.autoRefreshDebounce`: Auto refresh debounce interval in milliseconds. (Default: 3000)
+- `koverage.coverageFileNames`: Coverage file names to look for. (Default: ["lcov.info", "cov.xml", "coverage.xml", "jacoco.xml"])
+- `koverage.coverageFilePaths`: Coverage file paths to search in. (Default: ["**"])
+- `koverage.ignoredPathGlobs`: Ignored path globs. (Default: "**/{node_modules,venv,.venv,vendor}/**")
+- `koverage.lowCoverageThreshold`: Coverage threshold considered too low. (Default: 50)
+- `koverage.sufficientCoverageThreshold`: Coverage threshold considered sufficient. (Default: 70)
 => lowCoverageThreshold < level < sufficientCoverageThreshold is rendered as Warn icon
 
 ## Licencing

package.json

@@ -5,7 +5,8 @@
   "keywords": [
     "coverage",
     "lcov",
-    "jacoco"
+    "jacoco",
+    "clover"
   ],
   "repository": "https://github.com/tenninebt/vscode-koverage",
   "description": "View the code coverage per folder/file in the test view",
@@ -31,23 +32,36 @@
           "description": "command to run to generate coverage",
           "scope": "resource"
         },
+        "koverage.autoRefresh": {
+          "type": "boolean",
+          "default": true,
+          "description": "Whether to watch the coverage files and configurations",
+          "scope": "resource"
+        },
+        "koverage.autoRefreshDebounce": {
+          "type": "number",
+          "default": 3000,
+          "description": "Auto refresh debounce interval in milliseconds",
+          "scope": "resource"
+        },
         "koverage.coverageFileNames": {
           "type": "array",
           "default": [
             "lcov.info",
             "cov.xml",
+            "clover.xml",
             "coverage.xml",
             "jacoco.xml"
           ],
-          "description": "coverage file names for the extension to automatically look for",
+          "description": "coverage file names to look for",
           "scope": "resource"
         },
         "koverage.coverageFilePaths": {
           "type": "array",
           "default": [
             "**"
           ],
-          "description": "coverage file paths for the extensions to automatically search in",
+          "description": "coverage file paths to search in",
           "scope": "resource"
         },
         "koverage.ignoredPathGlobs": {
@@ -121,10 +135,11 @@
     }
   },
   "scripts": {
+    "clean": "rm -rf dist",
+    "compile": "yarn clean && tsc",
     "build:prod": "webpack --mode production",
     "build:dev": "webpack --mode development",
     "build:watch": "webpack --mode development --watch",
-    "compile": "tsc",
     "lint": "eslint ./src"
   },
   "devDependencies": {
@@ -140,6 +155,7 @@
     "eslint-plugin-n": "^15.7.0",
     "eslint-plugin-promise": "^6.1.1",
     "ovsx": "^0.8.0",
+    "ts-loader": "^9.5.0",
     "typescript": "^5.0.3",
     "vscode-test": "^1.3.0",
     "webpack": "^5.74.0",

src/ConfigStore.ts

@@ -6,6 +6,8 @@ export class ConfigStore {
   private readonly configurationKey: string = "koverage"
 
   private readonly _configChangedNotifier: rx.Subject<void>
+  public readonly configChangedNotifier: rx.Observable<void>
+
   private readonly _perFolderConfig: Map<vscode.Uri, rx.BehaviorSubject<Config>>
   public get(workspaceFolder: vscode.WorkspaceFolder): Config {
     // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
@@ -14,6 +16,7 @@ export class ConfigStore {
 
   constructor(private readonly logger: vscodeLogging.IVSCodeExtLogger) {
     this._configChangedNotifier = new rx.Subject<void>()
+    this.configChangedNotifier = this._configChangedNotifier.asObservable()
     this._perFolderConfig = new Map<vscode.Uri, rx.BehaviorSubject<Config>>()
 
     void this.readConfig()
@@ -93,10 +96,6 @@ export class ConfigStore {
       sufficientCoverageThreshold
     })
   }
-
-  public subscribe(next?: () => void, error?: (error: any) => void, complete?: () => void): rx.Subscription {
-    return this._configChangedNotifier.subscribe(next, error, complete)
-  }
 }
 
 export class Config {

src/CoverageLevel.ts

@@ -0,0 +1,10 @@
+
+export enum CoverageLevel {
+    Low = "low",
+    Medium = "medium",
+    High = "high"
+}
+
+export class CoverageLevelThresholds {
+    constructor(public readonly sufficientCoverageThreshold: number, public readonly lowCoverageThreshold: number) { }
+}