| === 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 | ||
| balloons | doug5, you about? | 16:20 |
|---|---|---|
| === chihchun is now known as chihchun_afk | ||
| elopio | balloons: hey, how can I run a single test from a click suite with adt-run? | 18:20 |
| doug5 | balloons, hellooo | 18:42 |
| doug5 | balloons, haven't started with my next task yet | 18:48 |
| === knome_ is now known as knome | ||
| doug5 | balloons, could you explain me what needs to be done here? https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/bottomEdge/+merge/241590 | 19:26 |
| 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:30 |
| balloons | elopio, yes you can | 19:39 |
| balloons | doug5, hey :-) | 19:39 |
| balloons | elopio, use ADT_AUTOPILOT_MODULE | 19:42 |
| 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:43 |
| elopio | balloons: pitti said I shouldn't use that env var. Is that the only way? | 19:46 |
| balloons | elopio, indeed you shouldn't. but it does work :-) | 19:52 |
| 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:55 |
| 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 | 19:56 |
| 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:17 |
| balloons | doug5, ohh, that bug is preventing you from working on it? | 20:18 |
| 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:19 |
| 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:20 |
| 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:24 |
| balloons | doug5, interesting mihir shows it has to do with online accounts. Do you have your gmail account synced on the desktop? | 20:25 |
| balloons | if so seems you can do exactly what he shows in the video. | 20:26 |
| 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:27 |
| doug5 | balloons, much better, thx :) | 20:29 |
| 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:31 |
| 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:32 |
| doug5 | balloons, ok, I see it, but when I click on it nothing happens | 20:33 |
| doug5 | balloons, ah no, got it :) | 20:34 |
| doug5 | balloons, thx | 20:34 |
| doug5 | balloons, I will check with mihir if we have to support both | 20:35 |
| 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:38 |
| balloons | you are welcome. So I can help work on that first part with you now if you have questions | 20:41 |
| doug5 | balloons, ok. Do you know which package contains qml DateTime ? | 20:45 |
| === roadmr is now known as roadmr_afk | ||
| === salem_ is now known as _salem | ||
| balloons | doug5, hmm, yep | 21:29 |
| balloons | ohh, qml datetime.. | 21:30 |
| balloons | I assume that's base qt | 21:30 |
| balloons | qt5-default | 21:30 |
| Letozaf_ | balloons, hey | 21:49 |
| balloons | Letozaf_, hello | 21:49 |
| 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:50 |
| 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:51 |
| 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:52 |
| 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:53 |
| Letozaf_ | balloons, ok | 21:54 |
| balloons | Letozaf_, so I do have a question about why the generic methods mention zip_dir | 21:55 |
| 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:56 |
| 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:57 |
| 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:58 |
| Letozaf_ | balloons, I first copy the zip file from the contents directory to a mocked home directory and then extract the files from it | 21:59 |
| 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:01 |
| balloons | does that make sense? I assume other tests also use go_to_place | 22:02 |
| Letozaf_ | balloons, I think I did not understand the type the directory name, you mean the fake home ? | 22:05 |
| 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:06 |
| 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:07 |
| 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:08 |
| * balloons notes there is a placessidebar and a placespage | 22:09 | |
| Letozaf_ | balloons, ok I understand so I will fix the tests | 22:09 |
| 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:10 |
| === roadmr_afk is now known as roadmr | ||
| balloons | Letozaf_, though, are you sure you even need to navigate to a directory? | 22:11 |
| balloons | it loads to home automatically | 22:11 |
| 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:12 |
| 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:13 |
| Letozaf_ | balloons, thanks | 22:14 |
| 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! | 22:15 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!