fix flyte remote bugs

[cosmicBboy]

Signed-off-by: Niels Bantilan [email protected]

TL;DR

Fix FlyteRemote bugs:

• don't override _closure property with self._closure = None in FlyteWorkflowExecution.__init__
• Remove unnecessary check in sync_workflow_execution for ResourceType.WORKFLOW

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

https://github.com/lyft/flyte/issues/

Follow-up issue

NA
OR
https://github.com/lyft/flyte/issues/

[codecov]

Codecov Report

Merging #737 (46e4b5d) into master (673e076) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #737   +/-   ##
=======================================
  Coverage   85.74%   85.75%           
=======================================
  Files         362      362           
  Lines       29996    29994    -2     
  Branches     2444     2443    -1     
=======================================
  Hits        25721    25721           
+ Misses       3632     3631    -1     
+ Partials      643      642    -1     

Impacted Files	Coverage Δ
flytekit/remote/workflow_execution.py	88.88% <ø> (-0.31%)	⬇️
flytekit/remote/remote.py	70.13% <100.00%> (+0.38%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 673e076...46e4b5d. Read the comment docs.