Skip to content

Commit c322b37

Browse files
mvandebergmvandeberg
committed
Fix active auth tests to use a testing authority #458 
1 parent c4d28cf commit c322b37

File tree

4 files changed

+39
-24
lines changed

4 files changed

+39
-24
lines changed

‎src/api/methods.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -287,7 +287,7 @@ export default [
287287
{
288288
"api": "database_api",
289289
"method": "verify_authority",
290-
"params": ["trx"]
290+
"params": ["trx", "auth"]
291291
},
292292
{
293293
"api": "database_api",

‎test/hf20-accounts.test.js

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,14 @@ import should from 'should';
33
import steem from '../src';
44

55
const username = process.env.STEEM_USERNAME || 'guest123';
6-
const password = process.env.STEEM_PASSWORD;
7-
const activeWif = steem.auth.toWif(username, password, 'active');
6+
const passwrod = process.env.STEEM_PASSWORD;
7+
const activeWif = '5K4RDXLLjoyvKttRj8jG9utT7GZmM9qNkePT6uRfWxKf19g322R';
8+
const activePub = steem.auth.wifToPublic( activeWif );
9+
const testAuth = {
10+
'weight_threshold': 1,
11+
'account_auths': [],
12+
'key_auths': [[activePub, 1]]
13+
};
814

915
describe('steem.hf20-accounts:', () => {
1016
it('has generated methods', () => {
@@ -20,9 +26,10 @@ describe('steem.hf20-accounts:', () => {
2026

2127
describe('claimAccount', () => {
2228

23-
/* Skip these tests. Steem-js test infrastructure not set up for testing active auths
24-
Blocked by Steem issue #3546
2529
it('signs and verifies auth', function(done) {
30+
let url = steem.config.get('uri');
31+
steem.api.setOptions({ url: url, useAppbaseApi: true });
32+
2633
let tx = {
2734
'operations': [[
2835
'claim_account', {
@@ -35,7 +42,7 @@ describe('steem.hf20-accounts:', () => {
3542

3643
steem.broadcast._prepareTransaction(tx).then(function(tx){
3744
tx = steem.auth.signTransaction(tx, [activeWif]);
38-
steem.api.verifyAuthorityAsync(tx).then(
45+
steem.api.verifyAuthorityAsync(tx, testAuth).then(
3946
(result) => {result.should.equal(true); done();},
4047
(err) => {done(err);}
4148
);
@@ -71,6 +78,5 @@ describe('steem.hf20-accounts:', () => {
7178
}, (err) => {done(err)});
7279
});
7380
});
74-
*/
7581
});
7682
});

‎test/hf21-sps.test.js

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,13 @@ import should from 'should';
44
import steem from '../src';
55

66
const username = process.env.STEEM_USERNAME || 'guest123';
7-
const password = process.env.STEEM_PASSWORD;
8-
const activeWif = steem.auth.toWif(username, password, 'active');
7+
const activeWif = '5K4RDXLLjoyvKttRj8jG9utT7GZmM9qNkePT6uRfWxKf19g322R';
8+
const activePub = steem.auth.wifToPublic( activeWif );
9+
const testAuth = {
10+
'weight_threshold': 1,
11+
'account_auths': [],
12+
'key_auths': [[activePub, 1]]
13+
};
914

1015
describe('steem.hf21-accounts:', () => {
1116
it('has generated methods', () => {
@@ -21,9 +26,9 @@ describe('steem.hf21-accounts:', () => {
2126
});
2227

2328
describe('create proposal ops', () => {
24-
/* Skip these tests. Steem-js test infrastructure not set up for testing active auths
25-
Blocked by Steem issue #3546
2629
it('signs and verifies create_proposal', function(done) {
30+
let url = steem.config.get('uri');
31+
steem.api.setOptions({ url: url, useAppbaseApi: true });
2732
let permlink = 'test';
2833

2934
let tx = {
@@ -45,7 +50,7 @@ describe('steem.hf21-accounts:', () => {
4550

4651
steem.broadcast._prepareTransaction(tx).then(function(tx){
4752
tx = steem.auth.signTransaction(tx, [activeWif]);
48-
steem.api.verifyAuthorityAsync(tx).then(
53+
steem.api.verifyAuthorityAsync(tx, testAuth).then(
4954
(result) => {result.should.equal(true); done();},
5055
(err) => {done(err);}
5156
);
@@ -73,6 +78,5 @@ describe('steem.hf21-accounts:', () => {
7378
);
7479
});
7580
})
76-
*/
7781
});
7882
});

‎test/smt.test.js

Lines changed: 16 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,13 @@ import should from 'should';
44
import steem from '../src';
55

66
const username = process.env.STEEM_USERNAME || 'guest123';
7-
const password = process.env.STEEM_PASSWORD;
8-
const activeWif = steem.auth.toWif(username, password, 'active');
7+
const activeWif = '5K4RDXLLjoyvKttRj8jG9utT7GZmM9qNkePT6uRfWxKf19g322R';
8+
const activePub = steem.auth.wifToPublic( activeWif );
9+
const testAuth = {
10+
'weight_threshold': 1,
11+
'account_auths': [],
12+
'key_auths': [[activePub, 1]]
13+
};
914

1015
describe('steem.smt:', () => {
1116

@@ -30,7 +35,7 @@ describe('steem.smt:', () => {
3035

3136
steem.broadcast._prepareTransaction(tx).then(function(tx){
3237
tx = steem.auth.signTransaction(tx, [activeWif]);
33-
steem.api.verifyAuthorityAsync(tx).then(
38+
steem.api.verifyAuthorityAsync(tx, testAuth).then(
3439
(result) => {result.should.equal(true); done();},
3540
(err) => {done(err);}
3641
);
@@ -98,7 +103,7 @@ describe('steem.smt:', () => {
98103

99104
steem.broadcast._prepareTransaction(tx).then(function(tx){
100105
tx = steem.auth.signTransaction(tx, [activeWif]);
101-
steem.api.verifyAuthorityAsync(tx).then(
106+
steem.api.verifyAuthorityAsync(tx, testAuth).then(
102107
(result) => {result.should.equal(true); done();},
103108
(err) => {done(err);}
104109
);
@@ -141,7 +146,7 @@ describe('steem.smt:', () => {
141146

142147
steem.broadcast._prepareTransaction(tx).then(function(tx){
143148
tx = steem.auth.signTransaction(tx, [activeWif]);
144-
steem.api.verifyAuthorityAsync(tx).then(
149+
steem.api.verifyAuthorityAsync(tx, testAuth).then(
145150
(result) => {result.should.equal(true); done();},
146151
(err) => {done(err);}
147152
);
@@ -169,7 +174,7 @@ describe('steem.smt:', () => {
169174

170175
steem.broadcast._prepareTransaction(tx).then(function(tx){
171176
tx = steem.auth.signTransaction(tx, [activeWif]);
172-
steem.api.verifyAuthorityAsync(tx).then(
177+
steem.api.verifyAuthorityAsync(tx, testAuth).then(
173178
(result) => {result.should.equal(true); done();},
174179
(err) => {done(err);}
175180
);
@@ -198,7 +203,7 @@ describe('steem.smt:', () => {
198203

199204
steem.broadcast._prepareTransaction(tx).then(function(tx){
200205
tx = steem.auth.signTransaction(tx, [activeWif]);
201-
steem.api.verifyAuthorityAsync(tx).then(
206+
steem.api.verifyAuthorityAsync(tx, testAuth).then(
202207
(result) => {result.should.equal(true); done();},
203208
(err) => {done(err);}
204209
);
@@ -224,7 +229,7 @@ describe('steem.smt:', () => {
224229

225230
steem.broadcast._prepareTransaction(tx).then(function(tx){
226231
tx = steem.auth.signTransaction(tx, [activeWif]);
227-
steem.api.verifyAuthorityAsync(tx).then(
232+
steem.api.verifyAuthorityAsync(tx, testAuth).then(
228233
(result) => {result.should.equal(true); done();},
229234
(err) => {done(err);}
230235
);
@@ -259,7 +264,7 @@ describe('steem.smt:', () => {
259264

260265
steem.broadcast._prepareTransaction(tx).then(function(tx){
261266
tx = steem.auth.signTransaction(tx, [activeWif]);
262-
steem.api.verifyAuthorityAsync(tx).then(
267+
steem.api.verifyAuthorityAsync(tx, testAuth).then(
263268
(result) => {result.should.equal(true); done();},
264269
(err) => {done(err);}
265270
);
@@ -296,7 +301,7 @@ describe('steem.smt:', () => {
296301

297302
steem.broadcast._prepareTransaction(tx).then(function(tx){
298303
tx = steem.auth.signTransaction(tx, [activeWif]);
299-
steem.api.verifyAuthorityAsync(tx).then(
304+
steem.api.verifyAuthorityAsync(tx, testAuth).then(
300305
(result) => {result.should.equal(true); done();},
301306
(err) => {done(err);}
302307
);
@@ -325,7 +330,7 @@ describe('steem.smt:', () => {
325330

326331
steem.broadcast._prepareTransaction(tx).then(function(tx){
327332
tx = steem.auth.signTransaction(tx, [activeWif]);
328-
steem.api.verifyAuthorityAsync(tx).then(
333+
steem.api.verifyAuthorityAsync(tx, testAuth).then(
329334
(result) => {result.should.equal(true); done();},
330335
(err) => {done(err);}
331336
);

0 commit comments

Comments
 (0)