Hi,

brlcad has left the following comment at Write CoreInterface unit test #3 https://www.google-melange.com/gci/task/view/google/gci2014/5900674122383360:


constructors


1. Those are five different constructors, so yes you should end up testing each of those constructor methods (either directly or indirectly). If the constructors call the 5 different Set functions, then you can get away with just testing the constructors or the Set functions, but the point is to make sure they are all exercised. This is a good one to discuss with andrei or daniel if you can get ahold of them to verify, but the intent of all unit testing is to test everything at least once.

2. A vector defines orientation and length, so while height could be described by a simple length, there'd still need to be a vector to describe the orientation. So it just uses a vector which can encode both. SemiPrincipalAxis is not declared as a size_t. There's a SemiPrincipalAxis() function that takes a size_t, and this is an integer index for a particular axis that was recorded with SetSemiPrincipalAxis(). See the Cone.h header for more details.


Greetings,
The Google Open Source Programs Team


---
You are receiving this message because you are subscribed to Write CoreInterface unit test #3. To stop receiving these messages, go to: https://www.google-melange.com/gci/task/view/google/gci2014/5900674122383360.

------------------------------------------------------------------------------
Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
from Actuate! Instantly Supercharge Your Business Reports and Dashboards
with Interactivity, Sharing, Native Excel Exports, App Integration & more
Get technology previously reserved for billion-dollar corporations, FREE
http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk
_______________________________________________
BRL-CAD Tracker mailing list
brlcad-tracker@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/brlcad-tracker

Reply via email to