Changeset 7421 for main/waeup.sirp/trunk/src/waeup/sirp/applicants

Timestamp:

21 Dec 2011, 11:00:19 (13 years ago)

Author:

Henrik Bettermann

Message:

We have to ensure that students are properly indexed after creation from applicants.

Location:

main/waeup.sirp/trunk/src/waeup/sirp/applicants

Files:

• applicant.py (modified) (2 diffs)
• tests/test_applicantcopier.py (modified) (1 diff)
• tests/test_browser.py (modified) (1 diff)

main/waeup.sirp/trunk/src/waeup/sirp/applicants/applicant.py

@@ -22 +22 @@
 from zope.component import getUtility, createObject, getUtility, getAdapter
 from zope.component.interfaces import IFactory
+from zope.event import notify
 from zope.catalog.interfaces import ICatalog
 from zope.securitypolicy.interfaces import IPrincipalRoleManager
@@ -114 +115 @@
         student['studycourse'].current_session = self.__parent__.year
         self._copyPassportImage(student)
+        # Update the catalog
+        notify(grok.ObjectModifiedEvent(student))
 
         # Save application slip

main/waeup.sirp/trunk/src/waeup/sirp/applicants/tests/test_applicantcopier.py

@@ -91 +91 @@
         self.assertEqual(self.browser.headers['Content-Type'],
                          'application/pdf')
+        # Has the student been properly indexed?
+        # Yes, we can find the student in department
+        self.browser.open('http://localhost/app/students')
+        self.browser.getControl(name="searchtype").value = ['depcode']
+        self.browser.getControl(name="searchterm").value = 'dep1'
+        self.browser.getControl("Search").click()
+        self.assertMatches('...John Tester...', self.browser.contents)
 
     def test_copier_wo_passport(self):

main/waeup.sirp/trunk/src/waeup/sirp/applicants/tests/test_browser.py

@@ -96 +96 @@
         self.certificate = certificate
         self.app['faculties']['fac1'] = Faculty()
-        self.app['faculties']['fac1']['dep1'] = Department()
+        # The code has explicitely to be set, otherwise we don't
+        # find created students in their department
+        self.app['faculties']['fac1']['dep1'] = Department(code='dep1')
         self.department = self.app['faculties']['fac1']['dep1']
         self.app['faculties']['fac1']['dep1'].certificates.addCertificate(