removed commented code and unused imports on swagger-core modules

Author: HugoMario

modules/swagger-core/src/main/java/io/swagger/v3/core/jackson/ModelResolver.java

@@ -328,9 +328,6 @@ public Schema resolve(AnnotatedType annotatedType, ModelConverterContext context
  }
 
  if (isPrimitive) {
- if (annotatedType.isSchemaProperty()) {
- //model.name(name);
- }
  XML xml = resolveXml(beanDesc.getClassInfo(), annotatedType.getCtxAnnotations(), resolvedSchemaAnnotation);
  if (xml != null) {
  model.xml(xml);
@@ -435,7 +432,6 @@ public Schema resolve(AnnotatedType annotatedType, ModelConverterContext context
  Schema mapModel = new MapSchema().additionalProperties(addPropertiesSchema);
  mapModel.name(name);
  model = mapModel;
- //return model;
  } else if (valueType != null) {
  if (ReflectionUtils.isSystemType(type) && !annotatedType.isSchemaProperty() && !annotatedType.isResolveAsRef()) {
  context.resolve(new AnnotatedType().type(valueType).jsonViewAnnotation(annotatedType.getJsonViewAnnotation()));
@@ -634,7 +630,6 @@ public Schema resolve(AnnotatedType annotatedType, ModelConverterContext context
  AnnotatedType aType = new AnnotatedType()
  .type(propType)
  .ctxAnnotations(annotations)
- //.name(propName)
  .parent(model)
  .resolveAsRef(annotatedType.isResolveAsRef())
  .jsonViewAnnotation(annotatedType.getJsonViewAnnotation())
@@ -654,8 +649,6 @@ public Schema resolve(AnnotatedType annotatedType, ModelConverterContext context
  return null;
  } else {
  return new Schema();
- //t.jsonUnwrappedHandler(null);
- //return context.resolve(t);
  }
  });
  property = clone(context.resolve(aType));

modules/swagger-core/src/main/java/io/swagger/v3/core/jackson/SwaggerModule.java

@@ -5,12 +5,6 @@
 public class SwaggerModule extends SimpleModule {
  private static final long serialVersionUID = 1L;
 
- /*
- /**********************************************************
- /* Life-cycle
- /**********************************************************
- */
-
  public SwaggerModule() {
  super(PackageVersion.VERSION);
  }

modules/swagger-core/src/main/java/io/swagger/v3/core/jackson/mixin/MediaTypeMixin.java

@@ -1,3 +1,19 @@
+/**
+ * Copyright 2021 SmartBear Software
+ * <p>
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ * <p>
+ * http://www.apache.org/licenses/LICENSE-2.0
+ * <p>
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 package io.swagger.v3.core.jackson.mixin;
 
 import com.fasterxml.jackson.annotation.JsonAnyGetter;

modules/swagger-core/src/main/java/io/swagger/v3/core/jackson/mixin/Schema31Mixin.java

@@ -19,7 +19,6 @@
 @JsonPropertyOrder(value = {"type", "format", "if", "then", "else"}, alphabetic = true)
 public abstract class Schema31Mixin {
 
- //@JsonValue
  @JsonIgnore
  public abstract Map<String, Object> getJsonSchema();
 

modules/swagger-core/src/main/java/io/swagger/v3/core/util/AnnotationsUtils.java

@@ -942,10 +942,7 @@ public static Optional<Header> getHeader(io.swagger.v3.oas.annotations.headers.H
  if (header.schema() != null) {
  if (header.schema().implementation().equals(Void.class)) {
  AnnotationsUtils.getSchemaFromAnnotation(header.schema(), jsonViewAnnotation).ifPresent(
- headerObject::setSchema
- //schema inline no need to add to components
- //components.addSchemas(schema.getType(), schema);
- );
+ headerObject::setSchema);
  }
  }
 
@@ -1445,7 +1442,6 @@ else if (tS != null && cS == null && cA == null) {
  }
 
  else if (tS != null && cS != null) {
- //return mergeSchemaAnnotations(cS, tS);
  return mergeSchemaAnnotations(tS, cS);
  }
 

modules/swagger-core/src/main/java/io/swagger/v3/core/util/Callback31Deserializer.java

@@ -1,18 +1,5 @@
 package io.swagger.v3.core.util;
 
-import com.fasterxml.jackson.core.JsonParser;
-import com.fasterxml.jackson.databind.DeserializationContext;
-import com.fasterxml.jackson.databind.JsonDeserializer;
-import com.fasterxml.jackson.databind.JsonNode;
-import com.fasterxml.jackson.databind.node.ObjectNode;
-import io.swagger.v3.oas.models.PathItem;
-
-import javax.security.auth.callback.Callback;
-import java.io.IOException;
-import java.util.Iterator;
-import java.util.LinkedHashMap;
-import java.util.Map;
-
 public class Callback31Deserializer extends CallbackDeserializer {
 
  public Callback31Deserializer() {

modules/swagger-core/src/test/java/io/swagger/v3/core/converting/ModelConverterTest.java

@@ -182,7 +182,6 @@ public void processModelWithPairProperties() {
  ModelConverters.getInstance().addConverter(asPropertyConverter);
  final Map<String, Schema> asProperty = readAll(ModelWithTuple2.class);
  ModelConverters.getInstance().removeConverter(asPropertyConverter);
- //assertEquals(asProperty.size(), 2);
  Map<String, Schema> values = asProperty.get("ModelWithTuple2").getProperties();
  Yaml.prettyPrint(values);
  for (Map.Entry<String, Schema> entry : values.entrySet()) {

modules/swagger-core/src/test/java/io/swagger/v3/core/converting/SwaggerSerializerTest.java

@@ -159,8 +159,6 @@ public void writeSpecWithParameterReferences() throws IOException {
  final OpenAPI swagger = new OpenAPI()
  .info(info)
  .addServersItem(new Server().url("http://petstore.swagger.io"))
- //.consumes("application/json")
- //.produces("application/json")
  .schema("Person", personModel);
 
  final QueryParameter parameter = (QueryParameter) new QueryParameter()
@@ -169,10 +167,8 @@ public void writeSpecWithParameterReferences() throws IOException {
  .schema(new IntegerSchema());
 
  final Operation get = new Operation()
- //.produces("application/json")
  .summary("finds pets in the system")
  .description("a longer description")
- //.tag("Pet Operations")
  .operationId("get pet by id")
  .addParametersItem(new Parameter().$ref("#/parameters/Foo"));
 

modules/swagger-core/src/test/java/io/swagger/v3/core/converting/override/SnakeCaseConverterTest.java

@@ -13,13 +13,10 @@
 import org.testng.annotations.Test;
 
 import javax.xml.bind.annotation.XmlRootElement;
-import java.lang.annotation.Annotation;
-import java.lang.reflect.Type;
 import java.util.Iterator;
 import java.util.LinkedHashMap;
 import java.util.Map;
 import java.util.Set;
-import java.util.function.BiFunction;
 
 import static io.swagger.v3.core.util.RefUtils.constructRef;
 import static io.swagger.v3.core.util.RefUtils.extractSimpleName;

modules/swagger-core/src/test/java/io/swagger/v3/core/matchers/SerializationMatchers.java

@@ -1,6 +1,5 @@
 package io.swagger.v3.core.matchers;
 
-import com.fasterxml.jackson.core.JsonProcessingException;
 import com.fasterxml.jackson.databind.JsonNode;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.databind.node.NumericNode;
@@ -46,7 +45,6 @@ private static void apply(Object objectToSerialize, String str, ObjectMapper map
  }
  if (!lhs.equals(new ObjectNodeComparator(), rhs)) {
  assertEquals(Yaml.pretty(lhs), Yaml.pretty(rhs));
- //fail(String.format("Serialized object:\n%s\ndoes not equal to expected serialized string:\n%s", lhs, rhs));
  }
  }
 
@@ -60,7 +58,6 @@ private static void apply31(Object objectToSerialize, String str, ObjectMapper m
  }
  if (!lhs.equals(new ObjectNodeComparator(), rhs)) {
  assertEquals(Yaml31.pretty(lhs), Yaml31.pretty(rhs));
- //fail(String.format("Serialized object:\n%s\ndoes not equal to expected serialized string:\n%s", lhs, rhs));
  }
  }