Merge pull request #29 from brain-facens/develop

Develop

Author: nata-vito

.github/workflows/pylint.yml

@@ -26,16 +26,14 @@ jobs:
     - name: Install dependencies
       run: |
         python -m pip install --upgrade pip
-        pip install flake8 pytest
+        pip install flake8 pytest pylint
         if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
-    - name: Lint with flake8
+    - name: Analysing the code with pylint
       run: |
-        # stop the build if there are Python syntax errors or undefined names
-        flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
-        # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
-        flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
-    - name: Run api
-      run: python src/field_vision_api/main.py     
-    - name: Test with pytest
+        pylint $(git ls-files '*.py')
+    - name: Build the Docker image
       run: |
-        pytest
+        docker build . --file Dockerfile --tag fiel-vision-api
+    - name: Run API docker image
+      run: |
+        docker run -d -p 8085:8085 fiel-vision-api

.github/workflows/python-app.yml

@@ -1,10 +1,10 @@
-FROM python:3.7
+FROM python:3.8
 
 WORKDIR /app
 
 RUN apt-get update && apt-get install -y libgtk2.0-dev && apt-get install ffmpeg libsm6 libxext6  -y
 
-RUN git clone https://github.com/PaddlePaddle/PaddleOCR.git
+RUN git clone https://github.com/brain-facens/FieldVision-AI.git
 
 RUN pip install --upgrade pip
 
@@ -22,7 +22,4 @@ ENV PYTHONPATH=/app/PaddleOCR
 
 EXPOSE 8080
 
-COPY python_scripts/utils.py /app
-COPY python_scripts/main.py /app
-
-CMD ["python", "main.py", "--filter", "first,second,third"]
+CMD ["python", "FieldVision-AI/src/field_vision_api/main.py"]

Dockerfile

@@ -5,7 +5,7 @@
 import pytest # W0611, pylint: disable=unused-import
 import requests
 
-def test_api_get_filter(url = "http://0.0.0.0:8085/v1/filter/") -> None:
+def test_api_get_filter(url = "http://localhost:8085/v1/filter/") -> None:
     """ 
     Test for endpoint filter result /v1/filter/.
 

src/README.md

@@ -4,7 +4,7 @@
 import pytest # W0611, pylint: disable=unused-import
 import requests
 
-def test_api_get_raw_result(url = "http://0.0.0.0:8085/v1/raw_result/"):
+def test_api_get_raw_result(url = "http://localhost:8085/v1/raw_result/"):
     """ 
     Test for endpoint raw result /v1/raw_result/.
 

tests/test_get_filter.py

@@ -4,7 +4,7 @@
 import pytest # W0611, pylint: disable=unused-import
 import requests
 
-def test_api_get_result(url = "http://0.0.0.0:8085/v1/result/"):
+def test_api_get_result(url = "http://localhost:8085/v1/result/"):
     """ 
     Test for endpoint raw result /v1/result/.
 

tests/test_get_raw_result.py

@@ -4,7 +4,7 @@
 import pytest # W0611, pylint: disable=unused-import
 import requests
 
-def test_api_get_root(url = "http://0.0.0.0:8085/"):
+def test_api_get_root(url = "http://localhost:8085/"):
     """ 
     Test for root endpoint /.
 

tests/test_get_result.py

@@ -4,7 +4,7 @@
 import pytest # W0611, pylint: disable=unused-import
 import requests
 
-def test_api_post_img(url = "http://0.0.0.0:8085/v1/post_image/"):
+def test_api_post_img(url = "http://localhost:8085/v1/post_image/"):
     """ 
     Test for endpoint /v1/post_image/.
 

tests/test_get_root.py

@@ -4,7 +4,7 @@
 import pytest # W0611, pylint: disable=unused-import
 import requests
 
-def test_api_put_result(url = "http://0.0.0.0:8085/v1/filter/"):
+def test_api_put_result(url = "http://localhost:8085/v1/filter/"):
     """ 
     Test for endpoint /v1/filter/.