silverfin / silverfin-cli

Command line tool for Silverfin template development
4 stars 1 forks source link

config.json file created on command 'silverfin run-test' #115

Closed latifatamisaft closed 5 months ago

latifatamisaft commented 7 months ago

Describe the bug When you use the command line to run a liquid test, you obviously want to make sure that you use the correct handle 🙈 If you go manage to get a typo in there, it:

Now it immediately caught my attention as I didn't have any files in my source control panel but easy to miss when you have multiple files in there..

To Reproduce Steps to reproduce the behavior:

  1. put the command "silverfin run-test -h _handle_with_atypo" in the terminal
  2. tadaa, you get a config.json file 😅

Expected behavior The error in the terminal is the only required action in this case. No config.json file should be created on the command 'run-test' and especially not if the handle doesn't exist.

Screenshots or Pull request Screenshot 2024-02-27 at 15 48 43

Desktop & CLI (please complete the following information):