-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Matlab/xUnit unit test framework clone #5
Comments
+1 |
@apjanke Octave#56325 was recently filed to add the Matlab |
Sure sounds like they are. I have commented on Savannah. |
Okay, I think I'm going to try this. It'll be a kinda large effort – the Matlab Unit Testing framework is large-ish, like fifty classes – but this would be a distinctive feature that could help get users interested in adopting Testify. Luckily, the Matlab Unit Testing API is very granular in its class/interface definitions, so it won't be hard to figure out what to do just from the public class documentation. The hard part will be test discovery: Matlab uses labels like
Without some tough parsing, I don't think we'll be able to reliably detect which functions and properties are actually inside the special |
Note to self: I talked to jwe in the #octave Freenode channel, and he says that Octave does not support classdef Events. (It parses them, but they're ignored and there's no functionality for them.) The Matlab Unit Testing API is partially defined in terms of classdef Events, so we won't be able to mimic it exactly. No big deal. It won't be hard to rig up event-handling functionality using regular classes (just like you do in most programming languages). Just means that advanced functionality (like user-defined custom TestRunner plugins) won't be portable between Octave and Matlab. I think this is a minor issue. |
The BIDS-Matlab folks are interested in maybe using this. Bumping up the priority on it. |
Maybe this package should implement a clone of Matlab's xUnit-based unit test framework after all?
If you're a potential user and interested in having this, add a comment on this issue to indicate your interest.
The text was updated successfully, but these errors were encountered: