Nov 6 | 7:20 PM |
Mark M. | has entered the room |
Mark M. | turned on guest access |
Nov 6 | 8:00 PM |
CodeApprentice | has entered the room |
CodeApprentice |
Hello
|
Mark M. |
hello, CodeApprentice!
|
CodeApprentice |
no one here, tonight.
|
Mark M. |
sorry I didn't have more for you on that bump request
|
CodeApprentice |
Or maybe I just missed them, eh?
|
CodeApprentice |
no problem. I just replied to your comment.
|
Mark M. |
no, been a quiet night
|
Mark M. |
I thought perhaps the view was already there
|
Nov 6 | 8:05 PM |
CodeApprentice |
I can probably just call solo.waitForView() to fix everything...trying that now.
|
Mark M. |
and that's why you weren't calling it
|
CodeApprentice |
not entirely sure why I took it out...I'm pretty sure that I had it in there originally.
|
CodeApprentice |
oh wait...you're right.
|
CodeApprentice |
The "Add" menu button is part of the view hierarchy...it's just not visible because the CAB covers the standard action bar.
|
Mark M. |
yeah, that's what I was thinking when pondering your question
|
Mark M. |
so your general approach is probably fine, but you need to figure out a better "wait" algorithm
|
Mark M. |
and that's where I figured Robotium's source might be useful
|
Nov 6 | 8:10 PM |
CodeApprentice |
yah, now that I think about it more, I remember trying to do waitForView() initially and my tests were passing when they should have.
|
CodeApprentice |
s/should/shouldn't/
|
CodeApprentice |
I needed to fix my tests to detect a bug which I recently discovered.
|
CodeApprentice |
and now that I fixed the bug, some of the tests are still failing even though they shouldn't
|
CodeApprentice |
anyways...I'll dig into the Robotium source code.
|
Mark M. |
hah
|
Mark M. |
looks like they just do a busy loop with sleep() calls
|
Mark M. | |
Nov 6 | 8:15 PM |
CodeApprentice |
too bad GitHub doesn't have navigation features like grepcode ;-(
|
Nov 6 | 8:20 PM |
Mark M. |
is there anything else that I can help you with?
|
CodeApprentice |
not right now.
|
CodeApprentice |
are you leaving?
|
Mark M. |
OK
|
Mark M. |
not for another 10 minutes :-)
|
CodeApprentice |
k
|
CodeApprentice |
I'm trying waitForView() just to make sure it doesn't work...
|
CodeApprentice |
hmmm...actually, it seems to work
|
CodeApprentice |
or else the race condition is favoring the test thread now rather than the UI thread.
|
Mark M. |
hmmm...
|
Mark M. |
looks like the waitForView() algorithm may ignore invisible views
|
Nov 6 | 8:25 PM |
CodeApprentice |
that would make sense
|
Mark M. |
so, if the regular action bar buttons are invisible, waitForView() will wait until they are visible again
|
Mark M. |
if I'm reading this correctly
|
CodeApprentice |
are you looking in the searchFor() method?
|
Mark M. |
yes
|
CodeApprentice |
yah, I see a `onlyVisible` flag.
|
CodeApprentice |
I definitely need to practice reading others' code more.
|
Nov 6 | 8:30 PM |
Mark M. |
anyway, that's a wrap for today's chat
|
Mark M. |
next one is Monday at 4pm US Eastern Time
|
Mark M. |
have a pleasant day!
|
CodeApprentice |
gnite
|
CodeApprentice | has left the room |
Mark M. | turned off guest access |