UI tesztek #19
UI tesztek #19
Conversation
A cypress beállítása közben az eredeti program nem tudott lefutni, így visszaállítottam a működő programot
teszt lefuttatása, sok hibát tapasztaltam, amik alapból benne voltak. Ezek javításának elkezdése, illetve egy új teszt kipróbálása.
Az automatikusan létrejött tesztek, átnézése, kitörlése
Az első tesztek, amik végre jól működnek
Próbáltam az összes componens valamennyi funkcióját letesztelni
LGietLaci37
requested review from
csaba012,
zolykovacs,
matesze41nons and
kolcsong
| import { DatasetListComponent } from './dataset-list.component'; | ||
| import { ActivatedRoute, convertToParamMap, Router } from '@angular/router'; | ||
| import { of, throwError } from 'rxjs'; |
There was a problem hiding this comment.
Unused import: throwError
| import { ActivatedRoute, convertToParamMap, Router } from '@angular/router'; | ||
| import { of, throwError } from 'rxjs'; | ||
| import { DatasetService } from '../dataset.service'; | ||
| import { MatDialogModule, MatDialog, MatDialogRef } from '@angular/material/dialog'; |
There was a problem hiding this comment.
Unused import: MatDialog, MatDialogRef
| import { MatToolbarModule } from '@angular/material/toolbar'; | ||
| import { MatIconModule } from '@angular/material/icon'; | ||
| import { BrowserAnimationsModule, NoopAnimationsModule } from '@angular/platform-browser/animations'; | ||
| import { PopUpWindowComponent } from '../pop-up-window/pop-up-window.component'; |
There was a problem hiding this comment.
Unused import: PopUpWindowComponent
|
|
||
| beforeEach(waitForAsync(() => { | ||
| mockDatasetService = jasmine.createSpyObj('DatasetService', ['getDatasetForResource']); | ||
| mockRouter = jasmine.createSpyObj('Router', ['navigate']); |
There was a problem hiding this comment.
Nem használod ezek után sehol, el lehet távolítani a mockRoutert innen és a 21-es sorból is
| import { MatFormFieldModule } from '@angular/material/form-field'; | ||
| import { MatCheckboxModule } from '@angular/material/checkbox'; | ||
| import { MatButtonModule } from '@angular/material/button'; | ||
| import { HttpClient, HttpClientModule } from '@angular/common/http'; |
There was a problem hiding this comment.
Unused import: HttpClient
| @@ -1,23 +1,85 @@ | |||
| import { ComponentFixture, TestBed } from '@angular/core/testing'; | |||
|
|
|||
| import { ReactiveFormsModule } from '@angular/forms'; | |||
There was a problem hiding this comment.
Unused import: ReactiveFormsModule
| import { MatIconModule } from '@angular/material/icon'; | ||
| import { RouterModule } from '@angular/router'; | ||
| import { MatButtonModule } from '@angular/material/button'; | ||
| import { ActivatedRoute } from '@angular/router'; |
There was a problem hiding this comment.
@angular/router imported multiple times.
A 9. és 11. sort vond össze: import { RouterModule, ActivatedRoute } from '@angular/router';
| ], | ||
| }).compileComponents(); | ||
|
|
||
| resourceService = TestBed.inject(ResourceService) as jasmine.SpyObj<ResourceService>; |
There was a problem hiding this comment.
Nem használod sehol a resourceService változót, így eltávolítható innen és a 16. sorból is.
| }).compileComponents(); | ||
|
|
||
| resourceService = TestBed.inject(ResourceService) as jasmine.SpyObj<ResourceService>; | ||
| matDialog = TestBed.inject(MatDialog) as jasmine.SpyObj<MatDialog>; |
There was a problem hiding this comment.
Nem használod sehol a matDialog változót, így eltávolítható innen és a 17. sorból is.
| @@ -2,15 +2,3 @@ import { TestBed } from '@angular/core/testing'; | |||
|
|
|||
There was a problem hiding this comment.
unused import TestBed, ResourceService
|
There was a problem hiding this comment.
Szépen strukturált tesztek, jó hogy @csaba012 javasolt módosításokat, így sokkal tisztább kód.
|
Ezt a branchet merge-lem a main-be. |
Tesztek a komponensekhez a .component.spec.ts fajlokban