Test environment #123

Merged
merged 4 commits into from Apr 12, 2013

Conversation

Projects
None yet
2 participants
@iirvine
Contributor

iirvine commented Apr 8, 2013

Hello! I thought it might be about time for this plugin to have some test coverage. To that end I've gone ahead and stubbed out a little test environment.

Mostly just following Leaflet's lead here - Mocha + expect + sinon + karma. npm install and npm test should be all that's needed to get it up and running. All that's left now is actually writing some tests....

iirvine added some commits Apr 8, 2013

Setting up test environment (Karma + Mocha) for Leaflet.Draw.
	-added Leaflet and testing dependencies to package.json
	-added spec folder + test libraries
	-added 'test' task to build.js and package.json
	-added karma config file (spec/karma.conf.js) and some miscellaneous spec helpers (leaflet-draw-include.js, leaflet-include.js, leaflet-deps.js)
@jacobtoye

This comment has been minimized.

Show comment Hide comment
@jacobtoye

jacobtoye Apr 12, 2013

Owner

Awesome! you rock @iirvine

I've been spread pretty thin lately with work commitments, however look forward to getting some test coverage!

Owner

jacobtoye commented Apr 12, 2013

Awesome! you rock @iirvine

I've been spread pretty thin lately with work commitments, however look forward to getting some test coverage!

jacobtoye added a commit that referenced this pull request Apr 12, 2013

@jacobtoye jacobtoye merged commit 96a981b into Leaflet:master Apr 12, 2013

@iirvine iirvine deleted the iirvine:test_env branch Apr 12, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment