模拟本地存储/覆盖

Question

伊夫下面的代码使用按预期工作的本地存储，我想为它创建单元测试的问题是，我在这个功能使用本地存储

这是我想测试

open: function(url) { 
      var that = this; 
      if (!localStorage.getItem(“alert”)) {    
       that_run(url) 
        .then(function(isBlocked) { 
         if (isBlocked) { 
          that._notify(); 
          localStorage.setItem(“alert”, true); 
         } else { 
          localStorage.setItem(“alert”, true); 
          that._exe(“page”,url) 
         } 
        }).done(); 
      } else { 
       that._exe(“page”,url) 
      } 
     }, 

这是正在测试的功能，但我想重写窗口是不好的做法，我的问题，如果我能写这个测试最好？

it.only("test for open", function () { 
    var url = "http://mytesturl”; 
    winlocalStorage.getItem = function(){ 
        return false; 
    }; 
    var oSimulateSpy = sandbox.spy(exc "_simulate"); 
    return orun.open(url).then(function(){ 
        expect(oSimulateSpy.called).to.be.true; 
    }); 
});  

我看到了这篇文章，这个使用功能的编程 https://stackoverflow.com/a/20153543/6124024 但我认为通过本地存储的参数是在这种情况下有点大材小用，因为该功能（开）从许多地方调用多次。 ..有没有更好的/更清洁的方式来处理这个问题？

Answer 1

因为你的代码被加载了各种各样的副作用，所以你在挣扎。因为您不会从this._exe或that_run中返回任何值，所以在我看来，这些功能也有副作用。您最好从这些函数返回值或承诺值，而不是依赖这些函数来改变更多的外部状态。

这里是写你的模块

// use "wrapper" function that configures this module 
export default function(storage) { 
    return { 
    // other functions ... 
    async open (url) { 
     if (storage.getItem('alert')) { 
     return this._exe('page', url) // make sure _exe returns promise 
     } 
     else if (await that_run(url)) { 
     storage.setItem('alert', true) 
     return this.notify() // make sure notify returns Promise 
     // or return Promise.resolve() 
     } 
     else { 
     storage.setItem('alert', true) 
     return this._exe('page', url) 
    } 
    } 
} 

在测试代码使用您的模块

// import mock storage adapter 
const MockStorage = require('./mock-storage'); 

// create mock storage adapter 
const mockStorage = new MockStorage(); 

// pass mock storage adapter to your module as a config argument 
const myModule = require('./my-module')(mockStorage); 

// don't forget to reset storage before each test 
beforeEach(function() { 
    mockStorage.reset(); 
}); 

it("test for open", async function() { 
    var url = "http://mytesturl"; 
    mockStorage.set('alert', false); 
    let someValue = await myModule.open(url); 
    assert.equal(someValue, someExpectation); 
    assert.equal(mockStorage.get('alert'), true); 
}); 

的模拟存储适配器可能是这样的

export default class MockStorage { 
    constructor() { 
    this.storage = new Map(); 
    } 
    setItem (key, value) { 
    this.storage.set(key, value); 
    } 
    getItem (key) { 
    return this.storage.get(key); 
    } 
    removeItem (key) { 
    this.storage.delete(key); 
    } 
    clear() { 
    this.constructor(); 
    } 
} 

然后，可能更好的办法，当你在生产代码中使用你的模块时，你可以通过真实的localStorage

// use window.localStorage as config argument in production code 
const myModule = require('./my-module')(window.localStorage);