fix: add integration tests using python

Author: n1rna

.github/workflows/ci.yml

@@ -32,12 +32,18 @@ jobs:
     - name: Download dependencies
       run: make deps
 
-    - name: Run tests
+    - name: Run unit tests
       run: make test
 
     - name: Generate coverage report
       run: make coverage
 
+    - name: Install uv
+      uses: astral-sh/setup-uv@v3
+
+    - name: Run integration tests
+      run: make test-integration
+
     - name: Upload coverage to Codecov
       uses: codecov/codecov-action@v3
       with:

.gitignore

@@ -1,3 +1,5 @@
 build/
 ai/
 **/*-test-*/
+**/__pycache__/
+**/*.pyc

Makefile

@@ -20,6 +20,7 @@ GOFMT=$(GOCMD) fmt
 GO_FILES=$(shell find . -name '*.go' -not -path "./vendor/*")
 
 .PHONY: all build clean test coverage deps fmt lint vet help install uninstall dev
+.PHONY: test-integration test-schema test-sheet test-project test-merge test-parallel
 
 all: clean build test ## Run clean, build, and test
 
@@ -32,12 +33,23 @@ clean: ## Clean build directory
 	@echo "Cleaning..."
 	@rm -rf $(BUILD_DIR)
 	@rm -rf $(COVERAGE_DIR)
+	@rm -rf tests/.venv
+	@rm -rf tests/.pytest_cache
+	@rm -rf tests/__pycache__
+	@find . -type d -name "__pycache__" -exec rm -rf {} + 2>/dev/null || true
+	@find . -type f -name "*.pyc" -delete 2>/dev/null || true
 	$(GOCLEAN)
 
-test: ## Run tests
-	@echo "Running tests..."
+test: ## Run unit tests
+	@echo "Running unit tests..."
 	$(GOTEST) -v ./...
 
+test-integration: ## Run integration tests
+	@echo "Running integration tests..."
+	@cd tests && uv run pytest
+
+test-all: test test-integration ## Run both unit and integration tests
+
 coverage: ## Generate test coverage report
 	@echo "Generating coverage report..."
 	@mkdir -p $(COVERAGE_DIR)

internal/command/apply.go

@@ -106,7 +106,7 @@ func (c *ApplyCommand) Run(cmd *cobra.Command, args []string) error {
 		if err != nil {
 			return err
 		}
-		if !quiet {
+		if !quiet && format != "json" {
 			printer.Info(fmt.Sprintf("Applying .env file: %s", envOrSheetName))
 		}
 	} else if standalone {
@@ -115,7 +115,7 @@ func (c *ApplyCommand) Run(cmd *cobra.Command, args []string) error {
 		if err != nil {
 			return err
 		}
-		if !quiet {
+		if !quiet && format != "json" {
 			printer.Info(fmt.Sprintf("Applying standalone config sheet: %s", envOrSheetName))
 		}
 	} else {
@@ -124,14 +124,17 @@ func (c *ApplyCommand) Run(cmd *cobra.Command, args []string) error {
 		if err != nil {
 			return err
 		}
-		if !quiet {
+		if !quiet && format != "json" {
 			printer.Info(fmt.Sprintf("Applying environment '%s'", envOrSheetName))
 		}
 	}
 
 	if dryRun {
 		// Show what would be applied
-		printer.Info("Environment variables that would be applied:")
+		// Only show info message for non-JSON formats
+		if format != "json" && !quiet {
+			printer.Info("Environment variables that would be applied:")
+		}
 		switch format {
 		case "env":
 			return printer.PrintEnvironmentExport(values)

internal/command/sheet.go

@@ -328,7 +328,14 @@ func (c *SheetCommand) runExport(cmd *cobra.Command, args []string) error {
 	}
 
 	format, _ := cmd.Flags().GetString("format")
-	printer := output.NewPrinter(output.FormatTable, false)
+	// Create printer with appropriate format for values export
+	var printerFormat output.Format
+	if format == "json" {
+		printerFormat = output.FormatJSON
+	} else {
+		printerFormat = output.FormatTable
+	}
+	printer := output.NewPrinter(printerFormat, false)
 
 	sheetName := args[0]
 	cs, err := manager.ConfigSheets.Get(sheetName)

internal/entities/schema.go

@@ -142,6 +142,11 @@ func (sm *SchemaManager) Update(nameOrUUID string, updater func(*Schema) error)
 // GetByReference loads a schema by reference, handling local:// and remote:// prefixes
 func (sm *SchemaManager) GetByReference(schemaRef string) (*Schema, error) {
 	switch {
+	case strings.HasPrefix(schemaRef, "#/schemas/"):
+		// JSON Pointer style reference: #/schemas/{uuid}
+		uuid := strings.TrimPrefix(schemaRef, "#/schemas/")
+		return sm.GetByID(uuid)
+
 	case strings.HasPrefix(schemaRef, "local://"):
 		// Local schema reference: local://schema-name
 		schemaName := strings.TrimPrefix(schemaRef, "local://")

internal/entities/validator.go

@@ -218,7 +218,7 @@ func (v *Validator) ValidateConfigSheet(sheet *ConfigSheet) error {
 
 	if sheet.Schema.IsReference() {
 		// Load referenced schema
-		schema, err = v.manager.Schemas.Get(sheet.Schema.Ref)
+		schema, err = v.manager.Schemas.GetByReference(sheet.Schema.Ref)
 		if err != nil {
 			return fmt.Errorf("failed to load referenced schema: %w", err)
 		}

internal/parser/schema.go

@@ -57,11 +57,16 @@ func (p *SchemaParser) ParseFile(path string) (*SchemaData, error) {
 		return nil, fmt.Errorf("failed to read file: %w", err)
 	}
 
-	var schemaObj entities.Schema
+	// Use a temporary struct for unmarshaling that matches the file structure
+	var fileData struct {
+		Description string              `json:"description" yaml:"description"`
+		Variables   []entities.Variable `json:"variables" yaml:"variables"`
+		Extends     []string            `json:"extends" yaml:"extends"`
+	}
 
 	// Try YAML first, then JSON, then dotenv as fallback
-	if err := yaml.Unmarshal(data, &schemaObj); err != nil {
-		if err := json.Unmarshal(data, &schemaObj); err != nil {
+	if err := yaml.Unmarshal(data, &fileData); err != nil {
+		if err := json.Unmarshal(data, &fileData); err != nil {
 			// Try parsing as dotenv file as fallback
 			dotenvParser := NewAnnotatedDotEnvParser()
 			_, schema, parseErr := dotenvParser.ParseFile(path)
@@ -77,17 +82,19 @@ func (p *SchemaParser) ParseFile(path string) (*SchemaData, error) {
 				Extends:     nil,
 			}, nil
 		}
+		// JSON parsing succeeded
 		return &SchemaData{
-			Description: schemaObj.Description,
-			Variables:   schemaObj.Variables,
-			Extends:     schemaObj.Extends,
+			Description: fileData.Description,
+			Variables:   fileData.Variables,
+			Extends:     fileData.Extends,
 		}, nil
 	}
 
+	// YAML parsing succeeded
 	return &SchemaData{
-		Description: schemaObj.Description,
-		Variables:   schemaObj.Variables,
-		Extends:     schemaObj.Extends,
+		Description: fileData.Description,
+		Variables:   fileData.Variables,
+		Extends:     fileData.Extends,
 	}, nil
 }