fix: improve test coverage

Author: vmarceau

parameters/validate_parameter.go

@@ -156,7 +156,6 @@ func ValidateParameterSchema(
 		hash := schema.GoLow().Hash()
 		if cached, ok := validationOptions.SchemaCache.Load(hash); ok && cached != nil && cached.CompiledSchema != nil {
 			jsch = cached.CompiledSchema
-			jsonSchema = cached.RenderedJSON
 		}
 	}
 

parameters/validate_parameter_test.go

@@ -7,11 +7,10 @@ import (
 	"testing"
 
 	"github.com/pb33f/libopenapi"
+	lowv3 "github.com/pb33f/libopenapi/datamodel/low/v3"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 
-	lowv3 "github.com/pb33f/libopenapi/datamodel/low/v3"
-
 	"github.com/pb33f/libopenapi-validator/cache"
 	"github.com/pb33f/libopenapi-validator/config"
 	"github.com/pb33f/libopenapi-validator/helpers"
@@ -964,6 +963,117 @@ func Test_GetRenderedSchema_NilCache(t *testing.T) {
 	assert.NotEmpty(t, result, "GetRenderedSchema should render schema even with nil cache")
 }
 
+// Test_GetRenderedSchema_CacheMiss verifies GetRenderedSchema renders fresh when cache entry has empty RenderedJSON.
+// This tests the code path where cache lookup succeeds but RenderedJSON is empty.
+func Test_GetRenderedSchema_CacheMiss(t *testing.T) {
+	spec := []byte(`{
+		"openapi": "3.1.0",
+		"info": {"title": "Test", "version": "1.0.0"},
+		"paths": {
+			"/test": {
+				"get": {
+					"parameters": [{
+						"name": "id",
+						"in": "query",
+						"schema": {"type": "integer"}
+					}],
+					"responses": {"200": {"description": "OK"}}
+				}
+			}
+		}
+	}`)
+
+	doc, err := libopenapi.NewDocument(spec)
+	require.NoError(t, err)
+
+	v3Model, errs := doc.BuildV3Model()
+	require.Nil(t, errs)
+
+	// Get the parameter schema
+	pathItem := v3Model.Model.Paths.PathItems.GetOrZero("/test")
+	param := pathItem.Get.Parameters[0]
+	schema := param.Schema.Schema()
+
+	// Create options with cache enabled
+	opts := config.NewValidationOptions()
+	require.NotNil(t, opts.SchemaCache)
+
+	// Store an entry with empty RenderedJSON to simulate cache miss scenario
+	hash := schema.GoLow().Hash()
+	opts.SchemaCache.Store(hash, &cache.SchemaCacheEntry{
+		Schema:       schema,
+		RenderedJSON: nil, // Empty - should trigger fresh rendering
+	})
+
+	// GetRenderedSchema should render fresh since RenderedJSON is empty
+	result := GetRenderedSchema(schema, opts)
+	assert.NotEmpty(t, result, "GetRenderedSchema should render fresh when cached RenderedJSON is empty")
+}
+
+// Test_ValidateSingleParameterSchema_CacheMissCompiledSchema tests the path where cache entry
+// exists but CompiledSchema is nil, forcing recompilation.
+func Test_ValidateSingleParameterSchema_CacheMissCompiledSchema(t *testing.T) {
+	spec := []byte(`{
+		"openapi": "3.1.0",
+		"info": {"title": "Test", "version": "1.0.0"},
+		"paths": {
+			"/test/{id}": {
+				"get": {
+					"parameters": [{
+						"name": "id",
+						"in": "path",
+						"required": true,
+						"schema": {"type": "integer", "minimum": 1}
+					}],
+					"responses": {"200": {"description": "OK"}}
+				}
+			}
+		}
+	}`)
+
+	doc, err := libopenapi.NewDocument(spec)
+	require.NoError(t, err)
+
+	v3Model, errs := doc.BuildV3Model()
+	require.Nil(t, errs)
+
+	// Get the parameter schema
+	pathItem := v3Model.Model.Paths.PathItems.GetOrZero("/test/{id}")
+	param := pathItem.Get.Parameters[0]
+	schema := param.Schema.Schema()
+
+	// Create options with cache enabled
+	opts := config.NewValidationOptions()
+	require.NotNil(t, opts.SchemaCache)
+
+	// Store an entry with nil CompiledSchema to force recompilation
+	hash := schema.GoLow().Hash()
+	opts.SchemaCache.Store(hash, &cache.SchemaCacheEntry{
+		Schema:         schema,
+		CompiledSchema: nil, // nil - should trigger recompilation
+	})
+
+	// Validate should still work by recompiling the schema
+	result := ValidateSingleParameterSchema(
+		schema,
+		int64(5), // valid integer
+		"Path parameter",
+		"The path parameter",
+		"id",
+		helpers.ParameterValidation,
+		helpers.ParameterValidationPath,
+		opts,
+		"/test/{id}",
+		"get",
+	)
+	assert.Empty(t, result, "Validation should pass for valid integer")
+
+	// Now verify the cache was populated with the compiled schema
+	cached, ok := opts.SchemaCache.Load(hash)
+	assert.True(t, ok, "Cache entry should exist")
+	assert.NotNil(t, cached.CompiledSchema, "CompiledSchema should be populated after validation")
+}
+
 // arrayValidationSpec is used to test array parameter validation with the updated function signatures
 var arrayValidationSpec = []byte(`{
 	"openapi": "3.1.0",