File

util.argparse: Fix bug (regression?) in argument parsing with --foo=bar After recent changes, '--foo bar' was working, but '--foo=bar' was not. The test had a typo (?) (bar != baz) and because util.argparse is not strict by default, the typo was not caught. The typo caused the code to take a different path, and bypassed the buggy handling of --foo=bar options. I've preserved the existing test (typo and all!) because it's still an interesting test, and ensures no unintended behaviour changes compared to the old code. However I've added a new variant of the test, with strict mode enabled and the typo fixed. This test failed due to the bug, and this commit introduces a fix.

local units = require "util.human.units";

describe("util.human.units", function ()
	describe("format", function ()
		it("formats numbers with SI units", function ()
			assert.equal("1 km", units.format(1000, "m"));
			assert.equal("1 GJ", units.format(1000000000, "J"));
			assert.equal("1 ms", units.format(1/1000, "s"));
			assert.equal("10 ms", units.format(10/1000, "s"));
			assert.equal("1 ns", units.format(1/1000000000, "s"));
			assert.equal("1 KiB", units.format(1024, "B", 'b'));
			assert.equal("1 MiB", units.format(1024*1024, "B", 'b'));
		end);
	end);
end);