After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 598947 - waittillguiexist() documentation should say if wildcards/regexes are allowed.
waittillguiexist() documentation should say if wildcards/regexes are allowed.
Status: RESOLVED FIXED
Product: LDTP
Classification: Other
Component: ldtp
1.7.0
Other Linux
: Normal normal
: ---
Assigned To: LDTP maintainers
LDTP Development Mailing List
Depends on:
Blocks:
 
 
Reported: 2009-10-19 15:34 UTC by Murray Cumming
Modified: 2009-11-11 05:37 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Murray Cumming 2009-10-19 15:34:49 UTC
I have some ldtp code for Glom whose code comments suggests that waittillguiexist() can use wildcards in the window_name parameter. If that is true then it should be documented:
http://ldtp.freedesktop.org/user-doc/waittillguiexist.html
Comment 1 Murray Cumming 2009-10-19 15:40:04 UTC
This is the code:
http://git.gnome.org/cgit/glom/tree/ldtp/common.py#n12

I am also surprised that it checks for a window with the title "WelcomeToGlom", though the actual window title is "Welcome to Glom" with spaces.
Comment 2 Murray Cumming 2009-10-19 15:40:55 UTC
By the way, I am looking at this code because it doesn't work - the waittillguiexist() calls timeout. However, I am using ldtp from git in jhbuild.
Comment 3 Armin Burgmeier 2009-10-19 15:55:51 UTC
> I am also surprised that it checks for a window with the title "WelcomeToGlom",
> though the actual window title is "Welcome to Glom" with spaces.

This is (was?) intended - see "Window name formats" in the LDTP tutorial from http://ldtp.freedesktop.org/wiki/Docs. In the "Glob pattern support" section it states that Wildcards are allowed. I don't know if things changed in LDTP git in the meanwhile though. I can try if the Glom scripts still work for me.
Comment 4 Murray Cumming 2009-10-19 16:04:55 UTC
(In reply to comment #3)
> > I am also surprised that it checks for a window with the title "WelcomeToGlom",
> > though the actual window title is "Welcome to Glom" with spaces.
> 
> This is (was?) intended - see "Window name formats" in the LDTP tutorial from
> http://ldtp.freedesktop.org/wiki/Docs. In the "Glob pattern support" section it
> states that Wildcards are allowed.

Sorry, I can't find that section. Do you have a more precise URL?

Still, even if it's documented somewhere, that should be mentioned everywhere that the format applies, probably by linking to some common page about the format.

> I don't know if things changed in LDTP git
> in the meanwhile though. I can try if the Glom scripts still work for me.

Thanks, if you have the time.
Comment 5 Armin Burgmeier 2009-10-19 16:13:39 UTC
(In reply to comment #4)
> Sorry, I can't find that section. Do you have a more precise URL?

It is on page 14 in http://download.freedesktop.org/ldtp/doc/ldtp-tutorial.pdf

> Still, even if it's documented somewhere, that should be mentioned everywhere
> that the format applies, probably by linking to some common page about the
> format.

I agree; just wanted to point out where I have the information from.

> > I don't know if things changed in LDTP git
> > in the meanwhile though. I can try if the Glom scripts still work for me.
> 
> Thanks, if you have the time.

Maybe not today, but I'll check this.
Comment 6 Armin Burgmeier 2009-10-20 17:30:12 UTC
So the ldtp tests for the latest glom work for me with the latest LDTP (when running ldtprunner test.xml... is this enough to reproduce the problem for you?)

Have you checked with accerciser that it recognizes the title of the dialog correctly? You should see something like http://arbur.net/stuff/accerciser-glom.png
Comment 7 Murray Cumming 2009-11-02 09:21:36 UTC
Yes, ldtprunner test.xml is enough to reproduce the problem (waittillguiexist() times out).

Yes, I see the same things (as your screenshot) in accerciser.

Anyway, the main point of this bug is still comment #4 about the lack of documentation.
Comment 8 Murray Cumming 2009-11-03 08:39:10 UTC
I have moved the Glom problem to https://bugzilla.gnome.org/show_bug.cgi?id=600523 , though it feels like a general problem in setting up ldtp.
Comment 9 Nagappan Alagappan 2009-11-04 03:13:31 UTC
I will fix the doc :)

Apologize for late response.

Thanks
Comment 10 Nagappan Alagappan 2009-11-11 05:37:01 UTC
Thanks for the bug report. Guess its fixed, could you please confirm, whether this is okay ?

I'm really sorry for the delay.

Thanks