SSENSE/vue-carousel

Chore: Increase Code Coverage!

Open

#143 opened on Feb 28, 2018

View on GitHub
 (19 comments) (0 reactions) (0 assignees)JavaScript (513 forks)batch import
good first issuehelp wantedroadmap

Repository metrics

Stars
 (1,705 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

The last few updates have had dropped coverage significantly - I'd like to re-add coveralls checks, but before we do that, it would be nice to have our code coverage/quality be above 80%. This is a great opportunity for someone to learn/practice writing unit tests. If anyone has any questions, feel free to comment here!

I'll leave this issue up for a few weeks, but if we have no biters I'll add them before we move to v7.0 🏁

Contributor guide

Research direction
Look at the existing test suite in the repository, identify components with low coverage, and write unit tests to increase coverage above 80%.
Tech stack
javascriptvue
Domain
frontend
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.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Vue.jsUnit testing
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
85