feat(kernel): annotate implemented interfaces on "ObjRef"s

package.json

@@ -6,7 +6,8 @@
     "bump": "bash scripts/bump.sh",
     "fetch-dotnet-snk": "bash scripts/fetch-dotnet-snk.sh",
     "package": "bash scripts/package.sh",
-    "test": "lerna run test --stream"
+    "test": "lerna run test --stream",
+    "test:update": "UPDATE_DIFF=1 lerna run test --stream"
   },
   "devDependencies": {
     "@types/node": "^10.14.22",

packages/jsii-calc-base-of-base/package.json

@@ -25,7 +25,8 @@
   "types": "lib/index.d.ts",
   "scripts": {
     "build": "jsii",
-    "test": "diff-test test/assembly.jsii .jsii"
+    "test": "diff-test test/assembly.jsii .jsii",
+    "test:update": "npm run build && UPDATE_DIFF=1 npm run test"
   },
   "devDependencies": {
     "jsii": "^0.19.0",

packages/jsii-calc-base/package.json

@@ -25,7 +25,8 @@
   "types": "lib/index.d.ts",
   "scripts": {
     "build": "jsii",
-    "test": "diff-test test/assembly.jsii .jsii"
+    "test": "diff-test test/assembly.jsii .jsii",
+    "test:update": "npm run build && UPDATE_DIFF=1 npm run test"
   },
   "dependencies": {
     "@scope/jsii-calc-base-of-base": "^0.19.0"

packages/jsii-calc-lib/package.json

@@ -27,7 +27,8 @@
   "types": "lib/index.d.ts",
   "scripts": {
     "build": "jsii",
-    "test": "diff-test test/assembly.jsii .jsii"
+    "test": "diff-test test/assembly.jsii .jsii",
+    "test:update": "npm run build && UPDATE_DIFF=1 npm run test"
   },
   "dependencies": {
     "@scope/jsii-calc-base": "^0.19.0"

packages/jsii-calc/lib/compliance.ts

@@ -1438,7 +1438,7 @@ export class EraseUndefinedHashValues {
     /**
      * We expect "prop2" to be erased
      */
-    public static prop2IsUndefined(): any {
+    public static prop2IsUndefined(): { [key: string]: any } {
         return {
             prop1: 'value1',
             prop2: undefined
@@ -1448,7 +1448,7 @@ export class EraseUndefinedHashValues {
     /**
      * We expect "prop1" to be erased
      */
-    public static prop1IsNull(): any {
+    public static prop1IsNull(): { [key: string]: any } {
         return {
             prop1: null,
             prop2: 'value2'
@@ -1900,3 +1900,23 @@ export class OverridableProtectedMember {
         return this.overrideReadOnly + this.overrideReadWrite;
     }
 }
+
+/**
+ * We can return an anonymous interface implementation from an override without losing the interface
+ * declarations.
+ */
+export interface IAnonymousImplementationProvider {
+    provide(): IAnonymouslyImplementMe;
+}
+export class AnonymousImplementationProvider implements IAnonymousImplementationProvider {
+    public provide(): IAnonymouslyImplementMe {
+        return {
+            value: 1337,
+            verb() { return 'to implement'; },
+        };
+    }
+}
+export interface IAnonymouslyImplementMe {
+    value: number;
+    verb(): string;
+}

packages/jsii-calc/test/assembly.jsii

@@ -1027,6 +1027,41 @@
       ],
       "name": "AllowedMethodNames"
     },
+    "jsii-calc.AnonymousImplementationProvider": {
+      "assembly": "jsii-calc",
+      "docs": {
+        "stability": "experimental"
+      },
+      "fqn": "jsii-calc.AnonymousImplementationProvider",
+      "initializer": {},
+      "interfaces": [
+        "jsii-calc.IAnonymousImplementationProvider"
+      ],
+      "kind": "class",
+      "locationInModule": {
+        "filename": "lib/compliance.ts",
+        "line": 1911
+      },
+      "methods": [
+        {
+          "docs": {
+            "stability": "experimental"
+          },
+          "locationInModule": {
+            "filename": "lib/compliance.ts",
+            "line": 1912
+          },
+          "name": "provide",
+          "overrides": "jsii-calc.IAnonymousImplementationProvider",
+          "returns": {
+            "type": {
+              "fqn": "jsii-calc.IAnonymouslyImplementMe"
+            }
+          }
+        }
+      ],
+      "name": "AnonymousImplementationProvider"
+    },
     "jsii-calc.AsyncVirtualMethods": {
       "assembly": "jsii-calc",
       "docs": {
@@ -3318,7 +3353,12 @@
           "name": "prop1IsNull",
           "returns": {
             "type": {
-              "primitive": "any"
+              "collection": {
+                "elementtype": {
+                  "primitive": "any"
+                },
+                "kind": "map"
+              }
             }
           },
           "static": true
@@ -3335,7 +3375,12 @@
           "name": "prop2IsUndefined",
           "returns": {
             "type": {
-              "primitive": "any"
+              "collection": {
+                "elementtype": {
+                  "primitive": "any"
+                },
+                "kind": "map"
+              }
             }
           },
           "static": true
@@ -3790,6 +3835,85 @@
       ],
       "name": "GreetingAugmenter"
     },
+    "jsii-calc.IAnonymousImplementationProvider": {
+      "assembly": "jsii-calc",
+      "docs": {
+        "stability": "experimental",
+        "summary": "We can return an anonymous interface implementation from an override without losing the interface declarations."
+      },
+      "fqn": "jsii-calc.IAnonymousImplementationProvider",
+      "kind": "interface",
+      "locationInModule": {
+        "filename": "lib/compliance.ts",
+        "line": 1908
+      },
+      "methods": [
+        {
+          "abstract": true,
+          "docs": {
+            "stability": "experimental"
+          },
+          "locationInModule": {
+            "filename": "lib/compliance.ts",
+            "line": 1909
+          },
+          "name": "provide",
+          "returns": {
+            "type": {
+              "fqn": "jsii-calc.IAnonymouslyImplementMe"
+            }
+          }
+        }
+      ],
+      "name": "IAnonymousImplementationProvider"
+    },
+    "jsii-calc.IAnonymouslyImplementMe": {
+      "assembly": "jsii-calc",
+      "docs": {
+        "stability": "experimental"
+      },
+      "fqn": "jsii-calc.IAnonymouslyImplementMe",
+      "kind": "interface",
+      "locationInModule": {
+        "filename": "lib/compliance.ts",
+        "line": 1919
+      },
+      "methods": [
+        {
+          "abstract": true,
+          "docs": {
+            "stability": "experimental"
+          },
+          "locationInModule": {
+            "filename": "lib/compliance.ts",
+            "line": 1921
+          },
+          "name": "verb",
+          "returns": {
+            "type": {
+              "primitive": "string"
+            }
+          }
+        }
+      ],
+      "name": "IAnonymouslyImplementMe",
+      "properties": [
+        {
+          "abstract": true,
+          "docs": {
+            "stability": "experimental"
+          },
+          "locationInModule": {
+            "filename": "lib/compliance.ts",
+            "line": 1920
+          },
+          "name": "value",
+          "type": {
+            "primitive": "number"
+          }
+        }
+      ]
+    },
     "jsii-calc.IAnotherPublicInterface": {
       "assembly": "jsii-calc",
       "docs": {
@@ -9699,5 +9823,5 @@
     }
   },
   "version": "0.19.0",
-  "fingerprint": "KFKFyrJ+3+cfNKWPiNetngWkPFI32ydxfJyaR43/wDI="
+  "fingerprint": "OpxBghTomKWICGI/VfSJmwyqks2F/envRslx0JSxGyU="
 }

packages/jsii-dotnet-jsonmodel/src/Amazon.JSII.JsonModel/Api/ObjectReference.cs

@@ -3,7 +3,7 @@
 namespace Amazon.JSII.JsonModel.Api
 {
     [JsonDictionary]
-    public class ObjectReference : JsonDictionaryBase<string, string>
+    public class ObjectReference : JsonDictionaryBase<string, object>
     {
     }
 }

packages/jsii-dotnet-runtime-test/test/Amazon.JSII.Runtime.IntegrationTests/Amazon.JSII.Runtime.IntegrationTests.csproj

@@ -1,4 +1,4 @@
-<Project Sdk="Microsoft.NET.Sdk">
+<Project Sdk="Microsoft.NET.Sdk">
 
     <PropertyGroup>
         <TargetFramework>netcoreapp2.1</TargetFramework>

packages/jsii-dotnet-runtime-test/test/Amazon.JSII.Runtime.IntegrationTests/ComplianceTests.cs

@@ -11,12 +11,14 @@
 
 [assembly: CollectionBehavior(DisableTestParallelization = true)]
 
+#pragma warning disable CS0612
+
 namespace Amazon.JSII.Runtime.IntegrationTests
 {
     /// <summary>
     /// Ported from packages/jsii-java-runtime/src/test/java/org/jsii/testing/ComplianceTest.java.
     /// </summary>
-    public class ComplianceTests : IClassFixture<ServiceContainerFixture>
+    public sealed class ComplianceTests : IClassFixture<ServiceContainerFixture>
     {
         class RuntimeException : Exception
         {
@@ -127,12 +129,18 @@ public void DynamicTypes()
             Assert.Equal(UnixEpoch.AddSeconds(1234), types.AnyProperty);
 
             // json (notice that when deserialized, it is deserialized as a map).
-            types.AnyProperty = new JObject(new JProperty("Goo",
-                new JArray(
-                    "Hello",
-                    new JObject(new JProperty("World", 123))
-                )
-            ));
+            types.AnyProperty = new Dictionary<string, object>
+            {
+                { "Goo", new object[]
+                    {
+                        "Hello",
+                        new Dictionary<string, object>
+                        {
+                            { "World", 123 }
+                        }             
+                    }
+                }
+            };
             var @object = (IDictionary<string, object>) types.AnyProperty;
             var array = (object[]) @object["Goo"];
             var innerObject = (IDictionary<string, object>) array[1];

packages/jsii-dotnet-runtime-test/test/Amazon.JSII.Runtime.IntegrationTests/ServiceContainerFixture.cs

@@ -4,7 +4,7 @@
 
 namespace Amazon.JSII.Runtime.IntegrationTests
 {
-    public class ServiceContainerFixture : IDisposable
+    public sealed class ServiceContainerFixture : IDisposable
     {
         public ServiceContainerFixture()
         {

packages/jsii-dotnet-runtime-test/test/Amazon.JSII.Runtime.IntegrationTests/XUnitLogger.cs

@@ -4,7 +4,7 @@
 
 namespace Amazon.JSII.Runtime.IntegrationTests
 {
-    public class XUnitLoggerFactory : ILoggerFactory
+    public sealed class XUnitLoggerFactory : ILoggerFactory
     {
         readonly ITestOutputHelper _output;
 
@@ -27,7 +27,7 @@ public void Dispose()
         }
     }
 
-    public class XUnitLogger : ILogger, IDisposable
+    public sealed class XUnitLogger : ILogger, IDisposable
     {
         readonly ITestOutputHelper _output;
         readonly string _categoryName;

packages/jsii-dotnet-runtime/src/Amazon.JSII.Runtime.UnitTests/AssemblyInfo.cs

@@ -1,4 +1,8 @@
-using Xunit;
+using System.Runtime.CompilerServices;
+using Castle.Core.Internal;
+using Xunit;
 
 // Each test in this assembly overrides the the global service provider, so they can't be run in parallel.
 [assembly: CollectionBehavior(CollectionBehavior.CollectionPerAssembly)]
+
+[assembly:InternalsVisibleTo(InternalsVisible.ToDynamicProxyGenAssembly2)]