aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* test(809): remove reliance on repo's filesystemSteve Moyer2022-06-224-16/+15
|
* test: use testing.TB as common interfaceSteve Moyer2022-06-213-9/+3
|
* test(809): remove remaining calls to InitGoRepo in testsSteve Moyer2022-06-163-34/+25
| | | | Note that examples still need to shown how a developer would use the library.
* Merge branch 'master' into refactor/guarantee-test-cleanupSteve Moyer2022-06-162-26/+32
|\
| * Merge pull request #808 from selesy/fix/778-git-bug-rm-brokenSteve Moyer2022-06-155-4/+133
| |\ | | | | | | fix(778): git bug rm broken
| * \ Merge pull request #795 from ↵Michael Muré2022-06-122-22/+27
| |\ \ | | | | | | | | | | | | | | | | MichaelMure/dependabot/go_modules/github.com/xanzy/go-gitlab-0.68.0 build(deps): bump github.com/xanzy/go-gitlab from 0.64.0 to 0.68.0
| | * | build(deps): bump github.com/xanzy/go-gitlab from 0.64.0 to 0.68.0dependabot[bot]2022-05-252-22/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/xanzy/go-gitlab](https://github.com/xanzy/go-gitlab) from 0.64.0 to 0.68.0. - [Release notes](https://github.com/xanzy/go-gitlab/releases) - [Changelog](https://github.com/xanzy/go-gitlab/blob/master/releases_test.go) - [Commits](https://github.com/xanzy/go-gitlab/compare/v0.64.0...v0.68.0) --- updated-dependencies: - dependency-name: github.com/xanzy/go-gitlab dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
| * | | Merge pull request #814 from ↵Michael Muré2022-06-122-4/+5
| |\ \ \ | | | | | | | | | | | | | | | | | | | | MichaelMure/dependabot/go_modules/github.com/stretchr/testify-1.7.2 build(deps): bump github.com/stretchr/testify from 1.7.1 to 1.7.2
| | * | | build(deps): bump github.com/stretchr/testify from 1.7.1 to 1.7.2dependabot[bot]2022-06-072-4/+5
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.7.1 to 1.7.2. - [Release notes](https://github.com/stretchr/testify/releases) - [Commits](https://github.com/stretchr/testify/compare/v1.7.1...v1.7.2) --- updated-dependencies: - dependency-name: github.com/stretchr/testify dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
* | | / refactor(809): eliminate need to defer CleanupTestRepos()Steve Moyer2022-06-1620-93/+60
| |_|/ |/| |
* | | test(778): simplify and guarantee backend cleanupSteve Moyer2022-06-151-18/+4
| | |
* | | refactor(778): test only command implementationsSteve Moyer2022-06-158-191/+94
| | |
* | | chore(808): rearrange imports to git-bug conventionSteve Moyer2022-06-071-2/+3
| | |
* | | test(808): document getCWD() and clean-up argumentsSteve Moyer2022-06-071-2/+8
| | |
* | | test(808): skip root help test that uses a golden fileSteve Moyer2022-06-071-4/+2
| | |
* | | fix(808): remove duplication stderr/stdout set-upSteve Moyer2022-06-061-3/+0
| | |
* | | test(808): make build tag compatible with Go 1.16Steve Moyer2022-06-061-0/+1
| | |
* | | test(808): do not run golden file tests on WindowsSteve Moyer2022-06-062-52/+63
| | |
* | | fix(808): simplify handling of Windows line terminationsSteve Moyer2022-06-061-2/+1
| | |
* | | chore(808): merge in LocalStorage namespace configurationSteve Moyer2022-06-062-3/+3
| | |
* | | Merge branch 'master' into fix/778-git-bug-rm-brokenSteve Moyer2022-06-0622-203/+277
|\| |
| * | Merge pull request #803 from MichaelMure/feat/configurable-local-storageMichael Muré2022-06-066-32/+90
| |\ \ | | | | | | | | Feat/configurable local storage
| | * | refactor: simplify creation of temp dir - after 1.15Steve Moyer2022-05-311-5/+1
| | | |
| | * | fix: remove only t.Parallel()Steve Moyer2022-05-311-2/+0
| | | |
| | * | gogit: close index before deleting it on diskMichael Muré2022-05-311-9/+8
| | | |
| | * | test: clean up temp dir and repo correctlySteve Moyer2022-05-312-5/+19
| | | |
| | * | test: add verification that localStorage.Root() resolves to the correct ↵Steve Moyer2022-05-261-3/+36
| | | | | | | | | | | | | | | | absolute filepath
| | * | refactor: use namespace instead of application of applicationNameSteve Moyer2022-05-265-28/+28
| | | |
| | * | chore: clean-up commented codeSteve Moyer2022-05-251-3/+0
| | | |
| | * | feat: make local storage configurableSteve Moyer2022-05-256-22/+43
| | |/
| * | Merge pull request #813 from MichaelMure/fix-data-race2Michael Muré2022-06-064-151/+144
| |\ \ | | | | | | | | Github: fix data race
| | * | github: fix data race when closing event channelMichael Muré2022-06-054-151/+144
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I believe the issue was twofold: When done importing, the calling context is likely still valid, so if the output channel is not read enough and reach capacity, some event producer down the line can be blocked trying to send in that channel. When closing it, this send is still trying to proceed, which is illegal in go. In rateLimitHandlerClient, there was a need to 2 different type of output channel: core.ExportResult and ImportEvent. To do so, the previous code was using a single channel type RateLimitingEvent and a series of goroutines to read/cast/send to the final channel. This could result in more async goroutine being stuck trying to send in an at-capacity channel. Instead, the code now use a simple synchronous callback to directly push to the final output channel. No concurrency needed anymore and the code is simpler. Any of those fixes could have resolved the data race, but both fixes is more correct.
| * | | Merge pull request #811 from MichaelMure/fix-data-raceMichael Muré2022-06-062-6/+6
| |\ \ \ | | |/ / | |/| | graphql: fix two invalid mutex lock leading to data races
| | * | graphql: fix two invalid mutex lock leading to data racesMichael Muré2022-06-052-6/+6
| |/ /
| * | Merge pull request #806 from MichaelMure/docsMichael Muré2022-05-313-2/+23
| |\ \ | | | | | | | | doc: more discoverable docs
| | * | doc: more discoverable docsMichael Muré2022-05-313-2/+23
| |/ /
| * | Merge pull request #805 from MichaelMure/completion-cleanupMichael Muré2022-05-318-12/+14
| |\ \ | | |/ | |/| misc: move all completions in a dedicated folder
| | * misc: move all completions in a dedicated folderMichael Muré2022-05-318-12/+14
| |/
* | fix(808): replace Windows line terminatorsSteve Moyer2022-06-061-1/+5
| |
* | fix(778): remove extra mutex lock when resolving bug prefixSteve Moyer2022-05-311-4/+0
| |
* | test(778): execute rm bug in testEnv (hangs)Steve Moyer2022-05-281-0/+15
| |
* | test(778): execute add user in testEnv and return userIDSteve Moyer2022-05-281-0/+33
| |
* | test(778): clear output after user creationSteve Moyer2022-05-281-1/+3
| |
* | test(778): verify user create results in an identity and cacheSteve Moyer2022-05-272-0/+40
| |
* | test(778): verify root command returns main help textSteve Moyer2022-05-272-0/+120
| |
* | test(778): allow alternate CWD via contextSteve Moyer2022-05-271-2/+16
| |
* | test(778): capture stderr and stdout during testsSteve Moyer2022-05-271-0/+3
|/
* Merge pull request #788 from hoijui/model_touchupMichael Muré2022-05-101-22/+28
|\ | | | | Model touchup
| * model: Multiple, minor readability and language improvementsRobin Vobruba2022-05-041-11/+11
| |
| * model: Removes now outdated statement about ops and rootRobin Vobruba2022-05-041-1/+0
| |