Skip to content

Commit 28dac87

Browse files
authored
Merge pull request #126 from marcobarilari/marco_patch-isQuestionToAsk
fix code that causes error in getIsQuestionToAsk
2 parents 987dd49 + af811a0 commit 28dac87

File tree

2 files changed

+24
-1
lines changed

2 files changed

+24
-1
lines changed

src/gui/createQuestionList.m

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,8 @@
3636
if cfg.subject.askGrpSess(1)
3737
questions.questionsToAsk{1, 1} = questions.group;
3838
questions.questionsToAsk{1, 2} = false;
39+
else
40+
questions.questionsToAsk{1, 2} = false;
3941
end
4042

4143
% the subject number
@@ -46,6 +48,8 @@
4648
if cfg.subject.askGrpSess(2)
4749
questions.questionsToAsk{3, 1} = questions.session;
4850
questions.questionsToAsk{3, 2} = true;
51+
else
52+
questions.questionsToAsk{3, 2} = false;
4953
end
5054

5155
% the run number

tests/test_createQuestionList.m

Lines changed: 20 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,25 @@ function test_createQuestionListBasic()
1919
'Enter the session number (i.e day ; 1-999): ', true
2020
'Enter the run number (1-999): ', true};
2121

22-
assertEqual(expectedCell(3, 1), questions.questionsToAsk(3, 1));
22+
assertEqual(expectedCell, questions.questionsToAsk);
23+
24+
end
25+
26+
27+
function test_createQuestionListRestricted()
28+
29+
%% set up
30+
cfg = struct();
31+
cfg.subject.askGrpSess = [false false];
32+
33+
questions = createQuestionList(cfg);
34+
35+
expectedCell = { ...
36+
[], false
37+
'Enter subject number (1-999): ', true
38+
[], false
39+
'Enter the run number (1-999): ', true};
40+
41+
assertEqual(expectedCell, questions.questionsToAsk);
2342

2443
end

0 commit comments

Comments
 (0)