Merge pull request #312 from canjs/remove-observe

Removing can-observe from test that doesn't need it (for IE 11 support)

Author: phillipskevin

package.json

@@ -17,7 +17,7 @@
     "postpublish": "git push --tags && git push",
     "testee": "testee test/test.html --browsers firefox",
     "test": "npm run detect-cycle && npm run jshint && npm run testee",
-    "jshint": "jshint ./*.js --config",
+    "jshint": "jshint ./*.js test/*.js --config",
     "release:pre": "npm version prerelease && npm publish --tag=pre",
     "release:patch": "npm version patch && npm publish",
     "release:minor": "npm version minor && npm publish",

test/component-can-bind-test.js

@@ -3,7 +3,6 @@ var Component = require("can-component");
 var helpers = require("./helpers");
 var QUnit = require("steal-qunit");
 var SimpleMap = require("can-simple-map");
-var stache = require("can-stache");
 var value = require("can-value");
 
 QUnit.module("can-component integration with can-bind");

test/component-define-test.js

@@ -220,7 +220,7 @@ QUnit.test("ViewModel properties default to DefineList if set to an Array (#225)
 });
 
 testHelpers.dev.devOnlyTest("filename should be passed to stache() for inline views", function() {
-	var MyComponent = Component.extend({
+	Component.extend({
 		tag: "my-filename-component",
 		ViewModel: {},
 		view: "{{scope.filename}}"

test/component-events-test.js

@@ -200,7 +200,7 @@ helpers.makeTests("can-component events", function(){
 
 
     QUnit.test('DOM trees not releasing when referencing CanMap inside CanMap in view (#1593)', function() {
-		var undo = domEvents.addEvent(removedEvent)
+		var undo = domEvents.addEvent(removedEvent);
 
         var baseTemplate = stache('{{#if show}}<my-outside></my-outside>{{/if}}'),
             show = new SimpleObservable(true),

test/component-helpers-test.js

@@ -1,14 +1,9 @@
 var QUnit = require("steal-qunit");
 
 var helpers = require("./helpers");
-var SimpleMap = require("can-simple-map");
 var stache = require("can-stache");
 var Component = require("can-component");
 
-var innerHTML = function(el){
-    return el && el.innerHTML;
-};
-
 helpers.makeTests("can-component helpers", function(){
 
     QUnit.test("helpers reference the correct instance (#515)", function () {

test/component-in-stache-test.js

@@ -5,9 +5,6 @@ var stache = require("can-stache");
 var canReflect = require("can-reflect");
 var QUnit = require("steal-qunit");
 
-var viewModel = require("can-view-model");
-var canDev = require("can-log/dev/dev");
-var testHelpers = require("can-test-helpers");
 var domMutate = require("can-dom-mutate");
 var globals = require("can-globals");
 
@@ -138,7 +135,7 @@ QUnit.test("Cleans up itself on the documentElement removal", function() {
 		globals.setKeyValue("document", realDoc);
 		QUnit.ok(true, "Called back without throwing");
 		start();
-	})
+	});
 
 	stop();
 

test/component-instantiation-test.js

@@ -179,7 +179,6 @@ QUnit.test("Components can be instantiated with viewModel", function() {
 			nullProp: null
 		}
 	});
-	var element = componentInstance.element;
 	var viewModel = componentInstance.viewModel;
 
 	// Initial values are correct
@@ -326,7 +325,7 @@ QUnit.test("component instantiation is not observable", function(){
 	});
 
 	var view = stache("<outer-component-creator/>");
-	frag = view();
+	view();
 
 	innerViewModel.innerValue = "SOME-VALUE";
 

test/component-slot-test.js

@@ -210,7 +210,6 @@ test("<can-slot> Context one-way binding works", function() {
 
 	equal(frag.firstChild.firstChild.innerHTML, 'Later Gator');
 });
-var queues = require("can-queues");
 
 test("<can-slot> Context two-way binding works", function() {
 	/*Passing in a custom context like <can-slot name='subject' {(context)}='value' />*/

test/component-tag-test.js

@@ -1,17 +1,9 @@
 var QUnit = require("steal-qunit");
 
-var helpers = require("./helpers")
+var helpers = require("./helpers");
 var SimpleMap = require("can-simple-map");
 var stache = require("can-stache");
 var Component = require("can-component");
-var canViewModel = require('can-view-model');
-var SetterObservable = require("can-simple-observable/setter/setter");
-var SimpleObservable = require("can-simple-observable");
-var domMutateNode = require('can-dom-mutate/node');
-
-var innerHTML = function(el){
-    return el && el.innerHTML;
-};
 
 helpers.makeTests("can-component tag", function(){
 

test/component-view-test.js

@@ -14,7 +14,6 @@ var queues = require("can-queues");
 var getFragment = require("can-fragment");
 var viewCallbacks = require("can-view-callbacks");
 var Scope = require("can-view-scope");
-var observe = require("can-observe");
 
 var innerHTML = function(el){
     return el && el.innerHTML;
@@ -267,8 +266,6 @@ helpers.makeTests("can-component views", function(doc, runTestInOnlyDocument){
         equal( innerHTML(frag.firstChild), '', 'child component is removed');
     });
 
-    var queues = require("can-queues");
-
     QUnit.test("references scopes are available to bindings nested in components (#2029)", function(){
 
         var renderer = stache('<export-er value:to="scope.vars.reference" />'+