FeatureLevelRemove- and SourceLevelRemove-Benchmarks in remove_paint_state.js are exactly the same · mapbox/mapbox-gl-js#10049

Métriques du dépôt

Unless I'm missing something, both benchmarks were already the same when they were added in #7761 by @peterqliu :

I haven't used these APIs yet, but I believe the SourceLevelRemove should have used something like this?

// No loop
this.map.removeFeatureState({source: 'land'});

Maybe also source-property level like this?

// No loop
this.map.removeFeatureState({source: 'land'}, 'bench');

Direction de recherche: Le problème identifie que les benchmarks FeatureLevelRemove et SourceLevelRemove dans bench/benchmarks/remove paint state.js (lignes 88-100 et 102-114) sont identiques, ce qui est probablement une erreur de copier coller. Le benchmark SourceLevelRemove doit utiliser `map.removeFeatureState({source: 'land'})` au lieu de boucler sur les entités. Consultez la documentation de l'API mapbox gl js pour `removeFeatureState` et envisagez si une suppression au niveau de la propriété de source (par exemple `map.removeFeatureState({source: 'land'}, 'bench')`) est plus appropriée. La correction consiste à modifier la fonction de benchmark SourceLevelRemove pour remplacer la boucle par l'appel API correct, en veillant à ce qu'elle teste la fonctionnalité prévue. Après avoir effectué la modification, exécutez les benchmarks pour vérifier leur exactitude.
Stack technique: javascript
Domaine: testing
Type d'issue: Bug
Difficulté: 2
Temps estimé: Moins d'une heure
Statut d'activité: Récente
Clarté: Claire
Prérequis: JavaScriptGit
Accessibilité débutant: 60