Add deprecated key to more definitions

changelog/@unreleased/deprecate-key-for-more-definitions.v2.yml

@@ -0,0 +1,5 @@
+type: improvement
+improvement:
+  description: More definitions take deprecated key
+  urls:
+    - https://github.com/palantir/conjure/pull/726

conjure-api/src/main/conjure/conjure-api.yml

@@ -111,21 +111,25 @@ types:
           typeName: TypeName
           alias: Type
           docs: optional<Documentation>
+          deprecated: optional<Documentation>
       EnumDefinition:
         fields:
           typeName: TypeName
           values: list<EnumValueDefinition>
           docs: optional<Documentation>
+          deprecated: optional<Documentation>
       ObjectDefinition:
         fields:
           typeName: TypeName
           fields: list<FieldDefinition>
           docs: optional<Documentation>
+          deprecated: optional<Documentation>
       UnionDefinition:
         fields:
           typeName: TypeName
           union: list<FieldDefinition>
           docs: optional<Documentation>
+          deprecated: optional<Documentation>
       EnumValueDefinition:
         fields:
           value: string
@@ -147,6 +151,7 @@ types:
           serviceName: TypeName
           endpoints: list<EndpointDefinition>
           docs: optional<Documentation>
+          deprecated: optional<Documentation>
       EndpointDefinition:
         fields:
           endpointName: EndpointName

conjure-core/src/test/java/com/palantir/conjure/defs/validator/EndpointDefinitionTest.java

@@ -137,10 +137,10 @@ public void testNoOptionalBinaryBodyParamValidator_throughAlias() {
 
         DealiasingTypeVisitor dealiasingVisitor = new DealiasingTypeVisitor(ImmutableMap.of(
                 typeName,
-                TypeDefinition.alias(AliasDefinition.of(
-                        typeName,
-                        Type.optional(OptionalType.of(Type.primitive(PrimitiveType.BINARY))),
-                        Documentation.of("")))));
+                TypeDefinition.alias(AliasDefinition.builder()
+                        .typeName(typeName)
+                        .alias(Type.optional(OptionalType.of(Type.primitive(PrimitiveType.BINARY))))
+                        .build())));
 
         assertThatThrownBy(() -> EndpointDefinitionValidator.validateAll(definition, dealiasingVisitor))
                 .isInstanceOf(IllegalStateException.class)
@@ -325,7 +325,9 @@ public void testComplexHeaderObject() {
 
         DealiasingTypeVisitor dealiasingVisitor = new DealiasingTypeVisitor(ImmutableMap.of(
                 typeName,
-                TypeDefinition.object(ObjectDefinition.of(typeName, ImmutableList.of(), Documentation.of("")))));
+                TypeDefinition.object(ObjectDefinition.builder()
+                        .typeName(typeName)
+                        .build())));
 
         assertThatThrownBy(() -> EndpointDefinitionValidator.validateAll(definition.build(), dealiasingVisitor))
                 .isInstanceOf(IllegalStateException.class)

conjure-core/src/test/resources/normalized.conjure.json

@@ -53,7 +53,8 @@
         "docs" : null,
         "deprecated" : null
       } ],
-      "docs" : null
+      "docs" : null,
+      "deprecated" : null
     }
   }, {
     "type" : "alias",
@@ -66,7 +67,8 @@
         "type" : "primitive",
         "primitive" : "STRING"
       },
-      "docs" : null
+      "docs" : null,
+      "deprecated" : null
     }
   }, {
     "type" : "alias",
@@ -79,7 +81,8 @@
         "type" : "primitive",
         "primitive" : "STRING"
       },
-      "docs" : null
+      "docs" : null,
+      "deprecated" : null
     }
   } ],
   "services" : [ {
@@ -98,7 +101,8 @@
       "deprecated" : null,
       "markers" : [ ]
     } ],
-    "docs" : null
+    "docs" : null,
+    "deprecated" : null
   }, {
     "serviceName" : {
       "name" : "TestService2",
@@ -115,7 +119,8 @@
       "deprecated" : null,
       "markers" : [ ]
     } ],
-    "docs" : null
+    "docs" : null,
+    "deprecated" : null
   } ],
   "extensions" : { }
 }