Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature fix event management #56

Merged
merged 3 commits into from
Apr 30, 2020
Merged

Conversation

pjjmunier
Copy link
Contributor

@pjjmunier pjjmunier commented Apr 24, 2020

Pull request checklist

Verifiez que votre Pull Request remplit les conditions suivantes :

  • Des tests ont été ajoutés pour les changements (corrections de bugs ou features)
  • De la documentation a été mise à jour ou ajoutée si nécessaire (corrections de bugs ou features)
  • Un build (npm run build) a été lancé localement et s'est correctement déroulé
  • Les exemples impactés par les modifications (npm run samples) ont été testés et validés localement
  • Les tests (npm run test) sont passés localement

Type de Pull request

Quel type de changement cette Pull Request introduit-elle :

  • Bugfix
  • Feature
  • Mise à jour du style du code (syntaxe, renommage de fonctions)
  • Refactoring (lisibilité/performance du code, sans changements fonctionnels)
  • Changement sur le processus de build
  • Contenu de la documentation
  • Autres (décrire ci-après) :

Quel est le comportement actuel (avant PR) :

  • la destruction des evenements est buggé dans la fonction switch2D3D

Quel est le nouveau comportement :

  • tous les evenements de l'ancienne carte sont oubliés dans la fonction switch2D3D
  • seuls les evenements de mapEventsOptions sont transférés à la nouvelle carte. Tous les evenements enregistrés apres l'initialisation (via la methode listen) ne seront pas transmis a la nouvelle carte
  • des fonctions d'oubli des evenements et de destruction de la carte sont créées (utilisation possible dans le portail)
  • la fonction 'listen' implemente un controle pour verifier que la fonction evenement est nommée. Voir si ce control n'est pas trop strict

Cette PR introduit-elle des breaking changes ?

  • Oui
  • Non

Autres informations

Copy link
Contributor

@elias75015 elias75015 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tout semble OK !

Les tests plantent sur les événements (pas lié à la PR je crois, je reproduis sur develop), à cause de timeout, aléatoirement j'ai l'impression...peux tu me dire si tu reproduis ? Peut être que c'est lié à mon environnement, je n'avais pas constaté ca avant....

Mais comme cette PR traite de la gestion des événements, si tu reproduis le problème, peut être qu'il serait pertinent de le traiter ici aussi ?

-- Test ItMap --
-- Init Events --
Should correctly launch maploaded event‣
Error: Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.
    at Test.Runnable._timeoutError (eval at module.exports (webpack-internal:///./node_modules/script-loader/addScript.js), <anonymous>:4618:10)
    at eval (eval at module.exports (webpack-internal:///./node_modules/script-loader/addScript.js), <anonymous>:4429:24)
Should correctly launch configured event‣
Error: Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.
    at Test.Runnable._timeoutError (eval at module.exports (webpack-internal:///./node_modules/script-loader/addScript.js), <anonymous>:4618:10)
    at eval (eval at module.exports (webpack-internal:///./node_modules/script-loader/addScript.js), <anonymous>:4429:24)
-- Events --
Should correctly launch zoomChanged event‣
Error: Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.
    at Test.Runnable._timeoutError (eval at module.exports (webpack-internal:///./node_modules/script-loader/addScript.js), <anonymous>:4618:10)
    at eval (eval at module.exports (webpack-internal:///./node_modules/script-loader/addScript.js), <anonymous>:4429:24)
Should correctly launch azimuthChanged event‣
Error: Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.
    at Test.Runnable._timeoutError (eval at module.exports (webpack-internal:///./node_modules/script-loader/addScript.js), <anonymous>:4618:10)
    at eval (eval at module.exports (webpack-internal:///./node_modules/script-loader/addScript.js), <anonymous>:4429:24)

@pjjmunier
Copy link
Contributor Author

Oui j'ai aussi des timeouts 4 sur develop et 2 sur cette branche.
Un problème avec les ressources je pense, le 1er message en console est :
Failed to load resource: the server responded with a status of 404 (Not Found)

@pjjmunier pjjmunier merged commit 6e174da into develop Apr 30, 2020
@pjjmunier pjjmunier deleted the feature-fix_event_management branch April 30, 2020 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants