提交 bb58edf5 编写于 作者: G gaoxi

修改编码规范问题

Signed-off-by: Ngaoxi <gaoxi785@huawei.com>
上级 964c1829
...@@ -1305,11 +1305,9 @@ export default function ObjectInterfaceTest() { ...@@ -1305,11 +1305,9 @@ export default function ObjectInterfaceTest() {
} }
contactsapi.addContact(contactData, (err, data) => { contactsapi.addContact(contactData, (err, data) => {
if (err) { if (err) {
console.info(`addContact callback: err->${JSON.stringify(err)}`);
done(); done();
return; return;
} }
console.info(`addContact callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) != -1).assertTrue(); expect(JSON.stringify(data) != -1).assertTrue();
done(); done();
}); });
...@@ -1324,7 +1322,6 @@ export default function ObjectInterfaceTest() { ...@@ -1324,7 +1322,6 @@ export default function ObjectInterfaceTest() {
} }
contactsapi.deleteContact('xxx', (err) => { contactsapi.deleteContact('xxx', (err) => {
if (err) { if (err) {
console.info(`deleteContact callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(err) == -1).assertTrue(); expect(JSON.stringify(err) == -1).assertTrue();
done(); done();
return; return;
...@@ -1345,12 +1342,10 @@ export default function ObjectInterfaceTest() { ...@@ -1345,12 +1342,10 @@ export default function ObjectInterfaceTest() {
} }
contactsapi.queryContact('xxx', (err, data) => { contactsapi.queryContact('xxx', (err, data) => {
if (err) { if (err) {
console.info(`queryContact callback: err->${JSON.stringify(err)}`);
expect(false).assertTrue(); expect(false).assertTrue();
done(); done();
return; return;
} }
console.info(`queryContact callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) == undefined).assertTrue(); expect(JSON.stringify(data) == undefined).assertTrue();
done(); done();
}); });
...@@ -1367,12 +1362,10 @@ export default function ObjectInterfaceTest() { ...@@ -1367,12 +1362,10 @@ export default function ObjectInterfaceTest() {
holderId: 0 holderId: 0
}, (err, data) => { }, (err, data) => {
if (err) { if (err) {
console.info(`queryContact callback: err->${JSON.stringify(err)}`);
expect(false).assertTrue(); expect(false).assertTrue();
done(); done();
return; return;
} }
console.info(`queryContact callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) == undefined).assertTrue(); expect(JSON.stringify(data) == undefined).assertTrue();
done(); done();
}); });
...@@ -1389,12 +1382,10 @@ export default function ObjectInterfaceTest() { ...@@ -1389,12 +1382,10 @@ export default function ObjectInterfaceTest() {
attributes: ["ATTR_EMAIL", "ATTR_NAME"] attributes: ["ATTR_EMAIL", "ATTR_NAME"]
}, (err, data) => { }, (err, data) => {
if (err) { if (err) {
console.info(`queryContact callback: err->${JSON.stringify(err)}`);
expect(false).assertTrue(); expect(false).assertTrue();
done(); done();
return; return;
} }
console.info(`queryContact callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) == undefined).assertTrue(); expect(JSON.stringify(data) == undefined).assertTrue();
done(); done();
}); });
...@@ -1413,12 +1404,10 @@ export default function ObjectInterfaceTest() { ...@@ -1413,12 +1404,10 @@ export default function ObjectInterfaceTest() {
attributes: ["ATTR_EMAIL", "ATTR_NAME"] attributes: ["ATTR_EMAIL", "ATTR_NAME"]
}, (err, data) => { }, (err, data) => {
if (err) { if (err) {
console.info(`queryContact callback: err->${JSON.stringify(err)}`);
expect(false).assertTrue(); expect(false).assertTrue();
done(); done();
return; return;
} }
console.info(`queryContact callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) == undefined).assertTrue(); expect(JSON.stringify(data) == undefined).assertTrue();
done(); done();
}); });
...@@ -1437,11 +1426,9 @@ export default function ObjectInterfaceTest() { ...@@ -1437,11 +1426,9 @@ export default function ObjectInterfaceTest() {
attributes: ["ATTR_EMAIL", "ATTR_NAME"] attributes: ["ATTR_EMAIL", "ATTR_NAME"]
}); });
promise.then((data) => { promise.then((data) => {
console.info(`queryContact success: data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) == undefined).assertTrue(); expect(JSON.stringify(data) == undefined).assertTrue();
done(); done();
}).catch((err) => { }).catch((err) => {
console.info(`queryContact fail: err->${JSON.stringify(err)}`);
expect(false).assertTrue(); expect(false).assertTrue();
done(); done();
}); });
...@@ -1456,12 +1443,10 @@ export default function ObjectInterfaceTest() { ...@@ -1456,12 +1443,10 @@ export default function ObjectInterfaceTest() {
} }
contactsapi.queryContacts((err, data) => { contactsapi.queryContacts((err, data) => {
if (err) { if (err) {
console.info(`queryContacts callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryContacts callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1478,12 +1463,10 @@ export default function ObjectInterfaceTest() { ...@@ -1478,12 +1463,10 @@ export default function ObjectInterfaceTest() {
holderId: 0 holderId: 0
}, (err, data) => { }, (err, data) => {
if (err) { if (err) {
console.info(`queryContacts callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryContacts callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1500,12 +1483,10 @@ export default function ObjectInterfaceTest() { ...@@ -1500,12 +1483,10 @@ export default function ObjectInterfaceTest() {
attributes: ["ATTR_EMAIL", "ATTR_NAME"] attributes: ["ATTR_EMAIL", "ATTR_NAME"]
}, (err, data) => { }, (err, data) => {
if (err) { if (err) {
console.info(`queryContacts callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryContacts callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1525,12 +1506,10 @@ export default function ObjectInterfaceTest() { ...@@ -1525,12 +1506,10 @@ export default function ObjectInterfaceTest() {
attributes: ["ATTR_EMAIL", "ATTR_NAME"] attributes: ["ATTR_EMAIL", "ATTR_NAME"]
}, (err, data) => { }, (err, data) => {
if (err) { if (err) {
console.info(`queryContacts callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryContacts callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1550,11 +1529,9 @@ export default function ObjectInterfaceTest() { ...@@ -1550,11 +1529,9 @@ export default function ObjectInterfaceTest() {
attributes: ["ATTR_EMAIL", "ATTR_NAME"] attributes: ["ATTR_EMAIL", "ATTR_NAME"]
}); });
promise.then((data) => { promise.then((data) => {
console.info(`queryContacts success: data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}).catch((err) => { }).catch((err) => {
console.info(`queryContacts fail: err->${JSON.stringify(err)}`);
expect(false).assertTrue(); expect(false).assertTrue();
done(); done();
}); });
...@@ -1569,12 +1546,10 @@ export default function ObjectInterfaceTest() { ...@@ -1569,12 +1546,10 @@ export default function ObjectInterfaceTest() {
} }
contactsapi.queryContactsByPhoneNumber('138xxxxxxxx', (err, data) => { contactsapi.queryContactsByPhoneNumber('138xxxxxxxx', (err, data) => {
if (err) { if (err) {
console.info(`queryContactsByPhoneNumber callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryContactsByPhoneNumber callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1591,12 +1566,10 @@ export default function ObjectInterfaceTest() { ...@@ -1591,12 +1566,10 @@ export default function ObjectInterfaceTest() {
holderId: 0 holderId: 0
}, (err, data) => { }, (err, data) => {
if (err) { if (err) {
console.info(`queryContactsByPhoneNumber callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryContactsByPhoneNumber callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1613,12 +1586,10 @@ export default function ObjectInterfaceTest() { ...@@ -1613,12 +1586,10 @@ export default function ObjectInterfaceTest() {
attributes: ["ATTR_EMAIL", "ATTR_NAME"] attributes: ["ATTR_EMAIL", "ATTR_NAME"]
}, (err, data) => { }, (err, data) => {
if (err) { if (err) {
console.info(`queryContactsByPhoneNumber callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryContactsByPhoneNumber callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1637,12 +1608,10 @@ export default function ObjectInterfaceTest() { ...@@ -1637,12 +1608,10 @@ export default function ObjectInterfaceTest() {
attributes: ["ATTR_EMAIL", "ATTR_NAME"] attributes: ["ATTR_EMAIL", "ATTR_NAME"]
}, (err, data) => { }, (err, data) => {
if (err) { if (err) {
console.info(`queryContactsByPhoneNumber callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryContactsByPhoneNumber callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1658,12 +1627,10 @@ export default function ObjectInterfaceTest() { ...@@ -1658,12 +1627,10 @@ export default function ObjectInterfaceTest() {
} }
contactsapi.queryContactsByEmail('xxx@email.com', (err, data) => { contactsapi.queryContactsByEmail('xxx@email.com', (err, data) => {
if (err) { if (err) {
console.info(`queryContactsByEmail callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryContactsByEmail callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1680,12 +1647,10 @@ export default function ObjectInterfaceTest() { ...@@ -1680,12 +1647,10 @@ export default function ObjectInterfaceTest() {
holderId: 0 holderId: 0
}, (err, data) => { }, (err, data) => {
if (err) { if (err) {
console.info(`queryContactsByEmail callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryContactsByEmail callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1703,12 +1668,10 @@ export default function ObjectInterfaceTest() { ...@@ -1703,12 +1668,10 @@ export default function ObjectInterfaceTest() {
attributes: ["ATTR_EMAIL", "ATTR_NAME"] attributes: ["ATTR_EMAIL", "ATTR_NAME"]
}, (err, data) => { }, (err, data) => {
if (err) { if (err) {
console.info(`queryContactsByEmail callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryContactsByEmail callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1728,12 +1691,10 @@ export default function ObjectInterfaceTest() { ...@@ -1728,12 +1691,10 @@ export default function ObjectInterfaceTest() {
attributes: ["ATTR_EMAIL", "ATTR_NAME"] attributes: ["ATTR_EMAIL", "ATTR_NAME"]
}, (err, data) => { }, (err, data) => {
if (err) { if (err) {
console.info(`queryContactsByEmail callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryContactsByEmail callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1748,12 +1709,10 @@ export default function ObjectInterfaceTest() { ...@@ -1748,12 +1709,10 @@ export default function ObjectInterfaceTest() {
} }
contactsapi.queryGroups((err, data) => { contactsapi.queryGroups((err, data) => {
if (err) { if (err) {
console.info(`queryGroups callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryGroups callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1770,12 +1729,10 @@ export default function ObjectInterfaceTest() { ...@@ -1770,12 +1729,10 @@ export default function ObjectInterfaceTest() {
holderId: 0 holderId: 0
}, (err, data) => { }, (err, data) => {
if (err) { if (err) {
console.info(`queryGroups callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryGroups callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1791,12 +1748,10 @@ export default function ObjectInterfaceTest() { ...@@ -1791,12 +1748,10 @@ export default function ObjectInterfaceTest() {
} }
contactsapi.queryHolders((err, data) => { contactsapi.queryHolders((err, data) => {
if (err) { if (err) {
console.info(`queryHolders callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryHolders callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1812,12 +1767,10 @@ export default function ObjectInterfaceTest() { ...@@ -1812,12 +1767,10 @@ export default function ObjectInterfaceTest() {
contactsapi.queryKey( /*id*/ contactsapi.queryKey( /*id*/
1, (err, data) => { 1, (err, data) => {
if (err) { if (err) {
console.info(`queryKey callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryKey callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1835,12 +1788,10 @@ export default function ObjectInterfaceTest() { ...@@ -1835,12 +1788,10 @@ export default function ObjectInterfaceTest() {
holderId: 1 holderId: 1
}, (err, data) => { }, (err, data) => {
if (err) { if (err) {
console.info(`queryKey callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryKey callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1856,12 +1807,10 @@ export default function ObjectInterfaceTest() { ...@@ -1856,12 +1807,10 @@ export default function ObjectInterfaceTest() {
} }
contactsapi.queryMyCard((err, data) => { contactsapi.queryMyCard((err, data) => {
if (err) { if (err) {
console.info(`queryMyCard callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryMyCard callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1879,12 +1828,10 @@ export default function ObjectInterfaceTest() { ...@@ -1879,12 +1828,10 @@ export default function ObjectInterfaceTest() {
attributes: ['ATTR_EMAIL', 'ATTR_NAME'] attributes: ['ATTR_EMAIL', 'ATTR_NAME']
}, (err, data) => { }, (err, data) => {
if (err) { if (err) {
console.info(`queryMyCard callback: err->${JSON.stringify(err)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
return; return;
} }
console.info(`queryMyCard callback: success data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
}); });
...@@ -1954,12 +1901,10 @@ export default function ObjectInterfaceTest() { ...@@ -1954,12 +1901,10 @@ export default function ObjectInterfaceTest() {
contactsapi.isLocalContact( /*id*/ contactsapi.isLocalContact( /*id*/
1, (err, data) => { 1, (err, data) => {
if (err) { if (err) {
console.info(`isLocalContact callback: err->${JSON.stringify(err)}`);
expect(false).assertTrue(); expect(false).assertTrue();
done(); done();
return; return;
} }
console.info(`isLocalContact callback: success data->${JSON.stringify(data)}`);
done(); done();
}); });
sleep(500) sleep(500)
...@@ -1974,12 +1919,10 @@ export default function ObjectInterfaceTest() { ...@@ -1974,12 +1919,10 @@ export default function ObjectInterfaceTest() {
contactsapi.isMyCard( /*id*/ contactsapi.isMyCard( /*id*/
1, (err, data) => { 1, (err, data) => {
if (err) { if (err) {
console.info(`isMyCard callback: err->${JSON.stringify(err)}`);
expect(false).assertTrue(); expect(false).assertTrue();
done(); done();
return; return;
} }
console.info(`isMyCard callback: success data->${JSON.stringify(data)}`);
done(); done();
}); });
sleep(500) sleep(500)
...@@ -1992,12 +1935,10 @@ export default function ObjectInterfaceTest() { ...@@ -1992,12 +1935,10 @@ export default function ObjectInterfaceTest() {
return; return;
} }
let sendCallback = function (err, data) { let sendCallback = function (err, data) {
console.log(`sendCallback: err->${JSON.stringify(err)}, data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
} }
let deliveryCallback = function (err, data) { let deliveryCallback = function (err, data) {
console.log(`deliveryCallback: err->${JSON.stringify(err)}, data->${JSON.stringify(data)}`);
expect(JSON.stringify(data) === null).assertFalse(); expect(JSON.stringify(data) === null).assertFalse();
done(); done();
} }
...@@ -2045,7 +1986,6 @@ export default function ObjectInterfaceTest() { ...@@ -2045,7 +1986,6 @@ export default function ObjectInterfaceTest() {
done(); done();
sleep(500) sleep(500)
}).catch((err) => { }).catch((err) => {
console.error(`selectContact fail: err->${JSON.stringify(err)}`);
done(); done();
}); });
}); });
...@@ -2074,11 +2014,9 @@ export default function ObjectInterfaceTest() { ...@@ -2074,11 +2014,9 @@ export default function ObjectInterfaceTest() {
if (result1 == -1 || result2 == -1) { if (result1 == -1 || result2 == -1) {
contactPermission = false; contactPermission = false;
done(); done();
return;
} }
contactPermission = true; contactPermission = true;
done(); done();
return;
} }
}); });
} }
\ No newline at end of file
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册