diff options
author | Kushal Pandya <kushalspandya@gmail.com> | 2017-06-05 09:12:15 +0000 |
---|---|---|
committer | Phil Hughes <me@iamphill.com> | 2017-06-05 09:12:15 +0000 |
commit | 7abe27b425c9e402bd1d7780ed51be023397ee2f (patch) | |
tree | 7084a728bc3c290ac05dad789decdfcd96d5d811 /spec/javascripts/lib | |
parent | f71106425cc6f62c8e19457cc600e41a668fb89e (diff) | |
download | gitlab-ce-7abe27b425c9e402bd1d7780ed51be023397ee2f.tar.gz |
Improve user experience around slash commands in instant comments
Diffstat (limited to 'spec/javascripts/lib')
-rw-r--r-- | spec/javascripts/lib/utils/ajax_cache_spec.js | 31 |
1 files changed, 31 insertions, 0 deletions
diff --git a/spec/javascripts/lib/utils/ajax_cache_spec.js b/spec/javascripts/lib/utils/ajax_cache_spec.js index e1747a82329..2c946802dcd 100644 --- a/spec/javascripts/lib/utils/ajax_cache_spec.js +++ b/spec/javascripts/lib/utils/ajax_cache_spec.js @@ -154,5 +154,36 @@ describe('AjaxCache', () => { .then(done) .catch(fail); }); + + it('makes Ajax call even if matching data exists when forceRequest parameter is provided', (done) => { + const oldDummyResponse = { + important: 'old dummy data', + }; + + AjaxCache.internalStorage[dummyEndpoint] = oldDummyResponse; + + ajaxSpy = (url) => { + expect(url).toBe(dummyEndpoint); + const deferred = $.Deferred(); + deferred.resolve(dummyResponse); + return deferred.promise(); + }; + + // Call without forceRetrieve param + AjaxCache.retrieve(dummyEndpoint) + .then((data) => { + expect(data).toBe(oldDummyResponse); + }) + .then(done) + .catch(fail); + + // Call with forceRetrieve param + AjaxCache.retrieve(dummyEndpoint, true) + .then((data) => { + expect(data).toBe(dummyResponse); + }) + .then(done) + .catch(fail); + }); }); }); |