=== chihchun_afk is now known as chihchun === chihchun is now known as chihchun_afk === chihchun_afk is now known as chihchun === chihchun is now known as chihchun_afk === _salem is now known as salem_ === roadmr is now known as roadmr_afk === chihchun_afk is now known as chihchun === roadmr_afk is now known as roadmr === Saviq_ is now known as Saviq [16:20] doug5, you about? === chihchun is now known as chihchun_afk [18:20] balloons: hey, how can I run a single test from a click suite with adt-run? [18:42] balloons, hellooo [18:48] balloons, haven't started with my next task yet === knome_ is now known as knome [19:26] balloons, could you explain me what needs to be done here? https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/bottomEdge/+merge/241590 [19:30] balloons, and I'm also getting this https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1386639 [19:30] Launchpad bug 1386639 in Ubuntu Calendar App "Can't run calendar app on desktop" [Undecided,New] [19:39] elopio, yes you can [19:39] doug5, hey :-) [19:42] elopio, use ADT_AUTOPILOT_MODULE [19:43] 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] balloons: pitti said I shouldn't use that env var. Is that the only way? [19:52] elopio, indeed you shouldn't. but it does work :-) [19:55] 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] 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] 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] if both are supported than of course you'd want the tests to do both [20:17] balloons, ok, do you know how to disable the calendar sync? (https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1386639) [20:17] Launchpad bug 1386639 in Ubuntu Calendar App "Can't run calendar app on desktop" [Undecided,New] [20:18] doug5, ohh, that bug is preventing you from working on it? [20:19] balloons, I cannot start the application anymore since I checked out the branch [20:19] doug5, let me try and see as well [20:19] balloons, don't know if the two things are related or I did something in the meanwhile [20:20] so I pulled trunk, then merged that branch into it [20:20] running calendar still seems fine. What erros do you get? [20:20] I did directly bzr branch lp:~mihirsoni/ubuntu-calendar-app/bottomEdge [20:24] doug5, ok, I can try that too.. should be more or less the same, but what output do you get? [20:24] balloons, exactly the error in the bug description [20:25] doug5, interesting mihir shows it has to do with online accounts. Do you have your gmail account synced on the desktop? [20:26] if so seems you can do exactly what he shows in the video. [20:27] I likely don't see the issue because I don't have an online account [20:27] balloons, could be...do you know how to disable the sync? [20:27] doug5, yes in online accounts, turn off evolution data sync [20:27] evolution data server I think it's called [20:29] balloons, much better, thx :) [20:31] doug5, :-) great. So anyways, I was telling mihir in the bug report, several other apps have tests that use the bottom edge [20:31] clock uses it as does file manager [20:32] 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] balloons, ok, I see it, but when I click on it nothing happens [20:34] balloons, ah no, got it :) [20:34] balloons, thx [20:35] balloons, I will check with mihir if we have to support both [20:38] 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] balloons, ok [20:38] balloons, thx for the support [20:41] you are welcome. So I can help work on that first part with you now if you have questions [20:45] balloons, ok. Do you know which package contains qml DateTime ? === roadmr is now known as roadmr_afk === salem_ is now known as _salem [21:29] doug5, hmm, yep [21:30] ohh, qml datetime.. [21:30] I assume that's base qt [21:30] qt5-default [21:49] balloons, hey [21:49] Letozaf_, hello [21:50] balloons, I was writing the test for extracting a tar file for filemanager app [21:50] balloons, but the feature has not been merged yet [21:50] balloons, as I am using scenarios I think I need it to or not _ [21:50] _ [21:50] ? [21:51] Letozaf_, where did we leave the zip tests? [21:51] balloons, I completed the test with the cancel unzipping the file test [21:52] balloons, tests passed [21:52] ahh I see :-) [21:52] balloons, now I added scenarios for both zip and tar, but tar has not been merged yet [21:53] ahh I understand now [21:53] :P [21:53] 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] so let's get the zip stuff merged though [21:53] let me check quickly [21:54] balloons, ok [21:55] Letozaf_, so I do have a question about why the generic methods mention zip_dir [21:56] is the change still needed? def go_to_place(self, object_name, zip_dir_path):? [21:56] balloons, right, I shouldn't mention zip as they ar both for tar and zip [21:57] 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] I guess maybe you just wanted to add dir_path to it? [21:57] balloons, I put the zip file in a mocked home dir [21:58] balloons, that's where the zip is and so I called it zip_dir_path [21:58] balloons, should I call it something else ? much clear ? [21:59] 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] right, that all makes sense. Then for go_to_place you sould probably just type the directory name right? [22:01] keep it generic? [22:01] I think changing the name to make it generic fixes everything [22:02] does that make sense? I assume other tests also use go_to_place [22:05] balloons, I think I did not understand the type the directory name, you mean the fake home ? [22:06] Letozaf_, it's clear I'm a little confused, so I'm going to look deeper at the code too :-) [22:06] balloons, I can wait I'm not in a hurry [22:06] my point is you changed go_to_place to have an extra argument [22:06] and that argument is only needed for your zip tests? [22:07] balloons, Oh now I understand [22:07] balloons, yes you are right [22:07] balloons, ok I will type that instead of passing the argument [22:08] well, I'm curious. Isn't there a common method that allows you to send a string to be navigated to [22:08] 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] balloons, ok I understand so I will fix the tests [22:10] balloons, placessidebar is when you are in tablet mode [22:10] 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] balloons, placespage when in phone mode [22:10] right === roadmr_afk is now known as roadmr [22:11] Letozaf_, though, are you sure you even need to navigate to a directory? [22:11] it loads to home automatically [22:12] balloons, oh... I didn't realize that I will check and fix if needed [22:12] yea, lol, you don't, I looked [22:12] 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] probably a test that could be added. check navigating to a folder [22:13] balloons, ok [22:13] make sense? [22:13] It makes sense to me now, lol [22:13] balloons, yes, I will fix everything tomorrow, it's bed time for me now :) [22:14] balloons, thanks [22:15] Letozaf_, thank you! and tehe, I managed to even find another test in there [22:15] balloons, lol [22:15] balloons, good night [22:15] Letozaf_, goodnight!