chore: rename tests add derived interface test (#1816)

* chore: rename tests add derived interface test * chore: added comment --------- Co-authored-by: Chris Pulman <[email protected]>
reactiveui · Sep 17, 2024 · d63d98d · d63d98d
1 parent fd48771
commit d63d98d
Show file tree

Hide file tree

Showing 6 changed files with 74 additions and 7 deletions.
diff --git a/Refit.GeneratorTests/InterfaceTests.cs b/Refit.GeneratorTests/InterfaceTests.cs
@@ -19,7 +19,7 @@ public interface IContainedInterface
     }
 
     [Fact]
-    public Task DerivedRefitInterfaceTest()
+    public Task RefitInterfaceDerivedFromRefitBaseTest()
     {
         return Fixture.VerifyForType(
             """
@@ -38,8 +38,9 @@ public interface IBaseInterface
     }
 
     [Fact]
-    public Task DerivedNonRefitInterfaceTest()
+    public Task RefitInterfaceDerivedFromBaseTest()
     {
+        // this currently generates invalid code see issue #1801 for more information
         return Fixture.VerifyForType(
             """
             public interface IGeneratedInterface : IBaseInterface
@@ -55,6 +56,21 @@ public interface IBaseInterface
             """);
     }
 
+    [Fact]
+    public Task InterfaceDerivedFromRefitBaseTest()
+    {
+        return Fixture.VerifyForType(
+            """
+            public interface IBaseInterface
+            {
+                [Get("/users")]
+                Task<string> Get();
+            }
+
+            public interface IDerivedInterface : IBaseInterface { }
+            """);
+    }
+
     [Fact]
     public Task NestedNamespaceTest()
     {

diff --git a/....DotNet6_0#IGeneratedClient.g.received.cs → ...efitBaseTest#IBaseInterface.g.verified.cs b/....DotNet6_0#IGeneratedClient.g.received.cs → ...efitBaseTest#IBaseInterface.g.verified.cs
@@ -1,4 +1,4 @@
-//HintName: IGeneratedClient.g.cs
+//HintName: IBaseInterface.g.cs
 #nullable disable
 #pragma warning disable
 namespace Refit.Implementation
@@ -13,16 +13,16 @@ partial class Generated
     [global::RefitInternalGenerated.PreserveAttribute]
     [global::System.Reflection.Obfuscation(Exclude=true)]
     [global::System.ComponentModel.EditorBrowsable(global::System.ComponentModel.EditorBrowsableState.Never)]
-    partial class RefitGeneratorTestIGeneratedClient
-        : global::RefitGeneratorTest.IGeneratedClient
+    partial class RefitGeneratorTestIBaseInterface
+        : global::RefitGeneratorTest.IBaseInterface
 
     {
         /// <inheritdoc />
         public global::System.Net.Http.HttpClient Client { get; }
         readonly global::Refit.IRequestBuilder requestBuilder;
 
         /// <inheritdoc />
-        public RefitGeneratorTestIGeneratedClient(global::System.Net.Http.HttpClient client, global::Refit.IRequestBuilder requestBuilder)
+        public RefitGeneratorTestIBaseInterface(global::System.Net.Http.HttpClient client, global::Refit.IRequestBuilder requestBuilder)
         {
             Client = client;
             this.requestBuilder = requestBuilder;
@@ -46,7 +46,7 @@ public RefitGeneratorTestIGeneratedClient(global::System.Net.Http.HttpClient cli
         }
 
         /// <inheritdoc />
-        async global::System.Threading.Tasks.Task<string> global::RefitGeneratorTest.IGeneratedClient.Get()
+        async global::System.Threading.Tasks.Task<string> global::RefitGeneratorTest.IBaseInterface.Get()
         {
             var ______arguments = global::System.Array.Empty<object>();
             var ______func = requestBuilder.BuildRestResultFuncForMethod("Get", global::System.Array.Empty<global::System.Type>() );

diff --git a/...napshots/InterfaceTests.InterfaceDerivedFromRefitBaseTest#IDerivedInterface.g.verified.cs b/...napshots/InterfaceTests.InterfaceDerivedFromRefitBaseTest#IDerivedInterface.g.verified.cs
@@ -0,0 +1,51 @@
+//HintName: IDerivedInterface.g.cs
+#nullable disable
+#pragma warning disable
+namespace Refit.Implementation
+{
+
+    partial class Generated
+    {
+
+    /// <inheritdoc />
+    [global::System.Diagnostics.CodeAnalysis.ExcludeFromCodeCoverage]
+    [global::System.Diagnostics.DebuggerNonUserCode]
+    [global::RefitInternalGenerated.PreserveAttribute]
+    [global::System.Reflection.Obfuscation(Exclude=true)]
+    [global::System.ComponentModel.EditorBrowsable(global::System.ComponentModel.EditorBrowsableState.Never)]
+    partial class RefitGeneratorTestIDerivedInterface
+        : global::RefitGeneratorTest.IDerivedInterface
+
+    {
+        /// <inheritdoc />
+        public global::System.Net.Http.HttpClient Client { get; }
+        readonly global::Refit.IRequestBuilder requestBuilder;
+
+        /// <inheritdoc />
+        public RefitGeneratorTestIDerivedInterface(global::System.Net.Http.HttpClient client, global::Refit.IRequestBuilder requestBuilder)
+        {
+            Client = client;
+            this.requestBuilder = requestBuilder;
+        }
+
+
+
+        /// <inheritdoc />
+        async global::System.Threading.Tasks.Task<string> global::RefitGeneratorTest.IBaseInterface.Get()
+        {
+            var ______arguments = global::System.Array.Empty<object>();
+            var ______func = requestBuilder.BuildRestResultFuncForMethod("Get", global::System.Array.Empty<global::System.Type>() );
+            try
+            {
+                return await ((global::System.Threading.Tasks.Task<string>)______func(this.Client, ______arguments)).ConfigureAwait(false);
+            }
+            catch (global::System.Exception ______ex)
+            {
+                throw ______ex;
+            }
+        }
+    }
+    }
+}
+
+#pragma warning restore
diff --git a/...aceTest#IGeneratedInterface.g.verified.cs → ...aseTest#IGeneratedInterface.g.verified.cs b/...aceTest#IGeneratedInterface.g.verified.cs → ...aseTest#IGeneratedInterface.g.verified.cs
diff --git a/...nterfaceTest#IBaseInterface.g.verified.cs → ...efitBaseTest#IBaseInterface.g.verified.cs b/...nterfaceTest#IBaseInterface.g.verified.cs → ...efitBaseTest#IBaseInterface.g.verified.cs
diff --git a/...aceTest#IGeneratedInterface.g.verified.cs → ...aseTest#IGeneratedInterface.g.verified.cs b/...aceTest#IGeneratedInterface.g.verified.cs → ...aseTest#IGeneratedInterface.g.verified.cs