Skip to content
This repository has been archived by the owner on Dec 18, 2022. It is now read-only.

Unit Tests #6

Open
transitive-bullshit opened this issue Aug 12, 2015 · 2 comments
Open

Unit Tests #6

transitive-bullshit opened this issue Aug 12, 2015 · 2 comments

Comments

@transitive-bullshit
Copy link

Just wanted to say that I love this library! High quality and solid all around.

Targeted unit tests are going to be very important in the future so we know when Snapchat breaks internal assumptions.

I'm working on a node.js port of SnapchatKit and would love to keep the two projects in sync going forwards.

Cheers!

@NSExceptional
Copy link
Owner

Thanks! That means a lot.

Oh, for sure. I should start working those in in my free time. How do you unit test a neworking library though? I'm not super familiar with unit tests, haha.

Tanner Bennett

On Aug 12, 2015, at 6:08 AM, Travis Fischer [email protected] wrote:

Just wanted to say that I love this library! High quality and solid all around.

Targeted unit tests are going to be very important in the future so we know when Snapchat breaks internal assumptions.

I'm working on a node.js port of SnapchatKit and would love to keep the two projects in sync going forwards.

Cheers!


Reply to this email directly or view it on GitHub.

@transitive-bullshit
Copy link
Author

Hmm, not the best person to ask either :)

http://nshipster.com/unit-testing/ seems like a good place to start

I'd say we have 3 types of tests here:

  • internal functionality that can be tested w/out any network
  • basic tests talking to the real API (e.g., /loq/ping or registering email)
  • tests which would benefit from a shim fake server implementation (we could use this same server implementation to test the objc, node, and php versions!)

@transitive-bullshit transitive-bullshit changed the title Thanks! Unit Tests Aug 18, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants