Recurse correctly in `util.deepcopy`

lunarmodules / luassert

Assertion library for Lua

MIT License

202 stars 76 forks source link

Recurse correctly in `util.deepcopy` #184

Closed RiskoZoSlovenska closed 2 years ago

Tieske commented 2 years ago

@RiskoZoSlovenska would you mind rebasing on master? I just fixed CI, and hopefully things go green on your PR. If you could add a test case, that would be great.

RiskoZoSlovenska commented 2 years ago

If you could add a test case, that would be great.

Unfortunately, I haven't been able to find a case with broken behaviour. util.deepcopy is only ever called in util.copyargs, and anything that goes through copyargs is only ever touched by utils.deepcompare, which doesn't care about metatables.