Jasmine.js:使用“运行”时的竞争条件

时间:2013-06-17 18:40:18

标签: javascript unit-testing jasmine race-condition

我在使用jasmine测试代码时发现了一个奇怪的行为。与我的规范中的其他测试一起执行时,一个测试失败。当单独调用时,测试通过。

测试断言脚本A.js依赖于提供方法“Create”的脚本B.js。我在测试内部创建了一个“Create”的间谍,并调用脚本A.js(A.init),它将加载一些数据(loadData再次返回一个promise),然后调用“Create”方法5次(一次loadData) - 承诺得到解决)。 A.init()返回另一个承诺!

当我使用Jasmine的“runs”方法并等到promise-init被解决时,我想断言B.Create被调用了5次。

执行测试时,同一规范中的其他一些测试将为B.Create方法设置自己的间谍。所以我认为这将创造一些竞争条件。

注释:每个测试都为create-method创建自己的间谍(var createSpy = spyOn(B, "Create");

所以归结为以下问题:

  • 我是否面临竞争条件?
  • 如何防止此问题?时间模拟(jasmine.Clock.useMock)不是一个真正的解决方案,因为我使用promise-fake模拟“loadData”方法。

更新-1 : Richard Dingwall在他的文章Parallel vs serial javascript async tests中概述了Jasmine并行执行测试,这是我问题的根源吗?

更新-2 这是失败的测试:

/* more code */

crmRestKitCreateSpy = spyOn( CrmRestKit, 'Create' )
   .andCallFake( function ( entitySchemaName, obj ) {

       return {
           then: function ( callback ) {
               // fake a create response where the id attribute is populated
               callback( _.extend( {}, obj, { AccountId: cloneId } ) );
           }
       };
   } );

/* more code */

it( 'links all child-clones to the new parent-clone', function () {

// arrange - inject spy
spyOn( CrmRestKit, 'ByQueryAll' ).andReturn(
        alfa.fake.promise.buildFakeResolvePromise( { d: fakeChildAcccounts, __next: false }, 750 )
);

// arrange 
includedOneToManyRel = [accountToAccountRel];

// action 
var promise = alfa.util.clonemachine.deepClone( fakeId, includedOneToManyRel );

waitsFor( function () {
    return promise.state() === 'resolved';
}, 800 );


runs( function () {

    expect( crmRestKitCreateSpy.callCount ).toBe( 5 );

    // assert - all child-clones reference the new parent-clone
    expect( crmRestKitCreateSpy.calls[1].args[1].ParentAccountId.Id ).toBe( cloneId );
    expect( crmRestKitCreateSpy.calls[2].args[1].ParentAccountId.Id ).toBe( cloneId );
    expect( crmRestKitCreateSpy.calls[3].args[1].ParentAccountId.Id ).toBe( cloneId );
    expect( crmRestKitCreateSpy.calls[4].args[1].ParentAccountId.Id ).toBe( cloneId );
} );

});

更新-3 我认为一个证据表明我面临着竞争条件。以下测试通过。所以我的测试受到其他运行测试的影响。

it( 'its a trape', function () {

            waitsFor( function () {

                return ( crmRestKitCreateSpy.callCount > 0 );
            }, 4000 );

            runs( function () {

                expect( crmRestKitCreateSpy.callCount ).toBeGreaterThan( 0 );
            } );
        } );

1 个答案:

答案 0 :(得分:0)

好吧,好像我正面临竞争条件:使用“运行”时,createSpy被多次测试使用(请参阅问题中的Update-3)。

事实证明,Jasmine的Clock-Mock解决了我的问题:

beforeEach( function () {

            // mock the ByQuery method
            crmRestKitByQuerySpy = spyOn( CrmRestKit, 'ByQuery' )
                .andCallFake( alfa.fake.promise.buildFakeResolvePromise( fakeChildAcccounts ) );

            // The Jasmine Mock Clock is available for a test suites that need the 
            // ability to use setTimeout or setInterval callbacks. It makes the 
            // timer callbacks synchronous, thus making them easier to test.
            jasmine.Clock.useMock();
        } );

it( 'it supports async execution - jasmine-timer-mock', function () {

            var deferedInMillisecond = 250;

            // arrange - inject the ByQueryAll method of the CrmRestKit
            spyOn( CrmRestKit, 'ByQueryAll' ).andReturn( alfa.fake.promise.buildFakeResolvePromise( {
                d: fakeChildAcccounts,
                __next: false
            }, deferedInMillisecond ) );

            // arrange 
            includedOneToManyRel = [accountToAccountRel];

            // action 
            var deepClonePromise = alfa.util.clonemachine.deepClone( fakeId, includedOneToManyRel );

            expect( deepClonePromise.state() === 'pending' );

            jasmine.Clock.tick( deferedInMillisecond + 1 );

            // assert - defere the assertion until the waitFor is completed
            expect( deepClonePromise.state() === 'resolved' );
        } );