bugfix/take-screenshot-twice (#51)

* Take screenshot twice when click an unselected device's take-screenshot-button

* Add GitHub actions

.github/workflows/publish.yml

@@ -0,0 +1,40 @@
+# This workflows will upload a Python Package using Twine when a release is created
+# For more information see: https://help.github.com/en/actions/language-and-framework-guides/using-python-with-github-actions#publishing-to-package-registries
+
+name: Publish to pypi
+
+on:
+  release:
+    types: [created]
+
+jobs:
+  build-n-publish:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+      - name: Set up Node
+        uses: actions/setup-node@v1
+        with:
+          node-version: "10"
+      - name: Install npm Dependencies
+        run: |
+          cd frontend
+          npm install
+          npm run build
+          cd ..
+      - name: Set up Python
+        uses: actions/setup-python@v1
+        with:
+          python-version: "3.7"
+      - name: Install python dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install setuptools wheel twine
+      - name: Build and publish
+        env:
+          TWINE_USERNAME: __token__
+          TWINE_PASSWORD: ${{ secrets.PYPI_PASSWORD }}
+        run: |
+          python setup.py sdist
+          twine upload dist/*

.github/workflows/unittest.yml

@@ -0,0 +1,22 @@
+name: Unit Test
+
+on: [push, pull_request]
+
+jobs:
+  unittest:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+      - name: Set up Python
+        uses: actions/setup-python@v1
+        with:
+          python-version: "3.7"
+      - name: Install python dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install -r requirements.txt
+      - name: Unit Test
+        run: |
+          cd tests
+          python -m pytest .

frontend/src/components/DeviceList.vue

@@ -31,10 +31,10 @@ export default {
   },
   methods: {
     onCellClick (deviceId) {
-      this.$store.commit('setFocusDeviceId', deviceId)
-      this.$store.commit('setDeviceInfo', this.devices[deviceId])
-      this.$store.dispatch('loadPackages')
-      this.$store.dispatch('takeScreenShot')
+      if (this.focusDeviceId !== deviceId) {
+        this.$store.commit('setFocusDeviceId', deviceId)
+        this.$store.dispatch('initAllDeviceInfo')
+      }
     },
     takeScreenShot () {
       if (this.focusDeviceId) {

frontend/src/store/index.js

@@ -73,15 +73,28 @@ export default new Vuex.Store({
     }
   },
   actions: {
-    loadDevices ({ state, commit }) {
+    loadDevices ({ state, commit, dispatch }) {
       api.getDevices().then(response => {
         commit('setDevices', response.data.device_list)
-        if (!state.devices.hasOwnProperty(state.focusDeviceId)) {
+        if (Object.keys(state.devices).length > 0) {
+          commit('setFocusDeviceId', Object.keys(state.devices)[0])
+          dispatch('initAllDeviceInfo')
+        } else {
           commit('setFocusDeviceId', null)
-          commit('setScreenShotUrl', null)
+          dispatch('clearAllDeviceInfo')
         }
       })
     },
+    initAllDeviceInfo ({state, commit, dispatch}) {
+      commit('setDeviceInfo', state.devices[state.focusDeviceId])
+      dispatch('loadPackages')
+      dispatch('takeScreenShot')
+    },
+    clearAllDeviceInfo ({commit, dispatch}) {
+      commit('setDeviceInfo', {})
+      commit('setScreenShotUrl', null)
+      commit('setPackages', [])
+    },
     takeScreenShot ({ state, commit }) {
       commit('setIsTakingScreen', true)
       api.getScreenShot(state.focusDeviceId).then(response => {

frontend/src/store/start.js

@@ -90,10 +90,10 @@ export default {
       api.launchApp(deviceId, packageName, state.launchActions)
         .then(response => {
           commit('setIsStartingApp', false)
-          bus.$emit('msg.success', 'Start App ' + state.focusPackageName + ' success!')
+          bus.$emit('msg.success', 'Start App ' + packageName + ' success!')
         })
         .catch(error => {
-          bus.$emit('msg.error', 'Start App ' + state.focusPackageName + ' error: ' + error.data.message)
+          bus.$emit('msg.error', 'Start App ' + packageName + ' error: ' + error.data.message)
         })
     }
   }

requirements.txt

@@ -1 +1 @@
-lyrebird
+-e .[dev]

setup.py

@@ -7,7 +7,7 @@
 
 setup(
     name='lyrebird-android',
-    version='0.5.7',
+    version='0.5.8',
     packages=['lyrebird_android'],
     url='https://github.com/meituan/lyrebird-android',
     author='HBQA',
@@ -29,5 +29,12 @@
     },
     install_requires=[
         'lyrebird'
-    ]
+    ],
+    extras_require={
+        'dev': [
+            "autopep8",
+            "pylint",
+            "pytest"
+        ]
+    }
 )