fix: Replace 'vCPU' with 'vcpus'

GCE shows vCPU in the description of the machine, not the vcpus. Refs: HMS-2301
RHEnVision · Oct 31, 2023 · 1c11b5b · 1c11b5b
1 parent d1cbba3
commit 1c11b5b
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/src/Utils/querySearch.js b/src/Utils/querySearch.js
@@ -2,9 +2,10 @@ const jsonata = require('jsonata');
 
 const OPERATORS = ['<', '>', '='];
 
-const parseQuery = (query) => {
+export const parseQuery = (query) => {
   query = query.replace('memory', 'memory_mib');
   query = query.replace('storage', 'storage_gb');
+  query = query.replace(/vcpu(\W|$)/i, 'vcpus$1');
   if (OPERATORS.every((operator) => !query.includes(operator))) {
     query = `name ~> /${query}/i`;
   }

diff --git a/src/Utils/querySearch.test.js b/src/Utils/querySearch.test.js
@@ -0,0 +1,9 @@
+import { parseQuery } from './querySearch';
+
+describe('parseQuery', () => {
+  test('replaces vcpu by vcpus', async () => {
+    expect(parseQuery('vcpu > 2')).toBe('types[vcpus > 2]');
+    expect(parseQuery('vcpu> 2')).toBe('types[vcpus> 2]');
+    expect(parseQuery('vCPU > 2')).toBe('types[vcpus > 2]');
+  });
+});