Fwd: [Juju-gui-peeps] Juju GUI service hiding handoff

[Nicola 'teknico' Larosa]

Gary Poster wrote:
> Forwarding to intended list

As agreed, I'm sending this back to hand it off at my end of day.

I did not make much progress over Benji's code, just some cleanup.

I added the integration tests code from his pastebin as commented code to
the bottom of test/test_environment_view.js .

The unit tests are not passing, they stop with an error at test #90 in
the mentioned file, the isGuiService utility filter is getting the wrong
object.

Code is at lp:~teknico/juju-gui/bug-1090716-wip/ .


> -------- Original Message --------
> Subject: [Juju-gui-peeps] Juju GUI service hiding handoff
> Date: Wed, 9 Jan 2013 21:49:59 +0000
> From: Benji York <benji.york at canonical.com>
> To: juju-gui-peeps at lists.launchpad.net
> 
> Hi all, the branch to hide the Juju GUI service and any relations 
> there-to is just about done, but I will be out until Monday, so I
> need to hand it off.
> 
> As far as I can tell the branch is functionally complete and the
> unit-y tests have all been written.  I was trying to come up with some
> sane integration tests when I ran out of time.  This branch has all
> the work sans integration tests: lp:~benji/juju-gui/bug-1090716-wip
> 
> Here is a paste of the diff containing the integration test that I
> was working toward.  It is far from working, but I hope it gives the
> flavor of what I was trying to do.  If anyone knows a better way to
> use d3's data structures in tests, it could probably be improved
> greatly: http://paste.ubuntu.com/1514637/
> 
> I think I will be able to check email frequently over the next two
> days, so hopefully I will be able to answer questions about the branch
> if necessary.

-- 
Nicola 'teknico' Larosa <https://launchpad.net/~teknico>