Skip to content

Params validitanion tests#944

Closed
Ra0to wants to merge 6 commits intodahlbyk:masterfrom
Ra0to:feature/param-validation-tests
Closed

Params validitanion tests#944
Ra0to wants to merge 6 commits intodahlbyk:masterfrom
Ra0to:feature/param-validation-tests

Conversation

@Ra0to
Copy link
Copy Markdown
Contributor

@Ra0to Ra0to commented Feb 13, 2023

During work on PR-943 I created tests for params validation. I'm not sure is they necessary or not. I found several problems in current suggestions and fixed them in this PR and linked above too.

It takes more much more time for tests completion.
On my machine:

  • Current master tests takes ~34s
  • This branch tests takes ~153s

Feel free to ask any improvements.

Fixed:
Subcommand 'grep' param 'open-file-in-pager' added missing s ->
'open-files-in-pager'.

Subcommand 'log':
    - Missing equal signs:
        - 'min-parents' -> 'min-parents='
        - 'max-parents' -> 'max-parents='
    - Missing s in 'remote' param -> 'remotes'

Subcommand 'pull' removed param 'update-head-ok' that doesn't exists.
@Ra0to
Copy link
Copy Markdown
Contributor Author

Ra0to commented Feb 13, 2023

Sorry, tests on my machine completes successfully. Going to investigate problem, closing PR for now.

@Ra0to Ra0to closed this Feb 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant