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

[PR #5030/74f2e1d2 backport][stable-5] passwordstore: Add some real gopass integration tests #5042

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jul 31, 2022

This is a backport of PR #5030 as merged into main (74f2e1d).

SUMMARY

Since the lookup_passwordstore module gained some real gopass support, add real unit tests for it.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

lookup_passwordstore

* passwordstore: Add some real go tests

This is work in progress.

* passwordstore: Fix gopass init

* Init gopass store in explicit path in integration test

* passwordstore: Show versions of tools in integration test

* passwordstore: Install gopass from different location on Debian

Part of integration tests

* passwordstore: Add changelog fragment for #5030

* passwordstore: Address review feedback

(cherry picked from commit 74f2e1d)
@ansibullbot ansibullbot added backport feature This issue/PR relates to a feature request integration tests/integration new_contributor Help guide this first time contributor tests tests labels Jul 31, 2022
@felixfontein felixfontein merged commit 53b2d69 into stable-5 Jul 31, 2022
@felixfontein felixfontein deleted the patchback/backports/stable-5/74f2e1d28b01e25fe2c894381b7fff1996cedfd9/pr-5030 branch July 31, 2022 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature This issue/PR relates to a feature request integration tests/integration new_contributor Help guide this first time contributor tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants