This repository was archived by the owner on Sep 30, 2024. It is now read-only.
chore/enterpriseportal: rewrite subscriptions API tests to be table-driven#63831
Merged
Conversation
c480ceb to
261838b
Compare
This was referenced Jul 15, 2024
Member
Author
Member
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @bobheadxi and the rest of your teammates on |
261838b to
31cb17d
Compare
03d1f57 to
0bfecda
Compare
31cb17d to
2791ba2
Compare
0bfecda to
f7e32bf
Compare
2791ba2 to
cf8baed
Compare
f7e32bf to
b218c2c
Compare
4d31019 to
47f201b
Compare
ec012c8 to
60e0588
Compare
47f201b to
dac145d
Compare
60e0588 to
e5bedf9
Compare
dac145d to
e74c7e1
Compare
jac
approved these changes
Jul 17, 2024
e5bedf9 to
431260c
Compare
e74c7e1 to
fc0331d
Compare
431260c to
8efaf03
Compare
fc0331d to
8289574
Compare
8efaf03 to
1e4bb8a
Compare
8289574 to
0a8d2b5
Compare
1e4bb8a to
3edaaf6
Compare
0a8d2b5 to
f1e4254
Compare
3edaaf6 to
6d652fa
Compare
f1e4254 to
b3bc977
Compare
3 tasks
Base automatically changed from
07-09-feat_enterprise-portal_db_layer_for_cody_gateway_access
to
main
July 19, 2024 21:06
b3bc977 to
de7ef25
Compare
de7ef25 to
044c0aa
Compare
This was referenced Jul 24, 2024
This was referenced Aug 2, 2024
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
As we start adding more stuff here, we need a bit more flexibility to add tests without it becoming overwhelming. The current copy-pasta is quite difficult to read, this standardizes the expected behaviour for the Update and List tests.
Test plan
CI