Bv fetch changes to fix, PD-238557 (#140)

* PD-238557, bug fix

* PD-238557, fixing test cases

* 2.9.3

lib/bvFetch/index.js

@@ -83,14 +83,15 @@ module.exports = function BvFetch ({ shouldCache, cacheName, cacheLimit }) {
       }
     }).then(() => {
       if (canBeCached) {
-        const clonedResponse = response.clone()
+        const clonedResponse = response.clone();
+        const sizeCheck = response.clone();
         const newHeaders = new Headers();
         clonedResponse.headers.forEach((value, key) => {
           newHeaders.append(key, value);
         });
         newHeaders.append('X-Bazaarvoice-Cached-Time', Date.now())
         // Get response text to calculate its size
-        clonedResponse.text().then(text => {
+        sizeCheck.text().then(text => {
         // Calculate size of response text in bytes
           const sizeInBytes = new Blob([text]).size;
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "bv-ui-core",
-  "version": "2.9.2",
+  "version": "2.9.3",
   "license": "Apache 2.0",
   "description": "Bazaarvoice UI-related JavaScript",
   "repository": {

test/unit/bvFetch/index.spec.js

@@ -140,16 +140,17 @@ describe('BvFetch', function () {
     bvFetchInstance.bvFetchFunc(url, options)
     .then(response => {
       // Check if response is fetched from network
-      expect(response).to.not.be.null;
-      console.log(response.body)
+      setTimeout(() => { 
+        expect(response).to.not.be.null;
+        console.log(response.body)
 
       // Check if caches.match was called
-      expect(cacheStub.calledOnce).to.be.false;
+        expect(cacheStub.calledOnce).to.be.false;
 
       // Check if response is not cached
-      const cachedResponse = cacheStorage.get(url);
-      expect(cachedResponse).to.be.undefined;
-
+        const cachedResponse = cacheStorage.get(url);
+        expect(cachedResponse).to.be.undefined; 
+      }, 500)
       done();
     })
     .catch(done);