added testing, removed old implementation ref, made local dev friendlier

Author: flowstate

.python-version

@@ -1 +1 @@
-3.11.10
+3.11

Makefile

@@ -1,7 +1,65 @@
-.PHONY: sync-deps test lint
+.PHONY: setup compile-deps sync-deps clean test lint init-tools local-dev first-time-setup update-deps dev-setup sync-all first-time-local-setup
 
+# Environment variable for local SDK path (optional)
+SOCKET_SDK_PATH ?= ../socket-sdk-python
+
+# Environment variable to control local development mode
+USE_LOCAL_SDK ?= false
+
+# === High-level workflow targets ===
+
+# First-time repo setup after cloning (using PyPI packages)
+first-time-setup: clean setup
+
+# First-time setup for local development (using local SDK)
+first-time-local-setup: 
+	$(MAKE) clean
+	$(MAKE) USE_LOCAL_SDK=true dev-setup
+
+# Update dependencies after changing pyproject.toml
+update-deps: compile-deps sync-deps
+
+# Setup for local development
+dev-setup: clean local-dev setup
+
+# Sync all dependencies after pulling changes
+sync-all: sync-deps
+
+# === Implementation targets ===
+
+# Creates virtual environment and installs pip-tools
+init-tools:
+	python -m venv .venv
+	. .venv/bin/activate && pip install pip-tools
+
+# Installs dependencies needed for local development
+# Currently: socket-sdk-python from test PyPI or local path
+local-dev: init-tools
+ifeq ($(USE_LOCAL_SDK),true)
+	. .venv/bin/activate && pip install -e $(SOCKET_SDK_PATH)
+endif
+
+# Creates/updates requirements.txt files with locked versions based on pyproject.toml
+compile-deps: local-dev
+	. .venv/bin/activate && pip-compile --output-file=requirements.txt pyproject.toml
+	. .venv/bin/activate && pip-compile --extra=dev --output-file=requirements-dev.txt pyproject.toml
+	. .venv/bin/activate && pip-compile --extra=test --output-file=requirements-test.txt pyproject.toml
+
+# Creates virtual environment and installs dependencies from pyproject.toml
+setup: compile-deps
+	. .venv/bin/activate && pip install -e ".[dev,test]"
+
+# Installs exact versions from requirements.txt into your virtual environment
 sync-deps:
-	pip-compile pyproject.toml -o requirements.txt
+	. .venv/bin/activate && pip-sync requirements.txt requirements-dev.txt requirements-test.txt
+ifeq ($(USE_LOCAL_SDK),true)
+	. .venv/bin/activate && pip install -e $(SOCKET_SDK_PATH)
+endif
+
+# Removes virtual environment and cache files
+clean:
+	rm -rf .venv
+	find . -type d -name "__pycache__" -exec rm -rf {} +
 
 test:
 	pytest

README.md

@@ -45,46 +45,61 @@ If you don't want to provide the Socket API Token every time then you can use th
 
 This project uses `pyproject.toml` as the primary dependency specification. 
 
-### Installing dependencies with your preferred tool:
-- **pip**: 
-  ```bash
-  pip install -r requirements.txt  # Install main dependencies
-  pip install -e ".[dev,test]"    # Install development and test dependencies
-  ```
-- **poetry**: 
-  ```bash
-  poetry install --all-extras  # Installs all dependencies including dev and test
-  ```
-- **Rye**: 
-  ```bash
-  rye sync --all-features  # Installs all dependencies including dev and test
-  ```
-
-### Changing dependencies:
-
-1. Update `pyproject.toml` with dependency changes
-2. Run `make sync-deps` to update `requirements.txt`
-   - Note: Requires dev dependencies to be installed (`pip-tools`)
+### Development Workflows
 
-### Running tests:
+The following Make targets provide streamlined workflows for common development tasks:
 
-#### Run all tests:
+#### Initial Setup (Choose One)
+
+1. Standard Setup (using PyPI packages):
 ```bash
-make test # Requires dev dependencies to be installed (`pytest`)
+pyenv local 3.11  # Ensure correct Python version
+make first-time-setup
 ```
-Note: For any of the `pytest` commands below, you can substitute `ptw` for `pytest` to run tests in watch mode.
 
-#### Run specific tests:
+2. Local Development Setup (for SDK development):
 ```bash
-# Run all tests in a file
-pytest tests/test_socketcli.py
-
-# Run all tests in a directory
-pytest tests/core
+pyenv local 3.11  # Ensure correct Python version
+SOCKET_SDK_PATH=~/path/to/socket-sdk-python make first-time-local-setup
 ```
+The default SDK path is `../socket-sdk-python` if not specified.
 
+#### Ongoing Development Tasks
 
-### Linting:
+After changing dependencies in pyproject.toml:
 ```bash
-make lint # Requires dev dependencies to be installed (`ruff`)
+make update-deps
+```
+
+After pulling changes:
+```bash
+make sync-all
+```
+
+### Available Make targets:
+
+High-level workflows:
+- `make first-time-setup`: Complete setup using PyPI packages
+- `make first-time-local-setup`: Complete setup for local SDK development
+- `make update-deps`: Update requirements.txt files and sync dependencies
+- `make sync-all`: Sync dependencies after pulling changes
+- `make dev-setup`: Setup for local development (included in first-time-local-setup)
+
+Implementation targets:
+- `make init-tools`: Creates virtual environment and installs pip-tools
+- `make local-dev`: Installs dependencies needed for local development
+- `make compile-deps`: Generates requirements.txt files with locked versions
+- `make setup`: Creates virtual environment and installs dependencies
+- `make sync-deps`: Installs exact versions from requirements.txt
+- `make clean`: Removes virtual environment and cache files
+- `make test`: Runs pytest suite
+- `make lint`: Runs ruff for code formatting and linting
+
+### Environment Variables
+
+- `SOCKET_SDK_PATH`: Path to local socket-sdk-python repository (default: ../socket-sdk-python)
+
+### Running tests:
+
+#### Run all tests:
 ```

pyproject.toml

@@ -13,8 +13,8 @@ dependencies = [
     'argparse',
     'GitPython',
     'packaging',
-    'python-dotenv',  # Add this
-    'socket-sdk-python>=2.0.0'
+    'python-dotenv', 
+    'socket-sdk-python>=2.0.1'
 ]
 readme = "README.md"
 description = "Socket Security CLI for CI/CD"
@@ -64,7 +64,15 @@ version = {attr = "socketsecurity.__version__"}
 
 [tool.coverage.run]
 source = ["socketsecurity"]
-omit = ["tests/*", "**/__init__.py"]
+branch = true
+include = [
+    "socketsecurity/**/*.py",
+    "socketsecurity/**/__init__.py"
+]
+omit = [
+    "socketsecurity/core/issues.py",  # Large data file
+    "socketsecurity/core/licenses.py"  # Large data file
+]
 
 [tool.coverage.report]
 exclude_lines = [
@@ -74,6 +82,8 @@ exclude_lines = [
     "raise NotImplementedError",
     "if TYPE_CHECKING:",
 ]
+show_missing = true
+skip_empty = true
 
 [tool.ruff]
 # Exclude a variety of commonly ignored directories.

pytest.ini

@@ -1,4 +1,7 @@
 [pytest]
 testpaths = tests/unit
-addopts = -vv --no-cov --tb=short -ra
+; addopts = -vv --no-cov --tb=short -ra
+addopts = -vv --tb=short -ra --cov=socketsecurity --cov-report=term-missing
 python_files = test_*.py
+asyncio_mode = strict
+asyncio_default_fixture_loop_scope = function