Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add some tests for the CLI completion #6784

Merged
merged 6 commits into from
Jul 18, 2022
Merged

Commits on Jul 12, 2022

  1. Configuration menu
    Copy the full SHA
    32effcc View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    260fb83 View commit details
    Browse the repository at this point in the history
  3. Harden the comparisons in the completion test

    - Don't use `printf` for the expected result, but just use bash's `$' '`
      litteral strings
    - Quote the `nix` call result
    - Invert the order in the comparisons (just because it feels more
      natural)
    thufschmitt committed Jul 12, 2022
    Configuration menu
    Copy the full SHA
    07e14d3 View commit details
    Browse the repository at this point in the history
  4. Test the tilde expansion for the flake completion

    Also add a disabled test for when the `--override-input` flag comes
    *before* the flake ref
    thufschmitt committed Jul 12, 2022
    Configuration menu
    Copy the full SHA
    21c443d View commit details
    Browse the repository at this point in the history

Commits on Jul 13, 2022

  1. Fix the “out of order” completion test

    `--override-input` id snarky because it takes two arguments, so it
    doesn't play well when completed in the middle of the CLI (since the
    argument just after gets interpreted as its second argument). So use
    `--update-input` instead
    thufschmitt committed Jul 13, 2022
    Configuration menu
    Copy the full SHA
    d34a333 View commit details
    Browse the repository at this point in the history
  2. Add some more completion tests

    - Test another command than `build`
    - Test with two input flakes
    thufschmitt committed Jul 13, 2022
    Configuration menu
    Copy the full SHA
    b052e7e View commit details
    Browse the repository at this point in the history