=== 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
balloonsdoug5, you about?16:20
=== chihchun is now known as chihchun_afk
elopioballoons: hey, how can I run a single test from a click suite with adt-run?18:20
doug5balloons, hellooo18:42
doug5balloons, haven't started with my next task yet18:48
=== knome_ is now known as knome
doug5balloons, could you explain me what needs to be done here? https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/bottomEdge/+merge/24159019:26
doug5balloons, and I'm also getting this https://bugs.launchpad.net/ubuntu-calendar-app/+bug/138663919:30
ubot5`Launchpad bug 1386639 in Ubuntu Calendar App "Can't run calendar app on desktop" [Undecided,New]19:30
balloonselopio, yes you can19:39
balloonsdoug5, hey :-)19:39
balloonselopio, use ADT_AUTOPILOT_MODULE19:42
balloonsfor instance ADT_AUTOPILOT_MODULE="-v music_app.tests.test_music.TestMainWindow.test_url_dispatcher_album_play" adt-run --click com.ubuntu.music --- ssh -s adb19:43
elopioballoons: pitti said I shouldn't use that env var. Is that the only way?19:46
balloonselopio, indeed you shouldn't. but it does work :-)19:52
balloonselopio, 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.. hmm19:55
balloonsdoug5, so with the bottomedge merge, a helper needs to be added to do the bottom edge swipe action to start off the new event tests19:56
balloonsI 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
balloonsif both are supported than of course you'd want the tests to do both19:56
doug5balloons, 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
balloonsdoug5, ohh, that bug is preventing you from working on it?20:18
doug5balloons, I cannot start the application anymore since I checked out the branch20:19
balloonsdoug5, let me try and see as well20:19
doug5balloons, don't know if the two things are related or I did something in the meanwhile20:19
balloonsso I pulled trunk, then merged that branch into it20:20
balloonsrunning calendar still seems fine. What erros do you get?20:20
doug5I did directly bzr branch lp:~mihirsoni/ubuntu-calendar-app/bottomEdge20:20
balloonsdoug5, ok, I can try that too.. should be more or less the same, but what output do you get?20:24
doug5balloons, exactly the error in the bug description20:24
balloonsdoug5, interesting mihir shows it has to do with online accounts. Do you have your gmail account synced on the desktop?20:25
balloonsif so seems you can do exactly what he shows in the video.20:26
balloonsI likely don't see the issue because I don't have an online account20:27
doug5balloons, could be...do you know how to disable the sync?20:27
balloonsdoug5, yes in online accounts, turn off evolution data sync20:27
balloonsevolution data server I think it's called20:27
doug5balloons, much better, thx :)20:29
balloonsdoug5, :-) great. So anyways, I was telling mihir in the bug report, several other apps have tests that use the bottom edge20:31
balloonsclock uses it as does file manager20:31
balloonsyou can use those for help and inspiration. Basically you perform a drag operation on the little stub at the bottom and open it up20:32
doug5balloons, ok, I see it, but when I click on it nothing happens20:33
doug5balloons, ah no, got it :)20:34
doug5balloons, thx20:34
doug5balloons, I will check with mihir if we have to support both20:35
balloonsdoug5, yep he should know what's what. The helper needs to exist either way, so writing it up is the first step20:38
doug5balloons, ok20:38
doug5balloons, thx for the support20:38
balloonsyou are welcome. So I can help work on that first part with you now if you have questions20:41
doug5balloons, ok. Do you know which package contains qml DateTime ?20:45
=== roadmr is now known as roadmr_afk
=== salem_ is now known as _salem
balloonsdoug5, hmm, yep21:29
balloonsohh, qml datetime..21:30
balloonsI assume that's base qt21:30
Letozaf_balloons, hey21:49
balloonsLetozaf_, hello21:49
Letozaf_balloons, I was writing the test for extracting a tar file for filemanager app21:50
Letozaf_balloons, but the feature has not been merged yet21:50
Letozaf_balloons, as I am using scenarios I think I need it to or not _21:50
balloonsLetozaf_, where did we leave the zip tests?21:51
Letozaf_balloons, I completed the test with the cancel unzipping the file test21:51
Letozaf_balloons, tests passed21:52
balloonsahh I see :-)21:52
Letozaf_balloons, now I added scenarios for both zip and tar, but tar has not been merged yet21:52
balloonsahh I understand now21:53
balloonsso right, we won't be able to merge the tar part. I would push to get it included in the main mp actually21:53
balloonsso let's get the zip stuff merged though21:53
balloonslet me check quickly21:53
Letozaf_balloons, ok21:54
balloonsLetozaf_, so I do have a question about why the generic methods mention zip_dir21:55
balloonsis 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 zip21:56
balloonsLetozaf_, right.. but also, aren't those methods used by other things? I'm confused why you need to write a zip_dir_path21:57
balloonsI guess maybe you just wanted to add dir_path to it?21:57
Letozaf_balloons, I put the zip file in a mocked home dir21:57
Letozaf_balloons, that's where the zip is and so I called it zip_dir_path21: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 it21:59
balloonsright, that all makes sense. Then for go_to_place you sould probably just type the directory name right?22:01
balloonskeep it generic?22:01
balloonsI think changing the name to make it generic fixes everything22:01
balloonsdoes that make sense? I assume other tests also use go_to_place22:02
Letozaf_balloons, I think I did not understand the type the directory name, you mean the fake home ?22:05
balloonsLetozaf_, 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 hurry22:06
balloonsmy point is you changed go_to_place to have an extra argument22:06
balloonsand that argument is only needed for your zip tests?22:06
Letozaf_balloons, Oh now I understand22:07
Letozaf_balloons, yes you are right22:07
Letozaf_balloons, ok I will type that instead of passing the argument22:07
balloonswell, I'm curious. Isn't there a common method that allows you to send a string to be navigated to22:08
balloonsand maybe that is the common method. If so however, I would call the argument you pass a path or something and not say zip specifically22:08
* balloons notes there is a placessidebar and a placespage22:09
Letozaf_balloons, ok I understand so I will fix the tests22:09
Letozaf_balloons, placessidebar is when you are in tablet mode22:10
balloonsand 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_path22:10
Letozaf_balloons, placespage when in phone mode22:10
=== roadmr_afk is now known as roadmr
balloonsLetozaf_, though, are you sure you even need to navigate to a directory?22:11
balloonsit loads to home automatically22:11
Letozaf_balloons, oh... I didn't realize that I will check and fix if needed22:12
balloonsyea, lol, you don't, I looked22:12
balloonsLetozaf_, however that change is handy to have so people can go to a specific folder22:12
* balloons wonders if there is a test for it..22:12
balloonsprobably a test that could be added. check navigating to a folder22:12
Letozaf_balloons, ok22:13
balloonsmake sense?22:13
balloonsIt makes sense to me now, lol22:13
Letozaf_balloons, yes, I will fix everything tomorrow, it's bed time for me now :)22:13
Letozaf_balloons, thanks22:14
balloonsLetozaf_, thank you! and tehe, I managed to even find another test in there22:15
Letozaf_balloons, lol22:15
Letozaf_balloons, good night22:15
balloonsLetozaf_, goodnight!22:15

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!