in progress - all tests run

Author: szuperaz

projects/customizations-example/src/app/icon/icon.component.spec.ts

@@ -1,4 +1,5 @@
 import { ComponentFixture, TestBed } from '@angular/core/testing';
+import { ChangeDetectionStrategy } from '@angular/core';
 
 import { IconComponent } from './icon.component';
 
@@ -9,7 +10,11 @@ describe('IconComponent', () => {
   beforeEach(async () => {
     await TestBed.configureTestingModule({
       declarations: [IconComponent],
-    }).compileComponents();
+    })
+      .overrideComponent(IconComponent, {
+        set: { changeDetection: ChangeDetectionStrategy.Default },
+      })
+      .compileComponents();
   });
 
   beforeEach(() => {

projects/customizations-example/src/app/message-action/message-action.component.spec.ts

@@ -1,4 +1,5 @@
 import { ComponentFixture, TestBed } from '@angular/core/testing';
+import { ChangeDetectionStrategy } from '@angular/core';
 
 import { MessageActionComponent } from './message-action.component';
 
@@ -9,7 +10,11 @@ describe('MessageActionComponent', () => {
   beforeEach(async () => {
     await TestBed.configureTestingModule({
       declarations: [MessageActionComponent],
-    }).compileComponents();
+    })
+      .overrideComponent(MessageActionComponent, {
+        set: { changeDetection: ChangeDetectionStrategy.Default },
+      })
+      .compileComponents();
   });
 
   beforeEach(() => {

projects/customizations-example/src/app/thread-header/thread-header.component.spec.ts

@@ -1,4 +1,5 @@
 import { ComponentFixture, TestBed } from '@angular/core/testing';
+import { ChangeDetectionStrategy } from '@angular/core';
 
 import { ThreadHeaderComponent } from './thread-header.component';
 
@@ -9,7 +10,11 @@ describe('ThreadHeaderComponent', () => {
   beforeEach(async () => {
     await TestBed.configureTestingModule({
       declarations: [ThreadHeaderComponent],
-    }).compileComponents();
+    })
+      .overrideComponent(ThreadHeaderComponent, {
+        set: { changeDetection: ChangeDetectionStrategy.Default },
+      })
+      .compileComponents();
   });
 
   beforeEach(() => {

projects/stream-chat-angular/src/lib/attachment-preview-list/attachment-preview-list.component.spec.ts

@@ -5,6 +5,7 @@ import { AttachmentUpload, CustomAttachmentPreviewListContext } from '../types';
 import { AttachmentPreviewListComponent } from './attachment-preview-list.component';
 import {
   AfterViewInit,
+  ChangeDetectionStrategy,
   Component,
   TemplateRef,
   ViewChild,
@@ -25,7 +26,11 @@ describe('AttachmentPreviewListComponent', () => {
     attachmentUploads$ = new BehaviorSubject<AttachmentUpload[]>([]);
     await TestBed.configureTestingModule({
       declarations: [AttachmentPreviewListComponent],
-    }).compileComponents();
+    })
+      .overrideComponent(AttachmentPreviewListComponent, {
+        set: { changeDetection: ChangeDetectionStrategy.Default },
+      })
+      .compileComponents();
   });
 
   beforeEach(() => {

projects/stream-chat-angular/src/lib/avatar-placeholder/avatar-placeholder.component.spec.ts

@@ -4,6 +4,7 @@ import { Channel } from 'stream-chat';
 import { AvatarComponent } from '../avatar/avatar.component';
 
 import { AvatarPlaceholderComponent } from './avatar-placeholder.component';
+import { ChangeDetectionStrategy } from '@angular/core';
 
 describe('AvatarPlaceholderComponent', () => {
   let component: AvatarPlaceholderComponent;
@@ -12,7 +13,11 @@ describe('AvatarPlaceholderComponent', () => {
   beforeEach(async () => {
     await TestBed.configureTestingModule({
       declarations: [AvatarPlaceholderComponent, AvatarComponent],
-    }).compileComponents();
+    })
+      .overrideComponent(AvatarPlaceholderComponent, {
+        set: { changeDetection: ChangeDetectionStrategy.Default },
+      })
+      .compileComponents();
   });
 
   beforeEach(() => {

projects/stream-chat-angular/src/lib/avatar/avatar.component.spec.ts

@@ -1,4 +1,4 @@
-import { SimpleChange } from '@angular/core';
+import { ChangeDetectionStrategy, SimpleChange } from '@angular/core';
 import { ComponentFixture, TestBed } from '@angular/core/testing';
 import { Observable, Subject, of } from 'rxjs';
 import { ChatClientService, ClientEvent } from '../chat-client.service';
@@ -33,6 +33,10 @@ describe('AvatarComponent', () => {
       providers: [
         { provide: ChatClientService, useValue: chatClientServiceMock },
       ],
+    }).overrideComponent(AvatarComponent, {
+      set: {
+        changeDetection: ChangeDetectionStrategy.Default,
+      },
     });
     fixture = TestBed.createComponent(AvatarComponent);
     component = fixture.componentInstance;
@@ -206,7 +210,7 @@ describe('AvatarComponent', () => {
     expect(component.initials).toBe('#');
   });
 
-  it(`should display other user's image in 1:1 chats`, () => {
+  it(`should display other user's image in 1:1 chats`, async () => {
     const channel = generateMockChannels()[0];
     channel.state.members = {
       otheruser: {
@@ -230,6 +234,9 @@ describe('AvatarComponent', () => {
     expect(queryImg()?.src).toContain('url/to/img');
 
     component.imageUrl = 'channel/img';
+    component.ngOnChanges({
+      imageUrl: {} as SimpleChange,
+    });
     fixture.detectChanges();
 
     expect(queryImg()?.src).toContain('channel/img');

projects/stream-chat-angular/src/lib/avatar/avatar.component.ts

@@ -79,6 +79,11 @@ export class AvatarComponent
     if (changes['channel']) {
       this.updateIsOnlineSubscription();
     }
+
+    if (changes['imageUrl'] || changes['channel']) {
+      this.isError = false;
+    }
+
     if (changes.type || changes.name || changes.channel) {
       this.setInitials();
     }

projects/stream-chat-angular/src/lib/channel-header/channel-header.component.spec.ts

@@ -8,6 +8,7 @@ import { ChannelService } from '../channel.service';
 import { ChatClientService } from '../chat-client.service';
 import { StreamI18nService } from '../stream-i18n.service';
 import { ChannelHeaderComponent } from './channel-header.component';
+import { ChangeDetectionStrategy } from '@angular/core';
 
 describe('ChannelHeaderComponent', () => {
   let fixture: ComponentFixture<ChannelHeaderComponent>;
@@ -36,6 +37,8 @@ describe('ChannelHeaderComponent', () => {
         { provide: ChannelService, useValue: channelServiceMock },
         { provide: ChatClientService, useValue: chatClientServiceMock },
       ],
+    }).overrideComponent(ChannelHeaderComponent, {
+      set: { changeDetection: ChangeDetectionStrategy.Default },
     });
     TestBed.inject(StreamI18nService).setTranslation();
     fixture = TestBed.createComponent(ChannelHeaderComponent);

projects/stream-chat-angular/src/lib/channel-list/channel-list.component.spec.ts

@@ -14,6 +14,7 @@ import { ChannelListComponent } from './channel-list.component';
 import { Subject, of } from 'rxjs';
 import { PaginatedListComponent } from '../paginated-list/paginated-list.component';
 import { Channel } from 'stream-chat';
+import { ChangeDetectionStrategy } from '@angular/core';
 
 describe('ChannelListComponent', () => {
   let channelServiceMock: MockChannelService;
@@ -46,6 +47,8 @@ describe('ChannelListComponent', () => {
         },
         ThemeService,
       ],
+    }).overrideComponent(ChannelListComponent, {
+      set: { changeDetection: ChangeDetectionStrategy.Default },
     });
     fixture = TestBed.createComponent(ChannelListComponent);
     nativeElement = fixture.nativeElement as HTMLElement;

projects/stream-chat-angular/src/lib/channel-preview/channel-preview.component.spec.ts

@@ -17,6 +17,7 @@ import { Observable, Subject, of } from 'rxjs';
 import { DateParserService } from '../date-parser.service';
 import { IconModule } from '../icon/icon.module';
 import { IconPlaceholderComponent } from '../icon/icon-placeholder/icon-placeholder.component';
+import { ChangeDetectionStrategy } from '@angular/core';
 
 describe('ChannelPreviewComponent', () => {
   let fixture: ComponentFixture<ChannelPreviewComponent>;
@@ -52,6 +53,8 @@ describe('ChannelPreviewComponent', () => {
         { provide: ChannelService, useValue: channelServiceMock },
         { provide: ChatClientService, useValue: chatClientServiceMock },
       ],
+    }).overrideComponent(ChannelPreviewComponent, {
+      set: { changeDetection: ChangeDetectionStrategy.Default },
     });
     fixture = TestBed.createComponent(ChannelPreviewComponent);
     component = fixture.componentInstance;
@@ -218,6 +221,46 @@ describe('ChannelPreviewComponent', () => {
     expect(avatar.location).toBe('channel-preview');
   });
 
+  it('should update title, avatarImage, and avatarName when channel is updated', () => {
+    // Setup initial channel
+    const channel = generateMockChannels()[0];
+    component.channel = channel;
+    fixture.detectChanges();
+
+    // Verify initial values
+    expect(component.title).toBe(channel.data?.name);
+    expect(component.avatarImage).toBe(channel.data?.image);
+    expect(component.avatarName).toBe(channel.data?.name);
+
+    // Update channel data
+    const updatedName = 'Updated Channel Name';
+    const updatedImage = 'https://example.com/updated-image.jpg';
+
+    // Simulate channel update event
+    channel.data = {
+      ...channel.data,
+      name: updatedName,
+      image: updatedImage,
+    };
+
+    channel.handleEvent('channel.updated', {
+      channel: channel,
+    });
+
+    fixture.detectChanges();
+
+    // Verify updated values
+    expect(component.title).toBe(updatedName);
+    expect(component.avatarImage).toBe(updatedImage);
+    expect(component.avatarName).toBe(updatedName);
+
+    // Verify UI is updated
+    expect(queryTitle()?.textContent).toContain(updatedName);
+    const avatar = queryAvatar();
+    expect(avatar.imageUrl).toBe(updatedImage);
+    expect(avatar.name).toBe(updatedName);
+  });
+
   it('should display channel display text', () => {
     const channel = generateMockChannels()[0];
     channel.data!.name = undefined;