diff options
Diffstat (limited to 'UNITTESTSTOADD.md')
| -rw-r--r-- | UNITTESTSTOADD.md | 511 |
1 files changed, 0 insertions, 511 deletions
diff --git a/UNITTESTSTOADD.md b/UNITTESTSTOADD.md deleted file mode 100644 index 2ebcd02..0000000 --- a/UNITTESTSTOADD.md +++ /dev/null @@ -1,511 +0,0 @@ -# Unit tests to consider adding - -This report lists functions that do not appear to be referenced in test files in the same package directory. Recommendations are heuristic. - -## main (cmd/hexai-lsp) - -- cmd/hexai-lsp/main.go — func main - - exported: false complexity: 5 has-control: true - - recommendation: add test (non-trivial logic) - -## hexaicli (internal/hexaicli) - -- internal/hexaicli/run.go — func Run - - exported: true complexity: 5 has-control: true - - recommendation: add test (exported) -- internal/hexaicli/run.go — func newClientFromConfig - - exported: false complexity: 6 has-control: true - - recommendation: add test (non-trivial logic) - -## llm (internal/llm) - -- internal/llm/copilot.go — method (copilotClient).Chat - - exported: true complexity: 24 has-control: true - - recommendation: add test (exported) -- internal/llm/copilot.go — method (copilotClient).CodeCompletion - - exported: true complexity: 18 has-control: true - - recommendation: add test (exported) -- internal/llm/copilot.go — method (copilotClient).DefaultModel - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) -- internal/llm/copilot.go — method (copilotClient).Name - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) -- internal/llm/copilot.go — func decodeCopilotChat - - exported: false complexity: 3 has-control: true - - recommendation: add test (non-trivial logic) -- internal/llm/copilot.go — method (copilotClient).ensureSession - - exported: false complexity: 19 has-control: true - - recommendation: add test (non-trivial logic) -- internal/llm/copilot.go — func handleCopilotNon2xx - - exported: false complexity: 6 has-control: true - - recommendation: add test (non-trivial logic) -- internal/llm/copilot.go — method (copilotClient).headersChat - - exported: false complexity: 3 has-control: false - - recommendation: add test (non-trivial logic) -- internal/llm/copilot.go — method (copilotClient).headersGhost - - exported: false complexity: 3 has-control: false - - recommendation: add test (non-trivial logic) -- internal/llm/copilot.go — func newCopilot - - exported: false complexity: 3 has-control: true - - recommendation: add test (non-trivial logic) -- internal/llm/copilot.go — func parseInt64 - - exported: false complexity: 3 has-control: false - - recommendation: add test (non-trivial logic) -- internal/llm/copilot.go — func parseJWTExp - - exported: false complexity: 8 has-control: true - - recommendation: add test (non-trivial logic) -- internal/llm/copilot.go — method (copilotClient).postJSON - - exported: false complexity: 4 has-control: true - - recommendation: add test (non-trivial logic) -- internal/llm/copilot.go — func randHex - - exported: false complexity: 4 has-control: true - - recommendation: add test (non-trivial logic) -- internal/llm/ollama.go — method (ollamaClient).Chat - - exported: true complexity: 20 has-control: true - - recommendation: add test (exported) -- internal/llm/ollama.go — method (ollamaClient).ChatStream - - exported: true complexity: 18 has-control: true - - recommendation: add test (exported) -- internal/llm/ollama.go — method (ollamaClient).DefaultModel - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) -- internal/llm/ollama.go — method (ollamaClient).Name - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) -- internal/llm/ollama.go — method (ollamaClient).doJSON - - exported: false complexity: 4 has-control: true - - recommendation: add test (non-trivial logic) -- internal/llm/ollama.go — func handleOllamaNon2xx - - exported: false complexity: 6 has-control: true - - recommendation: add test (non-trivial logic) -- internal/llm/ollama.go — method (ollamaClient).logStart - - exported: false complexity: 3 has-control: true - - recommendation: add test (non-trivial logic) -- internal/llm/ollama.go — func newOllama - - exported: false complexity: 3 has-control: true - - recommendation: add test (non-trivial logic) -- internal/llm/openai.go — method (openAIClient).Chat - - exported: true complexity: 21 has-control: true - - recommendation: add test (exported) -- internal/llm/openai.go — method (openAIClient).ChatStream - - exported: true complexity: 18 has-control: true - - recommendation: add test (exported) -- internal/llm/openai.go — method (openAIClient).DefaultModel - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) -- internal/llm/openai.go — method (openAIClient).Name - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) -- internal/llm/openai.go — func decodeOpenAIChat - - exported: false complexity: 3 has-control: true - - recommendation: add test (non-trivial logic) -- internal/llm/openai.go — method (openAIClient).doJSON - - exported: false complexity: 5 has-control: true - - recommendation: add test (non-trivial logic) -- internal/llm/openai.go — method (openAIClient).doJSONWithAccept - - exported: false complexity: 6 has-control: true - - recommendation: add test (non-trivial logic) -- internal/llm/openai.go — method (openAIClient).logStart - - exported: false complexity: 3 has-control: true - - recommendation: add test (non-trivial logic) -- internal/llm/openai.go — method (openAIClient).logf - - exported: false complexity: 1 has-control: false - - recommendation: optional (helper/trivial) -- internal/llm/openai.go — func newOpenAI - - exported: false complexity: 3 has-control: true - - recommendation: add test (non-trivial logic) -- internal/llm/provider.go — func NewFromConfig - - exported: true complexity: 3 has-control: true - - recommendation: add test (exported) -- internal/llm/provider.go — func WithMaxTokens - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) -- internal/llm/provider.go — func WithModel - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) -- internal/llm/provider.go — func WithStop - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) -- internal/llm/provider.go — func WithTemperature - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) -- internal/llm/util.go — func nilStringErr - - exported: false complexity: 1 has-control: false - - recommendation: optional (helper/trivial) - -## lsp (internal/lsp) - -- internal/lsp/context.go — method (Server).fullFileContext - - exported: false complexity: 3 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/document.go — method (Server).deleteDocument - - exported: false complexity: 3 has-control: false - - recommendation: add test (exported receiver) -- internal/lsp/document.go — method (Server).getDocument - - exported: false complexity: 3 has-control: false - - recommendation: add test (exported receiver) -- internal/lsp/document.go — func hasAny - - exported: false complexity: 2 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/document.go — method (Server).isDefiningNewFunction - - exported: false complexity: 10 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers.go — method (Server).compCacheTouchLocked - - exported: false complexity: 4 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers.go — method (Server).completionCacheGet - - exported: false complexity: 6 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers.go — method (Server).completionCacheKey - - exported: false complexity: 13 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers.go — method (Server).completionCachePut - - exported: false complexity: 6 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers.go — method (Server).fallbackCompletionItems - - exported: false complexity: 1 has-control: false - - recommendation: add test (exported receiver) -- internal/lsp/handlers.go — method (Server).handle - - exported: false complexity: 2 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers.go — func instructionFromSelection - - exported: false complexity: 3 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers.go — method (Server).isTriggerEvent - - exported: false complexity: 7 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers.go — method (Server).makeCompletionItems - - exported: false complexity: 6 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers.go — method (Server).reply - - exported: false complexity: 2 has-control: false - - recommendation: add test (exported receiver) -- internal/lsp/handlers_codeaction.go — method (Server).buildDocumentCodeAction - - exported: false complexity: 6 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_codeaction.go — method (Server).buildGoUnitTestCodeAction - - exported: false complexity: 9 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_codeaction.go — func deriveGoFuncName - - exported: false complexity: 7 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_codeaction.go — method (Server).diagnosticsInRange - - exported: false complexity: 7 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_codeaction.go — func exportName - - exported: false complexity: 4 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_codeaction.go — func fileExists - - exported: false complexity: 2 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_codeaction.go — func findGoFunctionAtLine - - exported: false complexity: 10 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_codeaction.go — method (Server).generateGoTestFunction - - exported: false complexity: 4 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_codeaction.go — func greaterPos - - exported: false complexity: 2 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_codeaction.go — method (Server).handleCodeAction - - exported: false complexity: 11 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_codeaction.go — method (Server).handleCodeActionResolve - - exported: false complexity: 4 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_codeaction.go — func lessPos - - exported: false complexity: 2 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_codeaction.go — method (Server).loadFileText - - exported: false complexity: 6 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_codeaction.go — func parseGoPackageName - - exported: false complexity: 2 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_codeaction.go — func rangesOverlap - - exported: false complexity: 5 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_codeaction.go — method (Server).resolveGoTest - - exported: false complexity: 30 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_completion.go — method (Server).buildCompletionMessages - - exported: false complexity: 5 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_completion.go — method (Server).buildDocString - - exported: false complexity: 1 has-control: false - - recommendation: add test (exported receiver) -- internal/lsp/handlers_completion.go — func extractTriggerInfo - - exported: false complexity: 4 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_completion.go — method (Server).handleCompletion - - exported: false complexity: 5 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_completion.go — method (Server).logCompletionContext - - exported: false complexity: 1 has-control: false - - recommendation: add test (exported receiver) -- internal/lsp/handlers_completion.go — func parseManualInvoke - - exported: false complexity: 4 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_completion.go — method (Server).postProcessCompletion - - exported: false complexity: 6 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_completion.go — method (Server).prefixHeuristicAllows - - exported: false complexity: 11 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_completion.go — method (Server).shouldSuppressForChatTriggerEOL - - exported: false complexity: 2 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_completion.go — method (Server).tryProviderNativeCompletion - - exported: false complexity: 18 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_completion.go — method (Server).waitForDebounce - - exported: false complexity: 3 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_completion.go — method (Server).waitForThrottle - - exported: false complexity: 4 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_document.go — method (Server).applyChatEdits - - exported: false complexity: 10 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_document.go — method (Server).buildChatHistory - - exported: false complexity: 10 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_document.go — method (Server).clientApplyEdit - - exported: false complexity: 6 has-control: false - - recommendation: add test (exported receiver) -- internal/lsp/handlers_document.go — method (Server).clientShowDocument - - exported: false complexity: 9 has-control: false - - recommendation: add test (exported receiver) -- internal/lsp/handlers_document.go — method (Server).deferShowDocument - - exported: false complexity: 1 has-control: false - - recommendation: add test (exported receiver) -- internal/lsp/handlers_document.go — method (Server).detectAndHandleChat - - exported: false complexity: 4 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_document.go — method (Server).docBeforeAfter - - exported: false complexity: 16 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_document.go — method (Server).handleDidChange - - exported: false complexity: 2 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_document.go — method (Server).handleDidClose - - exported: false complexity: 2 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_document.go — method (Server).handleDidOpen - - exported: false complexity: 2 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_document.go — method (Server).nextReqID - - exported: false complexity: 6 has-control: false - - recommendation: add test (exported receiver) -- internal/lsp/handlers_document.go — func stripTrailingTrigger - - exported: false complexity: 6 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_execute.go — method (Server).handleExecuteCommand - - exported: false complexity: 3 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_init.go — method (Server).handleExit - - exported: false complexity: 2 has-control: false - - recommendation: add test (exported receiver) -- internal/lsp/handlers_init.go — method (Server).handleInitialize - - exported: false complexity: 4 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_init.go — method (Server).handleInitialized - - exported: false complexity: 1 has-control: false - - recommendation: add test (exported receiver) -- internal/lsp/handlers_init.go — method (Server).handleShutdown - - exported: false complexity: 1 has-control: false - - recommendation: add test (exported receiver) -- internal/lsp/handlers_utils.go — func applyIndent - - exported: false complexity: 4 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_utils.go — func buildPrompts - - exported: false complexity: 4 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_utils.go — method (Server).collectPromptRemovalEdits - - exported: false complexity: 5 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_utils.go — func computeTextEditAndFilter - - exported: false complexity: 5 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_utils.go — func computeWordStart - - exported: false complexity: 3 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_utils.go — func extractRangeText - - exported: false complexity: 13 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_utils.go — func inParamList - - exported: false complexity: 4 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_utils.go — method (Server).incRecvCounters - - exported: false complexity: 4 has-control: false - - recommendation: add test (exported receiver) -- internal/lsp/handlers_utils.go — method (Server).incSentCounters - - exported: false complexity: 4 has-control: false - - recommendation: add test (exported receiver) -- internal/lsp/handlers_utils.go — func isBareDoubleSemicolon - - exported: false complexity: 5 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_utils.go — func isIdentBoundary - - exported: false complexity: 1 has-control: false - - recommendation: optional (helper/trivial) -- internal/lsp/handlers_utils.go — func isIdentChar - - exported: false complexity: 1 has-control: false - - recommendation: optional (helper/trivial) -- internal/lsp/handlers_utils.go — func labelForCompletion - - exported: false complexity: 3 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_utils.go — func leadingIndent - - exported: false complexity: 4 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_utils.go — func lineHasInlinePrompt - - exported: false complexity: 2 has-control: true - - recommendation: add test (non-trivial logic) -- internal/lsp/handlers_utils.go — method (Server).llmRequestOpts - - exported: false complexity: 3 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/handlers_utils.go — method (Server).logLLMStats - - exported: false complexity: 13 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/server.go — method (Server).Run - - exported: true complexity: 1 has-control: true - - recommendation: add test (exported) -- internal/lsp/transport.go — method (Server).readMessage - - exported: false complexity: 7 has-control: true - - recommendation: add test (exported receiver) -- internal/lsp/transport.go — method (Server).writeMessage - - exported: false complexity: 5 has-control: true - - recommendation: add test (exported receiver) - -## main (scripts) - -- scripts/scan_uncovered.go — func isExportedIdent - - exported: false complexity: 3 has-control: true - - recommendation: add test (non-trivial logic) -- scripts/scan_uncovered.go — func main - - exported: false complexity: 12 has-control: true - - recommendation: add test (non-trivial logic) -- scripts/scan_uncovered.go — func recommend - - exported: false complexity: 4 has-control: true - - recommendation: add test (non-trivial logic) -- scripts/scan_uncovered.go — func relPath - - exported: false complexity: 2 has-control: true - - recommendation: add test (non-trivial logic) -- scripts/scan_uncovered.go — func typeString - - exported: false complexity: 1 has-control: false - - recommendation: optional (helper/trivial) - -## appconfig (internal/appconfig) - -- internal/appconfig/config.go — func Load - - exported: true complexity: 6 has-control: true - - recommendation: add test (exported) -- internal/appconfig/config.go — func getConfigPath - - exported: false complexity: 3 has-control: true - - recommendation: add test (non-trivial logic) -- internal/appconfig/config.go — func loadFromEnv - - exported: false complexity: 27 has-control: true - - recommendation: add test (non-trivial logic) -- internal/appconfig/config.go — func loadFromFile - - exported: false complexity: 7 has-control: true - - recommendation: add test (non-trivial logic) -- internal/appconfig/config.go — method (App).mergeBasics - - exported: false complexity: 11 has-control: true - - recommendation: add test (exported receiver) -- internal/appconfig/config.go — method (App).mergeProviderFields - - exported: false complexity: 9 has-control: true - - recommendation: add test (exported receiver) -- internal/appconfig/config.go — method (App).mergeWith - - exported: false complexity: 2 has-control: false - - recommendation: add test (exported receiver) -- internal/appconfig/config.go — func newDefaultConfig - - exported: false complexity: 2 has-control: false - - recommendation: optional (helper/trivial) - -## hexailsp (internal/hexailsp) - -- internal/hexailsp/run.go — func buildClientIfNil - - exported: false complexity: 7 has-control: true - - recommendation: add test (non-trivial logic) -- internal/hexailsp/run.go — func ensureFactory - - exported: false complexity: 2 has-control: true - - recommendation: add test (non-trivial logic) -- internal/hexailsp/run.go — func makeServerOptions - - exported: false complexity: 1 has-control: false - - recommendation: optional (helper/trivial) -- internal/hexailsp/run.go — func normalizeLoggingConfig - - exported: false complexity: 2 has-control: true - - recommendation: add test (non-trivial logic) - -## logging (internal/logging) - -- internal/logging/chatlogger.go — method (ChatLogger).LogStart - - exported: true complexity: 4 has-control: true - - recommendation: add test (exported) -- internal/logging/chatlogger.go — func NewChatLogger - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) -- internal/logging/logging.go — func Bind - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) -- internal/logging/logging.go — func Logf - - exported: true complexity: 3 has-control: true - - recommendation: add test (exported) -- internal/logging/logging.go — func PreviewForLog - - exported: true complexity: 2 has-control: true - - recommendation: add test (exported) -- internal/logging/logging.go — func SetLogPreviewLimit - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) - -## main (.) - -- Magefile.go — func Build - - exported: true complexity: 3 has-control: false - - recommendation: add test (exported) -- Magefile.go — func BuildHexaiCLI - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) -- Magefile.go — func BuildHexaiLSP - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) -- Magefile.go — func Cover - - exported: true complexity: 10 has-control: true - - recommendation: add test (exported) -- Magefile.go — func CoverAll - - exported: true complexity: 10 has-control: true - - recommendation: add test (exported) -- Magefile.go — func Dev - - exported: true complexity: 3 has-control: true - - recommendation: add test (exported) -- Magefile.go — func DevInstall - - exported: true complexity: 2 has-control: true - - recommendation: add test (exported) -- Magefile.go — func Install - - exported: true complexity: 7 has-control: true - - recommendation: add test (exported) -- Magefile.go — func Lint - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) -- Magefile.go — func RunCLI - - exported: true complexity: 3 has-control: false - - recommendation: add test (exported) -- Magefile.go — func Test - - exported: true complexity: 2 has-control: true - - recommendation: add test (exported) -- Magefile.go — func Vet - - exported: true complexity: 1 has-control: false - - recommendation: add test (exported) -- Magefile.go — func totalCoveragePercent - - exported: false complexity: 8 has-control: true - - recommendation: add test (non-trivial logic) -- Magefile.go — func warnIfLowCoverage - - exported: false complexity: 6 has-control: true - - recommendation: add test (non-trivial logic) - -## main (cmd/hexai) - -- cmd/hexai/main.go — func main - - exported: false complexity: 4 has-control: true - - recommendation: add test (non-trivial logic) - |