rework unit tests (mocking the fetch())

.babelrc

@@ -12,8 +12,8 @@
         ["babel-plugin-inline-import", {
             "extensions": [
                 ".json",
+                ".geojson",
                 ".glsl",
-                ".gltf",
                 ".css"
             ]
         }],

package.json

@@ -76,6 +76,7 @@
     "@babel/preset-env": "^7.22.5",
     "@babel/register": "^7.22.5",
     "@types/three": "^0.159.0",
+    "@xmldom/xmldom": "^0.8.10",
     "babel-inline-import-loader": "^1.0.1",
     "babel-loader": "^9.1.3",
     "babel-plugin-inline-import": "^3.0.0",
@@ -107,6 +108,7 @@
     "puppeteer": "^21.6.0",
     "q": "^1.5.1",
     "replace-in-file": "^7.0.2",
+    "sinon": "^17.0.1",
     "three": "^0.159.0",
     "typescript": "^5.3.3",
     "webpack": "^5.89.0",

src/Core/Style.js

@@ -10,11 +10,10 @@ import itowns_stroke_single_before from './StyleChunk/itowns_stroke_single_befor
 
 export const cacheStyle = new Cache();
 
-const svg = document.createElementNS('http://www.w3.org/2000/svg', 'svg');
-const matrix = svg.createSVGMatrix();
+const matrix = document.createElementNS('http://www.w3.org/2000/svg', 'svg').createSVGMatrix();
+const canvas = document.createElement('canvas');
 
 const inv255 = 1 / 255;
-const canvas = (typeof document !== 'undefined') ? document.createElement('canvas') : {};
 
 function baseAltitudeDefault(properties, ctx) {
     return ctx?.coordinates?.z || 0;
@@ -1079,7 +1078,7 @@ const CustomStyle = {
     itowns_stroke_single_before,
 };
 
-const customStyleSheet = (typeof document !== 'undefined') ? document.createElement('style') : {};
+const customStyleSheet = document.createElement('style');
 customStyleSheet.type = 'text/css';
 
 Object.keys(CustomStyle).forEach((key) => {

src/Layer/C3DTilesLayer.js

@@ -30,8 +30,8 @@ const update = process3dTilesNode();
 
 /**
  * Find tileId of object
- *
  * @param {THREE.Object3D} object - object
+ *
  * @returns {number} tileId
  */
 function findTileID(object) {
@@ -41,14 +41,13 @@ function findTileID(object) {
         currentObject = currentObject.parent;
         result = currentObject.tileId;
     }
-
     return result;
 }
 
 /**
  * Check if object3d has feature
- *
  * @param {THREE.Object3D} object3d - object3d to check
+ *
  * @returns {boolean} - true if object3d has feature
  */
 function object3DHasFeature(object3d) {
@@ -231,11 +230,13 @@ class C3DTilesLayer extends GeometryLayer {
      * targets picked under specified coordinates. Intersects can be
      * computed with view.pickObjectsAt(..). See fillHTMLWithPickingInfo()
      * in 3dTilesHelper.js for an example.
+     *
      * @returns {C3DTileFeature} - the closest C3DTileFeature of the intersects array
      */
     getC3DTileFeatureFromIntersectsArray(intersects) {
         // find closest intersect with an attributes _BATCHID + face != undefined
         let closestIntersect = null;
+
         for (let index = 0; index < intersects.length; index++) {
             const i = intersects[index];
             if (i.object.geometry &&
@@ -277,7 +278,6 @@ class C3DTilesLayer extends GeometryLayer {
 
     /**
      * Initialize C3DTileFeatures from tileContent
-     *
      * @param {THREE.Object3D} tileContent - tile as THREE.Object3D
      */
     initC3DTileFeatures(tileContent) {
@@ -344,8 +344,8 @@ class C3DTilesLayer extends GeometryLayer {
     /**
      * Update style of the C3DTFeatures, an allowList of tile id can be passed to only update certain tile.
      * Note that this function only update THREE.Object3D materials, in order to see style changes you should call view.notifyChange()
-     *
      * @param {Array<number>|null} [allowTileIdList] - tile ids to allow in updateStyle computation if null all tiles are updated
+     *
      * @returns {boolean} true if style updated false otherwise
      */
     updateStyle(allowTileIdList = null) {

src/Process/3dTilesProcessing.js

@@ -272,7 +272,6 @@ export function process3dTilesNode(cullingTest = $3dTilesCulling, subdivisionTes
             return undefined;
         }
 
-
         // do proper culling
         const isVisible = cullingTest ? (!cullingTest(layer, context.camera, node, node.matrixWorld)) : true;
         node.visible = isVisible;