mirrors/zotero
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Set new API key on each setCredentials() call in sync preferences tests

Browse source
This commit is contained in:
Dan Stillman 2024-11-08 03:10:48 -05:00
parent e9a19cb7a1
commit 7c91f37348
1 changed files with 18 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

31
test/tests/preferences_syncTest.js
View file

@ -15,27 +15,32 @@ describe("Sync Preferences", function () {
describe("Settings", function () {
describe("Data Syncing", function () {
var getAPIKeyFromCredentialsStub, deleteAPIKey, indicatorElem;
var getAPIKeyFromCredentialsStub, deleteAPIKey, indicatorElem, apiKey, apiResponse;
var setCredentials = Zotero.Promise.coroutine(function* (username, password) {
var setCredentials = Zotero.Promise.coroutine(function* (username, password, isIncorrectPasword) {
let usernameElem = doc.getElementById('sync-username-textbox');
let passwordElem = doc.getElementById('sync-password');
usernameElem.value = username;
passwordElem.value = password;
// Triggered by `change` event for usernameElem and passwordElem;
yield win.Zotero_Preferences.Sync.linkAccount();
});
var apiKey = Zotero.Utilities.randomString(24);
var apiResponse = {
apiKey = Zotero.Utilities.randomString(24);
apiResponse = {
key: apiKey,
username: "Username",
username,
userID: 1,
access: {}
};
// Triggered by `change` event for usernameElem and passwordElem
if (isIncorrectPasword) {
getAPIKeyFromCredentialsStub.resolves(false);
}
else {
getAPIKeyFromCredentialsStub.resolves(apiResponse);
}
yield win.Zotero_Preferences.Sync.linkAccount();
});
before(function* () {
getAPIKeyFromCredentialsStub = sinon.stub(
Zotero.Sync.APIClient.prototype, 'createAPIKeyFromCredentials');
@ -58,7 +63,6 @@ describe("Sync Preferences", function () {
});
it("should set API key and display full controls with correct credentials", function* () {
getAPIKeyFromCredentialsStub.resolves(apiResponse);
yield setCredentials("Username", "correctPassword");
yield assert.eventually.equal(Zotero.Sync.Data.Local.getAPIKey(), apiKey);
@ -67,8 +71,7 @@ describe("Sync Preferences", function () {
it("should display dialog when credentials incorrect", function* () {
getAPIKeyFromCredentialsStub.resolves(false);
yield setCredentials("Username", "incorrectPassword");
yield setCredentials("Username", "incorrectPassword", true);
assert.isTrue(Zotero.alert.called);
yield assert.eventually.equal(Zotero.Sync.Data.Local.getAPIKey(), "");
@ -77,7 +80,6 @@ describe("Sync Preferences", function () {
it("should delete API key and display auth form when 'Unlink Account' clicked", function* () {
getAPIKeyFromCredentialsStub.resolves(apiResponse);
yield setCredentials("Username", "correctPassword");
yield assert.eventually.equal(Zotero.Sync.Data.Local.getAPIKey(), apiKey);
@ -89,7 +91,6 @@ describe("Sync Preferences", function () {
});
it("should not unlink on pressing cancel", function* () {
getAPIKeyFromCredentialsStub.resolves(apiResponse);
yield setCredentials("Username", "correctPassword");
waitForDialog(null, 'cancel');