mike-marcacci / node-redlock / 98

100%

master: 100%

Pull #63

travis-ci

Replace `table.getn` with length operator

`table.getn` was removed in Lua 5.2, in favor of the `#` operator - see e.g. see e.g. https://stackoverflow.com/questions/11890125/lua-table-library-removed.

While Redis itself still seems to support the old syntax, this change allows compatibility with mock libraries which may rely on newer Lua versions. In my case I ran into issues with https://github.com/stipsan/ioredis-mock.

For anyone looking for a solution in the meantime to use `redlock` with `ioredis-mock` or similar, I'm using:

```javascript
const replaceGetn = (lua: string) => {
  // table.getn(KEYS) --> #KEYS
  return lua.replace(/table\.getn\(([A-Za-z0-9]+)\)/g, '#$1');
}
const redlock = new Redlock(clients, {
  lockScript: replaceGetn,
  extendScript: replaceGetn,
});
```

303 of 384 branches covered (78.91%)

2034 of 2034 relevant lines covered (100.0%)

77.34 hits per line