-
Notifications
You must be signed in to change notification settings - Fork 240
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
Cleanup test code and add tests for rate limits #2602
Conversation
agent/src/index.test.ts
Outdated
public accessToken?: string | ||
public info: ClientInfo | ||
|
||
constructor(name: string, accessToken?: string) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI, you can use public/private readonly to avoid declaring separate properties. Similar to case classes
constructor(name: string, accessToken?: string) { | |
constructor(public readonly name: string, public readonly accessToken?: string) { |
@@ -103,7 +103,9 @@ recordings. | |||
To fix this problem, update the HTTP recordings with the following command: | |||
|
|||
```sh | |||
export SRC_ACCESS_TOKEN=sgp_YOUR_ACCESS_TOKEN # redacted in the recordings | |||
# tokens are redacted in the recordings | |||
export SRC_ACCESS_TOKEN_WITH_RATE_LIMIT=sgp_YOUR_ACCESS_TOKEN_WITH_RATE_LIMIT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Link to this message to find the token https://sourcegraph.slack.com/archives/C059N5FRYG3/p1702990080820699
889cd28
to
5565c36
Compare
super() | ||
|
||
this.name = name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be removed.
Test plan
No needed, those are just changes for the tests.
Changes: