Skip to content

Commit 472eebb

Browse files
authored
[JS] Fix: making methods static (#11182)
1 parent 3bdde2a commit 472eebb

File tree

3 files changed

+29
-21
lines changed

3 files changed

+29
-21
lines changed

javascript/node/selenium-webdriver/lib/virtual_authenticator.js

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -169,24 +169,34 @@ class Credential {
169169
* @param userHandle userHandle associated to the credential. Must be Base64 encoded string.
170170
* @param privateKey Base64 encoded PKCS
171171
* @param signCount initial value for a signature counter.
172+
* @deprecated This method has been made static. Call it with class name. Example, Credential.createResidentCredential()
172173
* @returns A resident credential
173174
*/
174175
createResidentCredential(id, rpId, userHandle, privateKey, signCount) {
175176
return new Credential(id, true, rpId, userHandle, privateKey, signCount)
176177
}
177178

179+
static createResidentCredential(id, rpId, userHandle, privateKey, signCount) {
180+
return new Credential(id, true, rpId, userHandle, privateKey, signCount)
181+
}
182+
178183
/**
179184
* Creates a non-resident (i.e. stateless) credential.
180185
* @param id Unique base64 encoded string.
181186
* @param rpId Relying party identifier.
182187
* @param privateKey Base64 encoded PKCS
183188
* @param signCount initial value for a signature counter.
189+
* @deprecated This method has been made static. Call it with class name. Example, Credential.createNonResidentCredential()
184190
* @returns A non-resident credential
185191
*/
186192
createNonResidentCredential(id, rpId, privateKey, signCount) {
187193
return new Credential(id, false, rpId, null, privateKey, signCount)
188194
}
189195

196+
static createNonResidentCredential(id, rpId, privateKey, signCount) {
197+
return new Credential(id, false, rpId, null, privateKey, signCount)
198+
}
199+
190200
toDict() {
191201
let credentialData = {
192202
credentialId: Buffer.from(this._id).toString('base64url'),

javascript/node/selenium-webdriver/test/lib/credentials_test.js

Lines changed: 15 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -51,14 +51,13 @@ describe('Credentials', function () {
5151

5252
it('can testRkEnabledCredential', function () {
5353
const { _id, rpId, userHandle, privateKey, signCount } = data
54-
const credential =
55-
new virtualAuthenticatorCredential().createResidentCredential(
56-
_id,
57-
rpId,
58-
userHandle,
59-
privateKey,
60-
signCount
61-
)
54+
const credential = virtualAuthenticatorCredential.createResidentCredential(
55+
_id,
56+
rpId,
57+
userHandle,
58+
privateKey,
59+
signCount
60+
)
6261

6362
let testCredentialId = new Uint8Array([1, 2, 3, 4])
6463

@@ -105,7 +104,7 @@ describe('Credentials', function () {
105104
it('can testRkDisabledCredential', function () {
106105
const { _id, rpId, privateKey, signCount } = data
107106
const credential =
108-
new virtualAuthenticatorCredential().createNonResidentCredential(
107+
virtualAuthenticatorCredential.createNonResidentCredential(
109108
_id,
110109
rpId,
111110
privateKey,
@@ -141,14 +140,13 @@ describe('Credentials', function () {
141140

142141
it('can testToDict', function () {
143142
const { _id, rpId, userHandle, privateKey, signCount } = data
144-
const credential =
145-
new virtualAuthenticatorCredential().createResidentCredential(
146-
_id,
147-
rpId,
148-
userHandle,
149-
privateKey,
150-
signCount
151-
)
143+
const credential = virtualAuthenticatorCredential.createResidentCredential(
144+
_id,
145+
rpId,
146+
userHandle,
147+
privateKey,
148+
signCount
149+
)
152150

153151
let credential_dict = credential.toDict()
154152
assert.equal(

javascript/node/selenium-webdriver/test/virtualAuthenticator_test.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,7 @@ suite(function (env) {
190190
*/
191191
driver = await createRkDisabledCTAP2Authenticator(driver)
192192
let credential =
193-
new virtualAuthenticatorCredential().createNonResidentCredential(
193+
virtualAuthenticatorCredential.createNonResidentCredential(
194194
new Uint8Array([1, 2, 3, 4]),
195195
'localhost',
196196
Buffer.from(BASE64_ENCODED_PK, 'base64').toString('binary'),
@@ -223,7 +223,7 @@ suite(function (env) {
223223
'RmgsJYxGP__fWN_S-j5sN4tT15XEpN_7QZnt14YvI6uvAgO0uJEboFaZlOEB'
224224

225225
let credential =
226-
new virtualAuthenticatorCredential().createNonResidentCredential(
226+
virtualAuthenticatorCredential.createNonResidentCredential(
227227
new Uint8Array([1, 2, 3, 4]),
228228
'localhost',
229229
Buffer.from(base64EncodedPK, 'base64').toString('binary'),
@@ -248,7 +248,7 @@ suite(function (env) {
248248
driver = await createRkEnabledCTAP2Authenticator(driver)
249249

250250
let credential =
251-
new virtualAuthenticatorCredential().createResidentCredential(
251+
virtualAuthenticatorCredential.createResidentCredential(
252252
new Uint8Array([1, 2, 3, 4]),
253253
'localhost',
254254
new Uint8Array([1]),
@@ -286,7 +286,7 @@ suite(function (env) {
286286
'RmgsJYxGP__fWN_S-j5sN4tT15XEpN_7QZnt14YvI6uvAgO0uJEboFaZlOEB'
287287

288288
let credential =
289-
new virtualAuthenticatorCredential().createResidentCredential(
289+
virtualAuthenticatorCredential.createResidentCredential(
290290
new Uint8Array([1, 2, 3, 4]),
291291
'localhost',
292292
new Uint8Array([1]),

0 commit comments

Comments
 (0)