Merge pull request #146 from alanconway/lint-fixes

NO-JIRA: Run linter

Author: openshift-merge-bot[bot]

web/src/__tests__/all-domains.spec.ts

@@ -72,5 +72,7 @@ it.each([
   },
 ])('convert URL<=>link', ({ url, query, constraint }) => {
   expect(allDomains.linkToQuery(new URIRef(url))).toEqual(Query.parse(query));
-  expect(allDomains.queryToLink(Query.parse(query), Constraint.fromAPI(constraint))).toEqual(new URIRef(url));
+  expect(allDomains.queryToLink(Query.parse(query), Constraint.fromAPI(constraint))).toEqual(
+    new URIRef(url),
+  );
 });

web/src/__tests__/k8s.spec.ts

@@ -111,7 +111,9 @@ describe('K8sNode.fromQuery', () => {
     },
     {
       query: `k8s:Pod:{"namespace":"x","name":"y","labels":{"a":"b","c":"d"},"fields": {"x":"y"}}`,
-      url: `k8s/ns/x/pods/y?labels=${encodeURIComponent('a=b,c=d')}&fields=${encodeURIComponent('x=y')}`,
+      url: `k8s/ns/x/pods/y?labels=${encodeURIComponent('a=b,c=d')}&fields=${encodeURIComponent(
+        'x=y',
+      )}`,
     },
     {
       query:

web/src/__tests__/log.spec.ts

@@ -13,7 +13,7 @@ describe('LogDomain.fromURL', () => {
     {
       url: `monitoring/logs?q=${encodeURIComponent(
         '{kubernetes_namespace_name="default",' +
-        'kubernetes_pod_name="foo",log_type="infrastructure"}',
+          'kubernetes_pod_name="foo",log_type="infrastructure"}',
       )}`,
       query:
         `log:infrastructure:{kubernetes_namespace_name="default",` +

web/src/__tests__/metric.spec.ts

@@ -7,25 +7,26 @@ import { Query, URIRef } from '../korrel8r/types';
  *
  */
 describe('metric', () => {
-  const metric = new MetricDomain()
+  const metric = new MetricDomain();
 
   describe('query-link round trip', () => {
     it.each([
       {
-        query: 'metric:metric:max_over_time(kube_pod_container_status_waiting_reason{job="kube-state-metrics",namespace=~"(openshift-.*|kube-.*|default)",reason="CrashLoopBackOff"}[5m]) <= 1',
-        link: 'monitoring/query-browser?query0=max_over_time%28kube_pod_container_status_waiting_reason%7Bjob%3D%22kube-state-metrics%22%2Cnamespace%3D%7E%22%28openshift-.*%7Ckube-.*%7Cdefault%29%22%2Creason%3D%22CrashLoopBackOff%22%7D%5B5m%5D%29+%3C%3D+1'
+        query:
+          'metric:metric:max_over_time(kube_pod_container_status_waiting_reason{job="kube-state-metrics",namespace=~"(openshift-.*|kube-.*|default)",reason="CrashLoopBackOff"}[5m]) <= 1',
+        link: 'monitoring/query-browser?query0=max_over_time%28kube_pod_container_status_waiting_reason%7Bjob%3D%22kube-state-metrics%22%2Cnamespace%3D%7E%22%28openshift-.*%7Ckube-.*%7Cdefault%29%22%2Creason%3D%22CrashLoopBackOff%22%7D%5B5m%5D%29+%3C%3D+1',
       },
       {
         // Verify issue: https://github.com/openshift/troubleshooting-panel-console-plugin/issues/127
-        query: 'metric:metric:(1 - sum(node_memory_MemFree_bytes + node_memory_Buffers_bytes + node_memory_Cached_bytes and on (instance) label_replace(kube_node_role{role="master"}, "instance", "$1", "node", "(.+)")) / sum(node_memory_MemTotal_bytes and on (instance) label_replace(kube_node_role{role="master"}, "instance", "$1", "node", "(.+)"))) * 100 > 60',
-        link: 'monitoring/query-browser?query0=%281+-+sum%28node_memory_MemFree_bytes+%2B+node_memory_Buffers_bytes+%2B+node_memory_Cached_bytes+and+on+%28instance%29+label_replace%28kube_node_role%7Brole%3D%22master%22%7D%2C+%22instance%22%2C+%22%241%22%2C+%22node%22%2C+%22%28.%2B%29%22%29%29+%2F+sum%28node_memory_MemTotal_bytes+and+on+%28instance%29+label_replace%28kube_node_role%7Brole%3D%22master%22%7D%2C+%22instance%22%2C+%22%241%22%2C+%22node%22%2C+%22%28.%2B%29%22%29%29%29+*+100+%3E+60'
+        query:
+          'metric:metric:(1 - sum(node_memory_MemFree_bytes + node_memory_Buffers_bytes + node_memory_Cached_bytes and on (instance) label_replace(kube_node_role{role="master"}, "instance", "$1", "node", "(.+)")) / sum(node_memory_MemTotal_bytes and on (instance) label_replace(kube_node_role{role="master"}, "instance", "$1", "node", "(.+)"))) * 100 > 60',
+        link: 'monitoring/query-browser?query0=%281+-+sum%28node_memory_MemFree_bytes+%2B+node_memory_Buffers_bytes+%2B+node_memory_Cached_bytes+and+on+%28instance%29+label_replace%28kube_node_role%7Brole%3D%22master%22%7D%2C+%22instance%22%2C+%22%241%22%2C+%22node%22%2C+%22%28.%2B%29%22%29%29+%2F+sum%28node_memory_MemTotal_bytes+and+on+%28instance%29+label_replace%28kube_node_role%7Brole%3D%22master%22%7D%2C+%22instance%22%2C+%22%241%22%2C+%22node%22%2C+%22%28.%2B%29%22%29%29%29+*+100+%3E+60',
       },
     ])('', ({ query, link }) => {
-      const q = Query.parse(query)
-      expect(metric.queryToLink(q).toString()).toEqual(link)
+      const q = Query.parse(query);
+      expect(metric.queryToLink(q).toString()).toEqual(link);
       expect(metric.linkToQuery(new URIRef(link))).toEqual(q);
-    })
-
+    });
   });
 
   describe('linkToQuery errors', () => {
@@ -40,7 +41,6 @@ describe('metric', () => {
       },
     ])('$url', ({ url, error }) => {
       expect(() => metric.linkToQuery(new URIRef(url))).toThrow(error);
-
     });
   });
 

web/src/__tests__/types.spec.ts

@@ -38,7 +38,9 @@ class FakeDomain extends Domain {
 
   queryToLink(query: Query, constraint?: Constraint): URIRef {
     if (!query || !query.class || query.class.domain != this.name) throw this.badQuery(query);
-    return new URIRef(`${query.class.domain}/${query.class.name}?${query.selector}`, { constraint: constraint });
+    return new URIRef(`${query.class.domain}/${query.class.name}?${query.selector}`, {
+      constraint: constraint,
+    });
   }
 }
 

web/src/components/Korrel8rPanel.tsx

@@ -54,10 +54,10 @@ export default function Korrel8rPanel() {
     persistedSearch?.queryStr
       ? persistedSearch
       : ({
-        ...defaultSearch,
-        queryStr: locationQuery?.toString(),
-        constraint: persistedSearch?.constraint,
-      } as Search),
+          ...defaultSearch,
+          queryStr: locationQuery?.toString(),
+          constraint: persistedSearch?.constraint,
+        } as Search),
   );
   const [result, setResult] = React.useState<Result | null>(null);
   const [showQuery, setShowQuery] = React.useState(false);

web/src/korrel8r/alert.ts

@@ -30,6 +30,6 @@ export class AlertDomain extends Domain {
 
   queryToLink(query: Query): URIRef {
     const selectors = keyValueList(JSON.parse(query.selector));
-    return new URIRef(`monitoring/alerts`, { alerts: selectors || undefined })
+    return new URIRef(`monitoring/alerts`, { alerts: selectors || undefined });
   }
 }

web/src/korrel8r/log.ts

@@ -47,6 +47,6 @@ export class LogDomain extends Domain {
       tenant: logClass,
       start: unixMilliseconds(constraint?.start),
       end: unixMilliseconds(constraint?.end),
-    })
+    });
   }
 }

web/src/korrel8r/metric.ts

@@ -21,6 +21,6 @@ export class MetricDomain extends Domain {
     if (!query.selector || query.selector.match(/{ *}/)) {
       throw this.badQuery(query, 'empty selector');
     }
-    return new URIRef('monitoring/query-browser', { query0: query.selector })    // FIXME use explicit URIRef everywhere.
+    return new URIRef('monitoring/query-browser', { query0: query.selector }); // FIXME use explicit URIRef everywhere.
   }
 }

web/src/korrel8r/netflow.ts

@@ -72,6 +72,6 @@ export class NetflowDomain extends Domain {
       filters: filters ? filters : undefined,
       startTime: unixSeconds(constraint?.start),
       endTime: unixSeconds(constraint?.end),
-    })
+    });
   }
 }