Permalink
Browse files

- fixed 1 leaked global

  • Loading branch information...
1 parent 2487e6a commit 470b89d95453b844b0c38634b6769c887ffe1822 @SamuraiJack SamuraiJack committed Aug 18, 2011
View
@@ -12,7 +12,7 @@ Release history for Joose:
[BUG FIXES]
- - none
+ - fixed 1 leaked global
3.018 2011-03-12 12:58
View
@@ -75,7 +75,7 @@ INSTALLATION
From [npm](http://github.com/isaacs/npm/)
- [sudo] npm install joose
+ [sudo] npm install joose --unsafe-perm=true
For other options please refer to: <http://joose.github.com/Joose/doc/html/Joose/Manual/Installation.html>
View
@@ -71,3 +71,6 @@ push_to = origin
tweet_url = http://joose.github.com/Joose
tweet = Released {{$DIST}}-{{$VERSION}} {{$URL}}
hash_tags = #npm #node.js
+
+[InstallRelease]
+install_command = sudo npm install . -g
@@ -47,7 +47,7 @@ Joose.Namespace.Manager = new Joose.Managed.Class('Joose.Namespace.Manager', {
return me.create(nsName.replace(/^\./, ''), metaClass, extend)
})
- props = extend || {}
+ var props = extend || {}
var parts = Joose.S.saneSplit(nsName, '.')
var object = this.getCurrent()
View
@@ -1,5 +1,4 @@
StartTest(function (t) {
- t.plan(29)
//==================================================================================================================================================================================
t.diag("Joose.Proto.Class")
@@ -103,4 +102,5 @@ StartTest(function (t) {
t.is("" + testClass1, 'a TestClass1', "testClass1 stringified correctly")
t.is("" + testClass1.meta,'a Joose.Proto.Class', "testClass1's meta stringified correctly")
+ t.done()
})
View
@@ -1,5 +1,4 @@
StartTest(function (t) {
- t.plan(38)
//==================================================================================================================================================================================
t.diag("Joose.Managed.PropertySet")
@@ -134,4 +133,5 @@ StartTest(function (t) {
t.ok(E1.haveProperty('A2'), "E1 still received A2 from B")
t.ok(E1.getProperty('A2') == B.getProperty('A2'), "E1 still received A2 from B")
+ t.done()
})
@@ -1,12 +1,15 @@
StartTest(function (t) {
- t.plan(53)
//==================================================================================================================================================================================
t.diag("Joose.Managed.PropertySet.Mutable")
t.ok(Joose.Managed.PropertySet.Mutable, "Joose.Managed.PropertySet.Mutable is here")
-
+//
+// t.fail('yo')
+//
+// throw "yo"
+
//==================================================================================================================================================================================
t.diag("Sanity")
@@ -189,4 +192,5 @@ StartTest(function (t) {
t.ok(!(F.getProperty('A1') instanceof Joose.Managed.Property.ConflictMarker), 'A1 of F is now not a conflict marker')
t.ok(F.getProperty('A1') == A.getProperty('A1'), 'A1 of F is now obtained from A')
+ t.done()
})
View
@@ -49,6 +49,8 @@ StartTest(function (t) {
})
+ t.expectGlobals('TestRole', 'TestRole2', 'TestClass')
+
t.ok(TestClass.my.res == 'class', "Symbiont is accesible via 'my' in prototype")
t.is(TestClass.my.result(), 'TestClass:class', "`result` method works correctly")
View
@@ -3,6 +3,8 @@ StartTest(function (t) {
//==================================================================================================================================================================================
t.diag("Modules")
+ t.autoCheckGlobals = false
+
t.ok(Joose.Namespace.Manager, "Joose.Namespace.Manager is here")
t.ok(Joose.Namespace.Keeper, "Joose.Namespace.Keeper is here")
@@ -1,7 +1,5 @@
StartTest(function (t) {
- t.plan(6)
-
//==================================================================================================================================================================================
t.diag("Initialization of advanced attributes in subclasses of Joose.Meta.Class")
@@ -70,5 +68,9 @@ StartTest(function (t) {
var testMeta2 = new TestMeta2('NewRole', {})
t.ok(testMeta2.attr == '123', "Attribute 'attr' was correctly initialized for roles")
+
+ t.expectGlobals('TestMeta', 'TestMeta2')
+
+ t.done()
})
@@ -1,7 +1,5 @@
StartTest(function (t) {
- t.plan(19)
-
//==================================================================================================================================================================================
t.diag("Advanced attributes and class's body")
@@ -111,5 +109,9 @@ StartTest(function (t) {
t.ok(bar.regex1.test(/123abc123/), 'Attribute was correctly initialized with RegExp #1')
t.ok(bar.regex2.test(/123abc123/), 'Attribute was correctly initialized with RegExp #2')
+
+ t.autoCheckGlobals = false
+
+ t.done()
})
@@ -96,6 +96,8 @@ StartTest(function (t) {
t.like(test.setMyAttr2.__CONTAIN__.toString(), /setRawValueTo/, 'Call to `setRawValueTo` has not been inlined')
t.like(test.getMyAttr2.__CONTAIN__.toString(), /getRawValueFrom/, 'Call to `setRawValueTo` has not been inlined')
+ t.expectGlobals('My', 'Test')
+
t.done()
})
View
@@ -3,6 +3,8 @@ StartTest(function (t) {
//==================================================================================================================================================================================
t.diag("Modules")
+ t.autoCheckGlobals = false
+
t.ok(Joose.Namespace.Manager, "Joose.Namespace.Manager is here")
t.ok(Joose.Namespace.Keeper, "Joose.Namespace.Keeper is here")
View
@@ -64,7 +64,7 @@ StartTest(function (t) {
}, Joose.is_IE ? "" : "Double declaration of [Test]", "Double class declaration is detected")
-
+ t.expectGlobals('Test', 'Base')
t.done()
})
@@ -59,6 +59,7 @@ StartTest(function (t) {
a2.method1()
+ t.expectGlobals('TestClass1', 'TestClass2')
t.done()
})
@@ -134,6 +134,7 @@ StartTest(function (t) {
t.ok(obj2.param2 == 10, '`Correct parameter #2')
t.ok(obj2.attribute1 == 'foo', '`Correct parameter #3')
+ t.expectGlobals('TestClass1', 'TestClass2')
t.done()
})
View
@@ -29,5 +29,7 @@ StartTest(function (t) {
}, Joose.is_IE ? '' : 'Attempt to consume an undefined Role into [TestClass2]', 'Empty Role is detected')
+ t.expectGlobals('TestClass1', 'TestClass2')
+
t.done()
})
View
@@ -1,10 +1,12 @@
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html>
<head>
- <link rel="stylesheet" type="text/css" href="/jsan/Task/ExtJS/resources/css/ext-all.css">
- <link rel="stylesheet" type="text/css" href="/jsan/Test/Run/static/css/all.css">
+ <link rel="stylesheet" type="text/css" href="../../ext-4.0.2a/resources/css/ext-all.css">
+ <link rel="stylesheet" type="text/css" href="/workspace/JavaScript/BuildFarm/Test-Run/resources/css/test-run-all.css">
- <script type="text/javascript" src="/jsan/Task/Test/Run/WebBundle.js"></script>
+ <script type="text/javascript" src="../../ext-4.0.2a/ext-all-debug.js"></script>
+
+ <script type="text/javascript" src="/workspace/JavaScript/BuildFarm/Test-Run/test-run-all.js"></script>
<script type="text/javascript" src="index.js"></script>
</head>
View
@@ -13,46 +13,86 @@ Harness.configure({
verbosity : 0,
- transparentEx : true,
-// runCore : 'sequential',
+// cachePreload : true,
+// transparentEx : true,
+
+ autoCheckGlobals : true,
+ expectedGlobals : [
+ 'Joose', 'Class', 'Role', 'Module'
+ ],
preload : [
- 'Task.Joose.Core'
+ '../lib/Task/Joose/Core.js'
]
})
Harness.start(
'001_helpers.t.js',
- '010_proto_class.t.js',
- '011_propertyset.t.js',
- '012_propertyset_mutable.t.js',
- '020_managed_class.t.js',
- '021_method_modifiers.t.js',
- '022_inheriting_from_proto_class.t.js',
- '023_builder_stem_inheritance.t.js',
- '024_builder_stem_inheritance.t.js',
- '031_managed_role.t.js',
- '032_role_application_basic.t.js',
- '033_role_application_sugar.t.js',
- '040_my_symbiont.t.js',
- '041_my_mutation.t.js',
- '045_role_builder.t.js',
- '050_helpers.t.js',
- '051_advanced_attribute.t.js',
- '052_advanced_attribute.t.js',
- '052_role_application_advanced.t.js',
- '052_adv_attr_in_metaclasses.t.js',
- '052_advanced_attribute_set_raw_inlining.t.js',
- '053_using_class_as_role.t.js',
- '054_meta_roles.t.js',
- '055_role_to_instance_application.t.js',
- '056_arbitrary_object_from_constructor.t.js',
- '060_modules.t.js',
- '061_modules.t.js',
- '070_reflection.t.js',
- '071_reflection_current_method.t.js',
- '080_non_joose_inheritance.t.js',
- '090_sanity_checks.t.js'
+ {
+ group : 'Proto class tests',
+
+ items : [
+ '010_proto_class.t.js',
+ '011_propertyset.t.js',
+ '012_propertyset_mutable.t.js'
+ ]
+ },
+ {
+ group : 'Managed class tests',
+
+ items : [
+ {
+ url : '020_managed_class.t.js',
+ forceDOMVisible : true
+ },
+ '021_method_modifiers.t.js',
+ '022_inheriting_from_proto_class.t.js',
+ '023_builder_stem_inheritance.t.js',
+ '024_builder_stem_inheritance.t.js',
+ '031_managed_role.t.js',
+ '032_role_application_basic.t.js',
+ '033_role_application_sugar.t.js',
+ {
+ url : '040_my_symbiont.t.js',
+ forceDOMVisible : true
+ },
+
+ '041_my_mutation.t.js',
+ '045_role_builder.t.js'
+ ]
+ },
+ {
+ group : 'Meta level',
+
+ items : [
+ '050_helpers.t.js',
+ '051_advanced_attribute.t.js',
+ '052_advanced_attribute.t.js',
+ '052_role_application_advanced.t.js',
+ '052_adv_attr_in_metaclasses.t.js',
+ '052_advanced_attribute_set_raw_inlining.t.js',
+ '053_using_class_as_role.t.js',
+ '054_meta_roles.t.js',
+ '055_role_to_instance_application.t.js',
+ '056_arbitrary_object_from_constructor.t.js',
+ {
+ group : 'Misc',
+
+ items : [
+ {
+ url : '060_modules.t.js',
+ forceDOMVisible : true
+ },
+
+ '061_modules.t.js',
+ '070_reflection.t.js',
+ '071_reflection_current_method.t.js',
+ '080_non_joose_inheritance.t.js',
+ '090_sanity_checks.t.js'
+ ]
+ }
+ ]
+ }
)

0 comments on commit 470b89d

Please sign in to comment.