server: split test code into sub-packages and files#107135
Merged
craig[bot] merged 2 commits intocockroachdb:masterfrom Jul 19, 2023
Merged
server: split test code into sub-packages and files#107135craig[bot] merged 2 commits intocockroachdb:masterfrom
craig[bot] merged 2 commits intocockroachdb:masterfrom
Conversation
Member
|
This change is |
303b7f8 to
5bfb191
Compare
a4060b8 to
626fb1b
Compare
626fb1b to
723ab3b
Compare
Release note: None
723ab3b to
d18e3f3
Compare
New sub-packages: - `storage_api`: unit tests for API endpoints specific to the KV/storage layer. - `application_api`: unit tests for API endpoints valid for application servers, including for secondary tenants. - `privchecker`: SQL authorization interface for API handlers. (previously: `adminPrivilegeChecker`) - `authserver`: HTTP/RPC authentication code. (previously: `authenticationServer`) - `srverrors`: error objects suitable for return from API handlers. - `srvtestutils`: common helpers for test code. Release note: None
d18e3f3 to
c5e3cf4
Compare
craig bot
pushed a commit
that referenced
this pull request
Jul 19, 2023
106928: jobsccl: remove some uses of TODOTestTenantDisabled, simplify test code r=yuzefovich a=knz First commits from #107135. Epic: CRDB-18499 Informs #76378. 107113: dev: perform `chmod` in `/tmp`, not `/artifacts` r=rail a=rickystewart This avoids an error in case your `artifacts` directory has some restrictive permissions. There's no functional difference here, we just copy to `/tmp` and do the `chmod` there before copying to the final destination directory. [Internal discussion](https://cockroachlabs.slack.com/archives/C019A8NANH0/p1689598801368169) Epic: none Release note: None Co-authored-by: Raphael 'kena' Poss <knz@thaumogen.net> Co-authored-by: Ricky Stewart <ricky@cockroachlabs.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Epic: CRDB-18499
Informs #76378.
New sub-packages:
storage_api: unit tests for API endpoints specificto the KV/storage layer.
application_api: unit tests for API endpoints valid forapplication servers, including for secondary tenants.
privchecker: SQL authorization interface for API handlers.(previously:
adminPrivilegeChecker)authserver: HTTP/RPC authentication code.(previously:
authenticationServer)srverrors: error objects suitable for return from API handlers.srvtestutils: common helpers for test code.