Room call push rule should be an underride, not an override.

[clokep]

#3181 added some tests which broke when #3179 merged in develop; I'm not really sure why that broke things, but it seems to be something about overriding the matrixClient.pushRules.

Anyway, consistently using this push rule as an underride rule, which is how it is already defined (and how MSC3914 defines it) fixes it:

matrix-js-sdk/src/pushprocessor.ts

Lines 92 to 110 in 933a0c9

	const DEFAULT_UNDERRIDE_RULES: IPushRule[] = [
	{
	// For homeservers which don't support MSC3914 yet
	rule_id: ".org.matrix.msc3914.rule.room.call",
	default: true,
	enabled: true,
	conditions: [
	{
	kind: ConditionKind.EventMatch,
	key: "type",
	pattern: "org.matrix.msc3401.call",
	},
	{
	kind: ConditionKind.CallStarted,
	},
	],
	actions: [PushRuleActionName.Notify, { set_tweak: TweakName.Sound, value: "default" }],
	},
	];

Checklist

• Tests written for new code (and old code if feasible)
• Linter and other CI checks pass
• Sign-off given on the changes (see CONTRIBUTING.md)

---

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