Added comments and fixed small bug.

plugins/org.yakindu.sct.refactoring/src/org/yakindu/sct/refactoring/refactor/impl/FoldIncomingActionsRefactoring.java

@@ -64,7 +64,7 @@ public class FoldIncomingActionsRefactoring extends AbstractRefactoring<State> {
 	public boolean isExecutable() {
 		// TODO check if there is at least one action on each transition
 		return super.isExecutable()
-				&& helper.oneIncomingTransitionEntersCompositeWithEntryActions(getContextObject());
+				&& !helper.oneIncomingTransitionEntersCompositeWithEntryActions(getContextObject());
 	}
 
 

plugins/org.yakindu.sct.refactoring/src/org/yakindu/sct/refactoring/refactor/impl/UnfoldEntryActionsRefactoring.java

@@ -56,7 +56,7 @@ public class UnfoldEntryActionsRefactoring extends AbstractRefactoring<State> {
 	@Override
 	public boolean isExecutable() {
 		return super.isExecutable()
-				&& helper.oneIncomingTransitionEntersCompositeWithEntryActions(getContextObject());
+				&& !helper.oneIncomingTransitionEntersCompositeWithEntryActions(getContextObject());
 	}
 
 	private void unfoldEntryActions() {

test-plugins/org.yakindu.sct.refactoring.tests/src/org/yakindu/sct/refactoring/refactor/FoldIncomingActionsRefactoringTest.java

@@ -21,6 +21,7 @@ import org.yakindu.sct.refactoring.refactor.impl.FoldIncomingActionsRefactoring;
 import com.google.common.collect.Lists;
 
 /**
+ * Test for {@link FoldIncomingActionsRefactoring}.
  * 
  * @author thomas kutz - Initial contribution and API
  * 

test-plugins/org.yakindu.sct.refactoring.tests/src/org/yakindu/sct/refactoring/refactor/FoldOutgoingActionsRefactoringTest.java

@@ -21,6 +21,7 @@ import org.yakindu.sct.refactoring.refactor.impl.FoldOutgoingActionsRefactoring;
 import com.google.common.collect.Lists;
 
 /**
+ * Tests for {@link FoldOutgoingActionsRefactoring}.
  * 
  * @author thomas kutz - Initial contribution and API
  * 

test-plugins/org.yakindu.sct.refactoring.tests/src/org/yakindu/sct/refactoring/refactor/RefactoringTest.java

@@ -1,66 +0,0 @@
-/**
- * Copyright (c) 2013 committers of YAKINDU and others.
- * All rights reserved. This program and the accompanying materials
- * are made available under the terms of the Eclipse Public License v1.0
- * which accompanies this distribution, and is available at
- * http://www.eclipse.org/legal/epl-v10.html
- * Contributors:
- * 	committers of YAKINDU - initial API and implementation
- * 
- */
-package org.yakindu.sct.refactoring.refactor;
-
-import junit.framework.Assert;
-
-import org.eclipse.emf.ecore.util.EcoreUtil;
-import org.eclipse.xtext.junit4.InjectWith;
-import org.eclipse.xtext.junit4.XtextRunner;
-import org.junit.runner.RunWith;
-import org.yakindu.sct.model.sgraph.State;
-import org.yakindu.sct.model.sgraph.Statechart;
-import org.yakindu.sct.model.sgraph.resource.AbstractSCTResource;
-import org.yakindu.sct.refactoring.refactor.util.TestHelper;
-import org.yakindu.sct.refactoring.refactor.util.TestInjectorProvider;
-import org.yakindu.sct.test.models.RefactoringTestModels;
-
-import com.google.inject.Inject;
-
-/**
- * 
- * @author thomas kutz - Initial contribution and API
- * 
- */
-@RunWith(XtextRunner.class)
-@InjectWith(TestInjectorProvider.class)
-public abstract class RefactoringTest {
-
-	@Inject
-	protected RefactoringTestModels models;
-
-	protected TestHelper helper = new TestHelper();
-
-	protected void compareStatecharts(Statechart initial, Statechart expected) {
-		if (!EcoreUtil.equals(initial, expected)) {
-			Assert.fail("Equality check on statecharts failed!");
-		}
-	}
-
-	protected void testRefactoringOnState(String pathToInitialSct,
-			String pathToExpectedSct, String stateName) {
-		Statechart initial = models
-				.loadStatechartFromResource(pathToInitialSct);
-		State stateB = helper.getStateByName(initial, stateName);
-
-		AbstractRefactoring<?> refactoring = getRefactoring(stateB);
-		((AbstractSCTResource) initial.eResource()).setSerializerEnabled(true);
-		refactoring.internalExecute();
-		((AbstractSCTResource) initial.eResource()).setSerializerEnabled(false);
-
-		Statechart expected = models
-				.loadStatechartFromResource(pathToExpectedSct);
-
-		compareStatecharts(initial, expected);
-	}
-
-	protected abstract AbstractRefactoring<?> getRefactoring(State state);
-}

test-plugins/org.yakindu.sct.refactoring.tests/src/org/yakindu/sct/refactoring/refactor/RenameRefactoringTest.java

@@ -36,6 +36,7 @@ import org.yakindu.sct.test.models.RefactoringTestModels;
 import com.google.inject.Inject;
 
 /**
+ * Tests for {@link RenameRefactoring}.
  * 
  * @author thomas kutz - Initial contribution and API
  * 

test-plugins/org.yakindu.sct.refactoring.tests/src/org/yakindu/sct/refactoring/refactor/StateBasedRefactoringTest.java

@@ -27,6 +27,7 @@ import org.yakindu.sct.test.models.RefactoringTestModels;
 import com.google.inject.Inject;
 
 /**
+ * Base test class for state based refactorings.
  * 
  * @author thomas kutz - Initial contribution and API
  * 

test-plugins/org.yakindu.sct.refactoring.tests/src/org/yakindu/sct/refactoring/refactor/UnfoldEntryActionsRefactoringTest.java

@@ -20,6 +20,7 @@ import org.yakindu.sct.refactoring.refactor.impl.UnfoldEntryActionsRefactoring;
 
 import com.google.common.collect.Lists;
 /**
+ * Tests for {@link UnfoldEntryActionsRefactoring}.
  * 
  * @author thomas kutz - Initial contribution and API
  * 

test-plugins/org.yakindu.sct.refactoring.tests/src/org/yakindu/sct/refactoring/refactor/UnfoldExitActionsRefactoringTest.java

@@ -21,6 +21,7 @@ import org.yakindu.sct.refactoring.refactor.impl.UnfoldExitActionsRefactoring;
 import com.google.common.collect.Lists;
 
 /**
+ * Tests for {@link UnfoldExitActionsRefactoring}.
  * 
  * @author thomas kutz - Initial contribution and API
  *