lambdacloud / LambdacloudSDK

SDK to record client logs
1 stars 2 forks source link

Android SDK增加获取地理信息以及电量功能 #26

Closed majun2408 closed 8 years ago

majun2408 commented 8 years ago

https://github.com/lambdacloud/lanyun/issues/1285 https://github.com/lambdacloud/lanyun/issues/1284 ps. LogAgent类中所添加方法不对外提供,仅为内部调用。

DeviceInfo:

  1. 添加获取地理信息功能(getLocation方法)
  2. 获取电量功能(getBatteryPower方法)
  3. 修改代码格式

    LogAgent:

  4. 添加sendLocation方法
    • 参数类型为Location
    • 发送日志格式为”日志类型[ldp_location],时间[ti meStamp],用户[userId],来源[客户端],设备平台[ANDROID],ldp_latitude[latitude],ldp_longitude[longitude]”
  5. 添加sendBatteryPower方法
    • 参数类型为String
    • 发送日志格式为”日志类型[ldp_battery_powe r],时间[timeStamp],用户[userId],来源[客户端],设备平台[ANDROID],ldp_battery_power[batteryPower]”

      DeviceInfoTest:

  6. 添加getBatteryPower方法的单元测试
  7. 添加getLocation方法的单元测试

    LogAgentTest:

  8. 由于在DeviceInfo.init中添加了sendAppList等方法中含有sendLog因此需要在 LogAgentTest 中添加以下代码来清空队列

    LogSpout logSpout = LogSpout.getInstance();
    logSpout.queue.clear();
majun2408 commented 8 years ago

@sky4star 杨哥我提交了代码,麻烦看一下吧。

sky4star commented 8 years ago

单元测试?

majun2408 commented 8 years ago

单元测试还没写完。。因为位置服务这个单元测试还没找到合适的解决方法。。。。

sky4star commented 8 years ago

嗯好的,你可以搜搜看git上面有没有可以参考的。 真机测试过了吧?

majun2408 commented 8 years ago

真机测试都过了。

sky4star commented 8 years ago

好的 ,解决了comments就可以提交了,辛苦了!

majun2408 commented 8 years ago

@sky4star 杨哥 你再看一下吧 要是没问题的话我就merge了

sky4star commented 8 years ago

还有一点问题,再改一下吧

majun2408 commented 8 years ago

@sky4star 杨哥关于那个单元测试没有对结果做测试的问题,我解决的方法是在getLocation或者getBatteryPower里面用System.out.print输出测试结果。目前俩个函数都是没有返回值的。测试结果是正确的。 还是说我们需要给getLocation或者getBatteryPower加上返回值然后再做测试这样吗?

sky4star commented 8 years ago

好的,这样就可以了。另外一种写法就是用回调函数,正常的代码是在回调中sendLog,测试代码是在回调中assert true

majun2408 commented 8 years ago

明白了 我去试试。

sky4star commented 8 years ago

shipit!