Leaflet: Tests for GeoJSON functions required

[johnd0e]

None of following GeoJSON functions are covered with tests yet:

• [ ] geometryToLayer
• [ ] coordsToLatLng
• [ ] coordsToLatLngs
• [ ] latLngToCoords
• [ ] latLngsToCoords
• [ ] asFeature

Sample test (for latLngToCoords):

describe("L.GeoJSON functions", function () {
    it("L.GeoJSON.latLngToCoords", function () {
        // todo: check alt too
        var arrLatLng = [-1.4837191022531273, 43.49222084042808];
        var arrLngLat = [arrLatLng[1], arrLatLng[0]];
        var latLng = L.latLng(arrLatLng);

        expect(L.GeoJSON.latLngToCoords(latLng)).to.eql([43.492221, -1.483719]);
        expect(L.GeoJSON.latLngToCoords(latLng, 0)).to.eql([43, -1]);
        // expect(L.GeoJSON.latLngToCoords(latLng, false)).to.eql(arrLngLat); // after #7100
    });

    // etc
});

[phloose]

@johnd0e i opened a ready-for-review pull request that adds the respective tests.

[devadeka]

If this is still open could I have a crack at it for hacktoberfest?
Ta

[devcer]

Can I work on this?

[phloose]

i opened a pull request (#7147) in may, which after an initial discussion did not get further attention. So maybe someone could review it?! @johnd0e @mourner ?