kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #16644
Commit breaks bounding box calculation
It’s actually not very important (I guess), but I found that this commit that broke
the pcb bounding box calculation (at least the calculated size doesn’t match the expected
for some reason).
http://bazaar.launchpad.net/~kicad-product-committers/kicad/product/view/head:/qa/testcases/test_002_board_class.py#L36
http://ci.kicad-pcb.org/job/kicad-qa/558/changes
http://ci.kicad-pcb.org/job/kicad-qa/558/console
05:13:18 ====================================================================== 05:13:18 FAIL: test_pcb_bounding_box (test_002_board_class.TestBoardClass) 05:13:18 ---------------------------------------------------------------------- 05:13:18 Traceback (most recent call last): 05:13:18 File "/var/lib/jenkins/workspace/kicad-qa/qa/testcases/test_002_board_class.py", line 54, in test_pcb_bounding_box 05:13:18 self.assertAlmostEqual(width, (30-10) + 0.5, 2) 05:13:18 AssertionError: 21.008002999999999 != 20.5 within 2 places 05:13:18 05:13:18 ----------------------------------------------------------------------
I will try to look at it, but first I need to fix QA to make sure it will catch actual testing errors like this, because if you see
at the end of the console, even if the test started failing long ago we had a big “SUCCESS”.
Miguel Ángel Ajo
Follow ups