Multiple set without any get calls, results in ttl mismatch

Scenario

const cache =  lru();
cache.set("foo", "bar");
// wait until TTL has passed
cache.set("foo", "baz");
const result = cache.get("foo");
console.log(typeof result === "undefined");

I do not expect the result to be “undefined”, but rather expect it to be the “baz” value. The second set() should IMO update the expiry property.

Failing test:

exports.multiSetWithTTL = {
	setUp: function (done) {
		this.cache = lru(4, 10);
		done();
	},
	test: function (test) {
		this.cache.set("foo", "bar");
		setTimeout(function(){
			this.cache.set("foo", "baz");
			test.equal(this.cache.get("foo"), "baz", "should return baz");
			test.done();
		}.bind(this), 11);
	}
};

Want a PR for this? This fix would require a breaking change? If this behaviour is expected, it should be documented in the README.

[sveisvei]

Thanx again.

[avoidwork]

i can likely land this corrected implementation over the weekend or on monday.