Prevent beeping when we run the receipt tests locally

[andybalaam]

Not sure whether this is the best way to do this, and/or whether we want to do it for all tests, but I couldn't find a better way to quieten the beeps, which have been driving me over the edge this week...

Relevant Cypress issue is: cypress-io/cypress#27148 - looks like they are not interested in fixing this in Cypress.

It's hard to do it in a more general way with this approach, because the test needs to be underway before we can overwrite the playAudioNotification method, because before the test starts it doesn't exist yet.

Part of element-hq/element-web#25449

---

This change is marked as an internal change (Task), so will not be included in the changelog.

[richvdh]

looks plausible to me, though cypress-io/cypress#27148 (comment) also looks like a compelling solution if it works

[t3chguy]

@richvdh it does but only on Chromium, didn't work on Electron for me and doesn't even attempt to work on Firefox

[t3chguy]

(I had a patch to do this locally as it also drove me mad)

[richvdh]

@richvdh it does but only on Chromium, didn't work on Electron for me and doesn't even attempt to work on Firefox

Ok, but don't we run the cypress tests in Chrome? I didn't know that running them in electron was even an option

[t3chguy]

@richvdh in CI yes, but sound in CI isn't likely to bother someone. test:cypress defaults to Electron. If you open headed mode test:cypress:open then it asks you how to run it.

[andybalaam]

@richvdh it does but only on Chromium, didn't work on Electron for me and doesn't even attempt to work on Firefox

Just to double check - does that mean you are OK with merging this PR @t3chguy ?

[t3chguy]

Just to double check - does that mean you are OK with merging this PR @t3chguy ?

I think it should be opt-in behaviour personally, at least on the headed cypress environment, given there you can use cy.pause() and interact with things manually so you'd expect it to behave as close to normal