Hi Omer,
there's no "index" for "currentLocationItem" + index, you should just call it "currentLocationItem". (LocationManagerSheet, L108)
Otherwise it's just fine, thanks a lot!
Now the only thing left is that "ubuntu_weather_app.tests.test_locationmanager.TestLocationManager.test_search_city" won't work on the device, since the "BackSpace" key is unknown. I've found a (litlle bit reckless) solution for that: Line 95 in http://bazaar.launchpad.net/~martin-borho/ubuntu-weather-app/device_tests_and_emulator/revision/83/tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py#
Perhaps you can incorporate that in a less hackish way, if you know one? Otherwise I could do that later on, no problem!
Thanks anyways!
Martin
« Back to merge proposal
Hi Omer,
there's no "index" for "currentLocatio nItem" + index, you should just call it "currentLocatio nItem". (LocationManage rSheet, L108)
Otherwise it's just fine, thanks a lot!
Now the only thing left is that "ubuntu_ weather_ app.tests. test_locationma nager.TestLocat ionManager. test_search_ city" won't work on the device, since the "BackSpace" key is unknown. I've found a (litlle bit reckless) solution for that: Line 95 in http:// bazaar. launchpad. net/~martin- borho/ubuntu- weather- app/device_ tests_and_ emulator/ revision/ 83/tests/ autopilot/ ubuntu_ weather_ app/tests/ test_locationma nager.py#
Perhaps you can incorporate that in a less hackish way, if you know one? Otherwise I could do that later on, no problem!
Thanks anyways!
Martin