Eli Mesika has posted comments on this change.
Change subject: [WIP] API: Adding Step mapping class
......................................................................
Patch Set 1: (10 inline comments)
....................................................
File
backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/StepMapper.java
Line 9: import org.ovirt.engine.core.common.job.StepEnum;
Line 10: import org.ovirt.engine.core.compat.Guid;
Line 11: import org.ovirt.engine.core.compat.NGuid;
Line 12:
Line 13: public class StepMapper {
Done
Line 14:
Line 15: @Mapping(from = org.ovirt.engine.core.common.job.Step.class, to =
org.ovirt.engine.api.model.Step.class)
Line 16: public static org.ovirt.engine.api.model.Step
map(org.ovirt.engine.core.common.job.Step entity,
Line 17: org.ovirt.engine.api.model.Step step) {
Line 16: public static org.ovirt.engine.api.model.Step
map(org.ovirt.engine.core.common.job.Step entity,
Line 17: org.ovirt.engine.api.model.Step step) {
Line 18:
Line 19: org.ovirt.engine.api.model.Step model = step != null ? step :
new org.ovirt.engine.api.model.Step();
Line 20: model.setId(String.valueOf(entity.getId()));
Done
Line 21: if (entity.getParentStepId() != null) {
Line 22: org.ovirt.engine.api.model.Step parentStep = new
org.ovirt.engine.api.model.Step();
Line 23: parentStep.setId(String.valueOf(entity.getParentStepId()));
Line 24: model.setParentStep(parentStep);
Line 29: model.setStepType(map(entity.getStepType()));
Line 30: model.setDescription(entity.getDescription());
Line 31: model.setNumber(entity.getStepNumber());
Line 32: model.setStatus(map(entity.getStatus(), null));
Line 33:
model.setStartTime(TypeConversionHelper.toXMLGregorianCalendar(entity.getStartTime(),
null));
Done
Line 34: if (entity.getEndTime() != null) {
Line 35:
model.setEndTime(TypeConversionHelper.toXMLGregorianCalendar(entity.getEndTime(),
null));
Line 36: }
Line 37: model.setExternal(entity.isExternal());
Line 46: target.setId(new Guid(step.getId()));
Line 47: if (step.isSetParentStep()) {
Line 48:
target.setParentStepId(NGuid.createGuidFromString(step.getParentStep().getId()));
Line 49: }
Line 50:
target.setJobId(Guid.createGuidFromString(step.getJob().getId()));
Done
Line 51: target.setStepType(map(step.getStepType()));
Line 52: target.setDescription(step.getDescription());
Line 53: target.setStepNumber(step.getNumber());
Line 54: target.setStatus(map(step.getStatus(), null));
Line 52: target.setDescription(step.getDescription());
Line 53: target.setStepNumber(step.getNumber());
Line 54: target.setStatus(map(step.getStatus(), null));
Line 55:
target.setStartTime(step.getStartTime().toGregorianCalendar().getTime());
Line 56: target.setEndTime(step.isSetStartTime() ?
step.getStartTime().toGregorianCalendar().getTime()
fixed
Line 57: : new
Date((Calendar.getInstance().getTimeInMillis())));
Line 58: target.setExternal(step.isExternal());
Line 59: return target;
Line 60: }
Line 58: target.setExternal(step.isExternal());
Line 59: return target;
Line 60: }
Line 61:
Line 62: @Mapping(from = org.ovirt.engine.api.model.Status.class,
Done moved to JobMapper class
Line 63: to =
org.ovirt.engine.core.common.job.JobExecutionStatus.class)
Line 64: public static org.ovirt.engine.core.common.job.JobExecutionStatus
map(org.ovirt.engine.api.model.Status status,
Line 65: org.ovirt.engine.core.common.job.JobExecutionStatus
incoming) {
Line 66: if
(JobExecutionStatus.STARTED.name().equals(status.getState().toUpperCase())) {
Line 95: st.setState(JobExecutionStatus.ABORTED.name());
Line 96: return st;
Line 97: }
Line 98: if (JobExecutionStatus.FAILED == status) {
Line 99: st.setState(JobExecutionStatus.FAILED.name());
same answer
Line 100: return st;
Line 101: }
Line 102: st.setState(JobExecutionStatus.UNKNOWN.name());
Line 103: return st;
Line 102: st.setState(JobExecutionStatus.UNKNOWN.name());
Line 103: return st;
Line 104: }
Line 105:
Line 106: @Mapping(from = String.class,
Done
Line 107: to = org.ovirt.engine.core.common.job.StepEnum.class)
Line 108: public static org.ovirt.engine.core.common.job.StepEnum
map(String type) {
Line 109: if (StepEnum.VALIDATING.name().equals(type.toUpperCase())) {
Line 110: return StepEnum.VALIDATING;
Line 117: }
Line 118: return StepEnum.UNKNOWN;
Line 119: }
Line 120:
Line 121: @Mapping(from = org.ovirt.engine.core.common.job.StepEnum.class,
Done
Line 122: to = String.class)
Line 123: public static String
map(org.ovirt.engine.core.common.job.StepEnum type) {
Line 124: if (StepEnum.VALIDATING == type) {
Line 125: return StepEnum.VALIDATING.toString();
....................................................
File
backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/StepMapperTest.java
Line 13: @Override
Line 14: protected void verify(Step model, Step transform) {
Line 15: assertNotNull(transform);
Line 16: assertEquals(model.getId(), transform.getId());
Line 17: assertEquals(model.getDescription(),
transform.getDescription());
Done
Line 18: }
Line 19:
Line 20: @Override
Line 21: protected Step postPopulate(Step model) {
--
To view, visit http://gerrit.ovirt.org/16161
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id1b95a094dc586e6ebbdacd44e0a034e91603274
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com>
Gerrit-Reviewer: Ori Liel <ol...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
