[Pkg-javascript-commits] [leaflet-markercluster] 35/128: Add unspiderfy test

[Jonas Smedegaard]

This is an automated email from the git hooks/post-receive script.

js pushed a commit to branch master
in repository leaflet-markercluster.

commit 2faaba2cb52621816106391b131e8840bc546e0e
Author: zverev <alexander.zverev at gmail.com>
Date:   Tue Jan 12 08:57:20 2016 +0300

    Add unspiderfy test
---
 spec/index.html               |   1 +
 spec/suites/unspiderfySpec.js | 169 ++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 170 insertions(+)

diff --git a/spec/index.html b/spec/index.html
index 93a5e27..ff372c9 100644
--- a/spec/index.html
+++ b/spec/index.html
@@ -61,6 +61,7 @@
 	<script type="text/javascript" src="suites/RemoveLayerSpec.js"></script>
 	<script type="text/javascript" src="suites/removeLayersSpec.js"></script>
 	<script type="text/javascript" src="suites/spiderfySpec.js"></script>
+	<script type="text/javascript" src="suites/unspiderfySpec.js"></script>
 	<script type="text/javascript" src="suites/zoomAnimationSpec.js"></script>
 
 	<script type="text/javascript" src="suites/RememberOpacity.js"></script>
diff --git a/spec/suites/unspiderfySpec.js b/spec/suites/unspiderfySpec.js
new file mode 100644
index 0000000..6faf07f
--- /dev/null
+++ b/spec/suites/unspiderfySpec.js
@@ -0,0 +1,169 @@
+describe('unspiderfy', function () {
+
+	/**
+	 * Avoid as much as possible creating and destroying objects for each test.
+	 * Instead, try re-using them, except for the ones under test of course.
+	 * PhantomJS does not perform garbage collection for the life of the page,
+	 * i.e. during the entire test process (Karma runs all tests in a single page).
+	 * http://stackoverflow.com/questions/27239708/how-to-get-around-memory-error-with-karma-phantomjs
+	 *
+	 * The `beforeEach` and `afterEach do not seem to cause much issue.
+	 * => they can still be used to initialize some setup between each test.
+	 * Using them keeps a readable spec/index.
+	 *
+	 * But refrain from re-creating div and map every time. Re-use those objects.
+	 */
+
+	/////////////////////////////
+	// SETUP FOR EACH TEST
+	/////////////////////////////
+
+	beforeEach(function () {
+
+		clock = sinon.useFakeTimers();
+
+	});
+
+	afterEach(function () {
+
+		if (group instanceof L.MarkerClusterGroup) {
+			group.removeLayers(group.getLayers());
+			map.removeLayer(group);
+		}
+
+		// group must be thrown away since we are testing it with a potentially
+		// different configuration at each test.
+		group = null;
+
+		clock.restore();
+		clock = null;
+
+	});
+
+
+	/////////////////////////////
+	// PREPARATION CODE
+	/////////////////////////////
+
+	var div, map, group, clock;
+
+	div = document.createElement('div');
+	div.style.width = '200px';
+	div.style.height = '200px';
+	document.body.appendChild(div);
+
+	map = L.map(div, {
+		maxZoom: 18
+	});
+
+	// Corresponds to zoom level 8 for the above div dimensions.
+	map.fitBounds(new L.LatLngBounds([
+		[1, 1],
+		[2, 2]
+	]));
+
+
+	/////////////////////////////
+	// TESTS
+	/////////////////////////////
+
+	it('Unspiderfies 2 Markers', function () {
+
+		group = new L.MarkerClusterGroup();
+
+		var marker = new L.Marker([1.5, 1.5]);
+		var marker2 = new L.Marker([1.5, 1.5]);
+
+		group.addLayer(marker);
+		group.addLayer(marker2);
+		map.addLayer(group);
+
+		marker.__parent.spiderfy();
+
+		clock.tick(1000);
+
+		group.unspiderfy();
+
+		clock.tick(1000);
+
+		expect(map.hasLayer(marker)).to.be(false);
+		expect(map.hasLayer(marker2)).to.be(false);
+	});
+
+	it('Unspiderfies 2 CircleMarkers', function () {
+
+		group = new L.MarkerClusterGroup();
+
+		var marker = new L.CircleMarker([1.5, 1.5]);
+		var marker2 = new L.CircleMarker([1.5, 1.5]);
+
+		group.addLayer(marker);
+		group.addLayer(marker2);
+		map.addLayer(group);
+
+		marker.__parent.spiderfy();
+
+		clock.tick(1000);
+
+		group.unspiderfy();
+
+		clock.tick(1000);
+
+		expect(map.hasLayer(marker)).to.be(false);
+		expect(map.hasLayer(marker2)).to.be(false);
+	});
+
+	it('Unspiderfies 2 Circles', function () {
+
+		group = new L.MarkerClusterGroup();
+
+		var marker = new L.Circle([1.5, 1.5], 10);
+		var marker2 = new L.Circle([1.5, 1.5], 10);
+
+		group.addLayer(marker);
+		group.addLayer(marker2);
+		map.addLayer(group);
+
+		marker.__parent.spiderfy();
+
+		clock.tick(1000);
+
+		group.unspiderfy();
+
+		clock.tick(1000);
+
+		expect(map.hasLayer(marker)).to.be(false);
+		expect(map.hasLayer(marker2)).to.be(false);
+	});
+
+	it('fires unspiderfied event on unspiderfy', function (done) {
+
+		group = new L.MarkerClusterGroup();
+
+		var marker = new L.Marker([1.5, 1.5]);
+		var marker2 = new L.Marker([1.5, 1.5]);
+
+		group.addLayers([marker, marker2]);
+		map.addLayer(group);
+
+		marker.__parent.spiderfy();
+
+		clock.tick(1000);
+
+		// Add event listener
+		group.on('unspiderfied', function (event) {
+			expect(event.target).to.be(group);
+			expect(event.cluster).to.be.a(L.Marker);
+			expect(event.markers[1]).to.be(marker);
+			expect(event.markers[0]).to.be(marker2);
+
+			done();
+		});
+
+		group.unspiderfy();
+
+		clock.tick(1000);
+
+	});
+
+});
\ No newline at end of file

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/leaflet-markercluster.git