Update based on PR comments - leave target as required while still in schema

Signed-off-by: Malikah Montgomery <malikah.montgomery@thoughtworks.com>

packages/catalog-model/src/schema/shared/common.schema.json

@@ -32,7 +32,7 @@
       "$id": "#relation",
       "type": "object",
       "description": "A directed relation from one entity to another.",
-      "required": ["type", "targetRef"],
+      "required": ["type", "target", "targetRef"],
       "additionalProperties": false,
       "properties": {
         "type": {

packages/catalog-model/src/validation/entitySchemaValidator.test.ts

@@ -365,9 +365,9 @@ describe('entitySchemaValidator', () => {
     expect(() => validator(entity)).toThrow(/relations/);
   });
 
-  it('does not reject missing relations.target', () => {
+  it('does reject missing relations.target', () => {
     delete entity.relations[0].target;
-    expect(() => validator(entity)).not.toThrow(/relations/);
+    expect(() => validator(entity)).toThrow(/relations/);
   });
 
   it('does reject missing relations.targetRef', () => {