Fix the type of optional[int] in dataclass

[pingsutw]

Signed-off-by: Kevin Su [email protected]

TL;DR

https://flyte-org.slack.com/archives/C037MMU74FN/p1660670745909239

@dataclass
class MyDataclass(DataClassJsonMixin):
    value: Optional[int] = None

my_dataclass = MyDataclass(value=1)

my_dataclass.value appears to be converted to float and becomes 1.0 instead of remaining int and 1 as I expected.

Explicitly convert the value to int as well.

Type

• Bug Fix
• Feature
• Plugin

Are all requirements met?

• Code completed
• Smoke tested
• Unit tests added
• Code documentation added
• Any pending items have an associated Issue

Complete description

How did you fix the bug, make the feature etc. Link to any design docs etc

Tracking Issue

flyteorg/flyte#2775

Follow-up issue

NA

[codecov]

Codecov Report

Merging #1135 (9f8a134) into master (73eaad1) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1135      +/-   ##
==========================================
+ Coverage   68.26%   68.28%   +0.01%     
==========================================
  Files         287      287              
  Lines       25829    25843      +14     
  Branches     2885     2887       +2     
==========================================
+ Hits        17633    17646      +13     
- Misses       7719     7720       +1     
  Partials      477      477              

Impacted Files	Coverage Δ
flytekit/core/type_engine.py	58.63% <100.00%> (+0.20%)	⬆️
tests/flytekit/unit/core/test_type_engine.py	98.34% <100.00%> (+0.01%)	⬆️
flytekit/extras/tasks/shell.py	77.71% <0.00%> (-0.45%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.