Skip to content

Commit

Permalink
Addressing failing tests
Browse files Browse the repository at this point in the history
  • Loading branch information
jeffersoncasimir committed Feb 12, 2025
1 parent 1f77e78 commit 9f58096
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion php/libraries/Candidate.class.inc
Original file line number Diff line number Diff line change
Expand Up @@ -1082,7 +1082,7 @@ class Candidate implements \LORIS\StudyEntities\AccessibleResource,
WHERE c.CandID=:candid",
['candid' => $this->getCandID()],
);
return $res['Description'] ?? '';
return $res['Description'] ?? "Not found for CandID: {$this->getCandID()}";
}

/**
Expand Down
2 changes: 1 addition & 1 deletion php/libraries/NDB_BVL_Feedback.class.inc
Original file line number Diff line number Diff line change
Expand Up @@ -396,7 +396,7 @@ class NDB_BVL_Feedback
$qparams['ComID'] = $this->_feedbackObjectInfo['CommentID'];
}
if (empty($select_inactive_threads)) {
$query .= " AND Active = 'Y'";
$query .= " AND fbt.Active = 'Y'";
}

$query .= " ORDER BY Status ASC";
Expand Down
6 changes: 3 additions & 3 deletions test/unittests/CandidateTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ protected function setUp(): void
$this->_factory->setDatabase($this->_dbMock);

$this->_candidateInfo = [
'ID' => 63,
'ID' => 111111,
'RegistrationCenterID' => 2,
'CandID' => new CandID('969664'),
'PSCID' => 'AAA0011',
Expand Down Expand Up @@ -1194,7 +1194,7 @@ public function testParticipantStatusDescription()
"participant_status",
[
0 => [
'CandidateID' => 63,
'CandidateID' => 111111,
'participant_status' => '2',
]
]
Expand All @@ -1203,7 +1203,7 @@ public function testParticipantStatusDescription()
$this->_DB->run("DROP TEMPORARY TABLE participant_status_options");
$this->_DB->run("DROP TEMPORARY TABLE participant_status");
$this->assertEquals(
$this->_candidate->getPSCID() . '--' . $this->_candidate->getData('ID'),
$result,
'description2'
);
}
Expand Down

0 comments on commit 9f58096

Please sign in to comment.