review changes part1

Author: dipinknair

tests/conftest.py

@@ -177,7 +177,7 @@ def mke_app_reset(request):
 _CHECK_PROCESS_RETURN_CODE = os.name == "nt"
 
 # set to true if you want to see all the subprocess stdout/stderr
-_PRINT_SUBPROCESS_OUTPUT_TO_CONSOLE = False
+_PRINT_SUBPROCESS_OUTPUT_TO_CONSOLE = True
 
 
 @pytest.fixture()

tests/embedding/test_app.py

@@ -263,7 +263,6 @@ def test_normal_appdata(pytestconfig, run_subprocess, rootdir):
 
 
 @pytest.mark.embedding_scripts
-@pytest.mark.python_env
 def test_building_gallery(pytestconfig, run_subprocess, rootdir):
     """Test for building gallery check.
 
@@ -284,7 +283,7 @@ def test_building_gallery(pytestconfig, run_subprocess, rootdir):
     stdout, _ = run_subprocess([sys.executable, embedded_gallery_py, version, "True"])
     stdout = stdout.decode()
 
-    # Assert Exception
+    # Assert stdout afte launching multiple app
     assert "Multiple App launched with building gallery flag on" in stdout
 
 

tests/scripts/build_gallery_test.py

@@ -43,12 +43,9 @@ def launch_app(version):
 
         pymechanical.BUILDING_GALLERY = True
 
-        try:
-            app2 = launch_app(version)
-            app3 = launch_app(version)
-            print("Multiple App launched with building gallery flag on")
-        except Exception as e:
-            print(e)
+        app2 = launch_app(version)
+        app3 = launch_app(version)
+        print("Multiple App launched with building gallery flag on")
 
     elif build_gallery_flag == "False":
         try: