Update py_region_test for new behavior and make sure the test is run …

…in CI.

.travis.yml

@@ -119,6 +119,7 @@ script:
   # legacy binary tests
   - "${TRAVIS_BUILD_DIR}/build/release/bin/connections_performance_test"
   - "${TRAVIS_BUILD_DIR}/build/release/bin/cpp_region_test"
+  - "${TRAVIS_BUILD_DIR}/build/release/bin/py_region_test"
   - "${TRAVIS_BUILD_DIR}/build/release/bin/helloregion"
   - "${TRAVIS_BUILD_DIR}/build/release/bin/hello_sp_tp"
   - "${TRAVIS_BUILD_DIR}/build/release/bin/prototest"

ci/bamboo/build-osx.sh

@@ -24,6 +24,7 @@ mkdir -p build/scripts
 cmake -DCMAKE_INSTALL_PREFIX=`pwd`/build/release -DPY_EXTENSIONS_DIR=`pwd`/bindings/py/nupic/bindings .
 make install
 ./build/release/bin/cpp_region_test
+./build/release/bin/py_region_test
 ./build/release/bin/unit_tests
 
 # Build wheel

src/test/integration/PyRegionTest.cpp

@@ -286,26 +286,19 @@ void testSecondTimeLeak()
   n.addRegion("r2", "py.TestNode", "");
 }
 
-void testFailOnRegisterDuplicateRegion()
+void testRegionDuplicateRegister()
 {
-  bool caughtException = false;
   Network::registerPyRegion("nupic.regions.TestDuplicateNodes",
                             "TestDuplicateNodes");
   try
   {
     Network::registerPyRegion("nupic.regions.TestDuplicateNodes",
                               "TestDuplicateNodes");
   } catch (std::exception& e) {
-    NTA_DEBUG << "Caught exception as expected: '" << e.what() << "'";
-    caughtException = true;
-  }
-  if (caughtException)
-  {
-    NTA_DEBUG << "testFailOnRegisterDuplicateRegion passed";
-  } else {
-    NTA_THROW << "testFailOnRegisterDuplicateRegion did not "
-              << "throw an exception as expected";
+    NTA_THROW << "testRegionDuplicateRegister failed with exception: '"
+              << e.what() << "'";
   }
+  NTA_DEBUG << "testRegionDuplicateRegister passed";
 }
 
 void testCreationParamTypes()