splunk / qbec

configure kubernetes objects on multiple clusters using jsonnet
https://qbec.io
Apache License 2.0
172 stars 37 forks source link

Update error message to include the input #278

Closed kalhanreddy closed 2 years ago

kalhanreddy commented 2 years ago

Updating error message so we can understand what is the command causing issue.

Example: Previous: exactly one environment required After change: exactly one environment required, but provided: [dev prod]

Testing: Ran make test and verified all tests pass locally(Note: Had to delete helm_test.go as i don't have helm installed locally)

codecov-commenter commented 2 years ago

Codecov Report

Merging #278 (5b2b823) into main (bd96b05) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #278   +/-   ##
=======================================
  Coverage   74.66%   74.66%           
=======================================
  Files          72       72           
  Lines        6948     6948           
=======================================
  Hits         5188     5188           
  Misses       1544     1544           
  Partials      216      216           
Impacted Files Coverage Δ
internal/commands/apply.go 76.73% <100.00%> (ø)
internal/commands/component.go 72.22% <100.00%> (ø)
internal/commands/delete.go 66.66% <100.00%> (ø)
internal/commands/diff.go 83.15% <100.00%> (ø)
internal/commands/env.go 94.70% <100.00%> (ø)
internal/commands/param.go 75.80% <100.00%> (ø)
internal/commands/show.go 92.30% <100.00%> (ø)
internal/commands/validate.go 91.66% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update bd96b05...5b2b823. Read the comment docs.