refactor(sortable): change getElementScope to increase code coverage

thgreasi · thgreasi · commit 0ad3250a25ce · 2015-01-15T07:51:21.000+02:00
diff --git a/src/sortable.js b/src/sortable.js
@@ -38,13 +38,15 @@ angular.module('ui.sortable', [])
           }
 
           function getElementScope(elementScopes, element) {
+            var result = null;
             for (var i = 0; i < elementScopes.length; i++) {
               var x = elementScopes[i];
               if (x.element[0] === element[0]) {
-                return x.scope;
+                result = x.scope;
+                break;
               }
             }
-            return null;
+            return result;
           }
 
           function afterStop(e, ui) {

Original file line number	Diff line number	Diff line change
`@@ -38,13 +38,15 @@ angular.module('ui.sortable', [])`
`38`	`38`	`}`
`39`	`39`
`40`	`40`	`function getElementScope(elementScopes, element) {`
	`41`	`+ var result = null;`
`41`	`42`	`for (var i = 0; i < elementScopes.length; i++) {`
`42`	`43`	`var x = elementScopes[i];`
`43`	`44`	`if (x.element[0] === element[0]) {`
`44`		`- return x.scope;`
	`45`	`+ result = x.scope;`
	`46`	`+ break;`
`45`	`47`	`}`
`46`	`48`	`}`
`47`		`- return null;`
	`49`	`+ return result;`
`48`	`50`	`}`
`49`	`51`
`50`	`52`	`function afterStop(e, ui) {`