search

felixge / node-ar-drone

A node.js client for controlling Parrot AR Drone 2.0 quad-copters.
http://nodecopter.com/
MIT License
1.76k stars 428 forks source link

Failed integration test: test/integration/video/PngEncoder/test-sample-encode.js #49

Open eschnou opened 11 years ago

eschnou commented 11 years ago

This test is skipped in travis due to a ffmpeg dependency but still fails locally. No clue if the issue is in the underlying code, or in the test. It seems the issue comes from the number of frames extracted from the test file not being 6 but 5.

[0:00:01 0 13/17 76.5% node test/integration/video/PngEncoder/test-sample-encode.js]

assert.js:102 throw new assert.AssertionError({ ^ AssertionError: 5 == 6 at process. (/data/eschenal/private/Workspace/node-ar-drone/test/integration/video/PngEncoder/test-sample-encode.js:23:10) at process.EventEmitter.emit (events.js:88:17)

yocontra commented 11 years ago

Travis will let you install stuff like ffmpeg before your tests are executed. We might want to look into that

felixge commented 11 years ago

@Contra yeah, that's probably what we want here.

janpieper commented 11 years ago

This could be related to #60.

felixge commented 11 years ago

@janpieper #78 seems more relevant / has more info at this point.

janpieper commented 11 years ago

After upgrading ffmpeg from 0.8.6 (AssertionError: 2 == 6) to 0.10.9, i get the same error as described by @eschnou (AssertionError: 5 == 6).