| fazer | ahayzen, I'm working on this bug: https://bugs.launchpad.net/ubuntu-weather-app/+bug/1452489 . I encountered something called an OptionSelector in the old tests. what exactly would that be in the reboot | 00:07 |
|---|---|---|
| ubot93 | Launchpad bug 1452489 in Ubuntu Weather App "[reboot] Autopilot Testcase Needed: Switch data providers" [High,Confirmed] | 00:07 |
| ahayzen | fazer, http://bazaar.launchpad.net/~ubuntu-weather-dev/ubuntu-weather-app/reboot/view/head:/app/ui/settings/DataProviderPage.qml#L32 | 00:08 |
| ahayzen | ExpandableListItem | 00:08 |
| ahayzen | ish | 00:08 |
| ahayzen | fazer, and that particular one maybe hard to test without the API keys | 00:09 |
| fazer | ok. cool. | 00:09 |
| fazer | ahayzen I'll try it without, if it doesn't work then I'll get the keys by following the steps. Shouldn't be too hard. | 00:10 |
| ahayzen | ok :-) | 00:10 |
| fazer | ahayzen, another question, why is the old tests checking to see if the precipitation units selector appears? What does that have to do with the data provider? | 00:13 |
| ahayzen | fazer, different providers give different info, i think before TWC only gave precipitation so it needed to be checked that that appeared | 00:14 |
| fazer | oh ok. Does that still need to be checked? Or do both providers give precipitation now? | 00:15 |
| ahayzen | good question :-) | 00:15 |
| fazer | :D | 00:15 |
| ahayzen | fazer, with TWC i get a few extra ones compared to OWM, "Chance of rain" and "UV Index" appear | 00:17 |
| fazer | ahayzen, So, I have to navigate to the precipitations page and check that those two options appear? Or is it unnecessary? | 00:18 |
| ahayzen | fazer, probably best, if you check the change units test, it does a similar thing | 00:18 |
| fazer | ahayzen, I don't understand. which part? | 00:22 |
| ahayzen | fazer, in the new tests, after changing the unit it goes back to the home page to check that the unit has actually changed | 00:22 |
| ahayzen | so you'll do similar to that, but check that the extra label appear | 00:23 |
| fazer | ok. thanks. | 00:23 |
| McIntireEvan | shoutout to all my other DC area people in the blizzard right now | 16:27 |
| fazer | ahayzen, need some help please. I have the same command in two different places with the only thing different being the object. But, it doesn't work the 2nd time. My branch is here: https://code.launchpad.net/~fazerlicourice/ubuntu-weather-app/add-test-switch-data-providers | 17:40 |
| ahayzen | fazer, which bit doesn't work ? | 17:41 |
| fazer | ahayzen: http://bazaar.launchpad.net/~fazerlicourice/ubuntu-weather-app/add-test-switch-data-providers/view/head:/tests/autopilot/ubuntu_weather_app/tests/test_settings_page.py#L53 | 17:43 |
| ahayzen | fazer, hmmm adding that there is going to make the other tests fail as well :-/ | 17:44 |
| fazer | why? | 17:44 |
| ahayzen | because the other tests assume that it starts on the unit page, you would then be changing the start point to the provider page :-/ | 17:45 |
| fazer | ok, let me try putting that before the unit page thing, but I don't think that'll change the fact that that line is not working | 17:46 |
| ahayzen | fazer, i would just put that into your test | 17:48 |
| fazer | ahayzen, I switched the order, so it goes to the provider page first but like I said, the get_data_provider_page isn't working | 17:48 |
| ahayzen | as the only reason stuff should really be going into the setpUp is if it is being used multiple times | 17:48 |
| ahayzen | it is really incorrect that the units stuff is there tbh, that should be in its own test as well | 17:49 |
| fazer | ok. but that still won't solve my problem. | 17:49 |
| ahayzen | fazer, can you see the provider page becoming visible after the click? | 17:50 |
| fazer | yes | 17:50 |
| fazer | it always fails when trying to get the page. | 17:52 |
| ahayzen | fazer, try putting visible=True into the query as well | 17:52 |
| fazer | you're referring to the call in settings page helper right? | 17:53 |
| ahayzen | yup where you get the page | 17:53 |
| fazer | didn't work. | 17:54 |
| ahayzen | what is the error? | 17:55 |
| fazer | state not found: object not found with name'DataProviderPAge' and properties visible: true | 17:56 |
| ahayzen | fazer, ah i know why :-) | 17:57 |
| fazer | care to share, s'il vous plait? | 17:58 |
| ahayzen | fazer, add this block of code to the DataProviderPage.qml http://bazaar.launchpad.net/~music-app-dev/music-app/trunk/view/head:/app/ui/Albums.qml#L54 | 17:58 |
| ahayzen | Line54-57 | 17:58 |
| fazer | is this necessary in like every single app? | 17:59 |
| ahayzen | (replacing "Albums" with "DataProviderPage" and "MusicPage" with "Page" | 17:59 |
| fazer | I remember doing this for the music app also | 17:59 |
| ahayzen | if you read the bug you'll see why, it is an issue with autopilot | 17:59 |
| ahayzen | bug 1341671 | 17:59 |
| ubot93 | bug 1341671 in Autopilot Qt Support "Versioned QML classes are not recognized by their public type name" [High,Confirmed] https://launchpad.net/bugs/1341671 | 18:00 |
| fazer | yup, I see it now. | 18:00 |
| fazer | ahayzen, ok, thanks. It worked. how can I get a key for TWC? | 18:03 |
| fazer | I got one for OWM | 18:03 |
| ahayzen | fazer, not sure if you can get a personal one, and it has been bought by another company so sortof in a transition | 18:04 |
| fazer | oh.... ok. | 18:05 |
| fazer | ahayzen, so this is pretty much impossible to test right now...? | 18:11 |
| ahayzen | fazer, unless you get a key :-) | 18:11 |
| fazer | but I can't.. | 18:11 |
| fazer | ahayzen, Is the weather underground the same as TWC? | 18:16 |
| ahayzen | fazer, don't think so was trying to work that out myself | 18:17 |
| ahayzen | fazer, the domain that we query is http://wxdata.weather.com/wxdata/ | 18:17 |
| fazer | ahaayzen, at the bottom og the homepage for the weather underground. There's a copyright by TWC. | 18:20 |
| fazer | Could it be the same API? | 18:20 |
| ahayzen | maybe, but they have a new API making the old one deprecated, so i don't know if it will work | 18:20 |
| fazer | you don't know if the weather underground api key will work? Or are you talking about something else | 18:24 |
| fazer | ahayzen ^ | 18:25 |
| ahayzen | fazer, pm'd you some info :-) | 18:27 |
| fazer | yup | 18:28 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!