Skip to content

Commit

Permalink
azure-client: enable read-client tests for Tinylicious (#21935)
Browse files Browse the repository at this point in the history
## Description

#20914 disabled a few Azure E2E tests because they were timing out in
T9s due to a T9s bug that was fixed in #20312. That bug has been patched
into T9s v4 and was also released in T9s v5. We can enable those tests
now.

Validated locally that these pass by running `pnpm test`
  • Loading branch information
znewton authored Jul 17, 2024
1 parent d0fd3f5 commit b88041f
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 18 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -213,12 +213,6 @@ for (const testOpts of testMatrix) {
* to resolve original member, and the original member should be able to observe the read-only member.
*/
it("can find read-only partner member", async function () {
// TODO: Fix tests when ran against local service - ADO:7876
const useAzure = process.env.FLUID_CLIENT === "azure";
if (!useAzure) {
this.skip();
}

let containerId: string;
let container: IFluidContainer;
let services: AzureContainerServices;
Expand Down Expand Up @@ -329,12 +323,6 @@ for (const testOpts of testMatrix) {
* the original read-only partner should observe memberAdded event and have correct partner count.
*/
it("can observe member leaving and joining in read-only mode", async function () {
// TODO: Fix tests when ran against local service - ADO:7876
const useAzure = process.env.FLUID_CLIENT === "azure";
if (!useAzure) {
this.skip();
}

let containerId: string;
let container: IFluidContainer;
if (isEphemeral) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -189,12 +189,6 @@ for (const testOpts of testMatrix) {
* a signal sent by any 1 client should be recieved by all 3 clients, regardless of read/write permissions.
*/
it("can send and receive read-only client signals", async function () {
// TODO: Fix tests when ran against local service - ADO:7876
const useAzure = process.env.FLUID_CLIENT === "azure";
if (!useAzure) {
this.skip();
}

const { signaler: writeSignaler, containerId } = await getOrCreateSignalerContainer(
undefined,
user1,
Expand Down

0 comments on commit b88041f

Please sign in to comment.