check variable types before trying to delete. fixes #96

[justinedelson]

Description

Deleting a secretString variable requires sending the type along with the deletion request. Failure to do so results in an error.

Fixing this also has the nice side-effect of not trying to delete variables that don't exist (thus saving some API calls).

Related Issue

#96

Motivation and Context

See bug report

How Has This Been Tested?

Updated tests

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• I have signed the Adobe Open Source CLA.
• My code follows the code style of this project.
• [] My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[codecov]

Codecov Report

Merging #97 into master will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #97      +/-   ##
==========================================
+ Coverage   92.32%   92.39%   +0.07%     
==========================================
  Files          25       25              
  Lines         860      868       +8     
  Branches      104      106       +2     
==========================================
+ Hits          794      802       +8     
  Misses         60       60              
  Partials        6        6              

Impacted Files	Coverage Δ
src/client.js	83.49% <100.00%> (ø)
...commands/cloudmanager/set-environment-variables.js	97.82% <100.00%> (+0.45%)	⬆️

---

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 08cfb1c...f248f11. Read the comment docs.