[00:07] <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:08] <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:09] <ahayzen> fazer, and that particular one maybe hard to test without the API keys
[00:09] <fazer> ok. cool.
[00:10] <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:13] <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:14] <ahayzen> fazer, different providers give different info, i think before TWC only gave precipitation so it needed to be checked that that appeared
[00:15] <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:17] <ahayzen> fazer, with TWC i get a few extra ones compared to OWM, "Chance of rain" and "UV Index" appear
[00:18] <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:22] <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:23] <ahayzen> so you'll do similar to that, but check that the extra label appear
[00:23] <fazer> ok. thanks.
[16:27] <McIntireEvan> shoutout to all my other DC area people in the blizzard right now
[17:40] <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:41] <ahayzen> fazer, which bit doesn't work ?
[17:43] <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:44] <ahayzen> fazer, hmmm adding that there is going to make the other tests fail as well :-/
[17:44] <fazer> why?
[17:45] <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:46] <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:48] <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:49] <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:50] <ahayzen> fazer, can you see the provider page becoming visible after the click?
[17:50] <fazer> yes
[17:52] <fazer> it always fails when trying to get the page.
[17:52] <ahayzen> fazer, try putting visible=True into the query as well
[17:53] <fazer> you're referring to the call in settings page helper right?
[17:53] <ahayzen> yup where you get the page
[17:54] <fazer> didn't work.
[17:55] <ahayzen> what is the error?
[17:56] <fazer> state not found: object not found with name'DataProviderPAge' and properties visible: true
[17:57] <ahayzen> fazer, ah i know why :-)
[17:58] <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:59] <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
[18:00] <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:03] <fazer> ahayzen, ok, thanks. It worked. how can I get a key for TWC?
[18:03] <fazer> I got one for OWM
[18:04] <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:05] <fazer> oh.... ok.
[18:11] <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:16] <fazer> ahayzen, Is the weather underground the same as TWC?
[18:17] <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:20] <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:24] <fazer> you don't know if the weather underground api key will work? Or are you talking about something else
[18:25] <fazer> ahayzen ^
[18:27] <ahayzen> fazer, pm'd you some info :-)
[18:28] <fazer> yup