[mapguide-commits] r9753 - sandbox/jng/flatten_exceptions/Server/src/UnitTesting

[svn_mapguide at osgeo.org]

Author: jng
Date: 2020-10-29 02:37:14 -0700 (Thu, 29 Oct 2020)
New Revision: 9753

Modified:
   sandbox/jng/flatten_exceptions/Server/src/UnitTesting/TestSiteService.cpp
Log:
Disable site manager tests wholesale until further notice

Modified: sandbox/jng/flatten_exceptions/Server/src/UnitTesting/TestSiteService.cpp
===================================================================
--- sandbox/jng/flatten_exceptions/Server/src/UnitTesting/TestSiteService.cpp	2020-10-29 08:28:44 UTC (rev 9752)
+++ sandbox/jng/flatten_exceptions/Server/src/UnitTesting/TestSiteService.cpp	2020-10-29 09:37:14 UTC (rev 9753)
@@ -863,13 +863,10 @@
 ///
 /// This test case adds a new server
 ///----------------------------------------------------------------------------
-#ifdef _WIN32
-//FIXME: On Windows, it appears ACE is actually validating the hostname/IP given
+
+//FIXME: It appears ACE is actually validating the hostname/IP given
 //so we can't put in fake hostnames/IPs for the purpose of exercising this API
 TEST_CASE("AddServer", "[Broken]")
-#else
-TEST_CASE("AddServer", "[SiteService]")
-#endif
 {
     //TODO: does not throw exceptions described in MgServerSiteService.cpp
     try
@@ -970,13 +967,10 @@
 ///
 /// This test case updates the description of the server that was added
 ///----------------------------------------------------------------------------
-#ifdef _WIN32
-//FIXME: On Windows, it appears ACE is actually validating the hostname/IP given
+
+//FIXME: It appears ACE is actually validating the hostname/IP given
 //so we can't put in fake hostnames/IPs for the purpose of exercising this API
 TEST_CASE("UpdateServer", "[Broken]")
-#else
-TEST_CASE("UpdateServer", "[SiteService]")
-#endif
 {
     //TODO: does not throw exceptions described in MgServerSiteService.cpp
     try
@@ -1038,13 +1032,10 @@
 ///
 /// This test case removes the server that was added before
 ///----------------------------------------------------------------------------
-#ifdef _WIN32
-//FIXME: On Windows, it appears ACE is actually validating the hostname/IP given
+
+//FIXME: It appears ACE is actually validating the hostname/IP given
 //so we can't put in fake hostnames/IPs for the purpose of exercising this API
 TEST_CASE("RemoveServer", "[Broken]")
-#else
-TEST_CASE("RemoveServer", "[SiteService]")
-#endif
 {
     //TODO: does not throw exceptions described in MgServerSiteService.cpp
     try