Fix Test Execution for Azurite

wsugarman commented 8 months ago

The PR includes a couple of quality-of-life improvements as well as fixes for running the current suite of tests with Azurite.

Fix tests with inline data to use DataTestMethod attribute
Use "retrieve" APIs instead of OData queries using the legacy Azure Storage SDK
Simplify local connection strings
Move all tests to the lowercase test folder

Below is the sample of YAML I used to run Azurite for the tests:

  - bash: |
     azurite --silent --location $(Agent.TempDirectory)/azurite &
     echo "##vso[task.setvariable variable=azuritePid]$!"
    displayName: 'Start Azurite'

  - task: VSTest@3
    displayName: 'Run unit and functional tests'
    inputs:
        # Some inputs ...
    env:
      DurableTaskTestStorageConnectionString: UseDevelopmentStorage=true

  - bash: kill $(azuritePid)
    displayName: 'Stop Azurite'
    condition: always()

cgillum commented 8 months ago

/azp run

azure-pipelines[bot] commented 8 months ago

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

nytian commented 7 months ago

Merged the PR into my own branch nytian/pipeline-test and then run in the CI. CI link All the tests are passed with a small change( downgrading msft.net.test.sdk to 15.8.*) so I will go ahead merge this PR. Thanks for the help and dedicated work, Will!

Azure / durabletask

Fix Test Execution for Azurite #1004