Refactoring python code location, adding tests (#66)

Author: samuelcolvin

.eslintrc.cjs

@@ -8,7 +8,7 @@ module.exports = {
     'plugin:react-hooks/recommended',
     'prettier',
   ],
-  ignorePatterns: ['node_modules'],
+  ignorePatterns: ['node_modules', 'dist'],
   parser: '@typescript-eslint/parser',
   plugins: ['react', '@typescript-eslint', 'react-refresh', 'simple-import-sort'],
   rules: {

.github/workflows/ci.yml

@@ -24,7 +24,7 @@ jobs:
         with:
           node-version: 18
 
-      - run: pip install -r python/requirements/all.txt
+      - run: pip install -r src/python-fastui/requirements/all.txt
 
       - run: npm install
 
@@ -34,7 +34,42 @@ jobs:
         env:
           SKIP: no-commit-to-branch
 
-  packages-build:
+  test:
+    name: test ${{ matrix.python-version }} on ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu, macos]
+        python-version: ['3.8', '3.9', '3.10', '3.11', '3.12']
+
+    runs-on: ${{ matrix.os }}-latest
+
+    env:
+      PYTHON: ${{ matrix.python-version }}
+      OS: ${{ matrix.os }}
+
+    steps:
+    - uses: actions/checkout@v3
+
+    - name: set up python
+      uses: actions/setup-python@v4
+      with:
+        python-version: ${{ matrix.python-version }}
+
+    - run: pip install -r src/python-fastui/requirements/test.txt
+    - run: pip install -r src/python-fastui/requirements/pyproject.txt
+    - run: pip install src/python-fastui
+
+    - run: make test
+
+    - run: coverage xml
+
+    - uses: codecov/codecov-action@v3
+      with:
+        file: ./coverage.xml
+        env_vars: PYTHON,OS
+
+  npm-build:
     runs-on: ubuntu-latest
 
     steps:
@@ -46,11 +81,11 @@ jobs:
 
       - run: npm install
       - run: npm run build
-      - run: tree packages
+      - run: tree src
 
   check: # This job does nothing and is only used for the branch protection
     if: always()
-    needs: [lint, packages-build]
+    needs: [lint, npm-build]
     runs-on: ubuntu-latest
 
     steps:

.gitignore

@@ -6,6 +6,7 @@ yarn-debug.log*
 yarn-error.log*
 pnpm-debug.log*
 lerna-debug.log*
+/**/*.egg-info
 
 node_modules
 dist
@@ -31,3 +32,4 @@ __pycache__/
 /frontend-dist/
 /scratch/
 /packages-dist/
+/.coverage

Makefile

@@ -1,44 +1,46 @@
 .DEFAULT_GOAL:=all
-paths = python
+path = src/python-fastui
 
 .PHONY: install
 install:
 	pip install -U pip pre-commit pip-tools
-	pip install -r python/requirements/all.txt
+	pip install -r $(path)/requirements/all.txt
+	pip install -e $(path)
 	pre-commit install
 
 .PHONY: update-lockfiles
 update-lockfiles:
 	@echo "Updating requirements files using pip-compile"
-	pip-compile -q --strip-extras -o python/requirements/lint.txt python/requirements/lint.in
-	pip-compile -q --strip-extras -o python/requirements/pyproject.txt pyproject.toml --extra=fastapi
-	pip install --dry-run -r python/requirements/all.txt
+	pip-compile -q --strip-extras -o $(path)/requirements/lint.txt $(path)/requirements/lint.in
+	pip-compile -q --strip-extras -o $(path)/requirements/test.txt $(path)/requirements/test.in
+	pip-compile -q --strip-extras -o $(path)/requirements/pyproject.txt $(path)/pyproject.toml --extra=fastapi
+	pip install --dry-run -r $(path)/requirements/all.txt
 
 .PHONY: format
 format:
-	ruff check --fix-only $(paths)
-	ruff format $(paths)
+	ruff check --fix-only $(path)
+	ruff format $(path)
 
 .PHONY: lint
 lint:
-	ruff check $(paths)
-	ruff format --check $(paths)
+	ruff check $(path)
+	ruff format --check $(path)
 
 .PHONY: typecheck
 typecheck:
-	pyright python/fastui
+	pyright
 
 .PHONY: test
 test:
-	coverage run -m pytest tests
+	coverage run -m pytest
 
 .PHONY: testcov
 testcov: test
 	coverage html
 
 .PHONY: dev
 dev:
-	uvicorn python.demo:app --reload
+	uvicorn demo:app --reload
 
 .PHONY: all
 all: testcov lint

python/demo/README.md demo/README.mdpython/demo/README.md renamed to demo/README.md

@@ -85,28 +85,19 @@ def form_content(kind: FormKind):
             return [
                 c.Heading(text='Login Form', level=2),
                 c.Paragraph(text='Simple login form with email and password.'),
-                c.ModelForm[LoginForm](
-                    submit_url='/api/forms/login',
-                    success_event=PageEvent(name='form_success'),
-                ),
+                c.ModelForm[LoginForm](submit_url='/api/forms/login'),
             ]
         case 'select':
             return [
                 c.Heading(text='Select Form', level=2),
                 c.Paragraph(text='Form showing different ways of doing select.'),
-                c.ModelForm[SelectForm](
-                    submit_url='/api/forms/select',
-                    success_event=PageEvent(name='form_success'),
-                ),
+                c.ModelForm[SelectForm](submit_url='/api/forms/select'),
             ]
         case 'big':
             return [
                 c.Heading(text='Large Form', level=2),
                 c.Paragraph(text='Form with a lot of fields.'),
-                c.ModelForm[BigModel](
-                    submit_url='/api/forms/big',
-                    success_event=PageEvent(name='form_success'),
-                ),
+                c.ModelForm[BigModel](submit_url='/api/forms/big'),
             ]
         case _:
             raise ValueError(f'Invalid kind {kind!r}')