From 3559d1979d18547e3d1a37dbb0bcaca655e344ae Mon Sep 17 00:00:00 2001 From: Jonas Schlitzer Date: Thu, 20 Sep 2018 18:21:02 +0200 Subject: [PATCH] fix reviewcontroller tests --- mubench.reviewsite/tests/ReviewControllerTest.php | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/mubench.reviewsite/tests/ReviewControllerTest.php b/mubench.reviewsite/tests/ReviewControllerTest.php index d77c27076..b1396f7dd 100644 --- a/mubench.reviewsite/tests/ReviewControllerTest.php +++ b/mubench.reviewsite/tests/ReviewControllerTest.php @@ -81,7 +81,7 @@ function test_determine_next_misuse_current_and_next_are_reviewed() $this->createConclusiveReviewState($misuse2); // current misuse = 1, reviewer = 1 - list($previous_misuse, $next_misuse, $next_reviewable_misuse, $misuse) = $this->reviewController->determineNavigationTargets($runs, Experiment::find(2), '-p-', '-v-', $misuse1->misuse_muid, $this->reviewer1, 30); + list($previous_misuse, $next_misuse, $next_reviewable_misuse, $misuse) = $this->reviewControllerHelper->reviewController->determineNavigationTargets($runs, Experiment::find(2), '-p-', '-v-', $misuse1->misuse_muid, $this->reviewer1, 30); self::assertEquals($misuse3->misuse_muid, $previous_misuse->misuse_muid); self::assertEquals($misuse2->misuse_muid, $next_misuse->misuse_muid); @@ -99,7 +99,7 @@ function test_determine_next_misuse_current_is_only_reviewable() $this->createConclusiveReviewState($misuse2); // current misuse = 3, reviewer = 1 - list($previous_misuse, $next_misuse, $next_reviewable_misuse, $misuse) = $this->reviewController->determineNavigationTargets($runs, Experiment::find(2), '-p-', '-v-', $misuse3->misuse_muid, $this->reviewer1, 30); + list($previous_misuse, $next_misuse, $next_reviewable_misuse, $misuse) = $this->reviewControllerHelper->reviewController->determineNavigationTargets($runs, Experiment::find(2), '-p-', '-v-', $misuse3->misuse_muid, $this->reviewer1, 30); self::assertEquals($misuse2->misuse_muid, $previous_misuse->misuse_muid); self::assertEquals($misuse1->misuse_muid, $next_misuse->misuse_muid); @@ -118,7 +118,7 @@ function test_determine_next_misuse_not_fully_reviewed() $this->reviewControllerHelper->createReview($misuse3, $this->reviewer2, "Yes"); // current misuse = 1, reviewer = 1 - list($previous_misuse, $next_misuse, $next_reviewable_misuse, $misuse) = $this->reviewController->determineNavigationTargets($runs, Experiment::find(2), '-p-', '-v-', $misuse1->misuse_muid, $this->reviewer1, 30); + list($previous_misuse, $next_misuse, $next_reviewable_misuse, $misuse) = $this->reviewControllerHelper->reviewController->determineNavigationTargets($runs, Experiment::find(2), '-p-', '-v-', $misuse1->misuse_muid, $this->reviewer1, 30); self::assertEquals($misuse3->misuse_muid, $previous_misuse->misuse_muid); self::assertEquals($misuse2->misuse_muid, $next_misuse->misuse_muid); @@ -136,7 +136,7 @@ function test_determine_nav_multiple_runs() $this->createConclusiveReviewState($misuse1); // current misuse = 2, reviewer = 1 - list($previous_misuse, $next_misuse, $next_reviewable_misuse, $misuse) = $this->reviewController->determineNavigationTargets($runs, Experiment::find(2), '-p1-', '-v-', $misuse2->misuse_muid, $this->reviewer1, 30); + list($previous_misuse, $next_misuse, $next_reviewable_misuse, $misuse) = $this->reviewControllerHelper->reviewController->determineNavigationTargets($runs, Experiment::find(2), '-p1-', '-v-', $misuse2->misuse_muid, $this->reviewer1, 30); self::assertEquals($misuse1->misuse_muid, $previous_misuse->misuse_muid); self::assertEquals($misuse1->misuse_muid, $next_misuse->misuse_muid);