add invalidating to lazyObservable

[thinhvoxuan]

lazyObserable sometimes need trigger to loading new data such as after edit profile, I need to run '/loadProfile' again

[mweststrate]

Thanks for the PR, looks useful!

Shouldn't it be called invalidate()? could you add unit tests as well?

Thanks!

[thinhvoxuan]

@mweststrate I name that function is 'refresh' instead of 'invalidate'. I also add test-case for refresh function but I never work. Could you give me any clue?

[mweststrate]

@voxuanthinh I think your implementation is correct, but you use a Promise in your test,and promises are not replayable, so you cannot test this behavior using Promises, a new then call will just resolve the already resolved value. You could test this by creating a new Promise each time instead:

sink => new Promise((resolve, reject) => {
        started = started + 1;
        resolve(i);
        i++;
    }).then(value =>  {
        sink(value)
    }),

[coveralls]

Coverage decreased (-0.3%) to 97.248% when pulling 057a9ae on voxuanthinh:master into ca9a1af on mobxjs:master.

[thinhvoxuan]

@mweststrate :) nice. thank.

[mweststrate]

Released as 1.1.2