Skip to content

Commit 8a58fa6

Browse files
tweaked addUser function and create checkAndGenerate function to which we will use integration test
1 parent 9fb2b25 commit 8a58fa6

File tree

2 files changed

+15
-9
lines changed

2 files changed

+15
-9
lines changed

app.js

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
const { generateText, createElement, validateInput } = require('./util');
1+
const { checkAndGenerate, createElement } = require('./util');
22

33
const initApp = () => {
44
// Initializes the app, registers the button click listener
@@ -12,18 +12,17 @@ const addUser = () => {
1212
const newUserNameInput = document.querySelector('input#name');
1313
const newUserAgeInput = document.querySelector('input#age');
1414

15-
if (
16-
!validateInput(newUserNameInput.value, true, false) ||
17-
!validateInput(newUserAgeInput.value, false, true)
18-
) {
15+
const outputText = checkAndGenerate(
16+
newUserNameInput.value,
17+
newUserAgeInput.value
18+
);
19+
20+
if (!outputText) {
1921
return;
2022
}
2123

2224
const userList = document.querySelector('.user-list');
23-
const outputText = generateText(
24-
newUserNameInput.value,
25-
newUserAgeInput.value
26-
);
25+
2726
const element = createElement('li', outputText, 'user-item');
2827
userList.appendChild(element);
2928
};

util.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,3 +25,10 @@ exports.validateInput = (text, notEmpty, isNumber) => {
2525
}
2626
return true;
2727
};
28+
29+
exports.checkAndGenerate = (name, age) => {
30+
if (!validateInput(name, true, false) || !validateInput(age, false, true)) {
31+
return false;
32+
}
33+
return generateText(name, age);
34+
};

0 commit comments

Comments
 (0)