fuwaneko / node-protobuf

Google Protocol Buffers wrapper for Node.js [UNMAINTAINED]
179 stars 42 forks source link

Adding some basic tests #22

Closed mshick closed 10 years ago

mshick commented 10 years ago

Adding a few simple tests to do the following:

mshick commented 10 years ago

Oops. Missed the fact we need to recompile the protobuf bin for this test to be useful... I'll explore how to do that locally (within the project itself, I mean). Maybe all it takes is 'npm install' but I'll need to test to be sure.

mshick commented 10 years ago

I now see that npm install builds the binary, so the test seems valid.

I do see that Travis is failing -- I wonder if you have any idea how to declare system resources in this way, since the protoc command is assumed to exist, or if this sort of thing simply needs to be excluded from Travis?

mshick commented 10 years ago

Okay, added the protobuf-compiler to the Travis config. Hopefully it will pass now. :pray:

fuwaneko commented 10 years ago

Added proper test suite.