andreamazz/AMScrollingNavbar

Add test suite

Open

#131 opened on May 24, 2015

View on GitHub
 (1 comment) (0 reactions) (0 assignees)Swift (632 forks)batch import
enhancementhelp wanted

Repository metrics

Stars
 (6,074 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

I never really tackled the testing of this library since a lot of the gesture driven behaviours are pretty hard to test, but it's time to.

Tooling: Quick + Nimble + Snapshot

Areas to look at:

  • Add test suite
  • Add coveralls.io
  • Improve test coverage

Contributor guide

Research direction
Explore the existing codebase and identify gesture driven behaviors that need testing. Look at the list of areas: add test suite, coveralls.io, improve coverage. Start by writing unit tests for public API methods using Quick and Nimble, and snapshot tests for UI states.
Tech stack
swift
Domain
mobile
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
SwiftQuickNimbleSnapshot testing
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60