Merge pull request #3822 from microsoft/lamil/fixTests

Lamil/fix tests

samples/csharp/assistants/hospitality-assistant/VirtualAssistantSample/Adapters/DefaultAdapter.cs

@@ -68,7 +68,6 @@ namespace VirtualAssistantSample.Adapters
             Use(new EventDebuggerMiddleware());
             Use(new SetSpeakMiddleware());
             Use(new ProactiveStateMiddleware(proactiveState));
-            Use(new FeedbackMiddleware(conversationState, telemetryClient, new FeedbackOptions()));
         }
 
         private async Task HandleTurnErrorAsync(ITurnContext turnContext, Exception exception)

sdk/csharp/Directory.Build.props

@@ -36,10 +36,7 @@
     <DebugType>full</DebugType>
     <DebugSymbols>true</DebugSymbols>
   </PropertyGroup>
-  
-  <PropertyGroup>
-    <PackageIcon>icon.png</PackageIcon>
-  </PropertyGroup>
+
   <ItemGroup>
     <None Include="images\icon.png" Pack="true" PackagePath="\"/>
   </ItemGroup>
@@ -47,9 +44,7 @@
   <PropertyGroup>
     <!-- Rules found at: https://aka.ms/Microsoft-NuGet-Compliance -->
     <PackageProjectUrl>https://github.com/Microsoft/botframework-solutions</PackageProjectUrl>
- 
-<PackageLicenseUrl>https://raw.githubusercontent.com/microsoft/botframework-solutions/master/LICENSE</PackageLicenseUrl>
-    <PackageIconUrl>https://raw.githubusercontent.com/microsoft/botframework-sdk/main/icon.png</PackageIconUrl>
+    <PackageIconUrl>https://raw.githubusercontent.com/microsoft/botframework-sdk/master/icon.png</PackageIconUrl>
     <PackageLicenseUrl>https://raw.githubusercontent.com/microsoft/botframework-solutions/master/LICENSE</PackageLicenseUrl>
     <PackageRequireLicenseAcceptance>true</PackageRequireLicenseAcceptance>
     <PackageIcon>icon.png</PackageIcon>
@@ -65,8 +60,4 @@
     <NoWarn>$(NoWarn);NU5125;NU1701;8002</NoWarn>
   </PropertyGroup>
   
-  <ItemGroup Condition="'$(GeneratePackageOnBuild)' == 'True'">
-    <None Include="icon.png" Pack="true" PackagePath="\" />
-  </ItemGroup>
-  
 </Project>