verdnatura/salix
verdnatura
/
salix
3
5
Fork 2
Code Issues Pull Requests 70 Releases Wiki Activity

2883-invoiceIn-Booking #786

Merged
carlosjr merged 17 commits from 2883-invoiceIn-Booking into dev 2022-01-24 15:29:57 +00:00
Conversation 0 Commits 17 Files Changed 17 +2 -2
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit dad4119300 - Show all commits

2
modules/invoiceIn/back/methods/invoice-in/getTotals.js
View File

@ -38,6 +38,6 @@ module.exports = Self => {
WHERE
ii.id = ?`, [id, id]);
return (result);
return result;
jgallego marked this conversation as resolved Outdated
carlosjr commented 2022-01-21 08:24:01 +00:00
Outdated
Review
Copy Link

remove the ()

remove the ()
};
};

2
modules/invoiceIn/front/descriptor/index.spec.js
View File

@ -37,7 +37,7 @@ describe('vnInvoiceInDescriptor', () => {
});
describe('checkToBook()', () => {
it(`should show both warning messages`, () => {
it(`should show a warning before book`, () => {
jgallego marked this conversation as resolved Outdated
carlosjr commented 2022-01-21 08:41:28 +00:00
Outdated
Review
Copy Link

the description of the test states two warning messages will be shown but the test doesn't check that

the description of the test states two warning messages will be shown but the test doesn't check that
controller.$.confirmToBookAnyway = {show: () => {}};
jest.spyOn(controller.$.confirmToBookAnyway, 'show');