Re: [ovs-dev] [flow-stats 14/14] netflow: Add basic unit tests.

2011-12-19 Thread Ben Pfaff
Thanks, I fixed both issues. On Mon, Dec 19, 2011 at 02:31:36PM -0800, Ethan Jackson wrote: > The includes in test-netflow.c are not in alphabetical order > (particularly util.h). > > I think you need a comma before "interval" when printing > hdr->sampling_interval. > > Otherwise looks good. >

Re: [ovs-dev] [flow-stats 14/14] netflow: Add basic unit tests.

2011-12-19 Thread Ethan Jackson
The includes in test-netflow.c are not in alphabetical order (particularly util.h). I think you need a comma before "interval" when printing hdr->sampling_interval. Otherwise looks good. Ethan On Thu, Dec 8, 2011 at 14:01, Ben Pfaff wrote: > These tests would have caught the flow statistics bu

[ovs-dev] [flow-stats 14/14] netflow: Add basic unit tests.

2011-12-08 Thread Ben Pfaff
These tests would have caught the flow statistics bug introduced by commit 501f8d1fd75 (ofproto-dpif: Batch interacting with the dpif on flow miss operations.) --- tests/automake.mk |4 + tests/ofproto-dpif.at | 128 +++ tests/test-netflow.c | 323 +++