From edd1fe9a05770a78fb7e49d23bb41c0c7422bb70 Mon Sep 17 00:00:00 2001
From: carlitosan <82187351+carlitosan@users.noreply.github.com>
Date: Wed, 8 Dec 2021 16:31:08 -0800
Subject: [PATCH] remove testing code

Signed-off-by: carlitosan <82187351+carlitosan@users.noreply.github.com>
---
 .../Code/Editor/View/Windows/Tools/UpgradeTool/Controller.cpp  | 2 ++
 .../Code/Editor/View/Windows/Tools/UpgradeTool/Modifier.cpp    | 3 +--
 .../Code/Editor/View/Windows/Tools/UpgradeTool/Scanner.cpp     | 2 +-
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/Gems/ScriptCanvas/Code/Editor/View/Windows/Tools/UpgradeTool/Controller.cpp b/Gems/ScriptCanvas/Code/Editor/View/Windows/Tools/UpgradeTool/Controller.cpp
index f14bc0f965..b666ef32f5 100644
--- a/Gems/ScriptCanvas/Code/Editor/View/Windows/Tools/UpgradeTool/Controller.cpp
+++ b/Gems/ScriptCanvas/Code/Editor/View/Windows/Tools/UpgradeTool/Controller.cpp
@@ -216,6 +216,8 @@ namespace ScriptCanvasEditor
             else
             {
                 VE_LOG("Failed to modify %s: %s", result.asset.Path().c_str(), result.errorMessage.data());
+                AZ_Warning(ScriptCanvas::k_VersionExplorerWindow.data()
+                    , false, "Failed to modify %s: %s", result.asset.Path().c_str(), result.errorMessage.data());
             }
 
             for (auto* item : FindTableItems(info))
diff --git a/Gems/ScriptCanvas/Code/Editor/View/Windows/Tools/UpgradeTool/Modifier.cpp b/Gems/ScriptCanvas/Code/Editor/View/Windows/Tools/UpgradeTool/Modifier.cpp
index e46f0ada24..a5515248e5 100644
--- a/Gems/ScriptCanvas/Code/Editor/View/Windows/Tools/UpgradeTool/Modifier.cpp
+++ b/Gems/ScriptCanvas/Code/Editor/View/Windows/Tools/UpgradeTool/Modifier.cpp
@@ -128,8 +128,7 @@ namespace ScriptCanvasEditor
         void Modifier::AssetCompilationSuccess([[maybe_unused]] const AZStd::string& assetPath)
         {
             AZStd::lock_guard<AZStd::recursive_mutex> lock(m_mutex);
-            // test failure path m_successNotifications.insert(assetPath);
-            m_failureNotifications.insert(assetPath);
+            m_successNotifications.insert(assetPath);
         }
 
         void Modifier::AssetCompilationFailed(const AZStd::string& assetPath)
diff --git a/Gems/ScriptCanvas/Code/Editor/View/Windows/Tools/UpgradeTool/Scanner.cpp b/Gems/ScriptCanvas/Code/Editor/View/Windows/Tools/UpgradeTool/Scanner.cpp
index c1551af9f9..061025faf5 100644
--- a/Gems/ScriptCanvas/Code/Editor/View/Windows/Tools/UpgradeTool/Scanner.cpp
+++ b/Gems/ScriptCanvas/Code/Editor/View/Windows/Tools/UpgradeTool/Scanner.cpp
@@ -83,7 +83,7 @@ namespace ScriptCanvasEditor
         {
             AZStd::string name = asset.Path().c_str();
 
-            if ((m_config.filter && m_config.filter(asset) == ScanConfiguration::Filter::Exclude) || !name.contains("LY_SC_UnitTest_FunctionContainer"))
+            if (m_config.filter && m_config.filter(asset) == ScanConfiguration::Filter::Exclude)
             {
                 VE_LOG("Scanner: Excluded: %s ", ModCurrentAsset().Path().c_str());
                 m_result.m_filteredAssets.push_back(ModCurrentAsset().Describe());