[generator] Add support for interface constants.

jpobst · jpobst · commit eba0dc660387 · 2019-06-25T14:53:00.000-05:00
diff --git a/tools/generator/CodeGenerationOptions.cs b/tools/generator/CodeGenerationOptions.cs
@@ -70,6 +70,7 @@ internal CodeGenerator CreateCodeGenerator (TextWriter writer)
 		public bool UseShortFileNames { get; set; }
 		public IList<GenBase> Gens {get;set;}
 		public int ProductVersion { get; set; }
+		public bool SupportDefaultInterfaceMethods { get; set; }
 
 		bool? buildingCoreAssembly;
 		public bool BuildingCoreAssembly {
diff --git a/tools/generator/CodeGenerator.cs b/tools/generator/CodeGenerator.cs
@@ -63,7 +63,8 @@ static void Run (CodeGeneratorOptions options, DirectoryAssemblyResolver resolve
 				UseGlobal             = options.GlobalTypeNames,
 				IgnoreNonPublicType   = true,
 				UseShortFileNames     = options.UseShortFileNames,
-				ProductVersion        = options.ProductVersion
+				ProductVersion        = options.ProductVersion,
+				SupportDefaultInterfaceMethods = options.SupportDefaultInterfaceMethods,
 			};
 
 			// Load reference libraries
diff --git a/tools/generator/CodeGeneratorOptions.cs b/tools/generator/CodeGeneratorOptions.cs
@@ -39,6 +39,7 @@ public CodeGeneratorOptions ()
 		public string               MappingReportFile { get; set; }
 		public bool                 OnlyRunApiXmlAdjuster { get; set; }
 		public string               ApiXmlAdjusterOutput { get; set; }
+		public bool                 SupportDefaultInterfaceMethods { get; set; }
 
 		public static CodeGeneratorOptions Parse (string[] args)
 		{
@@ -85,6 +86,9 @@ public static CodeGeneratorOptions Parse (string[] args)
 				{ "sdk-platform|api-level=",
 					"SDK Platform {VERSION}/API level.",
 					v => opts.ApiLevel = v },
+				{ "default-interface-methods",
+					"For internal use.",
+					v => opts.SupportDefaultInterfaceMethods = v != null },
 				{ "preserve-enums",
 					"For internal use.",
 					v => opts.PreserveEnums = v != null },
diff --git a/tools/generator/Java.Interop.Tools.Generator.CodeGeneration/CodeGenerator.cs b/tools/generator/Java.Interop.Tools.Generator.CodeGeneration/CodeGenerator.cs
@@ -439,61 +439,26 @@ internal virtual void WriteField (Field field, string indent, GenBase type)
 		public void WriteInterface (InterfaceGen @interface, string indent, GenerationInfo gen_info)
 		{
 			opt.ContextTypes.Push (@interface);
+
 			// interfaces don't nest, so generate as siblings
 			foreach (GenBase nest in @interface.NestedTypes) {
 				WriteType (nest, indent, gen_info);
 				writer.WriteLine ();
 			}
 
-			var staticMethods = @interface.Methods.Where (m => m.IsStatic);
-			if (@interface.Fields.Any () || staticMethods.Any ()) {
-				string name = @interface.HasManagedName
-					? @interface.Name.Substring (1) + "Consts"
-					: @interface.Name.Substring (1);
-				writer.WriteLine ("{0}[Register (\"{1}\"{2}, DoNotGenerateAcw=true)]", indent, @interface.RawJniName, @interface.AdditionalAttributeString ());
-				writer.WriteLine ("{0}public abstract class {1} : Java.Lang.Object {{", indent, name);
-				writer.WriteLine ();
-				writer.WriteLine ("{0}\tinternal {1} ()", indent, name);
-				writer.WriteLine ("{0}\t{{", indent);
-				writer.WriteLine ("{0}\t}}", indent);
-
-				var seen = new HashSet<string> ();
-				bool needsClassRef = WriteFields (@interface.Fields, indent + "\t", @interface, seen) || staticMethods.Any ();
-				foreach (var iface in @interface.GetAllImplementedInterfaces ().OfType<InterfaceGen> ()) {
-					writer.WriteLine ();
-					writer.WriteLine ("{0}\t// The following are fields from: {1}", indent, iface.JavaName);
-					bool v = WriteFields (iface.Fields, indent + "\t", iface, seen);
-					needsClassRef = needsClassRef || v;
-				}
-
-				foreach (var m in @interface.Methods.Where (m => m.IsStatic))
-					WriteMethod (m, indent + "\t", @interface, true);
-
-				if (needsClassRef) {
-					writer.WriteLine ();
-					WriteClassHandle (@interface, indent + "\t", name);
-				}
+			WriteInterfaceImplementedMembersAlternative (@interface, indent);
 
-				writer.WriteLine ("{0}}}", indent, @interface.Name);
-				writer.WriteLine ();
+			// If this interface is just fields and we can't generate any of them
+			// then we don't need to write the interface
+			if (@interface.IsConstSugar && @interface.GetGeneratableFields (opt).Count () == 0)
+				return;
 
-				if (!@interface.HasManagedName) {
-					writer.WriteLine ("{0}[Register (\"{1}\"{2}, DoNotGenerateAcw=true)]", indent, @interface.RawJniName, @interface.AdditionalAttributeString ());
-					writer.WriteLine ("{0}[global::System.Obsolete (\"Use the '{1}' type. This type will be removed in a future release.\")]", indent, name);
-					writer.WriteLine ("{0}public abstract class {1}Consts : {1} {{", indent, name);
-					writer.WriteLine ();
-					writer.WriteLine ("{0}\tprivate {1}Consts ()", indent, name);
-					writer.WriteLine ("{0}\t{{", indent);
-					writer.WriteLine ("{0}\t}}", indent);
-					writer.WriteLine ("{0}}}", indent);
-					writer.WriteLine ();
-				}
-			}
+			WriteInterfaceDeclaration (@interface, indent);
 
+			// If this interface is just constant fields we don't need to write all the invoker bits
 			if (@interface.IsConstSugar)
 				return;
 
-			WriteInterfaceDeclaration (@interface, indent);
 			if (!@interface.AssemblyQualifiedName.Contains ('/'))
 				WriteInterfaceExtensionsDeclaration (@interface, indent, null);
 			WriteInterfaceInvoker (@interface, indent);
@@ -545,11 +510,15 @@ public void WriteInterfaceDeclaration (InterfaceGen @interface, string indent)
 
 			if (@interface.IsDeprecated)
 				writer.WriteLine ("{0}[ObsoleteAttribute (@\"{1}\")]", indent, @interface.DeprecatedComment);
-			writer.WriteLine ("{0}[Register (\"{1}\", \"\", \"{2}\"{3})]", indent, @interface.RawJniName, @interface.Namespace + "." + @interface.FullName.Substring (@interface.Namespace.Length + 1).Replace ('.', '/') + "Invoker", @interface.AdditionalAttributeString ());
+
+			if (!@interface.IsConstSugar)
+				writer.WriteLine ("{0}[Register (\"{1}\", \"\", \"{2}\"{3})]", indent, @interface.RawJniName, @interface.Namespace + "." + @interface.FullName.Substring (@interface.Namespace.Length + 1).Replace ('.', '/') + "Invoker", @interface.AdditionalAttributeString ());
+
 			if (@interface.TypeParameters != null && @interface.TypeParameters.Any ())
 				writer.WriteLine ("{0}{1}", indent, @interface.TypeParameters.ToGeneratedAttributeString ());
-			writer.WriteLine ("{0}{1} partial interface {2} : {3} {{", indent, @interface.Visibility, @interface.Name,
-				@interface.Interfaces.Count == 0 || sb.Length == 0 ? "IJavaObject" : sb.ToString ());
+			writer.WriteLine ("{0}{1} partial interface {2}{3} {{", indent, @interface.Visibility, @interface.Name,
+				@interface.IsConstSugar ? string.Empty : @interface.Interfaces.Count == 0 || sb.Length == 0 ? " : IJavaObject" : " : " + sb.ToString ());
+			WriteInterfaceFields (@interface, indent + "\t");
 			writer.WriteLine ();
 			WriteInterfaceProperties (@interface, indent + "\t");
 			WriteInterfaceMethods (@interface, indent + "\t");
@@ -707,6 +676,74 @@ public void WriteInterfaceExtensionsDeclaration (InterfaceGen @interface, string
 			writer.WriteLine ();
 		}
 
+		public void WriteInterfaceFields (InterfaceGen iface, string indent)
+		{
+			// Interface fields are only supported with DIM
+			if (!opt.SupportDefaultInterfaceMethods)
+				return;
+
+			var seen = new HashSet<string> ();
+			var fields = iface.GetGeneratableFields (opt).ToList ();
+
+			WriteFields (fields, indent, iface, seen);
+		}
+
+		public void WriteInterfaceImplementedMembersAlternative (InterfaceGen @interface, string indent)
+		{
+			// Historically .NET has not allowed interface implemented fields or constants, so we
+			// initially worked around that by moving them to an abstract class, generally
+			// IMyInterface -> MyInterfaceConsts
+			// This was later expanded to accomodate static interface methods, creating a more appropriately named class
+			// IMyInterface -> MyInterface
+			// In this case the XXXConsts class is [Obsolete]'d and simply inherits from the newer class
+			// in order to maintain backward compatibility.
+			var staticMethods = @interface.Methods.Where (m => m.IsStatic);
+
+			if (@interface.Fields.Any () || staticMethods.Any ()) {
+				string name = @interface.HasManagedName
+					? @interface.Name.Substring (1) + "Consts"
+					: @interface.Name.Substring (1);
+				writer.WriteLine ("{0}[Register (\"{1}\"{2}, DoNotGenerateAcw=true)]", indent, @interface.RawJniName, @interface.AdditionalAttributeString ());
+				writer.WriteLine ("{0}public abstract class {1} : Java.Lang.Object {{", indent, name);
+				writer.WriteLine ();
+				writer.WriteLine ("{0}\tinternal {1} ()", indent, name);
+				writer.WriteLine ("{0}\t{{", indent);
+				writer.WriteLine ("{0}\t}}", indent);
+
+				var seen = new HashSet<string> ();
+				bool needsClassRef = WriteFields (@interface.Fields, indent + "\t", @interface, seen) || staticMethods.Any ();
+				foreach (var iface in @interface.GetAllImplementedInterfaces ().OfType<InterfaceGen> ()) {
+					writer.WriteLine ();
+					writer.WriteLine ("{0}\t// The following are fields from: {1}", indent, iface.JavaName);
+					bool v = WriteFields (iface.Fields, indent + "\t", iface, seen);
+					needsClassRef = needsClassRef || v;
+				}
+
+				foreach (var m in @interface.Methods.Where (m => m.IsStatic))
+					WriteMethod (m, indent + "\t", @interface, true);
+
+				if (needsClassRef) {
+					writer.WriteLine ();
+					WriteClassHandle (@interface, indent + "\t", name);
+				}
+
+				writer.WriteLine ("{0}}}", indent, @interface.Name);
+				writer.WriteLine ();
+
+				if (!@interface.HasManagedName) {
+					writer.WriteLine ("{0}[Register (\"{1}\"{2}, DoNotGenerateAcw=true)]", indent, @interface.RawJniName, @interface.AdditionalAttributeString ());
+					writer.WriteLine ("{0}[global::System.Obsolete (\"Use the '{1}' type. This type will be removed in a future release.\")]", indent, name);
+					writer.WriteLine ("{0}public abstract class {1}Consts : {1} {{", indent, name);
+					writer.WriteLine ();
+					writer.WriteLine ("{0}\tprivate {1}Consts ()", indent, name);
+					writer.WriteLine ("{0}\t{{", indent);
+					writer.WriteLine ("{0}\t}}", indent);
+					writer.WriteLine ("{0}}}", indent);
+					writer.WriteLine ();
+				}
+			}
+		}
+
 		public void WriteInterfaceInvoker (InterfaceGen @interface, string indent)
 		{
 			writer.WriteLine ("{0}[global::Android.Runtime.Register (\"{1}\", DoNotGenerateAcw=true{2})]", indent, @interface.RawJniName, @interface.AdditionalAttributeString ());
diff --git a/tools/generator/Java.Interop.Tools.Generator.ObjectModel/InterfaceGen.cs b/tools/generator/Java.Interop.Tools.Generator.ObjectModel/InterfaceGen.cs
@@ -1,4 +1,5 @@
 using System;
+using System.Collections;
 using System.Collections.Generic;
 using System.IO;
 using System.Linq;
@@ -27,6 +28,15 @@ public override string DefaultValue {
 
 		public bool HasManagedName => hasManagedName;
 
+		// These are fields that we currently support generating on the interface with DIM
+		public IEnumerable<Field> GetGeneratableFields (CodeGenerationOptions options)
+		{
+			if (!options.SupportDefaultInterfaceMethods)
+				return Enumerable.Empty<Field> ();
+
+			return Fields.Where (f => !f.NeedsProperty && !(f.DeprecatedComment?.Contains ("constant will be removed") == true));
+		}
+
 		public bool IsConstSugar {
 			get { 
 				if (Methods.Count > 0 || Properties.Count > 0)
diff --git a/tools/generator/Tests/Unit-Tests/CodeGeneratorExpectedResults/JavaInterop1/WriteConstSugarInterfaceFields.txt b/tools/generator/Tests/Unit-Tests/CodeGeneratorExpectedResults/JavaInterop1/WriteConstSugarInterfaceFields.txt
@@ -0,0 +1,18 @@
+// Metadata.xml XPath interface reference: path="/api/package[@name='java.code']/interface[@name='IMyInterface']"
+public partial interface IMyInterface {
+
+	// Metadata.xml XPath field reference: path="/api/package[@name='java.code']/interface[@name='IMyInterface']/field[@name='MyConstantField']"
+	[Register ("MyConstantField")]
+	public const int MyConstantField = (int) 7;
+
+	// Metadata.xml XPath field reference: path="/api/package[@name='java.code']/interface[@name='IMyInterface']/field[@name='MyConstantStringField']"
+	[Register ("MyConstantStringField")]
+	public const string MyConstantStringField = (string) "hello";
+
+	// Metadata.xml XPath field reference: path="/api/package[@name='java.code']/interface[@name='IMyInterface']/field[@name='MyDeprecatedField']"
+	[Register ("MyDeprecatedField")]
+	[Obsolete ("")]
+	public const int MyDeprecatedField = (int) 7;
+
+}
+
diff --git a/tools/generator/Tests/Unit-Tests/CodeGeneratorExpectedResults/JavaInterop1/WriteInterfaceFields.txt b/tools/generator/Tests/Unit-Tests/CodeGeneratorExpectedResults/JavaInterop1/WriteInterfaceFields.txt
@@ -0,0 +1,14 @@
+// Metadata.xml XPath interface reference: path="/api/package[@name='java.code']/interface[@name='IMyInterface']"
+[Register ("java/code/IMyInterface", "", "java.code.IMyInterfaceInvoker")]
+public partial interface IMyInterface : IJavaObject {
+
+	// Metadata.xml XPath field reference: path="/api/package[@name='java.code']/interface[@name='IMyInterface']/field[@name='MyConstantField']"
+	[Register ("MyConstantField")]
+	public const int MyConstantField = (int) 7;
+
+	// Metadata.xml XPath method reference: path="/api/package[@name='java.code']/interface[@name='IMyInterface']/method[@name='DoSomething' and count(parameter)=0]"
+	[Register ("DoSomething", "()V", "GetDoSomethingHandler:java.code.IMyInterfaceInvoker, ")]
+	void DoSomething ();
+
+}
+
diff --git a/tools/generator/Tests/Unit-Tests/CodeGeneratorExpectedResults/XamarinAndroid/WriteConstSugarInterfaceFields.txt b/tools/generator/Tests/Unit-Tests/CodeGeneratorExpectedResults/XamarinAndroid/WriteConstSugarInterfaceFields.txt
@@ -0,0 +1,18 @@
+// Metadata.xml XPath interface reference: path="/api/package[@name='java.code']/interface[@name='IMyInterface']"
+public partial interface IMyInterface {
+
+	// Metadata.xml XPath field reference: path="/api/package[@name='java.code']/interface[@name='IMyInterface']/field[@name='MyConstantField']"
+	[Register ("MyConstantField")]
+	public const int MyConstantField = (int) 7;
+
+	// Metadata.xml XPath field reference: path="/api/package[@name='java.code']/interface[@name='IMyInterface']/field[@name='MyConstantStringField']"
+	[Register ("MyConstantStringField")]
+	public const string MyConstantStringField = (string) "hello";
+
+	// Metadata.xml XPath field reference: path="/api/package[@name='java.code']/interface[@name='IMyInterface']/field[@name='MyDeprecatedField']"
+	[Register ("MyDeprecatedField")]
+	[Obsolete ("")]
+	public const int MyDeprecatedField = (int) 7;
+
+}
+
diff --git a/tools/generator/Tests/Unit-Tests/CodeGeneratorExpectedResults/XamarinAndroid/WriteInterfaceFields.txt b/tools/generator/Tests/Unit-Tests/CodeGeneratorExpectedResults/XamarinAndroid/WriteInterfaceFields.txt
@@ -0,0 +1,14 @@
+// Metadata.xml XPath interface reference: path="/api/package[@name='java.code']/interface[@name='IMyInterface']"
+[Register ("java/code/IMyInterface", "", "java.code.IMyInterfaceInvoker")]
+public partial interface IMyInterface : IJavaObject {
+
+	// Metadata.xml XPath field reference: path="/api/package[@name='java.code']/interface[@name='IMyInterface']/field[@name='MyConstantField']"
+	[Register ("MyConstantField")]
+	public const int MyConstantField = (int) 7;
+
+	// Metadata.xml XPath method reference: path="/api/package[@name='java.code']/interface[@name='IMyInterface']/method[@name='DoSomething' and count(parameter)=0]"
+	[Register ("DoSomething", "()V", "GetDoSomethingHandler:java.code.IMyInterfaceInvoker, ")]
+	void DoSomething ();
+
+}
+
diff --git a/tools/generator/Tests/Unit-Tests/DefaultInterfaceMethodsTests.cs b/tools/generator/Tests/Unit-Tests/DefaultInterfaceMethodsTests.cs
@@ -0,0 +1,65 @@
+using System;
+using generatortests.Unit_Tests;
+using MonoDroid.Generation;
+using NUnit.Framework;
+
+namespace generatortests
+{
+	[TestFixture]
+	class JavaInteropDefaultInterfaceMethodsTests : DefaultInterfaceMethodsTests
+	{
+		protected override Xamarin.Android.Binder.CodeGenerationTarget Target => Xamarin.Android.Binder.CodeGenerationTarget.JavaInterop1;
+	}
+
+	[TestFixture]
+	class XamarinAndroidDefaultInterfaceMethodsTests : DefaultInterfaceMethodsTests
+	{
+		protected override Xamarin.Android.Binder.CodeGenerationTarget Target => Xamarin.Android.Binder.CodeGenerationTarget.XamarinAndroid;
+	}
+
+	abstract class DefaultInterfaceMethodsTests : CodeGeneratorTestBase
+	{
+		[Test]
+		public void WriteInterfaceFields ()
+		{
+			// This is an interface that has both fields and method declarations
+			var iface = SupportTypeBuilder.CreateEmptyInterface("java.code.IMyInterface");
+
+			iface.Fields.Add (new TestField ("int", "MyConstantField").SetConstant ().SetValue ("7"));
+			iface.Methods.Add (new TestMethod (iface, "DoSomething").SetAbstract ());
+
+			iface.Validate (options, new GenericParameterDefinitionList ());
+
+			options.ContextTypes.Push (iface);
+			generator.WriteInterfaceDeclaration (iface, string.Empty);
+			options.ContextTypes.Pop ();
+
+			Assert.AreEqual (GetTargetedExpected (nameof (WriteInterfaceFields)), writer.ToString ().NormalizeLineEndings ());
+		}
+
+		[Test]
+		public void WriteConstSugarInterfaceFields ()
+		{
+			// This is an interface that only has fields (IsConstSugar)
+			// We treat   a little differenly because they don't need to interop with Java
+			var iface = SupportTypeBuilder.CreateEmptyInterface ("java.code.IMyInterface");
+
+			// These interface fields are supported and should be in the output
+			iface.Fields.Add (new TestField ("int", "MyConstantField").SetConstant ().SetValue ("7"));
+			iface.Fields.Add (new TestField ("java.lang.String", "MyConstantStringField").SetConstant ().SetValue ("\"hello\""));
+			iface.Fields.Add (new TestField ("int", "MyDeprecatedField").SetConstant ().SetValue ("7").SetDeprecated ());
+
+			// These interface fields are not supported and should be ignored
+			iface.Fields.Add (new TestField ("int", "MyDeprecatedEnumField").SetConstant ().SetValue ("MyEnumValue").SetDeprecated ("This constant will be removed in the future version."));
+			iface.Fields.Add (new TestField ("int", "MyStaticField").SetStatic ().SetValue ("7"));
+
+			iface.Validate (options, new GenericParameterDefinitionList ());
+
+			options.ContextTypes.Push (iface);
+			generator.WriteInterfaceDeclaration (iface, string.Empty);
+			options.ContextTypes.Pop ();
+
+			Assert.AreEqual (GetTargetedExpected (nameof (WriteConstSugarInterfaceFields)), writer.ToString ().NormalizeLineEndings ());
+		}
+	}
+}
diff --git a/tools/generator/Tests/Unit-Tests/SupportTypes.cs b/tools/generator/Tests/Unit-Tests/SupportTypes.cs
@@ -101,6 +101,12 @@ public TestField SetVisibility (string visibility)
 			return this;
 		}
 
+		public TestField SetValue (string value)
+		{
+			this.value = value;
+			return this;
+		}
+
 		public override bool IsDeprecated => isDeprecated;
 
 		public override string DeprecatedComment => deprecatedComment;
@@ -352,7 +358,7 @@ public static TestCtor CreateConstructor (GenBase parent, string methodName, Cod
 			return ctor;
 		}
 
-		public static TestInterface CreateEmptyInterface (string interfaceName, CodeGenerationOptions options)
+		public static TestInterface CreateEmptyInterface(string interfaceName)
 		{
 			var iface = new TestInterface (null, interfaceName);
 
@@ -361,7 +367,7 @@ public static TestInterface CreateEmptyInterface (string interfaceName, CodeGene
 
 		public static TestInterface CreateInterface (string interfaceName, CodeGenerationOptions options)
 		{
-			var iface = CreateEmptyInterface (interfaceName, null);
+			var iface = CreateEmptyInterface(interfaceName);
 
 			iface.Properties.Add (CreateProperty (iface, "Count", "int", options));
 			iface.Properties.Add (CreateProperty (iface, "Key", "java.lang.String", options));
@@ -401,7 +407,6 @@ public static Property CreateProperty (GenBase parent, string propertyName, stri
 				Setter = CreateMethod (parent, $"set_{propertyName}", options, "void", isStatic, isAbstract, parameters: new Parameter ("value", propertyType, propertyType, false))
 			};
 
-
 			return prop;
 		}
 
diff --git a/tools/generator/Tests/generator-Tests.csproj b/tools/generator/Tests/generator-Tests.csproj
