Merge pull request #400 from dreamit-de/399-incmutationtest

399 Add incremental mutation tests and runs

.github/workflows/ci.yaml

@@ -4,3 +4,5 @@ on: [push, workflow_dispatch]
 jobs:
   graphqlserver:
     uses: dreamit-de/workflows/.github/workflows/nodejs-ci-libs.yml@main
+    with:
+      runMutationTests: true

.prettierignore

@@ -0,0 +1,2 @@
+# Stryker incremental mutation file
+stryker-incremental.json

package.json

@@ -1,6 +1,6 @@
 {
     "name": "@dreamit/graphql-server",
-    "version": "4.11.0",
+    "version": "4.11.1",
     "description": "A GraphQL server written in NodeJS/Typescript.",
     "scripts": {
         "build": "tsup-node",
@@ -10,6 +10,7 @@
         "lint": "eslint src/*.ts src/**/*.ts tests/*.ts tests/**/*.ts",
         "lintfix": "eslint --fix src/*.ts src/**/*.ts tests/*.ts tests/**/*.ts",
         "mutationtest": "stryker run",
+        "mutationtestCI": "stryker run --incremental --incrementalFile stryker-incremental.json",
         "prepack": "npm run build",
         "test": "vitest run --coverage"
     },

src/error/DetermineGraphQLOrFetchError.ts

@@ -17,8 +17,7 @@ export function determineGraphQLOrFetchError(error: unknown): string {
             error.message.includes('network timeout') ||
             error.message.includes('invalid redirect URL') ||
             error.message.includes(
-                'uri requested responds with a redirect' +
-                    ', redirect mode is set to error',
+                'uri requested responds with a redirect, redirect mode is set to error',
             ) ||
             error.message.includes('maximum redirect reached') ||
             error.message.includes('Cannot follow redirect') ||

src/metrics/NoMetricsClient.ts

@@ -5,9 +5,7 @@ import { MetricsClient } from '@dreamit/graphql-server-base'
  * Metrics client that does not collect any metrics.
  */
 export class NoMetricsClient implements MetricsClient {
-    constructor() {
-        this.initMetrics()
-    }
+    constructor() {}
     initMetrics(): void {}
     setAvailability(): void {}
     increaseRequestThroughput(): void {}

src/metrics/SimpleMetricsClient.ts

@@ -20,9 +20,9 @@ export class SimpleMetricsClient implements MetricsClient {
     readonly requestThroughputMetricName: string
     readonly availabilityMetricName: string
     readonly errorsMetricName: string
-    graphQLServerAvailabilityGauge!: number
-    requestThroughput!: number
-    graphQLServerErrorCounter!: Map<string, number>
+    graphQLServerAvailabilityGauge = 0
+    requestThroughput = 0
+    graphQLServerErrorCounter: Map<string, number> = new Map<string, number>()
 
     constructor(
         requestThroughputMetricName = 'graphql_server_request_throughput',
@@ -36,24 +36,9 @@ export class SimpleMetricsClient implements MetricsClient {
     }
 
     initMetrics(): void {
-        this.createRequestThroughputCounter()
-        this.createServerAvailabilityGauge()
-        this.createServerErrorCounter()
         this.initErrorCounterLabels()
     }
 
-    createServerErrorCounter(): void {
-        this.graphQLServerErrorCounter = new Map<string, number>()
-    }
-
-    createServerAvailabilityGauge(): void {
-        this.graphQLServerAvailabilityGauge = 0
-    }
-
-    createRequestThroughputCounter(): void {
-        this.requestThroughput = 0
-    }
-
     /**
      * Initializes the error counter.
      * When evaluating time series this can help