Apply edits as suggested in review of pull #446
This commit is contained in:
parent
afc2c95238
commit
61470ad068
|
@ -374,16 +374,11 @@ Key.prototype.encrypt = function(passphrase) {
|
||||||
if (this.isPrivate()) {
|
if (this.isPrivate()) {
|
||||||
var keys = this.getAllKeyPackets();
|
var keys = this.getAllKeyPackets();
|
||||||
for (var i = 0; i < keys.length; i++) {
|
for (var i = 0; i < keys.length; i++) {
|
||||||
try {
|
|
||||||
keys[i].encrypt(passphrase);
|
keys[i].encrypt(passphrase);
|
||||||
} catch (e) {
|
|
||||||
return false;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
throw new Error("Nothing to encrypt in a public key");
|
throw new Error("Nothing to encrypt in a public key");
|
||||||
}
|
}
|
||||||
return true;
|
|
||||||
};
|
};
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
|
@ -716,9 +716,13 @@ var pgp_desktop_priv =
|
||||||
var armor2 = key.armor();
|
var armor2 = key.armor();
|
||||||
expect(armor1).to.equal(armor2);
|
expect(armor1).to.equal(armor2);
|
||||||
expect(key.decrypt('passphrase')).to.be.true;
|
expect(key.decrypt('passphrase')).to.be.true;
|
||||||
|
expect(key.primaryKey.isDecrypted).to.be.true;
|
||||||
expect(key.encrypt('new_passphrase')).to.be.true;
|
expect(key.encrypt('new_passphrase')).to.be.true;
|
||||||
|
expect(key.primaryKey.isDecrypted).to.be.false;
|
||||||
expect(key.decrypt('passphrase')).to.be.false;
|
expect(key.decrypt('passphrase')).to.be.false;
|
||||||
|
expect(key.primaryKey.isDecrypted).to.be.false;
|
||||||
expect(key.decrypt('new_passphrase')).to.be.true;
|
expect(key.decrypt('new_passphrase')).to.be.true;
|
||||||
|
expect(key.primaryKey.isDecrypted).to.be.true;
|
||||||
var armor3 = key.armor();
|
var armor3 = key.armor();
|
||||||
expect(armor3).to.not.equal(armor1);
|
expect(armor3).to.not.equal(armor1);
|
||||||
done();
|
done();
|
||||||
|
|
Loading…
Reference in New Issue
Block a user