Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Conversation

ggilmore
Copy link
Contributor

@ggilmore ggilmore commented Jun 17, 2024

This is a small change that ensures that the user/repository not found error types fulfill the errcode.NotFound interface. This is a building block for https://app.graphite.dev//github/pr/sourcegraph/sourcegraph/63302

Test plan

Unit tests

Changelog

  • Added unit tests to ensure that the database userNotFound and RepositoryNotFound error types fulfuill the errcode.NotFound interface.

@ggilmore ggilmore force-pushed the 06-14-fix_graphql_database_fix_repo_permission_sync_count_to_ignore_soft_deleted_repositories branch from d4a0c3b to 8b8b223 Compare June 17, 2024 19:49
@ggilmore ggilmore force-pushed the graphite-ggilmore06-17-feature_internal_database_add_test_to_ensure_that_not_found_errors_fulfiill_errcode.notfound branch from 4d5e82c to c3c0cf5 Compare June 17, 2024 19:49
@ggilmore ggilmore requested a review from a team June 17, 2024 19:54
@ggilmore ggilmore marked this pull request as ready for review June 17, 2024 19:58
Copy link
Member

@eseliger eseliger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

Copy link

graphite-app bot commented Jun 17, 2024

TV gif. Steve Irwin the Crocodile Hunter looking down at a body of water, turns around and gives a double thumbs-up, mouthing 'that's good.' (Added via Giphy)

@ggilmore ggilmore force-pushed the 06-14-fix_graphql_database_fix_repo_permission_sync_count_to_ignore_soft_deleted_repositories branch from 8b8b223 to e06238d Compare June 17, 2024 20:19
@ggilmore ggilmore force-pushed the graphite-ggilmore06-17-feature_internal_database_add_test_to_ensure_that_not_found_errors_fulfiill_errcode.notfound branch from c3c0cf5 to fd80f0c Compare June 17, 2024 20:19
@ggilmore ggilmore force-pushed the 06-14-fix_graphql_database_fix_repo_permission_sync_count_to_ignore_soft_deleted_repositories branch from e06238d to cee1f50 Compare June 17, 2024 20:27
@ggilmore ggilmore force-pushed the graphite-ggilmore06-17-feature_internal_database_add_test_to_ensure_that_not_found_errors_fulfiill_errcode.notfound branch from fd80f0c to 68da82c Compare June 17, 2024 20:28
@ggilmore ggilmore changed the base branch from 06-14-fix_graphql_database_fix_repo_permission_sync_count_to_ignore_soft_deleted_repositories to graphite-base/63299 June 17, 2024 20:35
@graphite-app graphite-app bot changed the base branch from graphite-base/63299 to main June 17, 2024 20:35
@ggilmore ggilmore force-pushed the graphite-ggilmore06-17-feature_internal_database_add_test_to_ensure_that_not_found_errors_fulfiill_errcode.notfound branch from 68da82c to b7026c4 Compare June 17, 2024 20:35
@ggilmore ggilmore merged commit fe56852 into main Jun 17, 2024
14 checks passed
@ggilmore ggilmore deleted the graphite-ggilmore06-17-feature_internal_database_add_test_to_ensure_that_not_found_errors_fulfiill_errcode.notfound branch June 17, 2024 20:43
Copy link
Contributor Author

Merge activity

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-signed team/product-platform team/source Tickets under the purview of Source - the one Source to graph it all
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants