Session: Add tests + fix query

pull/4011/head
Julio 4 years ago
parent 08a2626ffc
commit 53c06be1f5
  1. 8
      src/CoreBundle/Repository/SessionRepository.php
  2. 4
      tests/CoreBundle/Repository/SessionRepositoryTest.php

@ -235,10 +235,10 @@ class SessionRepository extends ServiceEntityRepository
$qb = $this->getEntityManager()->createQueryBuilder();
$qb
->select('sru')
->from(User::class, 'u')
->innerJoin(SessionRelUser::class, 'sru', Join::WITH, 'sru.user = u.id')
->innerJoin(AccessUrlRelUser::class, 'uru', Join::WITH, 'uru.user = u.id')
->andWhere('sru.session = :session AND uru.url = :url')
->from(SessionRelUser::class, 'sru')
->innerJoin('sru.user', 'u')
->innerJoin('u.portals', 'p')
->andWhere('sru.session = :session AND p.url = :url')
->setParameters([
'session' => $session,
'url' => $url,

@ -397,6 +397,7 @@ class SessionRepositoryTest extends AbstractApiTest
->setVisibility(Session::INVISIBLE)
;
$em->persist($session);
$this->assertSame(0, $session->getCourses()->count());
$session->addCourse($course);
@ -436,5 +437,8 @@ class SessionRepositoryTest extends AbstractApiTest
$sessions = $sessionRepo->getUsersByAccessUrl($session, $url, [Session::STUDENT]);
$this->assertCount(2, $sessions);
$sessions = $sessionRepo->getUsersByAccessUrl($session, $url, [Session::STUDENT, Session::DRH]);
$this->assertCount(2, $sessions);
}
}

Loading…
Cancel
Save