Fixed project structure and enabled unit tests on PR

Author: merlimat

.github/workflows/run-tests.yaml

@@ -16,31 +16,33 @@ name: Run tests
 
 on:
   pull_request:
-#    branches:
-#      - main
+    branches:
+      - main
+  push:
+    branches:
+      - main
 
 jobs:
   build_and_test:
     runs-on: ubuntu-latest
 
     steps:
     - name: Checkout repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v5
 
     - name: Set up Python
       uses: actions/setup-python@v5
       with:
         python-version: '3.13'
 
-    - name: Install build dependencies
-      run: |
-        python -m pip install --upgrade uv
+    - name: Install uv
+      uses: astral-sh/setup-uv@v6
+
+    - name: Install dependencies
+      run: uv sync --dev
 
     - name: Build distribution packages
       run: uv build
 
-#    - name: Publish package to PyPI
-#      uses: pypa/gh-action-pypi-publish@release/v1
-#      with:
-        # No need to specify username or password due to trusted publishing
-        # repository_url: https://test.pypi.org/legacy/ # Uncomment to publish to TestPyPI
+    - name: Run tests
+      run: uv run pytest

pyproject.toml

@@ -19,6 +19,9 @@ description = "Oxia client for Python"
 authors = [
     { name = "StreamNative"}
 ]
+
+requires-python = ">=3.13"
+
 dependencies = [
     "betterproto2-compiler",
     "grpcio",
@@ -29,6 +32,10 @@ dependencies = [
 dev = [
     "grpcio-tools>=1.75.1",
     "pydoctor>=25.10.1",
+    "pytest>=8.4.2",
     "testcontainers>=4.13.1",
 ]
 
+[build-system]
+requires = ["uv_build>=0.8.22,<0.9.0"]
+build-backend = "uv_build"

oxia/__init__.py src/oxia/__init__.pyoxia/__init__.py renamed to src/oxia/__init__.py

@@ -16,7 +16,6 @@
 Oxia Python Client SDK
 """
 
-import oxia.ex as ex
 from oxia.client import (
     Client,
 )

oxia/client.py src/oxia/client.pyoxia/client.py renamed to src/oxia/client.py

@@ -177,7 +177,7 @@ def get(self, key: str,
         # // Returns ErrorKeyNotFound if the record does not exist
         if partition_key is None and \
                 (comparison_type != oxia.defs.ComparisonType.EQUAL
-                     or use_index is not None):
+                 or use_index is not None):
 
             results = []
             for shard, stub in self._service_discovery.get_all_shards():

oxia/defs.py src/oxia/defs.pyoxia/defs.py renamed to src/oxia/defs.py

@@ -11,9 +11,6 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
-import functools
-from opcode import cmp_op
-
 
 def _cmp(a: str, b: str) -> int:
     if a < b: