New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

get returns reference to inner array #9

Open

mrstux opened this issue Jul 5, 2018 · 1 comment

mrstux commented Jul 5, 2018 •

edited

Loading

This is perhaps expected, but is it correct?

	it( 'results of prevous get should not be modified by future set', () => {
		let map = new Multimap( [['a',1]]);
		
		let aList = map.get('a');
		assert.strictEqual(aList.length, 1);
		
		map.set('a', 2);
		
		// aList.length is now 2
		assert.strictEqual(aList.length, 1);
	});

mrstux closed this as completed

mrstux reopened this

Collaborator

KoryNunn commented Nov 12, 2019

Hey @mrstux, I think this is a reasonable test, would you be able to add it and do a PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment