Closed arbulu89 closed 3 years ago
Merging #182 (a3685ff) into master (12ec919) will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #182 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 24 24
Lines 697 702 +5
=========================================
+ Hits 697 702 +5
Impacted Files | Coverage Δ | |
---|---|---|
app/controllers/deploys_controller.rb | 100.00% <100.00%> (ø) |
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 12ec919...a3685ff. Read the comment docs.
Fix to manage properly the terraform stdout content.
In the case of an error, the
content
was set with the error message, so it was not getting the correct data to create the progress information. Besides that, the text for failed scenario was not set.Rarely, the terraform stdout was coming with
nil
value. Theblank
method adds a safe guard to that scenario, returning an empty progress information.