Error Report: `func config` Command Testing (LFX Mentorship)

[RayyanSeliya]

Summary

Comprehensive testing of the func config command identified 25+ issues across environment variables, git configuration, labels, and volumes management, with the primary concerns being critical crashes where the envs remove command panics with nil pointer dereference for all removal attempts, making the feature completely broken; missing validations where duplicate environment variable names are allowed creating invalid configurations, duplicate volume mount paths are accepted leading to deployment failures, and invalid namespace formats, builder types, and provider names are accepted without early validation; and late validation problems where invalid URLs, unsupported builders, and invalid size formats are only caught after lengthy interactive prompts, wasting user time. Key findings include the critical bug where removing environment variables crashes the application regardless of whether the variable exists or not; the data integrity issue where duplicate env names and mount paths are allowed without warnings, creating configurations that will fail at deployment; and the incomplete implementation where git config listing shows "Not implemented yet" despite being a documented feature. Additionally, case-sensitive output formats cause confusion when users provide JSON or YAML instead of lowercase, flags are silently ignored without warnings (like --source for EmptyDir volumes), and the verbose flag produces no additional output across all subcommands.

Full Detailed Report: Link to Google Docs -https://docs.google.com/document/d/1f7kISQIGJ3h9JrQAunTVaiyJj45Jf8aQCLpauFnQ70k/edit?usp=sharing

Resolved Issues

The following issues have been marked as resolved in the detailed report:

(No issues marked as resolved yet)