Skip to content

Constructor argument matching supports simple int to (float|double) types #239

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
sigbjorn wants to merge 16 commits into from
Closed

Constructor argument matching supports simple int to (float|double) types #239

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
2a2c9c7
added intial test that illustrates how int->double conversion works i…
sigbjorn Jun 30, 2016
78dca29
ease windows development, - default settings after checkout reflectin…
sigbjorn Jul 3, 2016
af1b937
added method tests for simple promotions int->float, int->double
sigbjorn Jul 3, 2016
24baadf
extending test to cover negative outcome of bool->double|float
sigbjorn Jul 3, 2016
bde4258
adding default ct to challenge the matching mechanism
sigbjorn Jul 3, 2016
c8c8d8c
merged extra tests
sigbjorn Jul 3, 2016
a65d793
adding test, and print-out warning for the not yet implemented feature
sigbjorn Jul 3, 2016
e698b57
added more test for illustrating super/sub-class and argument passing…
sigbjorn Jul 3, 2016
1fd75b3
fixed py 2.7 syntax for super, plus fixed typo
sigbjorn Jul 4, 2016
fb9b21e
reverted the .sln to unmodified state
sigbjorn Jul 4, 2016
9e1d72b
reverting changes to runtime and clrmodule projects
sigbjorn Jul 4, 2016
0e2cd12
Reverting changes to Console, embed_tests and testing .csproj files
sigbjorn Jul 4, 2016
e58d5b2
Hopefully fixed py 2.6 issues
sigbjorn Jul 4, 2016
8602823
Merge branch 'ct_arg_match_fix' of https://github.com/sigbjorn/python…
sigbjorn Jul 4, 2016
bdbaa62
SimplePromotableType simplified, returns true if src is int-type and …
sigbjorn Jul 23, 2016
478b632
Merge branch 'master' into ct_arg_match_fix
sigbjorn Jul 30, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Hopefully fixed py 2.6 issues
  • Loading branch information
@sigbjorn
sigbjorn committed Jul 4, 2016
commit e58d5b25cf35b9cc28b00425bfd6b24c39eab0d6
5 changes: 4 additions & 1 deletion src/tests/test_constructors.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -121,8 +121,11 @@ def testConstructorRaiseExceptionIfNoMatch(self):


if constructor_throw_on_arg_match_is_fixed:
with self.assertRaises(TypeError):
try:
o = ToDoubleConstructorTest('a','not a number','c') # this should raise exception, because there are no match!
except TypeError:
return
self.fail("exception should be raised for non-matching constructor atempt")
else:
print("\n\n*** Warning: failing arg match on constructors are currently silently accepted if there is a null constructor\n")

Expand Down
20 changes: 11 additions & 9 deletions src/tests/test_method.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -219,10 +219,13 @@ def testSimpleTypePromotionIntToDouble(self):
sum_of_a_plus_b = object.TestSimpleIntToDoubleTypePromotion(2.0,2.0)
self.assertAlmostEqual(sum_of_a_plus_b,2.0+2.0)

with self.assertRaises(TypeError):
#with self.assertRaises(TypeError), wont work on py 2.6, so ..
try:
should_fail = object.TestSimpleIntToDoubleTypePromotion(2,'x2.0')
#with self.assertRaises(TypeError):
# should_fail = object.TestSimpleIntToDoubleTypePromotion(2,True)
except TypeError:
return
self.fail("A string (not convertible to number), should throw type error")
# TODO: consider this for consistence: bool True-> 1.0, False -> 0.0

def testSimpleTypePromotionIntToFloat(self):
object = MethodTest()
Expand All @@ -235,13 +238,12 @@ def testSimpleTypePromotionIntToFloat(self):
self.assertAlmostEqual(sum_of_a_plus_b,2+2)
sum_of_a_plus_b = object.TestSimpleIntToFloatTypePromotion(2.0,2.0)
self.assertAlmostEqual(sum_of_a_plus_b,2.0+2.0)

with self.assertRaises(TypeError):
try:
should_fail = object.TestSimpleIntToFloatTypePromotion(2,'x2.0')
# bool True-> 1.0, False -> 0.0
#with self.assertRaises(TypeError):
# should_fail = object.TestSimpleIntToFloatTypePromotion(2,True)
# print("And the unexpected result is...",should_fail)
except TypeError:
return
self.fail("A string (not convertible to number), should throw type error")
# TODO: consider this for consistence: bool True-> 1.0, False -> 0.0

def testMethodCallStructConversion(self):
"""Test struct conversion in method call."""
Expand Down