diff --git a/openapi/tests/user/config_validation_test.go b/openapi/tests/user/config_validation_test.go index efbbc102..75a2cd16 100644 --- a/openapi/tests/user/config_validation_test.go +++ b/openapi/tests/user/config_validation_test.go @@ -10,11 +10,11 @@ import ( func TestConfigValidationLogic(t *testing.T) { // Test cases for different configuration scenarios testCases := []struct { - name string - clientID string - clientSecret string - shouldPass bool - expectedError string + name string + clientID string + clientSecret string + shouldPass bool + expectedError string }{ { name: "valid_direct_values", @@ -58,13 +58,13 @@ func TestConfigValidationLogic(t *testing.T) { // This is a conceptual test - in practice, we'd test the actual validation logic t.Logf("Testing scenario: %s", tc.name) t.Logf("ClientID: %s, ClientSecret: %s", tc.clientID, tc.clientSecret) - + if tc.shouldPass { t.Logf("Expected: Should pass validation") } else { t.Logf("Expected: Should fail with error: %s", tc.expectedError) } - + // This test documents the expected behavior assert.True(t, true, "Validation logic should be tested through integration tests") }) @@ -90,7 +90,7 @@ func TestEnvVarNameExtraction(t *testing.T) { for _, tc := range testCases { t.Run(tc.input, func(t *testing.T) { t.Logf("Input: %s, Expected: %s", tc.input, tc.expected) - + // This test documents the expected behavior of extractEnvVarName // In practice, we'd need to make the function public or test it through integration assert.True(t, true, "Function behavior should be tested through integration tests") diff --git a/openapi/tests/user/env_var_extraction_test.go b/openapi/tests/user/env_var_extraction_test.go index aa17b42c..cec85397 100644 --- a/openapi/tests/user/env_var_extraction_test.go +++ b/openapi/tests/user/env_var_extraction_test.go @@ -10,7 +10,7 @@ import ( func TestExtractEnvVarName(t *testing.T) { // Import the user package to access the function // Note: This test assumes the function is exported or we can test it indirectly - + testCases := []struct { input string expected string @@ -28,7 +28,7 @@ func TestExtractEnvVarName(t *testing.T) { // Since extractEnvVarName is not exported, we'll test the behavior indirectly // by checking if the error message contains the correct variable name t.Logf("Testing input: %s, expected: %s", tc.input, tc.expected) - + // This is a conceptual test - in practice, we'd need to make the function public // or test it through the public API assert.True(t, true, "Function behavior should be tested through integration tests") @@ -40,7 +40,7 @@ func TestExtractEnvVarName(t *testing.T) { func TestEnvVarNameExtractionIntegration(t *testing.T) { // This test verifies that the error message correctly identifies the missing environment variable // by checking the actual error message format - + // Test with a custom environment variable name testCases := []struct { name string @@ -53,7 +53,7 @@ func TestEnvVarNameExtractionIntegration(t *testing.T) { expected: "SIGNIN_CLIENT_ID", }, { - name: "CUSTOM_CLIENT_ID", + name: "CUSTOM_CLIENT_ID", envVar: "CUSTOM_CLIENT_ID", expected: "CUSTOM_CLIENT_ID", },