Re: [gnome-desktop-testing] Mago logging

[Michael Fletcher <m fletcher theplanet ca>]

* Add a main index page linking to the test suites
* Show test logs from previous mago invocations

Cool. Those would be really nice features to have. The first, i.e. would be easy to implement and indeed would be helpful.

+1

There are a lot of options for implementing it. The first one that came to mind was wrapping the ooldtp context object and doing a little bit of magic there. Perhaps instead of:

save_dialog = ooldtp.context(self.SAVE_DLG)

you might invoke:

save_dialog = magoldtp.context(self.SAVE_DLG, logger)

Mmm, I don't think I like the idea of having mago wrappers for all ooldtp and ldtp methods. It would be too complex.

I agree with Ara.  These would be difficult to maintain.

 

If we don't like the wording of the LDTP messages to the user, we might think about proposing changes in the LDTP code, instead. Nagappan (LDTP's main dev) is always happy to accept contributions.

Changing the message returned by LDTP looks pretty straightforward.



I agree that displaying the detail of each test would be really useful.  I'd suggest adding an Observer pattern to LDTP and hooking mago into that.  Its more difficult because you need to get patches accepted in two different projects however in the long run it would be easier to maintain for both projects.

What I mean by observer is something like this (and pardon my python):

class LDTPObserver:
   def button_clicked(self, component_name)
       print("Button " + component_Name + " was clicked.")

   def button_error(self, component_name)
       print("Button " + component_name + " did not exist.")

   # etc ....

o = LDTPObserver()

ldtp.register_observer(o)
ldtp.click('frmFoo','btnBar')