vue-leaflet/Vue2Leaflet

Add unit tests for LPopup

Open

#461 opened on Nov 7, 2019

View on GitHub
 (0 comments) (0 reactions) (0 assignees)JavaScript (1,942 stars) (380 forks)batch import
beginner-friendlyconfirmedhelp wanted

Description

As per title, by following the guidelines of the rest of the test we need to add unit tests for LPopup

Contributor guide

Tech stack
javascriptvue
Domain
frontendtesting
Issue type
test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Familiarity with Vue testing utilities
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
80
Research direction
Look at existing test files in the repository to understand the testing framework and conventions used. The issue mentions following guidelines from other tests, so review tests for similar components like LMap or LTileLayer to replicate the pattern for LPopup. Check the Vue2Leaflet repository's test directory for examples and ensure the new tests cover basic rendering and interaction.