0

更新 2: 事实证明,我的新测试最有可能在首次加载时响应上一个测试的 url 参数。这显然不是我想要的,可能意味着我需要更改路由。是在 afterEach 中重置路由参数的唯一解决方案吗?有没有更好的方法来处理这个?

更新:

经过更多的挖掘,我不再认为这是一个异步问题,但我不确定问题是什么。我认为这与路由或订阅有关。这是我投入组件中的一些控制台日志中失败的测试场景的示例。

//This is an emulated copy of what I see in the console

new request: 
> {pageId: 3, testCount: 1}
retrievePage: making the request with:
> {pageId: 3, testCount: 1}
new request:
> {id: 2, testCount: 2}
retrieveItem: making the request with:
> {id: 2, testCount: 2}
retrieveItem: made the request with: 
> {id: 2, testCount: 2}
FAILED MyComponent when the url matches item/:id should load the appropriate item
retrievePage: made the request with:
> {pageId: 3, testCount: 1}
> error properties: Object({ originalStack: 'Error: Expected spy service.retrievePage not to have been called.....

我更新了下面的代码片段以反映我添加的控制台日志。

正如您从日志中看到的那样,由于某种原因,参数订阅在第一次测试中第二次运行(第一次测试运行并正常完成,记录了“新请求”和“制作...”/“制作.. ." 消息仅一次)。我不知道是否有办法确定正在运行哪个测试组件(如果由于某种原因生成了新组件并响应先前测试中的参数)或什么。我不确定问题是什么或如何进一步调试。

原始问题

我在测试运行一些异步代码的组件时遇到问题。由于保密协议,我无法发布实际代码,但我提供了一个简单的表示,它尽可能接近我的真实代码。

我有一个像这样运行一些异步代码的组件(仅包含相关位,默认导入/设置是隐含的):

...
import { ActivatedRoute } from '@angular/router';
import { Subscription } from 'rxjs';
import { MyService } from 'services/my-service';

@Component({
    ...
})
export class MyComponent implements OnInit, OnDestroy {
    private paramsSubscription: Subscription;

    constructor(private route: ActivatedRoute, private service: MyService) {}

    ngOnInit(): void {
        this.paramsSubscription = this.route.params.subscribe(async params => {
            console.log('new request: ', params);
            let result;
            let itemId = params.id;
            
            if (params.pageId) {
                
                console.log('retrievePage: making the request with: ', params);
                let page = await this.service.retrievePage(params.pageId).toPromise();
                console.log('retrievePage: made the request with: ', params);
                itemId = page.items[0].id;
            }

            if (itemId) {
                console.log('retrieveItem: making the request with: ', params);
                result = await this.service.retrieveItem(itemId).toPromise();
                console.log('retrieveItem: made the request with: ', params);
            }
            ...
        });
    }
    ngOnDestroy() {
        if (this.paramsSubscription) this.paramsSubscription.unsubscribe();
    }
}

然后我有一个这样的单元测试(也是示例代码):

...
import { fakeAsync, tick } from '@angular/core/testing';
import { of, ReplaySubject } from 'rxjs';
import { ActivatedRoute, Params } from '@angular/router';
import { RouterTestingModule } from '@angular/router/testing';
import { MyService } from 'services/my-service';
import { MyComponent } from './my.component';

describe('MyComponent', () => {
    let component: MyComponent;
    let fixture: ComponentFixture<MyComponent>;

    let service = jasmine.createSpyObj('MyService', {
        retrieveItem: of([]), 
        retrievePage: of([])
    });
    let route: ActivatedRoute;
    let routeParams = new ReplaySubject<Params>(1);
    
    let testCount = 0;

    let item = {
        id: 2,
        ...
    };

    let page = {
        id: 3,
        items: [item]
        ...
    };

    beforeEach(async(() => {
        TestBed.configureTestingModule({
            declarations: [MyComponent],
            imports: [RouterTestingModule],
            providers: [
                { provide: MyService, useValue: service },
                { provide: ActivatedRoute, useValue: 
                    jasmine.createSpyObj('ActivatedRoute',[], {
                        params: routeParams.asObservable()
                    }) 
                },
            ]
        }).compileComponents();
    }));

    beforeEach(() => {
        fixture = TestBed.createComponent(MyComponent);
        component = fixture.componentInstance;

        service = TestBed.inject(MyService);
        route = TestBed.inject(ActivatedRoute);
        
        testCount++;

        fixture.detectChanges();
    });
    
    afterEach(() => {
        service.retrieveItem.calls.reset();
        service.retrievePage.calls.reset();
    });

    it('should create', () => {
        expect(component).toBeTruthy();
    });
    

    describe('when the url matches: item/:id', () => {
        beforeEach(fakeAsync(() => {
            routeParams.next({id: item.id, testCount: testCount});
            tick();
            fixture.detectChanges();
        }));

        it('should load the appropriate item', () => {
            expect(service.retrieveItem).toHaveBeenCalledWith(item.id);
            expect(service.retrievePage).not.toHaveBeenCalled();
        });
        ...
    });

    describe('when the url matches: item', () => {
        beforeEach(fakeAsync(() => {
            routeParams.next({testCount: testCount});
            tick();
            fixture.detectChanges();
        }));

        it('should load the first item', () => {
            expect(service.retrievePage).not.toHaveBeenCalled();
            expect(service.retrieveItem).toHaveBeenCalledWith(1);
        });
        ...
    });

    describe('when the url matches: item/:pageId', () => {
        beforeEach(fakeAsync(() => {
            routeParams.next({pageId: page.id, testCount: testCount});
            tick();
            fixture.detectChanges();
        }));

        it('should load the item from the page', () => {
            expect(service.retrievePage).toHaveBeenCalledWith(page.id);
            expect(service.retrieveItem).toHaveBeenCalledWith(page.items[0].id);
        });
        ...
    });
});

我遇到的问题是在我的单元测试中,来自我的组件的 await 调用在进入下一个单元测试之前尚未完成,因此我的期望调用无法正常运行。当我的测试以完美的顺序运行时,一切都很好。但有时我的测试以不同的顺序运行,比如 1、3、2。当他们这样做时,测试#2 失败,因为它期望使用 id 2 调用retrieveItem,但之前的调用使用 id 1 触发了它。(这只是一个人为的例子,我的实际代码比这更复杂)

我想知道的是,我如何告诉 jasmine 等待在我的组件中触发的调用返回。我已经尝试查看有关 jasmine 的异步代码的教程https://jasmine.github.io/tutorials/async但这些似乎都提到了从我的测试中手动进行异步调用,这不是我正在尝试的去做。

带有 tick() 的 fakeAsync 是我试图让它等待路由和一切完成,但它似乎并没有解决我的问题。有谁知道我做错了什么或如何解决我的问题?

我也试过:

  • 在刻度内放置一个值,例如刻度(10)
  • 使用fixture.whenStable().then(() => { //expect stuff})

另外,如果我添加 routeParams.next({}); 对我的 afterEach() 来说,它解决了这个问题,但我觉得这违背了测试随机顺序的目的,因为这意味着我的代码只有在你没有来自任何参数的情况下才会起作用,在我的实际实现中你可以在这里使用您的网址中的任何各种参数(例如,我可以从 pageId: 1 转到 id: 2 而没有中间步骤)。

更多信息:在我的真实代码中,它是这样发生的:

//This is pseudo code
params.subscribe
    if(params.first) param1 = params.first;
    if(params.second) param2 = params.second;
    if (param1) {
        item = function1();
        param2 = item.param2;
    }
    if(param2) {
        otherItem = await function2();
    } else if (item) {
        result = await function3();
    }

我的测试是检查在 activateRoute 传递了各种参数时是否调用了 function1、function2 和 function3。我在我的组件中添加了控制台日志,以确认 test2 将启动,并且在 test2 启动后调用来自 test1 的 function2。因此,我确保未调用 function2 的检查失败,因为它从 test1 延迟。

非常感谢任何建议或提示。

4

1 回答 1

0

因此,经过更多的挖掘和谷歌搜索,我自己找到了解决问题的方法,所以我在这里分享它,以防其他人像我一样困惑。

https://angular.io/guide/testing-components-scenarios#testing-with-activatedroutestub

在使用激活路由时的角度文档中,我注意到他们首先触发路由,然后创建组件。通过观察应用程序的正常工作方式(而不是在测试期间),这更符合 Angular 的实际功能 - 路由发生在组件创建之前。基于我的解决方案,我最终将测试修改为如下所示:

...
import { async } from '@angular/core/testing';
import { of, ReplaySubject } from 'rxjs';
import { ActivatedRoute, Params } from '@angular/router';
import { RouterTestingModule } from '@angular/router/testing';
import { MyService } from 'services/my-service';
import { MyComponent } from './my.component';

describe('MyComponent', () => {
    let component: MyComponent;
    let fixture: ComponentFixture<MyComponent>;

    let service = jasmine.createSpyObj('MyService', {
        retrieveItem: of([]), 
        retrievePage: of([])
    });
    let route: ActivatedRoute;
    let routeParams = new ReplaySubject<Params>(1);
    
    let testCount = 0;

    let item = {
        id: 2,
        ...
    };

    let page = {
        id: 3,
        items: [item]
        ...
    };

    beforeEach(async(() => {
        TestBed.configureTestingModule({
            declarations: [MyComponent],
            imports: [RouterTestingModule],
            providers: [
                { provide: MyService, useValue: service },
                { provide: ActivatedRoute, useValue: 
                    jasmine.createSpyObj('ActivatedRoute',[], {
                        params: routeParams.asObservable()
                    }) 
                },
            ]
        }).compileComponents();
    }));

    beforeEach(() => {
        service = TestBed.inject(MyService);
        route = TestBed.inject(ActivatedRoute);
    });
    
    function createComponent() {
        fixture = TestBed.createComponent(MyComponent);
        component = fixture.componentInstance;

        fixture.detectChanges();
    }
    
    afterEach(() => {
        service.retrieveItem.calls.reset();
        service.retrievePage.calls.reset();
    });

    it('should create', () => {
        createComponent();
        expect(component).toBeTruthy();
    });
    

    describe('when the url matches: item/:id', () => {
        beforeEach(async () => {
            routeParams.next({id: item.id, testCount: testCount});
            createComponent();
        });

        it('should load the appropriate item', () => {
            expect(service.retrieveItem).toHaveBeenCalledWith(item.id);
            expect(service.retrievePage).not.toHaveBeenCalled();
        });
        ...
    });

    describe('when the url matches: item', () => {
        beforeEach(async () => {
            routeParams.next({testCount: testCount});
            createComponent();
        });

        it('should load the first item', () => {
            expect(service.retrievePage).not.toHaveBeenCalled();
            expect(service.retrieveItem).toHaveBeenCalledWith(1);
        });
        ...
    });

    describe('when the url matches: item/:pageId', () => {
        beforeEach(async () => {
            routeParams.next({pageId: page.id, testCount: testCount});
            createComponent();
        ));

        it('should load the item from the page', () => {
            expect(service.retrievePage).toHaveBeenCalledWith(page.id);
            expect(service.retrieveItem).toHaveBeenCalledWith(page.items[0].id);
        });
        ...
    });
});

于 2021-07-21T17:55:43.690 回答