Skip to content

CAT-XP6 [AS,KL] StopScriptBrickTest - removed unnecessary code#2196

Closed
kirshanluhana wants to merge 0 commit intoCatrobat:feature/MobapTestRefactoringfrom
kirshanluhana:StopScriptBrickTest
Closed

CAT-XP6 [AS,KL] StopScriptBrickTest - removed unnecessary code#2196
kirshanluhana wants to merge 0 commit intoCatrobat:feature/MobapTestRefactoringfrom
kirshanluhana:StopScriptBrickTest

Conversation

@kirshanluhana
Copy link

Checks if Brick has correct name
Checks if list has correct option names

@catrobatjenkins
Copy link
Contributor

Can a catrobat member please verify this patch? If it doesn't look evil comment with "ok to test" to start the testrun.

1 similar comment
@catrobatjenkins
Copy link
Contributor

Can a catrobat member please verify this patch? If it doesn't look evil comment with "ok to test" to start the testrun.

@84n4n4
Copy link
Contributor

84n4n4 commented Mar 22, 2017

ok to test

@84n4n4
Copy link
Contributor

84n4n4 commented Mar 22, 2017

please change pull request destination to feature/MobapTestRefactoring

@84n4n4
Copy link
Contributor

84n4n4 commented Mar 22, 2017

please fix code style issues:

StopScriptBrickTest.java:27 27 Normal ImportOrderCheck Imports
StopScriptBrickTest.java:51 51 Normal IndentationCheck Indentation

@kirshanluhana kirshanluhana changed the base branch from develop to feature/MobapTestRefactoring March 22, 2017 14:21
@84n4n4
Copy link
Contributor

84n4n4 commented Mar 22, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants