docker test fix (#1184)

* Docker tests fixes

* Updated dedicated github workflow

* Updated consumption workflow

* Reverted change to docker tests

* Removed setup webhost from consumption docker tests

* Removing the index url from the workflows

---------

Co-authored-by: Gavin Aguiar <gavin@GavinPC>

Author: gavin-aguiar

.github/workflows/ci_consumption_workflow.yml

@@ -1,7 +1,7 @@
 # This workflow will run all tests in tests/consumption_tests in Docker using a consumption image
 # For more information see: https://help.github.com/actions/language-and-framework-guides/using-python-with-github-actions
 
-name: CI Consumption tests
+name: CI Consumption E2E tests
 
 on:
   workflow_dispatch:
@@ -28,10 +28,7 @@ jobs:
           python-version: ${{ matrix.python-version }}
       - name: Install dependencies
         run: |
-          python -m pip install --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple -U -e .[dev]
-          python setup.py build
-          python setup.py webhost --branch-name=dev
-          python setup.py extension
+          python -m pip install -U -e .[dev]
       - name: Running 3.7 Tests
         if: matrix.python-version == 3.7
         env:

.github/workflows/ci_docker_con_workflow.yml

@@ -29,10 +29,7 @@ jobs:
           python-version: ${{ matrix.python-version }}
       - name: Install dependencies
         run: |
-          python -m pip install --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple -U -e .[dev]
-          python setup.py build
-          python setup.py webhost --branch-name=dev
-          python setup.py extension
+          python -m pip install -U -e .[dev]
       - name: Running 3.7 Tests
         if: matrix.python-version == 3.7
         env:

.github/workflows/ci_docker_ded_workflow.yml

@@ -30,10 +30,7 @@ jobs:
           python-version: ${{ matrix.python-version }}
       - name: Install dependencies
         run: |
-          python -m pip install --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple -U -e .[dev]
-          python setup.py build
-          python setup.py webhost --branch-name=dev
-          python setup.py extension
+          python -m pip install -U -e .[dev]
       - name: Running 3.7 Tests
         if: matrix.python-version == 3.7
         env:

tests/utils/constants.py

@@ -16,5 +16,5 @@
 PYAZURE_WEBHOST_DEBUG = "PYAZURE_WEBHOST_DEBUG"
 
 # CI test constants
-CONSUMPTION_DOCKER_TEST = "false"
-DEDICATED_DOCKER_TEST = "false"
+CONSUMPTION_DOCKER_TEST = "CONSUMPTION_DOCKER_TEST"
+DEDICATED_DOCKER_TEST = "DEDICATED_DOCKER_TEST"

tests/utils/testutils.py

@@ -219,13 +219,13 @@ def setUpClass(cls):
         else:
             cls.host_stdout = tempfile.NamedTemporaryFile('w+t')
 
-        _setup_func_app(TESTS_ROOT / script_dir)
         try:
             if is_envvar_true(CONSUMPTION_DOCKER_TEST):
                 cls.webhost = WebHostConsumption(script_dir).spawn_container()
             elif is_envvar_true(DEDICATED_DOCKER_TEST):
                 cls.webhost = WebHostDedicated(script_dir).spawn_container()
             else:
+                _setup_func_app(TESTS_ROOT / script_dir)
                 cls.webhost = start_webhost(script_dir=script_dir,
                                             stdout=cls.host_stdout)
         except Exception: