Rubinius::Debugger::Frame specs #957

Closed
wants to merge 3 commits into from

3 participants

@txus
Rubinius member

Hi! I've started working on the debugger specs, since apparently it has been heavily rewritten and the current specs are totally out of date. So I've started with Frame, since it's a simple class.

To set a controlled environment for the Frame, I had to come up with some hackish instance_evals. For example, since Rubinius::Location is mainly instantiated from C++, it has neither setters nor an initialize method to set variables, method, etc. (off-topic question: would it make sense to make this public API? Even if it is just to make testing easier).

@steveklabnik
Rubinius member

I'd love to merge this in, but it needs rebased off of the current work. Would you mind doing that for me?

@txus
Rubinius member

Of course :) I'll have to reread a bit the commits since it's been a while and I had it a bit abandoned. Will rebase!

@steveklabnik
Rubinius member

No worries, thanks!

@henrikhodne
Rubinius member

@txus Any progress on this?

@txus txus closed this Sep 27, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment