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.
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
[Alerting] Secret Service #28894
[Alerting] Secret Service #28894
Changes from 172 commits
256fda8
fb6bff6
ebc65e8
0687917
42e1cdd
1969aa6
68314ce
4ba0fb7
70b0630
a853ba3
7d3163f
8317c82
41e15cc
eb54ceb
c7b2aa7
8f2b399
23d64df
1932993
417fd9f
128ad90
baa34ec
bb839ae
eda401e
df909a9
4f7312b
4e64a7b
ec22f45
d11c05a
f897f1c
2c7b603
680e3f1
5bfb756
5cd0ef3
a30f897
fad7e9d
d31c589
94d1777
494aeb6
dba327c
430f65e
e4a8b3e
6f9e0e6
25de9ae
698ff47
e531739
11683d4
c3f6a90
a90e579
1b3e235
3be7373
1d15b48
7d9bdc8
49315c7
9f0fd42
39e80d6
8ca9bb7
846bc3d
5877fb9
32f0203
d62b678
a638188
e54caee
4fed095
4ff5c8b
6bd8f56
726c2ab
02bc63d
70d8ce8
349f1e2
9b86952
f282b25
c1ebef8
59e6d03
acc006e
5dcb051
ef931da
22da72d
ce9357e
780e415
f2f2887
5856a5a
3befab4
69b6b28
36d48f5
7861d2f
3ab122d
8e6908f
3bed5c8
57f9628
c0b2971
a427cb7
e909579
ecfb8c0
d986da3
0c95fcc
2c2212e
96da1c9
68f8673
5d124ad
c8cbd31
6183622
9afd2be
6d79462
a742044
f131b05
df6f979
c71ead4
ce64482
1febac6
b477eab
47b4173
3e6d7c8
9aa914f
bcc3a2f
7f1a9d0
ac2e47e
9e1e2e4
9a90a60
68d62cb
7d8311a
92b07d4
aca559b
0054bfb
e7c60df
6d38575
3eb98fc
35c883b
dee6e7c
88215cf
bc66231
d74479d
e54596c
287004f
128aa88
a4fa204
2ae5ace
2597af2
1c09cb4
ba0c33f
26f46ac
42b6b80
8956c7d
fd7ec07
95113cb
042cd26
9fcd6c8
ed94270
d0d77de
0f059d7
a765571
9cff873
c29ea09
8327938
8b23d91
b6a9b7a
aa0c863
e2c24e5
e5696aa
6eefd5e
712ba8e
c49dc27
633e1fb
b1b57ba
c49ec5f
7f5ed81
3d8a5d9
0442daa
50a8d72
d97c4e9
c36dc33
af82e74
70ffcbd
5c9e1d0
6829bf0
3f267d9
b15f935
aefde7a
77ddb00
3ebae23
a944c56
401bbb4
b42a9ed
035ef81
b864f58
088e035
d750223
1d19557
9b61aff
51aaa9c
8e69307
d94554a
a747a09
00d6a44
93e2300
40b71c7
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
question: should we follow this new "mock" convention here as well and move these mocks closer to the entities they mock? Just to have less work when this plugin is being migrated to the new platform...
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.
nit: what is the benefit of having "named"
TestSecretService
function here comparing to arrow functions you use forbeforeAll
/it
?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.
nit: hmm, you have a dedicated test exactly for this, maybe we can remove this
expect
from here?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.
optional nit: if the logic inside of the mock implementation doesn't depend on arguments and there is no requirement to return new value on every call, I'd say
jest.fn().mockReturnValue
would make intention clearer.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.
optional nit:
expect.any(SecretService)
?question: can we also test key auto-generation flow?
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.
nit: maybe
server: Legacy.Server
(import { Legacy } from 'kibana';
) instead ofserver: any
?