Skip to content

Commit 578d4ef

Browse files
authored
refactor: remove missing asObservable usages (#20357)
1 parent 1f6b90c commit 578d4ef

File tree

2 files changed

+10
-12
lines changed

2 files changed

+10
-12
lines changed

src/material-experimental/mdc-autocomplete/autocomplete.spec.ts

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,12 @@ import {Overlay, OverlayContainer} from '@angular/cdk/overlay';
44
import {_supportsShadowDom} from '@angular/cdk/platform';
55
import {ScrollDispatcher} from '@angular/cdk/scrolling';
66
import {
7-
MockNgZone,
87
clearElement,
98
createKeyboardEvent,
109
dispatchEvent,
1110
dispatchFakeEvent,
1211
dispatchKeyboardEvent,
12+
MockNgZone,
1313
typeInElement,
1414
} from '@angular/cdk/testing/private';
1515
import {
@@ -37,22 +37,20 @@ import {
3737
import {FormControl, FormsModule, ReactiveFormsModule} from '@angular/forms';
3838
import {MatOption, MatOptionSelectionChange} from '@angular/material-experimental/mdc-core';
3939
import {MatFormField, MatFormFieldModule} from '@angular/material-experimental/mdc-form-field';
40-
import {By} from '@angular/platform-browser';
4140
import {MatInputModule} from '@angular/material-experimental/mdc-input';
41+
import {By} from '@angular/platform-browser';
4242
import {NoopAnimationsModule} from '@angular/platform-browser/animations';
4343
import {EMPTY, Observable, Subject, Subscription} from 'rxjs';
4444
import {map, startWith} from 'rxjs/operators';
45-
46-
4745
import {
4846
getMatAutocompleteMissingPanelError,
49-
MAT_AUTOCOMPLETE_DEFAULT_OPTIONS,
50-
MAT_AUTOCOMPLETE_SCROLL_STRATEGY,
5147
MatAutocomplete,
5248
MatAutocompleteModule,
5349
MatAutocompleteOrigin,
5450
MatAutocompleteSelectedEvent,
5551
MatAutocompleteTrigger,
52+
MAT_AUTOCOMPLETE_DEFAULT_OPTIONS,
53+
MAT_AUTOCOMPLETE_SCROLL_STRATEGY,
5654
} from './index';
5755

5856

@@ -1566,7 +1564,7 @@ describe('MDC-based MatAutocomplete', () => {
15661564
let spacer = document.createElement('div');
15671565
let fixture = createComponent(SimpleAutocomplete, [{
15681566
provide: ScrollDispatcher,
1569-
useValue: {scrolled: () => scrolledSubject.asObservable()}
1567+
useValue: {scrolled: () => scrolledSubject}
15701568
}]);
15711569

15721570
fixture.detectChanges();
@@ -2234,7 +2232,7 @@ describe('MDC-based MatAutocomplete', () => {
22342232
const fixture = createComponent(SimpleAutocomplete, [
22352233
{
22362234
provide: ScrollDispatcher,
2237-
useValue: {scrolled: () => scrolledSubject.asObservable()}
2235+
useValue: {scrolled: () => scrolledSubject}
22382236
},
22392237
{
22402238
provide: MAT_AUTOCOMPLETE_SCROLL_STRATEGY,

src/material-experimental/mdc-dialog/dialog.spec.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,12 +37,12 @@ import {BrowserAnimationsModule, NoopAnimationsModule} from '@angular/platform-b
3737
import {numbers} from '@material/dialog';
3838
import {Subject} from 'rxjs';
3939
import {
40-
MAT_DIALOG_DATA,
41-
MAT_DIALOG_DEFAULT_OPTIONS,
4240
MatDialog,
4341
MatDialogModule,
4442
MatDialogRef,
45-
MatDialogState
43+
MatDialogState,
44+
MAT_DIALOG_DATA,
45+
MAT_DIALOG_DEFAULT_OPTIONS
4646
} from './index';
4747

4848
describe('MDC-based MatDialog', () => {
@@ -63,7 +63,7 @@ describe('MDC-based MatDialog', () => {
6363
{provide: Location, useClass: SpyLocation},
6464
{
6565
provide: ScrollDispatcher,
66-
useFactory: () => ({scrolled: () => scrolledSubject.asObservable()})
66+
useFactory: () => ({scrolled: () => scrolledSubject})
6767
},
6868
],
6969
});

0 commit comments

Comments
 (0)