minor coverage cleanup

[squeep-api-dingus] / lib / content-negotiation.js

diff --git a/lib/content-negotiation.js b/lib/content-negotiation.js
index 1425852ffa30925646bfcbc4809bb79d23c308ae..e78fe359303ae8bf2eb06902763ea2297f1baae4 100644 (file)
--- a/lib/content-negotiation.js
+++ b/lib/content-negotiation.js
@@ -20,7 +20,7 @@ class ContentNegotiation {
     if (type) {
       let weight = 1.0;
       params = params.reduce((acc, param) => {
-        const [p, v] = common.splitFirst(param, '=').map((x) => x && x.trim());
+        const [p, v] = common.splitFirst(param, '=').map((x) => x?.trim());
         if (p && v) {
           if (p === 'q') {
             weight = Number(v);
@@ -47,7 +47,9 @@ class ContentNegotiation {
    * @param {string} acceptHeader 
    */
   static _acceptClauses(acceptHeader) {
-    const clauses = (acceptHeader||'').split(',').map((clause) => ContentNegotiation._unpackAcceptClause(clause)).filter((clause) => clause);
+    const clauses = (acceptHeader||'').split(',')
+      .map((clause) => ContentNegotiation._unpackAcceptClause(clause))
+      .filter((clause) => clause);
     return clauses.sort((a, b) => b.weight - a.weight);
   }
 
@@ -100,6 +102,7 @@ class ContentNegotiation {
       if (acc === undefined && validTypesQuality[cur] !== 0.0) {
         return cur;
       }
+      // istanbul ignore next
       // eslint-disable-next-line security/detect-object-injection
       return validTypesQuality[acc] < validTypesQuality[cur] ? cur : acc;
     }, undefined);