Skip to content

Commit 44be756

Browse files
committed
fix: skip poor call quality warning tests
1 parent 1771468 commit 44be756

1 file changed

Lines changed: 9 additions & 5 deletions

File tree

apps/webapp/src/script/repositories/calling/CallingRepository.test.ts

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -519,19 +519,21 @@ describe('CallingRepository', () => {
519519
spyOn(Warnings, 'hideWarning');
520520
});
521521

522-
it('shows poor call quality warning when parsed quality is medium', () => {
522+
// skipping test for now. Once we have correct stats about network
523+
// quality then we will enable the feature and tests again.
524+
it.skip('shows poor call quality warning when parsed quality is medium', () => {
523525
callingRepository['updateCallQuality'](conversationId, userId, remoteClientId, qualityInfo(QUALITY.MEDIUM));
524526

525527
expect(Warnings.showWarning).toHaveBeenCalledWith(Warnings.TYPE.CALL_QUALITY_POOR);
526528
});
527529

528-
it('shows poor call quality warning when parsed quality is poor', () => {
530+
it.skip('shows poor call quality warning when parsed quality is poor', () => {
529531
callingRepository['updateCallQuality'](conversationId, userId, remoteClientId, qualityInfo(QUALITY.POOR));
530532

531533
expect(Warnings.showWarning).toHaveBeenCalledWith(Warnings.TYPE.CALL_QUALITY_POOR);
532534
});
533535

534-
it('shows poor call quality warning when parsed quality is network problem', () => {
536+
it.skip('shows poor call quality warning when parsed quality is network problem', () => {
535537
callingRepository['updateCallQuality'](
536538
conversationId,
537539
userId,
@@ -542,7 +544,7 @@ describe('CallingRepository', () => {
542544
expect(Warnings.showWarning).toHaveBeenCalledWith(Warnings.TYPE.CALL_QUALITY_POOR);
543545
});
544546

545-
it('shows poor call quality warning when parsed quality is reconnecting', () => {
547+
it.skip('shows poor call quality warning when parsed quality is reconnecting', () => {
546548
callingRepository['updateCallQuality'](conversationId, userId, remoteClientId, qualityInfo(QUALITY.RECONNECTING));
547549

548550
expect(Warnings.showWarning).toHaveBeenCalledWith(Warnings.TYPE.CALL_QUALITY_POOR);
@@ -586,7 +588,9 @@ describe('CallingRepository', () => {
586588
expect(Warnings.hideWarning).not.toHaveBeenCalled();
587589
});
588590

589-
it('handles partially missing fields in qualityInfo JSON', () => {
591+
// skipping test for now. Once we have correct stats about network
592+
// quality then we will enable the feature and tests again.
593+
it.skip('handles partially missing fields in qualityInfo JSON', () => {
590594
const json = JSON.stringify({
591595
quality: QUALITY.POOR,
592596
// missing jitter, connection, peer etc.

0 commit comments

Comments
 (0)