Re: [POLICYREP] [PATCH 1/1] policyrep role implementation

[Karl MacMillan <kmacmillan@mentalrootkit.com>]

On Tue, 2007-07-17 at 14:06 -0400, Joshua Brindle wrote:
> Joshua Brindle wrote:
> 
> > --- policyrep-roles.orig/libpolicyrep/tests/libpolicyrep-test.cpp
> > +++ policyrep-roles/libpolicyrep/tests/libpolicyrep-test.cpp
> > @@ -39,7 +39,11 @@ void test()
> >          t->attributes().insert("userdomain");
> >          
> >          mod->append_child(t);
> > -        
> > +
> > +	RolePtr r(new Role("foo"));
> > +	r->types().insert("foo");
> > +	mod->append_child(r);       
> > + 
> >  	std::cout << "============ basic test ============" << std::endl;
> >          output_tree(std::cout, pol);
> >          
> 
> One thing I'd like to note, while this simple testing mechanism is 
> working for now as we add more policy components and do more complex 
> things we are going to need something more sophisticated. Should we move 
> to cppunit or boost testing? We are already using boost so it might be 
> worthwhile to use their test suite (though I've never used it so I can't 
> vouch for it.) <http://www.boost.org/libs/test/doc/index.html>
> 

Fine by me - we can at least try it and go for something else if needed.

Karl


--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.