Message5576
Dear, John.
(****)
> In your patch for roundup/hyperdb.py you have:
> but there is no translation function/variable/... in the current
> roundup code.
It seems to be related to my patch "[issue2550871] Extending
translation ability (for date at least)".
> I removed that argument and it seems to have stopped the tests
> from crashing. I also reverted a change you made to the testNumber
> function in test/test_hyperdbvals.py.
> - self.assertEqual(self._test('number', ' 10 '), 10)
> + self.assertEqual(self._test('number', ' 1 '), 1)
> I reinstated the test against 10 and removed the test against 1.
I have no objection. I've set 10 to distinct from boolean values which
may be presented as 1/0. I've thought that there may be a bad case when
test for 1 passed but internally parsed as Boolean value. This is the
reason why I use 10. But it doesn't matter.
> I also added test for:
> self.assertEqual(self._test('integer', ' 0 '), 0)
> self.assertNotEqual(self._test('integer', ' -100.2 '), -100.2)
> to testInteger since I assume it should truncate the .2, and I wanted to
> test 0 as the midpoint of the range.
I really welcome extending the test cases.
> I also updated the docs to include Integer in the places where Number
> was mentioned.
It's the most hard work for me because of my poor English. Really
thank you that you made it.
> I am currently running:
> python run_tests.py -k 'not test_postgresql'
> if it all passes I plan on checking it in unless you have an issue
> with my changes above.
If review of patch "[issue2550871] Extending
translation ability" is delayed for indefinitely then support of Integer
may be applied as described above. If progress on [issue2550871] is
coming then Integer support may be applied without changes
mentioned in (****).
> ________________________________________________
> Roundup tracker <issues@roundup-tracker.org>
> <http://issues.roundup-tracker.org/issue2550886>
> ________________________________________________ |
|
Date |
User |
Action |
Args |
2016-06-04 10:41:07 | antmail | set | recipients:
+ antmail, rouilj |
2016-06-04 10:41:07 | antmail | link | issue2550886 messages |
2016-06-04 10:41:06 | antmail | create | |
|