Skip to content

Commit c0dddca

Browse files
author
SadiJr
committed
Merge branch 'issue-1180' into '4.18.0.0-scclouds'
Realização do _backport_ da nova mudança na API `deleteTemplate` Closes apache#1180 See merge request scclouds/scclouds!754
2 parents 9f7f294 + bfdc1ef commit c0dddca

2 files changed

Lines changed: 12 additions & 7 deletions

File tree

api/src/main/java/org/apache/cloudstack/api/command/user/template/DeleteTemplateCmd.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
// under the License.
1717
package org.apache.cloudstack.api.command.user.template;
1818

19+
import org.apache.commons.lang3.BooleanUtils;
1920
import org.apache.log4j.Logger;
2021

2122
import org.apache.cloudstack.api.APICommand;
@@ -67,7 +68,7 @@ public Long getZoneId() {
6768
}
6869

6970
public boolean isForced() {
70-
return (forced != null) ? forced : true;
71+
return BooleanUtils.toBooleanDefaultIfNull(forced, false);
7172
}
7273
/////////////////////////////////////////////////////
7374
/////////////// API Implementation///////////////////

test/integration/component/test_templates.py

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -123,12 +123,16 @@ def setUp(self):
123123

124124
def tearDown(self):
125125
try:
126-
# Clean up, terminate the created templates
127-
cleanup_resources(self.apiclient, self.cleanup)
128-
129-
except Exception as e:
130-
raise Exception("Warning: Exception during cleanup : %s" % e)
131-
return
126+
# Clean up the created templates
127+
for temp in self.cleanup:
128+
cmd = deleteTemplate.deleteTemplateCmd()
129+
cmd.id = temp.id
130+
cmd.zoneid = self.zone.id
131+
cmd.forced = True
132+
self.apiclient.deleteTemplate(cmd)
133+
except Exception as e:
134+
raise Exception("Warning: Exception during cleanup : %s" % e)
135+
return
132136

133137
@classmethod
134138
def setUpClass(cls):

0 commit comments

Comments
 (0)