Skip legacy B2C local-account Todo UI test in WebAppUiTests

[Copilot]

The B2C local-account Todo E2E test depends on deprecated legacy infra (webappsapistests Key Vault / fabrikamb2c tenant). This PR disables only that test using the project’s existing xUnit skip pattern.

• Targeted change

  • Updated a single attribute on Susi_B2C_LocalAccount_TodoAppFunctionsCorrectlyAsync in tests/E2E Tests/WebAppUiTests/B2CWebAppCallsWebApiLocally.cs.
  • No test logic, setup, or other files were modified.

• Skip reason alignment

  • Uses an explicit infrastructure deprecation reason, consistent with existing skip usage in this test project.

- [Fact]
+ [Fact(Skip = "Legacy B2C test infrastructure (webappsapistests Key Vault / fabrikamb2c tenant) is no longer valid.")]

Original prompt

In tests/E2E Tests/WebAppUiTests/B2CWebAppCallsWebApiLocally.cs, disable the single test method Susi_B2C_LocalAccount_TodoAppFunctionsCorrectlyAsync by changing its [Fact] attribute to include a Skip reason.

The change should be on line 42 only:

-    [Fact]
+    [Fact(Skip = "Legacy B2C test infrastructure (webappsapistests Key Vault / fabrikamb2c tenant) is no longer valid.")]

This follows the existing skip pattern already used in the same test project (e.g., TestingWebAppLocally.cs line 48 and WebAppCallsApiCallsGraphLocally.cs line 186).

Do NOT modify anything else — no other files, no other lines, no deletions. Only this single attribute change.

The following is the prior conversation context from the user's chat exploration (may be truncated):

User: Please search the repo for all tests that depend on the legacy B2C test setup backed by Key Vault webappsapistests.

I already confirmed one example:

• Susi_B2C_LocalAccount_TodoAppFunctionsCorrectlyAsync

I need the full list of tests that use any of these:

• Key Vault URI: https://webappsapistests.vault.azure.net

• secret names:

  • IdWeb-B2C-user
  • IdWeb-B2C-password
  • IdWeb-B2C-Client-ClientSecret

Please return:

1. all matching test method names
2. file paths
3. whether they are UI/E2E/integration tests
4. any shared helper/base class that centralizes this setup
5. any appsettings or config files tied to the same B2C setup

Also check for equivalent patterns, not just exact string matches, such as:

• constants like KeyvaultEmailName, KeyvaultPasswordName, KeyvaultClientSecretName
• _keyvaultUri
• B2C local account Todo app test flows
• any references to test@id4s.authsdk.org

Goal: identify every test that should be reviewed or potentially disabled if this old B2C infra is no longer valid.

Assistant:

I'll conduct a thorough search across the repository to find all tests and configuration tied to the legacy B2C test infrastructure. Let me search for each indicator systematically.

Assistant: Now let me check for any other test files that might reference these B2C patterns and the UiTestHelpers shared class:

Assistant: Now let me also check the DevApps config files for the B2C setup:

Assistant: Now I have a comprehensive picture. Let me verify there are no other test files referencing this B2C Key Vault setup that I might have missed:

Assistant: Now I have a comprehensive picture. Let me verify no other test files reference webappsapistests or the B2C keyvault secrets by checking one more pattern:

Assistant: Based on the thorough search across the repository, here is the complete inventory of everything tied to the legacy B2C test setup backed by Key Vault webappsapistests.

---

1. Matching Test Methods

There is exactly one test method that directly depends on the webappsapistests Key Vault and the B2C secrets:

#	Test Method	File	Type
1	Susi_B2C_LocalAccount_TodoAppFunctionsCorrectlyAsync	tests/E2E Tests/WebAppUiTests/B2CWebAppCallsWebApiLocally.cs	UI / E2E test (Playwright-based, [Fact], Windows-only)

No other test class or test method in the repository references webappsapistests.vault.azure.net, IdWeb-B2C-user, IdWeb-B2C-password, or IdWeb-B2C-Client-ClientSecret. The string test@id4s.authsdk.org also has zero matches in the repository.

---

2. File Paths (Full Blast Radius)

Test file

    private const string KeyvaultEmailName = "IdWeb-B2C-user";
    private const string KeyvaultPasswordName = "IdWeb-B2C-password";
    private const string KeyvaultClientSecretName = "IdWeb-B2C-Client-ClientSecret";
    private const string NameOfUser = "unknown";
    private const uint TodoListClientPort = 5000;
    private const uint TodoListServicePort = 44332;
    private const string TraceClassName = "B2CWebAppCallsWebApiLocally";
    private readonly LocatorAssertionsToBeVisibleOptions _assertVisibleOptions = new() { Timeout = 25000 };
    private readonly string _devAppPath = Path.Join("DevApps", "B2CWebAppCallsWebApi");
    private readonly Uri _keyvaultUri = new("https://webappsapistests.vault.azure.net");

DevApp (SUT) launched by the test

File	Purpose
tests/DevApps/B2CWebAppCallsWebApi/Client/appsettings.json	B2C client config (fabrikamb2c tenant, client ID fdb91ff5-...)
tests/DevApps/B2CWebAppCallsWebApi/TodoListService/appsettings.json	B2C service config (fabrikamb2c tenant, client ID 90c0fe63-...)
tests/DevApps/B2CWebAppCallsWebApi/Client/Startup.cs	Client app startup (reads AzureAdB2C section)
tests/DevApps/B2CWebAppCallsWebApi/Client/TodoListClient.csproj	Client project
`tests/DevApps/B2CWebAp...

This pull request was created from Copilot chat.

[Avery-Dunn]

Approved, but you should make a Github issue to track re-enabling skipped tests (if there isn't one already)