Add ChangedOperationId class

Author: issy

core/src/main/java/org/openapitools/openapidiff/core/model/BackwardIncompatibleProp.java

@@ -10,6 +10,7 @@ public enum BackwardIncompatibleProp {
   EXTENSION_CONTENT_TYPES_DECREASED("incompatible.extension.content.types.decreased", false),
   EXTENSION_CONTENT_TYPE_DELETED("incompatible.extension.content.type.%s.deleted", false),
   OPENAPI_ENDPOINTS_DECREASED("incompatible.openapi.endpoints.decreased", true),
+  OPERATION_ID_CHANGED("incompatible.operation.id.changed", false),
   REQUEST_BODY_REQUIRED_INCREASED("incompatible.request.body.increased", true),
   REQUEST_CONTENT_DECREASED("incompatible.request.content.decreased", true),
   REQUEST_ENUM_DECREASED("incompatible.request.enum.decreased", true),

core/src/main/java/org/openapitools/openapidiff/core/model/ChangedOperation.java

@@ -7,6 +7,7 @@
 import java.util.Arrays;
 import java.util.List;
 import java.util.Objects;
+import org.openapitools.openapidiff.core.model.schema.ChangedOperationId;
 
 public class ChangedOperation implements ComposedChanged {
   private Operation oldOperation;
@@ -15,7 +16,7 @@ public class ChangedOperation implements ComposedChanged {
   private PathItem.HttpMethod httpMethod;
   private ChangedMetadata summary;
   private ChangedMetadata description;
-  private ChangedMetadata operationId;
+  private ChangedOperationId operationId;
   private boolean deprecated;
   private ChangedParameters parameters;
   private ChangedRequestBody requestBody;
@@ -92,7 +93,7 @@ public ChangedMetadata getDescription() {
     return this.description;
   }
 
-  public ChangedMetadata getOperationId() {
+  public ChangedOperationId getOperationId() {
     return this.operationId;
   }
 
@@ -150,7 +151,7 @@ public ChangedOperation setDescription(final ChangedMetadata description) {
     return this;
   }
 
-  public ChangedOperation setOperationId(final ChangedMetadata operationId) {
+  public ChangedOperation setOperationId(final ChangedOperationId operationId) {
     this.operationId = operationId;
     return this;
   }

core/src/main/java/org/openapitools/openapidiff/core/model/schema/ChangedOperationId.java

@@ -0,0 +1,40 @@
+package org.openapitools.openapidiff.core.model.schema;
+
+import static org.openapitools.openapidiff.core.model.BackwardIncompatibleProp.OPERATION_ID_CHANGED;
+
+import java.util.Objects;
+import org.openapitools.openapidiff.core.model.Changed;
+import org.openapitools.openapidiff.core.model.DiffContext;
+import org.openapitools.openapidiff.core.model.DiffResult;
+
+public class ChangedOperationId implements Changed {
+
+  private final String left;
+  private final String right;
+  private final DiffContext context;
+
+  public ChangedOperationId(String left, String right, DiffContext context) {
+    this.left = left;
+    this.right = right;
+    this.context = context;
+  }
+
+  @Override
+  public DiffResult isChanged() {
+    if (Objects.equals(left, right)) {
+      return DiffResult.NO_CHANGES;
+    }
+    if (OPERATION_ID_CHANGED.enabled(context)) {
+      return DiffResult.INCOMPATIBLE;
+    }
+    return DiffResult.METADATA;
+  }
+
+  public String getLeft() {
+    return this.left;
+  }
+
+  public String getRight() {
+    return this.right;
+  }
+}