[16:20] <balloons> doug5, you about?
[18:20] <elopio> balloons: hey, how can I run a single test from a click suite with adt-run?
[18:42] <doug5> balloons, hellooo
[18:48] <doug5> balloons, haven't started with my next task yet
[19:26] <doug5> balloons, could you explain me what needs to be done here? https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/bottomEdge/+merge/241590
[19:30] <doug5> balloons, and I'm also getting this https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1386639
[19:30] <ubot5`> Launchpad bug 1386639 in Ubuntu Calendar App "Can't run calendar app on desktop" [Undecided,New]
[19:39] <balloons> elopio, yes you can
[19:39] <balloons> doug5, hey :-)
[19:42] <balloons> elopio, use ADT_AUTOPILOT_MODULE
[19:43] <balloons> for instance ADT_AUTOPILOT_MODULE="-v music_app.tests.test_music.TestMainWindow.test_url_dispatcher_album_play" adt-run --click com.ubuntu.music --- ssh -s adb
[19:46] <elopio> balloons: pitti said I shouldn't use that env var. Is that the only way?
[19:52] <balloons> elopio, indeed you shouldn't. but it does work :-)
[19:55] <balloons> elopio, we would have to talk to pitti about what the actual plan is/was. In the beginning it was cast aside to focus on making things work. I think actually the official way to do it is to modify the manifest.. hmm
[19:56] <balloons> doug5, so with the bottomedge merge, a helper needs to be added to do the bottom edge swipe action to start off the new event tests
[19:56] <balloons> I believe the old action header button is deprecated and you swipe now, but it's possible both are supported. I'd have to launch the app again to see ;-)
[19:56] <balloons> if both are supported than of course you'd want the tests to do both
[20:17] <doug5> balloons, ok, do you know how to disable the calendar sync? (https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1386639)
[20:17] <ubot5`> Launchpad bug 1386639 in Ubuntu Calendar App "Can't run calendar app on desktop" [Undecided,New]
[20:18] <balloons> doug5, ohh, that bug is preventing you from working on it?
[20:19] <doug5> balloons, I cannot start the application anymore since I checked out the branch
[20:19] <balloons> doug5, let me try and see as well
[20:19] <doug5> balloons, don't know if the two things are related or I did something in the meanwhile
[20:20] <balloons> so I pulled trunk, then merged that branch into it
[20:20] <balloons> running calendar still seems fine. What erros do you get?
[20:20] <doug5> I did directly bzr branch lp:~mihirsoni/ubuntu-calendar-app/bottomEdge
[20:24] <balloons> doug5, ok, I can try that too.. should be more or less the same, but what output do you get?
[20:24] <doug5> balloons, exactly the error in the bug description
[20:25] <balloons> doug5, interesting mihir shows it has to do with online accounts. Do you have your gmail account synced on the desktop?
[20:26] <balloons> if so seems you can do exactly what he shows in the video.
[20:27] <balloons> I likely don't see the issue because I don't have an online account
[20:27] <doug5> balloons, could be...do you know how to disable the sync?
[20:27] <balloons> doug5, yes in online accounts, turn off evolution data sync
[20:27] <balloons> evolution data server I think it's called
[20:29] <doug5> balloons, much better, thx :)
[20:31] <balloons> doug5, :-) great. So anyways, I was telling mihir in the bug report, several other apps have tests that use the bottom edge
[20:31] <balloons> clock uses it as does file manager
[20:32] <balloons> you can use those for help and inspiration. Basically you perform a drag operation on the little stub at the bottom and open it up
[20:33] <doug5> balloons, ok, I see it, but when I click on it nothing happens
[20:34] <doug5> balloons, ah no, got it :)
[20:34] <doug5> balloons, thx
[20:35] <doug5> balloons, I will check with mihir if we have to support both
[20:38] <balloons> doug5, yep he should know what's what. The helper needs to exist either way, so writing it up is the first step
[20:38] <doug5> balloons, ok
[20:38] <doug5> balloons, thx for the support
[20:41] <balloons> you are welcome. So I can help work on that first part with you now if you have questions
[20:45] <doug5> balloons, ok. Do you know which package contains qml DateTime ?
[21:29] <balloons> doug5, hmm, yep
[21:30] <balloons> ohh, qml datetime..
[21:30] <balloons> I assume that's base qt
[21:30] <balloons> qt5-default
[21:49] <Letozaf_> balloons, hey
[21:49] <balloons> Letozaf_, hello
[21:50] <Letozaf_> balloons, I was writing the test for extracting a tar file for filemanager app
[21:50] <Letozaf_> balloons, but the feature has not been merged yet
[21:50] <Letozaf_> balloons, as I am using scenarios I think I need it to or not _
[21:50] <Letozaf_> _
[21:50] <Letozaf_> ?
[21:51] <balloons> Letozaf_, where did we leave the zip tests?
[21:51] <Letozaf_> balloons, I completed the test with the cancel unzipping the file test
[21:52] <Letozaf_> balloons, tests passed
[21:52] <balloons> ahh I see :-)
[21:52] <Letozaf_> balloons, now I added scenarios for both zip and tar, but tar has not been merged yet
[21:53] <balloons> ahh I understand now
[21:53] <Letozaf_> :P
[21:53] <balloons> so right, we won't be able to merge the tar part. I would push to get it included in the main mp actually
[21:53] <balloons> so let's get the zip stuff merged though
[21:53] <balloons> let me check quickly
[21:54] <Letozaf_> balloons, ok
[21:55] <balloons> Letozaf_, so I do have a question about why the generic methods mention zip_dir
[21:56] <balloons> is the change still needed? def go_to_place(self, object_name, zip_dir_path):?
[21:56] <Letozaf_> balloons, right, I shouldn't mention zip as they ar both for tar and zip
[21:57] <balloons> Letozaf_, right.. but also, aren't those methods used by other things? I'm confused why you need to write a zip_dir_path
[21:57] <balloons> I guess maybe you just wanted to add dir_path to it?
[21:57] <Letozaf_> balloons, I put the zip file in a mocked home dir
[21:58] <Letozaf_> balloons, that's where the zip is and so I called it zip_dir_path
[21:58] <Letozaf_> balloons, should I call it something else ? much clear ?
[21:59] <Letozaf_> balloons, I first copy the zip file from the contents directory to a mocked home directory and then extract the files from it
[22:01] <balloons> right, that all makes sense. Then for go_to_place you sould probably just type the directory name right?
[22:01] <balloons> keep it generic?
[22:01] <balloons> I think changing the name to make it generic fixes everything
[22:02] <balloons> does that make sense? I assume other tests also use go_to_place
[22:05] <Letozaf_> balloons, I think I did not understand the type the directory name, you mean the fake home ?
[22:06] <balloons> Letozaf_, it's clear I'm a little confused, so I'm going to look deeper at the code too :-)
[22:06] <Letozaf_> balloons, I can wait I'm not in a hurry
[22:06] <balloons> my point is you changed go_to_place to have an extra argument
[22:06] <balloons> and that argument is only needed for your zip tests?
[22:07] <Letozaf_> balloons, Oh now I understand
[22:07] <Letozaf_> balloons, yes you are right
[22:07] <Letozaf_> balloons, ok I will type that instead of passing the argument
[22:08] <balloons> well, I'm curious. Isn't there a common method that allows you to send a string to be navigated to
[22:08] <balloons> and maybe that is the common method. If so however, I would call the argument you pass a path or something and not say zip specifically
[22:09]  * balloons notes there is a placessidebar and a placespage
[22:09] <Letozaf_> balloons, ok I understand so I will fix the tests
[22:10] <Letozaf_> balloons, placessidebar is when you are in tablet mode
[22:10] <balloons> and yes it looks like that is a good spot for it. So I think that is the common method and the argument just needs to be named dir_path
[22:10] <Letozaf_> balloons, placespage when in phone mode
[22:10] <balloons> right
[22:11] <balloons> Letozaf_, though, are you sure you even need to navigate to a directory?
[22:11] <balloons> it loads to home automatically
[22:12] <Letozaf_> balloons, oh... I didn't realize that I will check and fix if needed
[22:12] <balloons> yea, lol, you don't, I looked
[22:12] <balloons> Letozaf_, however that change is handy to have so people can go to a specific folder
[22:12]  * balloons wonders if there is a test for it..
[22:12] <balloons> probably a test that could be added. check navigating to a folder
[22:13] <Letozaf_> balloons, ok
[22:13] <balloons> make sense?
[22:13] <balloons> It makes sense to me now, lol
[22:13] <Letozaf_> balloons, yes, I will fix everything tomorrow, it's bed time for me now :)
[22:14] <Letozaf_> balloons, thanks
[22:15] <balloons> Letozaf_, thank you! and tehe, I managed to even find another test in there
[22:15] <Letozaf_> balloons, lol
[22:15] <Letozaf_> balloons, good night
[22:15] <balloons> Letozaf_, goodnight!