GNU
/
easejs
1
0
Fork 0
Code Releases Activity

[#25] MemberBuilder/MethodTest - removed unnecessary test

closure/master
Mike Gerwitz 2011-11-02 19:12:15 -04:00
parent 110d937838
commit de78a472f0
1 changed files with 2 additions and 38 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
test/MemberBuilder/MethodTest.js
View File

@ -32,46 +32,10 @@ require( 'common' ).testCase(
); );
this.sut = this.require( 'MemberBuilder' )( this.sut = this.require( 'MemberBuilder' )(
stubFactory, stubFactory stubFactory, stubFactory,
this.mockValidate = this.getMock( 'MemberBuilderValidator' )
); );
this.members = this.sut.initMembers(); this.members = this.sut.initMembers();
}, },
/**
* Unlike languages like C++, ease.js does not automatically mark overridden
* methods as virtual. C# and some other languages offer a 'seal' keyword or
* similar in order to make overridden methods non-virtual. In that sense,
* ease.js will "seal" overrides by default.
*/
'Overridden methods are not virtual by default': function()
{
var name = 'foo';
// declare a virtual method
this.sut.buildMethod( this.members, {}, name, function() {},
{ virtual: true }, function() {}, 1, {}
);
// override it (non-virtual)
this.sut.buildMethod( this.members, {}, name, function() {},
{ override: true }, function() {}, 1, {}
);
// attempt to override again (should fail)
try
{
this.sut.buildMethod( this.members, {}, name, function() {},
{ override: true }, function() {}, 1, {}
);
}
catch ( e )
{
this.incAssertCount();
return;
}
assert.fail( "Overrides should not be declared as virtual by default" );
},
} ); } );