For http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=910, Tomas's comment says that the Marshal attribute was not significant. Do you still want to use the attribute in the test case?
The Hosting tests are not really Interop, and could be under tests\Hosting, but doesn't really matter. Also, there are DLR hosting tests. Do take a look to see how much of that needs to be tested specially from an IronRuby point of view. Why are you adding tests for the individual methods in IronRuby::Clr::Float? I would just add one test that it is the same as System::Float, and a smoke test for one of the methods. Same for IronRuby::Clr::Integer, etc. Rest looks good. -----Original Message----- From: Jim Deville Sent: Friday, July 24, 2009 12:23 PM To: IronRuby External Code Reviewers; Dino Viehland Cc: [email protected] Subject: Code Review: bugclosing tfpt review "/shelveset:bugclosing;REDMOND\jdeville" Comment : YAML does not properly deserialize Time values Pasted from <http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=910> overriding unsafe methods Pasted from <http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=964> Better error message for running ir.exe on pre-.Net 2.0 SP1 Pasted from <http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=468> rand doesn't work collectly Pasted from <http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=1537> Rubygems failures Pasted from <http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=1626> Adding a random object to Time which responds to to_f Pasted from <http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=1211> ArgumentError when calling System.String..ctor(Char[], int, int) Pasted from <http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=1678> Unverifiable code generated by mspec :lang Pasted from <http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=1533> -X:PrivateBinding does not enable referring to classes that have internal visibility Pasted from <http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=1506> Notes on these: * To get better error messages on pre-.NET 2.0sp1, I moved the error checking code from PythonConsoleHost up to ConsoleHost * As part of writing name_mangling specs, I created stubs for all of the IronRuby module. I'll start filling those specs out going forward. _______________________________________________ Ironruby-core mailing list [email protected] http://rubyforge.org/mailman/listinfo/ironruby-core
