search

SeaCloudsEU / SeaCloudsPlatform

Seamless adaptive multi-cloud management of service-based applications
http://www.seaclouds-project.eu/
Apache License 2.0
18 stars 20 forks source link

Fixes issue reinserting credentials for PaaS nodes into DAM. #298

Closed rosogon closed 8 years ago

rosogon commented 8 years ago

The insertion of credentials into DAM in the last step of wizard was not working for PaaS providers. This PR fixes this.

rosogon commented 8 years ago

@kiuby88 , @perezp : Could you check this?

codecov-io commented 8 years ago

Current coverage is 41.43%

Merging #298 into master will decrease coverage by -6.71% as of 8350a2f

@@            master    #298   diff @@
======================================
  Files          219     365    +146
  Stmts         9045   13234   +4189
  Branches      1322    1672    +350
  Methods          0       0        
======================================
+ Hit           4355    5483   +1128
- Partial        355     432     +77
- Missed        4335    7319   +2984

Review entire Coverage Diff as of 8350a2f

Powered by Codecov. Updated on successful CI builds.

kiuby88 commented 8 years ago

It looks good. I have tested in runtime too. +1 Thanks @rosogon