set default map style option to an empty style #6748

[stepankuzmin]

This PR sets the default map style option to an empty style according to #6748
I'm not sure about hardcoding version, maybe there is a more convenient way?

• briefly describe the changes in this PR
• manually test the debug page

[stepankuzmin]

I've noticed, that unit test that checks if the load event is emitted after a style is set is now broken because a style is now always set.

t.test('emits load event after a style is set', (t) => {
    t.stub(Map.prototype, '_detectMissingCSS');
    const map = new Map({ container: window.document.createElement('div') });

    map.on('load', fail);

    setTimeout(() => {
        map.off('load', fail);
        map.on('load', pass);
        map.setStyle(createStyle());
    }, 1);

    function fail() { t.ok(false); }
    function pass() { t.end(); }
});

Should I rewrite this test, or it should work as conceived?

[andrewharvey]

I'm 👍 for this change.

Should I rewrite this test, or it should work as conceived?

I would say rewrite the test, as if the map would now have a default empty style then map.on('load') can be triggered even without setStyle.

[vakila]

Closing this out as it has moved to #8924, thanks @stepankuzmin @arindam1993 !