removed has:annotations since that makes little sense - they are always there

Signed-off-by: Fredrik Adelöw <freben@gmail.com>

plugins/catalog/src/alpha/filter/matrchers/createHasMatcher.test.ts

@@ -24,7 +24,6 @@ describe('createHasMatcher', () => {
   const empty2 = {
     metadata: {
       labels: {},
-      annotations: {},
       links: [],
     },
   } as unknown as Entity;
@@ -33,11 +32,6 @@ describe('createHasMatcher', () => {
       labels: { a: 'b' },
     },
   } as unknown as Entity;
-  const annotations = {
-    metadata: {
-      annotations: { a: 'b' },
-    },
-  } as unknown as Entity;
   const links = {
     metadata: { links: [{}] },
   } as unknown as Entity;
@@ -53,10 +47,6 @@ describe('createHasMatcher', () => {
     expect(createHasMatcher(['labels'], err)(empty2)).toBe(false);
     expect(createHasMatcher(['labels'], err)(labels)).toBe(true);
 
-    expect(createHasMatcher(['annotations'], err)(empty1)).toBe(false);
-    expect(createHasMatcher(['annotations'], err)(empty2)).toBe(false);
-    expect(createHasMatcher(['annotations'], err)(annotations)).toBe(true);
-
     expect(createHasMatcher(['links'], err)(empty1)).toBe(false);
     expect(createHasMatcher(['links'], err)(empty2)).toBe(false);
     expect(createHasMatcher(['links'], err)(links)).toBe(true);
@@ -69,7 +59,6 @@ describe('createHasMatcher', () => {
     expect(createHasMatcher(['labels', 'links'], err)(empty2)).toBe(false);
     expect(createHasMatcher(['labels', 'links'], err)(labels)).toBe(true);
     expect(createHasMatcher(['labels', 'links'], err)(links)).toBe(true);
-    expect(createHasMatcher(['labels', 'links'], err)(annotations)).toBe(false);
 
     expect(err).not.toHaveBeenCalled();
   });
@@ -81,7 +70,7 @@ describe('createHasMatcher', () => {
         throw e;
       }),
     ).toThrowErrorMatchingInlineSnapshot(
-      `"'bar' is not a valid parameter for 'has' filter expressions, expected one of 'labels','annotations','links'"`,
+      `"'bar' is not a valid parameter for 'has' filter expressions, expected one of 'labels','links'"`,
     );
     expect(err).not.toHaveBeenCalled();
 
@@ -90,12 +79,12 @@ describe('createHasMatcher', () => {
     expect(err).toHaveBeenCalledTimes(2);
     expect(err).toHaveBeenCalledWith(
       expect.objectContaining({
-        message: `'foo' is not a valid parameter for 'has' filter expressions, expected one of 'labels','annotations','links'`,
+        message: `'foo' is not a valid parameter for 'has' filter expressions, expected one of 'labels','links'`,
       }),
     );
     expect(err).toHaveBeenCalledWith(
       expect.objectContaining({
-        message: `'bar' is not a valid parameter for 'has' filter expressions, expected one of 'labels','annotations','links'`,
+        message: `'bar' is not a valid parameter for 'has' filter expressions, expected one of 'labels','links'`,
       }),
     );
     expect(matcher(empty1)).toBe(false);

plugins/catalog/src/alpha/filter/matrchers/createHasMatcher.ts

@@ -21,9 +21,6 @@ const allowedMatchers: Record<string, EntityMatcherFn> = {
   labels: entity => {
     return Object.keys(entity.metadata.labels ?? {}).length > 0;
   },
-  annotations: entity => {
-    return Object.keys(entity.metadata.annotations ?? {}).length > 0;
-  },
   links: entity => {
     return (entity.metadata.links ?? []).length > 0;
   },

plugins/catalog/src/alpha/filter/parseFilterExpression.test.ts

@@ -89,7 +89,7 @@ describe('parseFilterExpression', () => {
     expect(run('has:labels,links')(annotations)).toBe(false);
 
     expect(() => run('has:labels,bar')).toThrowErrorMatchingInlineSnapshot(
-      `"'bar' is not a valid parameter for 'has' filter expressions, expected one of 'labels','annotations','links'"`,
+      `"'bar' is not a valid parameter for 'has' filter expressions, expected one of 'labels','links'"`,
     );
   });