Skip to content

t1308-config-set: fix a test that has a typo #387

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

Closed
wants to merge 1 commit into from

Conversation

dscho
Copy link
Member

@dscho dscho commented Oct 10, 2019

I am currently trying to whittle down the number of open PRs at https://github.com/git/git, and this is one of the patches I deem valuable enough (and complete enough) to put through to the Git mailing list even when the original contributor has gone silent.

Change test 'find value_list for a key from a configset' to redirect the
result to 'expect' instead of 'except' which was a typo.

With this change, the test case actually fails because it uses
`configset_get_value`. Clearly, this was intended to be
`configset_get_value_multi` since the test expects a list of values
instead of a single value, so let's fix that, too.

Originally contributed as git#451.

Signed-off-by: Tanay Abhra <[email protected]>
Signed-off-by: Johannes Schindelin <[email protected]>
@dscho
Copy link
Member Author

dscho commented Oct 10, 2019

/submit

@gitgitgadget
Copy link

gitgitgadget bot commented Oct 10, 2019

Submitted as [email protected]

@gitgitgadget
Copy link

gitgitgadget bot commented Oct 11, 2019

This branch is now known as ta/t1308-typofix.

@gitgitgadget
Copy link

gitgitgadget bot commented Oct 11, 2019

This patch series was integrated into pu via git@760c9a7.

@gitgitgadget gitgitgadget bot added the pu label Oct 11, 2019
@gitgitgadget
Copy link

gitgitgadget bot commented Oct 12, 2019

This patch series was integrated into pu via git@f24c35c.

@gitgitgadget
Copy link

gitgitgadget bot commented Oct 15, 2019

This patch series was integrated into pu via git@f9a6540.

@gitgitgadget
Copy link

gitgitgadget bot commented Oct 15, 2019

This patch series was integrated into next via git@0228b44.

@gitgitgadget gitgitgadget bot added the next label Oct 15, 2019
@gitgitgadget
Copy link

gitgitgadget bot commented Oct 16, 2019

This patch series was integrated into pu via git@7db8972.

@gitgitgadget
Copy link

gitgitgadget bot commented Oct 18, 2019

This patch series was integrated into pu via git@cdc196d.

@gitgitgadget
Copy link

gitgitgadget bot commented Oct 18, 2019

This patch series was integrated into pu via git@9b83a94.

@gitgitgadget
Copy link

gitgitgadget bot commented Oct 18, 2019

This patch series was integrated into next via git@9b83a94.

@gitgitgadget
Copy link

gitgitgadget bot commented Oct 18, 2019

This patch series was integrated into master via git@9b83a94.

@gitgitgadget gitgitgadget bot added the master label Oct 18, 2019
@gitgitgadget gitgitgadget bot closed this Oct 18, 2019
@gitgitgadget
Copy link

gitgitgadget bot commented Oct 18, 2019

Closed via 9b83a94.

@dscho dscho deleted the config-set-test-fix branch October 18, 2019 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants