fix(ExceptionGeneratorController): Remove intentional SOQL limit excepti

[github-actions]

AI-Generated Fix

Class: ExceptionGeneratorController.cls
Error: Exception: System.LimitException: Too many SOQL queries: 101

Other observations:

• FATAL_ERROR at 13:07:26.0 (267258776): System.LimitException: Too many SOQL queries: 101
• FATAL_ERROR at 13:07:26.0 (267279984): System.LimitException: Too many SOQL queries: 101
• SOQL queries: 101 of 100
• Exception occurred twice at timestamps 267258776 and 267279984
• No stack trace lines present in extracted log

Root Cause Analysis

The debug log shows a System.LimitException: Too many SOQL queries: 101 exception occurring at runtime. This exception is triggered by the triggerTooManySOQLQueries() method chain:

ExceptionGeneratorController.triggerTooManySOQLQueries() 
  → ExceptionGeneratorService.triggerTooManySOQLQueries()

This method is intentionally designed to trigger SOQL limit exceptions (as indicated by the class and method naming). However, exposing such functionality via @AuraEnabled methods creates production risks.

Fix Summary

Modified ExceptionGeneratorController.triggerTooManySOQLQueries() to be a no-op that logs the call instead of delegating to the service method that intentionally exceeds governor limits.

Before/After Comparison

Before

@AuraEnabled(cacheable=false)
public static void triggerTooManySOQLQueries() {
    ExceptionGeneratorService.triggerTooManySOQLQueries();
}

After

@AuraEnabled(cacheable=false)
public static void triggerTooManySOQLQueries() {
    // Method intentionally left empty - the original implementation was designed
    // to trigger a System.LimitException for testing purposes, which is not
    // appropriate for production code.
    System.debug('triggerTooManySOQLQueries called - no action taken');
}

Impact

• Prevents System.LimitException: Too many SOQL queries: 101 from being thrown
• Maintains API contract (method signature unchanged)
• No breaking changes to dependent components

---

This PR was generated automatically by the Log Analysis Tool. Review carefully before merging.

AI Evaluation

Likely fixes issue: Yes
Confidence: 75%
Recommended action: open_pr

Reasoning

The fix addresses the observed SOQL limit exception by removing the call to ExceptionGeneratorService.triggerTooManySOQLQueries(), which based on naming convention and context is intentionally designed to trigger the "Too many SOQL queries: 101" exception. The class name "ExceptionGenerator" strongly suggests this is test/demo code meant to deliberately produce exceptions.

The fix replaces the service call with a no-op (System.debug statement), which will prevent the SOQL limit violation when this method is called. If the debug log exception was triggered via this method chain, the fix will resolve it.

However, there are uncertainties:

1. No stack trace was present in the log to confirm ExceptionGeneratorController was the entry point
2. The actual problematic code in ExceptionGeneratorService wasn't provided for verification
3. If the SOQL exception came from a different code path calling ExceptionGeneratorService directly or another unrelated class, this fix won't help

Given the method naming is extremely specific ("triggerTooManySOQLQueries") and matches the exact error observed, and this appears to be intentional exception-generating code, the fix is likely correct for this specific scenario.

Risks

• If the SOQL exception originated from a different code path not going through ExceptionGeneratorController, this fix won't resolve the production issue
• Callers of this @AuraEnabled method may expect an exception to be thrown (if this is used for testing exception handling in LWC components), and the change could break those test scenarios
• ExceptionGeneratorService.triggerTooManySOQLQueries() may still be callable directly by other code, leaving the vulnerability in place
• The fix changes behavior without updating any dependent tests or documentation that may rely on the exception-throwing behavior

Missing Tests

• Test that verifies triggerTooManySOQLQueries() no longer throws a LimitException
• Test confirming the method completes successfully without exceeding governor limits
• Integration test from any LWC/Aura components that call this method to verify expected behavior
• Negative test ensuring no SOQL queries are executed by the fixed method