Remove unused vars from workflows

[t-young31]

Resolves #61

I hit the same issue, so thought I'd propose a solution (tested and works ok)

What is being addressed

Workflows include inputs that are unused. The checks and references to TEST_APP_ID, TEST_WORKSPACE_APP_ID, TEST_WORKSPACE_APP_SECRET and MS_TEAMS_WEBHOOK_URI are removed. Also uses the value of $ec rather than the literal string, which I think is a 🐞

[LizaShak]

LGTM. @marrobi I think that we should also create a PR in the main repo to remove the legacy use of it. In e2e_tests/helpers.py lines 117-120. WDYT? If you'd like I can do that.

[marrobi]

LGTM. @marrobi I think that we should also create a PR in the main repo to remove the legacy use of it. In e2e_tests/helpers.py lines 117-120. WDYT? If you'd like I can do that.

@LizaShak I'm not 100% what those lines are for? If not used anywhere then they should go...

[tamirkamara]

@marrobi @LizaShak This PR is likely to overwritten when the next release happens as we only compare in the TRE->Deployment direction.

[marrobi]

@tamirkamara so what are you suggesting? Should these variables made optional in the microsoft/AzureTRE repository instead? If can provide some guidance then maybe we can work to resolve.

[jonnyry]

$ec bug described in microsoft/AzureTRE#3778