- Pollen-commits - mailman.cloud.codelutin.com

r3486 - trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl
by tchemit＠users.chorem.org 15 Jun '12

15 Jun '12

Author: tchemit Date: 2012-06-15 23:13:46 +0200 (Fri, 15 Jun 2012) New Revision: 3486 Url: http://chorem.org/repositories/revision/pollen/3486 Log: add new methods in securit services Modified: trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/SecurityService.java Modified: trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/SecurityService.java =================================================================== --- trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/SecurityService.java 2012-06-15 20:50:50 UTC (rev 3485) +++ trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/SecurityService.java 2012-06-15 21:13:46 UTC (rev 3486) @@ -22,6 +22,9 @@ */ package org.chorem.pollen.services.impl; +import com.google.common.base.Preconditions; +import com.google.common.collect.Sets; +import org.apache.commons.lang3.StringUtils; import org.chorem.pollen.PollenTechnicalException; import org.chorem.pollen.bean.PollUri; import org.chorem.pollen.business.persistence.Poll; @@ -33,6 +36,8 @@ import org.chorem.pollen.services.exceptions.UnauthorizedPollAccessException; import org.nuiton.topia.TopiaException; +import java.util.Set; + /** * Service to manager security. * @@ -41,6 +46,37 @@ */ public class SecurityService extends PollenServiceSupport { + /** + * To define meaning of a accountId. + * + * @author tchemit <chemit(a)codelutin.com> + * @since 1.4 + */ + public enum AccountIdRole { + + /** + * Creator of the poll. + * <p/> + * This role can access to everything, but can not vote. + */ + CREATOR, + /** A user that has voted on a poll. */ + VOTER, + /** + * A user that was invited to a restricted poll. + * <p/> + * It might have alreay voted or not. + */ + RESTRICTED_VOTER, + /** When accountId does not exists for a poll. */ + UNDEFINED + } + + public static final Set<AccountIdRole> NONE_FREE_ACCOUNT_ID_ROLES = Sets.newHashSet( + AccountIdRole.RESTRICTED_VOTER, + AccountIdRole.CREATOR + ); + public void checkPoll(PollUri uri) throws PollNotFoundException { getExistingPoll(uri); @@ -119,7 +155,74 @@ } } + private boolean isVoterAccountId(PollAccountDAO dao, PollUri uri) { + try { + + return dao.existsByPollVoteAccountId(uri.getPollId(), uri.getAccountId()); + + } catch (TopiaException e) { + throw new PollenTechnicalException( + "Could not check pollAccount existence from poll '" + + uri.getPollId() + "' and account '" + uri.getAccountId() + "'", e); + } + } + + private boolean isRestrictPollAccountId(PollAccountDAO dao, PollUri uri) { + try { + + PollAccount result = + dao.getRestrictedPollAccount(uri.getPollId(), uri.getAccountId()); + + return result != null; + + } catch (TopiaException e) { + throw new PollenTechnicalException( + "Could not check pollAccount existence from poll '" + + uri.getPollId() + "' and account '" + uri.getAccountId() + "'", e); + } + } + protected Poll getExistingPoll(PollUri uri) throws PollNotFoundException { return newService(PollService.class).getExistingPollByPollId(uri.getPollId()); } + + public AccountIdRole getAccountIdRole(Poll poll, String accountId) { + + Preconditions.checkNotNull(poll); + + AccountIdRole result = AccountIdRole.UNDEFINED; + + if (StringUtils.isNotBlank(accountId)) { + + if (accountId.equals(poll.getCreator().getAccountId())) { + + result = AccountIdRole.CREATOR; + } else { + + PollAccountDAO dao = getDAO(PollAccount.class); + + PollUri pollUri = PollUri.newPollUri(poll.getPollId(), accountId); + + if (poll.getPollType() == PollType.FREE) { + + // + + boolean found = isVoterAccountId(dao, pollUri); + + if (found) { + result = AccountIdRole.VOTER; + } + } else { + + boolean found = isRestrictPollAccountId(dao, pollUri); + + if (found) { + result = AccountIdRole.RESTRICTED_VOTER; + } + } + } + } + return result; + } + }

1 0

r3485 - trunk/pollen-ui-struts2/src/main/webapp/WEB-INF/decorators
by tchemit＠users.chorem.org 15 Jun '12

15 Jun '12

Author: tchemit Date: 2012-06-15 22:50:50 +0200 (Fri, 15 Jun 2012) New Revision: 3485 Url: http://chorem.org/repositories/revision/pollen/3485 Log: fix bad javascript Modified: trunk/pollen-ui-struts2/src/main/webapp/WEB-INF/decorators/layout-default.jsp Modified: trunk/pollen-ui-struts2/src/main/webapp/WEB-INF/decorators/layout-default.jsp =================================================================== --- trunk/pollen-ui-struts2/src/main/webapp/WEB-INF/decorators/layout-default.jsp 2012-06-15 19:13:01 UTC (rev 3484) +++ trunk/pollen-ui-struts2/src/main/webapp/WEB-INF/decorators/layout-default.jsp 2012-06-15 20:50:50 UTC (rev 3485) @@ -28,7 +28,10 @@ <%@ taglib prefix="sj" uri="/struts-jquery-tags" %> <%@ taglib prefix="d" uri="http://www.opensymphony.com/sitemesh/decorator" %> <html xmlns="http://www.w3.org/1999/xhtml"> - +<script type="text/javascript"> + var StrutsUtils = new Object(); + StrutsUtils.showValidationErrors = function(){}; +</script> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"/> <title>Pollen - <d:title default="Pollen"/></title>

1 0

r3484 - trunk/src/site/rst
by tchemit＠users.chorem.org 15 Jun '12

15 Jun '12

Author: tchemit Date: 2012-06-15 21:13:01 +0200 (Fri, 15 Jun 2012) New Revision: 3484 Url: http://chorem.org/repositories/revision/pollen/3484 Log: fix rst headers Modified: trunk/src/site/rst/security.rst trunk/src/site/rst/tests.rst Modified: trunk/src/site/rst/security.rst =================================================================== --- trunk/src/site/rst/security.rst 2012-06-15 17:12:40 UTC (rev 3483) +++ trunk/src/site/rst/security.rst 2012-06-15 19:13:01 UTC (rev 3484) @@ -1,3 +1,26 @@ +.. - +.. * #%L +.. * Pollen +.. * $Id$ +.. * $HeadURL$ +.. * %% +.. * Copyright (C) 2009 - 2012 CodeLutin +.. * %% +.. * This program is free software: you can redistribute it and/or modify +.. * it under the terms of the GNU Affero General Public License as published by +.. * the Free Software Foundation, either version 3 of the License, or +.. * (at your option) any later version. +.. * +.. * This program is distributed in the hope that it will be useful, +.. * but WITHOUT ANY WARRANTY; without even the implied warranty of +.. * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +.. * GNU General Public License for more details. +.. * +.. * You should have received a copy of the GNU Affero General Public License +.. * along with this program. If not, see <http://www.gnu.org/licenses/>. +.. * #L% +.. - + ~~~~~~~~~~~~~~~~~~~~~~~~ Security model in Pollen ~~~~~~~~~~~~~~~~~~~~~~~~ Property changes on: trunk/src/site/rst/security.rst ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL Added: svn:eol-style + native Modified: trunk/src/site/rst/tests.rst =================================================================== --- trunk/src/site/rst/tests.rst 2012-06-15 17:12:40 UTC (rev 3483) +++ trunk/src/site/rst/tests.rst 2012-06-15 19:13:01 UTC (rev 3484) @@ -1,3 +1,26 @@ +.. - +.. * #%L +.. * Pollen +.. * $Id$ +.. * $HeadURL$ +.. * %% +.. * Copyright (C) 2009 - 2012 CodeLutin +.. * %% +.. * This program is free software: you can redistribute it and/or modify +.. * it under the terms of the GNU Affero General Public License as published by +.. * the Free Software Foundation, either version 3 of the License, or +.. * (at your option) any later version. +.. * +.. * This program is distributed in the hope that it will be useful, +.. * but WITHOUT ANY WARRANTY; without even the implied warranty of +.. * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +.. * GNU General Public License for more details. +.. * +.. * You should have received a copy of the GNU Affero General Public License +.. * along with this program. If not, see <http://www.gnu.org/licenses/>. +.. * #L% +.. - + ~~~~~~~~~~ Tests d'UI ~~~~~~~~~~

1 0

r3483 - trunk/pollen-services/src/main/java/org/chorem/pollen/bean
by tchemit＠users.chorem.org 15 Jun '12

15 Jun '12

Author: tchemit Date: 2012-06-15 19:12:40 +0200 (Fri, 15 Jun 2012) New Revision: 3483 Url: http://chorem.org/repositories/revision/pollen/3483 Log: add some usefull methods on PollUri Modified: trunk/pollen-services/src/main/java/org/chorem/pollen/bean/PollUri.java Modified: trunk/pollen-services/src/main/java/org/chorem/pollen/bean/PollUri.java =================================================================== --- trunk/pollen-services/src/main/java/org/chorem/pollen/bean/PollUri.java 2012-06-15 16:06:58 UTC (rev 3482) +++ trunk/pollen-services/src/main/java/org/chorem/pollen/bean/PollUri.java 2012-06-15 17:12:40 UTC (rev 3483) @@ -24,7 +24,6 @@ import com.google.common.base.Preconditions; import org.apache.commons.lang3.StringUtils; -import org.chorem.pollen.business.persistence.Poll; import java.io.Serializable; @@ -85,6 +84,14 @@ this.accountId = accountId; } + public boolean isPollIdNotBlank() { + return StringUtils.isNotBlank(pollId); + } + + public boolean isAccountIdNotBlank() { + return StringUtils.isNotBlank(accountId); + } + public String getUri() { String result = pollId; if (StringUtils.isNotEmpty(accountId)) {

1 0

r3482 - trunk
by tchemit＠users.chorem.org 15 Jun '12

15 Jun '12

Author: tchemit Date: 2012-06-15 18:06:58 +0200 (Fri, 15 Jun 2012) New Revision: 3482 Url: http://chorem.org/repositories/revision/pollen/3482 Log: updates to topia 2.6.11 Modified: trunk/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2012-06-15 15:12:06 UTC (rev 3481) +++ trunk/pom.xml 2012-06-15 16:06:58 UTC (rev 3482) @@ -178,7 +178,7 @@ <projectId>pollen</projectId>  - <topiaVersion>2.6.11-SNAPSHOT</topiaVersion> + <topiaVersion>2.6.11</topiaVersion> <eugenePluginVersion>2.4.2</eugenePluginVersion> <nuitonI18nVersion>2.4.1</nuitonI18nVersion>

1 0

r3481 - in trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui: . its
by ymartel＠users.chorem.org 15 Jun '12

15 Jun '12

Author: ymartel Date: 2012-06-15 17:12:06 +0200 (Fri, 15 Jun 2012) New Revision: 3481 Url: http://chorem.org/repositories/revision/pollen/3481 Log: refs #606 : extract check of current url Modified: trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/PollenFixtures.java trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/CreateFreeTextPollSIT.java trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/PollenBaseWebDriverIT.java Modified: trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/PollenFixtures.java =================================================================== --- trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/PollenFixtures.java 2012-06-15 15:09:26 UTC (rev 3480) +++ trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/PollenFixtures.java 2012-06-15 15:12:06 UTC (rev 3481) @@ -147,6 +147,10 @@ return baseUrl() + "poll/create"; } + public String savePollURL() { + return baseUrl() + "poll/save"; + } + public String summaryURL() { return baseUrl() + "poll/summary"; } Modified: trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/CreateFreeTextPollSIT.java =================================================================== --- trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/CreateFreeTextPollSIT.java 2012-06-15 15:09:26 UTC (rev 3480) +++ trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/CreateFreeTextPollSIT.java 2012-06-15 15:12:06 UTC (rev 3481) @@ -95,8 +95,7 @@ submit.click(); // All good, should be on summary page ! - String currentUrl = driver.getCurrentUrl(); - Assert.assertTrue(currentUrl.startsWith(fixtures.summaryURL())); + checkCurrentUrl(fixtures.summaryURL(), false); } /** @@ -130,7 +129,7 @@ submit.click(); // Title is missing, should stay on create poll page - Assert.assertTrue(driver.getCurrentUrl().startsWith(fixtures.createPollURL())); + checkCurrentUrl(fixtures.createPollURL(), false); // Check an error message has been displayed WebElement titleError = findElement(By.className("errorMessage")); @@ -172,7 +171,7 @@ submit.click(); // No choices, should stay on create poll page - Assert.assertTrue(driver.getCurrentUrl().startsWith(fixtures.createPollURL())); + checkCurrentUrl(fixtures.createPollURL(), false); // Check that title field is not lost titleElement = findElement(By.name("poll.title")); @@ -230,7 +229,7 @@ submit.click(); // No choices, should stay on create poll page - Assert.assertTrue(driver.getCurrentUrl().startsWith(fixtures.createPollURL())); + checkCurrentUrl(fixtures.createPollURL(), false); // Check an error message has been displayed WebElement dateError = findElement(By.className("errorMessage")); @@ -307,7 +306,7 @@ submit.click(); // No choices, should stay on create poll page - Assert.assertTrue(driver.getCurrentUrl().startsWith(fixtures.createPollURL())); + checkCurrentUrl(fixtures.createPollURL(), false); // Check an error message has been displayed WebElement dateError = findElement(By.className("errorMessage")); @@ -367,7 +366,7 @@ submit.click(); // No choices, should stay on create poll page - Assert.assertTrue(driver.getCurrentUrl().startsWith(fixtures.createPollURL())); + checkCurrentUrl(fixtures.createPollURL(), false); // Check an error message has been displayed WebElement dateError = findElement(By.className("errorMessage")); @@ -437,7 +436,7 @@ submit.click(); // No choices, should stay on create poll page - Assert.assertTrue(driver.getCurrentUrl().startsWith(fixtures.createPollURL())); + checkCurrentUrl(fixtures.createPollURL(), false); // Check an error message has been displayed WebElement dateError = findElement(By.className("errorMessage")); @@ -473,7 +472,7 @@ submit.click(); // No choices, should stay on create poll page - Assert.assertTrue(driver.getCurrentUrl().startsWith(fixtures.createPollURL())); + checkCurrentUrl(fixtures.createPollURL(), false); } Modified: trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/PollenBaseWebDriverIT.java =================================================================== --- trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/PollenBaseWebDriverIT.java 2012-06-15 15:09:26 UTC (rev 3480) +++ trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/PollenBaseWebDriverIT.java 2012-06-15 15:12:06 UTC (rev 3481) @@ -113,6 +113,16 @@ expectedUrl = fallBackUrl; } + checkCurrentUrl(expectedUrl, strict); + } + + /** + * Check that the current URL is the expected one. + * If it is a strict comparison, the URL should be the same that the wanted. + * If not, it only checks that the current URL start with the wanted url. + * + */ + protected void checkCurrentUrl(String expectedUrl, boolean strict) { if (strict) { Assert.assertEquals(expectedUrl, driver.getCurrentUrl()); } else {

1 0

r3480 - in trunk: doc src/site src/site/rst
by tchemit＠users.chorem.org 15 Jun '12

15 Jun '12

Author: tchemit Date: 2012-06-15 17:09:26 +0200 (Fri, 15 Jun 2012) New Revision: 3480 Url: http://chorem.org/repositories/revision/pollen/3480 Log: add some tech docs Added: trunk/src/site/rst/security.rst trunk/src/site/rst/tests.rst Removed: trunk/doc/tests.rst Modified: trunk/src/site/site_en.xml Deleted: trunk/doc/tests.rst =================================================================== --- trunk/doc/tests.rst 2012-06-15 14:45:42 UTC (rev 3479) +++ trunk/doc/tests.rst 2012-06-15 15:09:26 UTC (rev 3480) @@ -1,119 +0,0 @@ -~~~~~~~~~~ -Tests d'UI -~~~~~~~~~~ - -Abstract -======== - -- Tests Pollen by ui. -- Needs a db with - - 1 admin user *admin/admin* - - 1 none admin user *user/user* - - 1 none admin user *user2/user2* - - 1 poll of each votecounting type (Normal, pourcentage, Condorcet, Nombre) + some votes on it - - 1 poll with restricted mode - -Scenarii -======== - -Security --------- - -- authentication avec mauvais login/password (KO) -- authentication avec bon login/password (OK) - -- access aux pages connectés en mode déconnecté (KO) -- access aux pages connectés en mode connecté (OK) - -- access aux pages admin en mode déconnecté (KO) -- access aux pages admin avec un user non admin (KO) -- access aux pages admin avec un user admin (OK) - -- access aux pages d'un sondage avec un mauvais pollId (KO) -- access aux pages d'un sondage avec un bon pollId (OK) - -- access aux pages d'un sondage en mode créateur - - en mode admin (pas de accountId dans l'url) : avec un user non admin (KO) - - en mode admin (pas de accountId dans l'url) : avec un admin (OK) - - en mode créateur (avec mauvais accountId dans l'url) (KO) - - en mode créateur (avec bon accountId dans l'url) (OK) - -- access aux résultats d'un sondage (non restreint) (OK) -- access aux résultats d'un sondage restreint en anonyme (KO) -- access aux résultats d'un sondage restreint en extérieur (mauvais accountId) (KO) -- access aux résultats d'un sondage restreint en extérieur (mauvais utilisateur connecté) (KO) -- access aux résultats d'un sondage restreint avec bon accountId (OK) -- access aux résultats d'un sondage restreint avec bon utilisateur connecté (OK) -- access aux résultats d'un sondage restreint avec accountId de créateur (OK) -- access aux résultats d'un sondage restreint avec un admin connecté (OK) - -Création d'un sondage (free + text) ------------------------------------ - -- creation avec champs obligatoires remplis (OK) -- creation avec Titre non rempli (KO) -- creation avec aucun choix proposé (KO) -- creation avec deux choix identiques (KO) -- creation avec Date de fin < Date de début (KO) -- creation avec Date de fin antérieur à maintenant (KO) (TODO) -- creation avec Ajout de choix Date de fin < Date de début (KO) -- creation avec Ajout de choix Date de fin antérieur à maintenant (KO) -- creation avec Ajout de choix cochez et remplit + decochez (les dates doivent être supprimées) (OK) (TODO) -- creation avec email invalide (KO) -- creation avec heure avant la fin de sondage <=0 (KO) (TODO) -- TODO Mettre un warning si date de fin proche... (new feature) -- creation avec nombre de choix <=0 (KO) (TODO : pffff quand on veut voter on a un message Le nombre de choix maximal est de -5....) -- TODO Par défaut mettre au moins 1 - -- TODO Quand on déchoche alors il faut desactiver les champs concernés (et pas uniquement les cacher) - -Création d'un sondage (free + date) ------------------------------------ - -- creation avec aucun choix proposé (KO) -- creation avec deux choix identiques (KO) -- creation d'un sondage de Date avec mauvais format (KO) - -Création d'un sondage (free + image) ------------------------------------- - -- creation avec aucun choix proposé (KO) -- creation d'un sondage d'Image avec mauvais mimetype (KO) (TODO) -- creation avec deux images au nom identiques (KO) (TODO) -- TODO Mettre une image avec une croix ROUGE!!! quand pas chargée - -Création d'un sondage (restreint + text) ----------------------------------------- - -- création sans participant (KO) -- création avec deux participants ayant le même nom (KO) -- création avec deux participants ayant le même email (KO) -- création avec un participants ayant juste un email et pas de nom (KO) (TODO : pollenTechincalException) -- création avec un participants ayant juste un nom et pas d'email (KO) -- TODO problème d'enregistrement des plusieurs votants, le dernier est conservé et dupliqué. -- TODO L'email envoyé n'est pas le bon (il manque le token d'account) + la sécurité ne va pas car je ne dois pas avoir accès à ce sondage - -Création d'un sondage (group + text) ----------------------------------------- - -- création sans nom de groupe (KO) -- création avec deux participants ayant le même nom (KO) -- création avec deux participants ayant le même email (KO) -- création avec un participants ayant juste un email et pas de nom (KO) (TODO : pollenTechincalException) -- création avec un participants ayant juste un nom et pas d'email (KO) -- TODO problème d'enregistrement des plusieurs votants, le dernier est conservé et dupliqué. -- TODO Revoir l'enregistrement (le groupe 2 est perdu...) - - -Edition d'un sondage ---------------------- - -- edition avec champs titre vidé (OK) -- edition avec modification date de fin antérieure à maintenant (KO) -- edition avec suppression de tous les choix (KO) - -Page de résumé d'un sondage ---------------------------- - -- TODO supprimer les // dans les urls - Added: trunk/src/site/rst/security.rst =================================================================== --- trunk/src/site/rst/security.rst (rev 0) +++ trunk/src/site/rst/security.rst 2012-06-15 15:09:26 UTC (rev 3480) @@ -0,0 +1,161 @@ +~~~~~~~~~~~~~~~~~~~~~~~~ +Security model in Pollen +~~~~~~~~~~~~~~~~~~~~~~~~ + +Abstract +======== + +- New way of dealing security in Pollen (1.5 ?) + +Roles +===== + +Pollen defines different roles for different usages. Some roles can be found +by the stored Pollen session (in servlet session), others can be found by the +accountId in some urls (example http://pollen/action/pollId:accountId) + +Anonymous +--------- + +This is a not loggued user with nothing is url (no accountId) which can +identify himself. + +Connected user +-------------- + +When a user has been registred inside Pollen and is connected in Pollen. + +We can find him back by using the pollen user account stored in servlet session. + +Admin +----- + +Connected user with more privileges to manage everything in Pollen. + +We can find him back by using the pollen user account stored in servlet session. + +Poll creator +------------ + +Person which has created a poll. It acts as an admin for his very own poll. + +We can find him back by using a accountId in url. + +Poll voter +---------- + +A person which has voted to a poll. + +We can find him back by using a accountId in url. + +Poll participant +---------------- + +A person that can vote on a none free poll. + +We can find him back by using a accountId in url. + +Resources and permissions +========================= + +User +---- + +A user (*UserAccount*) is a Pollen connected user. + +- user:modify +- user:delete + +VotingList +---------- + +A voting list (*VotingList*) is a list of person (name + email) to simplify +creation of restricted poll (you can import in one time a such list for a poll). + +- votingList:modify +- votingList:delete + +Poll +---- + +A poll (*Poll*) is life in Pollen! + +- poll:create +- poll:modify +- poll:vote +- poll:close +- poll:delete +- poll:clone +- poll:export +- poll:votecounting +- poll:comment +- poll:add choice + +Choice +------ + +A choice (*Choice*) is a choice on which to vote in a poll. + +- choice:modify +- choice:delete + +Comment +------- + +A comment (*Comment*) is a comment to be added on a poll. + +- comment:modify +- comment:delete + +Usage of wildcard +----------------- + +To simplify usage of permissions, we will use some wildcard. For example, an +admin can do anything and has all permissions: :: + + user:*:* + poll:*:* (only if poll is free, result public and continuous,...) + choice:*:* + comment:*:* + +A poll creator will have all permissions on his poll: :: + + poll:*:pollId (only if poll is free, result public and continuous,...) + choice:*:pollId (only if choice can add + comment:*:pollId + +But these examples are not quite exact. In facts, a admin or a poll creator could have + +How to implement this security model +==================================== + +Based on Shiro (http://shiro.apache.org/) + +- using authentication mecanism to find roles (http://shiro.apache.org/authentication-features.html) +- using permission mecanism to find role permissions (http://shiro.apache.org/permissions.html) + +Global workflow +--------------- + +When a user wants to acces a resource (page), find out first the user role, +then compute his. + +To tune this we will use a chache of permissions, beacuse it does seem very +realist to recompute at each request the cache for a poll or a connected user. + +From this first cache: + +- key re userAccount or accountId, +- values are his set of permissions, + +we will build one form each poll: + +- key is pollId +- values are users (on which permissions where build) involved by this poll. + +At each poll modification, we then invalidate poll cache for the pollId entry +and then also simply invalidate the cache of all users involved by this poll. + + + + Copied: trunk/src/site/rst/tests.rst (from rev 3478, trunk/doc/tests.rst) =================================================================== --- trunk/src/site/rst/tests.rst (rev 0) +++ trunk/src/site/rst/tests.rst 2012-06-15 15:09:26 UTC (rev 3480) @@ -0,0 +1,119 @@ +~~~~~~~~~~ +Tests d'UI +~~~~~~~~~~ + +Abstract +======== + +- Tests Pollen by ui. +- Needs a db with + - 1 admin user *admin/admin* + - 1 none admin user *user/user* + - 1 none admin user *user2/user2* + - 1 poll of each votecounting type (Normal, pourcentage, Condorcet, Nombre) + some votes on it + - 1 poll with restricted mode + +Scenarii +======== + +Security +-------- + +- authentication avec mauvais login/password (KO) +- authentication avec bon login/password (OK) + +- access aux pages connectés en mode déconnecté (KO) +- access aux pages connectés en mode connecté (OK) + +- access aux pages admin en mode déconnecté (KO) +- access aux pages admin avec un user non admin (KO) +- access aux pages admin avec un user admin (OK) + +- access aux pages d'un sondage avec un mauvais pollId (KO) +- access aux pages d'un sondage avec un bon pollId (OK) + +- access aux pages d'un sondage en mode créateur + - en mode admin (pas de accountId dans l'url) : avec un user non admin (KO) + - en mode admin (pas de accountId dans l'url) : avec un admin (OK) + - en mode créateur (avec mauvais accountId dans l'url) (KO) + - en mode créateur (avec bon accountId dans l'url) (OK) + +- access aux résultats d'un sondage (non restreint) (OK) +- access aux résultats d'un sondage restreint en anonyme (KO) +- access aux résultats d'un sondage restreint en extérieur (mauvais accountId) (KO) +- access aux résultats d'un sondage restreint en extérieur (mauvais utilisateur connecté) (KO) +- access aux résultats d'un sondage restreint avec bon accountId (OK) +- access aux résultats d'un sondage restreint avec bon utilisateur connecté (OK) +- access aux résultats d'un sondage restreint avec accountId de créateur (OK) +- access aux résultats d'un sondage restreint avec un admin connecté (OK) + +Création d'un sondage (free + text) +----------------------------------- + +- creation avec champs obligatoires remplis (OK) +- creation avec Titre non rempli (KO) +- creation avec aucun choix proposé (KO) +- creation avec deux choix identiques (KO) +- creation avec Date de fin < Date de début (KO) +- creation avec Date de fin antérieur à maintenant (KO) (TODO) +- creation avec Ajout de choix Date de fin < Date de début (KO) +- creation avec Ajout de choix Date de fin antérieur à maintenant (KO) +- creation avec Ajout de choix cochez et remplit + decochez (les dates doivent être supprimées) (OK) (TODO) +- creation avec email invalide (KO) +- creation avec heure avant la fin de sondage <=0 (KO) (TODO) +- TODO Mettre un warning si date de fin proche... (new feature) +- creation avec nombre de choix <=0 (KO) (TODO : pffff quand on veut voter on a un message Le nombre de choix maximal est de -5....) +- TODO Par défaut mettre au moins 1 + +- TODO Quand on déchoche alors il faut desactiver les champs concernés (et pas uniquement les cacher) + +Création d'un sondage (free + date) +----------------------------------- + +- creation avec aucun choix proposé (KO) +- creation avec deux choix identiques (KO) +- creation d'un sondage de Date avec mauvais format (KO) + +Création d'un sondage (free + image) +------------------------------------ + +- creation avec aucun choix proposé (KO) +- creation d'un sondage d'Image avec mauvais mimetype (KO) (TODO) +- creation avec deux images au nom identiques (KO) (TODO) +- TODO Mettre une image avec une croix ROUGE!!! quand pas chargée + +Création d'un sondage (restreint + text) +---------------------------------------- + +- création sans participant (KO) +- création avec deux participants ayant le même nom (KO) +- création avec deux participants ayant le même email (KO) +- création avec un participants ayant juste un email et pas de nom (KO) (TODO : pollenTechincalException) +- création avec un participants ayant juste un nom et pas d'email (KO) +- TODO problème d'enregistrement des plusieurs votants, le dernier est conservé et dupliqué. +- TODO L'email envoyé n'est pas le bon (il manque le token d'account) + la sécurité ne va pas car je ne dois pas avoir accès à ce sondage + +Création d'un sondage (group + text) +---------------------------------------- + +- création sans nom de groupe (KO) +- création avec deux participants ayant le même nom (KO) +- création avec deux participants ayant le même email (KO) +- création avec un participants ayant juste un email et pas de nom (KO) (TODO : pollenTechincalException) +- création avec un participants ayant juste un nom et pas d'email (KO) +- TODO problème d'enregistrement des plusieurs votants, le dernier est conservé et dupliqué. +- TODO Revoir l'enregistrement (le groupe 2 est perdu...) + + +Edition d'un sondage +--------------------- + +- edition avec champs titre vidé (OK) +- edition avec modification date de fin antérieure à maintenant (KO) +- edition avec suppression de tous les choix (KO) + +Page de résumé d'un sondage +--------------------------- + +- TODO supprimer les // dans les urls + Property changes on: trunk/src/site/rst/tests.rst ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL Added: svn:eol-style + native Modified: trunk/src/site/site_en.xml =================================================================== --- trunk/src/site/site_en.xml 2012-06-15 14:45:42 UTC (rev 3479) +++ trunk/src/site/site_en.xml 2012-06-15 15:09:26 UTC (rev 3480) @@ -103,6 +103,11 @@ <item name="English" href="index.html"/> </menu> + <menu name="Technical"> + <item name="Security" href="security.html"/> + <item name="UI Tests" href="tests.html"/> + </menu> + <menu ref="reports"/> <footer>

1 0

r3479 - trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its
by ymartel＠users.chorem.org 15 Jun '12

15 Jun '12

Author: ymartel Date: 2012-06-15 16:45:42 +0200 (Fri, 15 Jun 2012) New Revision: 3479 Url: http://chorem.org/repositories/revision/pollen/3479 Log: refs #606 : add tests for free text poll creation with bad number of choices Modified: trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/CreateFreeTextPollSIT.java Modified: trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/CreateFreeTextPollSIT.java =================================================================== --- trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/CreateFreeTextPollSIT.java 2012-06-15 07:30:44 UTC (rev 3478) +++ trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/CreateFreeTextPollSIT.java 2012-06-15 14:45:42 UTC (rev 3479) @@ -387,4 +387,94 @@ } + /** + * This test create a free text poll with an invalid email. + * When submit, an error text should be displayed : email should be valid + * + * @throws Exception + */ + @Test + public void createFreeTextPollWithInvalidMaxVotedChoices() throws Exception { + + // Go on create poll page + gotoUrl(fixtures.createPollURL()); + + // Set title + WebElement titleElement = findElement(By.name("poll.title")); + Assert.assertEquals("input", titleElement.getTagName()); + Assert.assertTrue(titleElement.isDisplayed()); + String pollTitle = "My Poll"; + titleElement.sendKeys(pollTitle); + + // Set first choice + WebElement choiceOneElement = findElement(By.name("textChoice_0.name")); + Assert.assertEquals("input", choiceOneElement.getTagName()); + Assert.assertTrue(choiceOneElement.isDisplayed()); + String choiceOne = "Choix numero un"; + choiceOneElement.sendKeys(choiceOne); + + // Go to option panel : click on it + WebElement optionsClick = findElement(By.xpath("//a[@href=\"#toptions\"]")); + optionsClick.click(); + + // Get the maxChoice element : should be disable while limitChoice is not selected + WebElement maxChoiceElement = findElement(By.name("maxChoices")); + Assert.assertEquals("input", maxChoiceElement.getTagName()); + Assert.assertTrue(maxChoiceElement.isDisplayed()); + // Get the checkbox and check it to set number of max choice after + WebElement limitChoiceElement = findElement(By.name("limitChoice")); + Assert.assertEquals("input", limitChoiceElement.getTagName()); + Assert.assertTrue(limitChoiceElement.isEnabled()); + limitChoiceElement.click(); + // Set a negative number of max choice + maxChoiceElement = findElement(By.name("maxChoices")); + Assert.assertTrue(maxChoiceElement.isEnabled()); + maxChoiceElement.clear(); + maxChoiceElement.sendKeys("-1"); + + // Submit form + WebElement submit = findElement(By.name("action:save")); + submit.click(); + + // No choices, should stay on create poll page + Assert.assertTrue(driver.getCurrentUrl().startsWith(fixtures.createPollURL())); + + // Check an error message has been displayed + WebElement dateError = findElement(By.className("errorMessage")); + Assert.assertNotNull(dateError); + + // Go on general tab and check that Title and and choice fields values are not lost + WebElement generalClick = findElement(By.xpath("//a[@href=\"#tgeneral\"]")); + generalClick.click(); + + titleElement = findElement(By.name("poll.title")); + Assert.assertTrue(titleElement.isDisplayed()); + Assert.assertEquals(pollTitle, titleElement.getAttribute("value")); + + choiceOneElement = findElement(By.name("textChoice_0.name")); + Assert.assertTrue(choiceOneElement.isDisplayed()); + Assert.assertEquals(choiceOne, choiceOneElement.getAttribute("value")); + + // Second part: try to set "0" for max choice + + // Go back to option panel : click on it + optionsClick = findElement(By.xpath("//a[@href=\"#toptions\"]")); + optionsClick.click(); + + // Set a negative number of max choice + maxChoiceElement = findElement(By.name("maxChoices")); + Assert.assertEquals("input", maxChoiceElement.getTagName()); + Assert.assertTrue(maxChoiceElement.isEnabled()); + maxChoiceElement.clear(); + maxChoiceElement.sendKeys("0"); + + // Submit form + submit = findElement(By.name("action:save")); + submit.click(); + + // No choices, should stay on create poll page + Assert.assertTrue(driver.getCurrentUrl().startsWith(fixtures.createPollURL())); + + } + }

1 0

r3478 - trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its
by ymartel＠users.chorem.org 15 Jun '12

15 Jun '12

Author: ymartel Date: 2012-06-15 09:30:44 +0200 (Fri, 15 Jun 2012) New Revision: 3478 Url: http://chorem.org/repositories/revision/pollen/3478 Log: fix htmlunit usage Modified: trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/PollenBaseWebDriverIT.java Modified: trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/PollenBaseWebDriverIT.java =================================================================== --- trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/PollenBaseWebDriverIT.java 2012-06-15 07:29:58 UTC (rev 3477) +++ trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/PollenBaseWebDriverIT.java 2012-06-15 07:30:44 UTC (rev 3478) @@ -40,6 +40,7 @@ import org.openqa.selenium.WebElement; import org.openqa.selenium.chrome.ChromeDriver; import org.openqa.selenium.chrome.ChromeDriverService; +import org.openqa.selenium.firefox.FirefoxDriver; import org.openqa.selenium.htmlunit.HtmlUnitDriver; import org.openqa.selenium.remote.DesiredCapabilities; import org.openqa.selenium.remote.RemoteWebDriver;

1 0

r3477 - trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its
by ymartel＠users.chorem.org 15 Jun '12

15 Jun '12

Author: ymartel Date: 2012-06-15 09:29:58 +0200 (Fri, 15 Jun 2012) New Revision: 3477 Url: http://chorem.org/repositories/revision/pollen/3477 Log: fix htmlunit usage Modified: trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/PollenBaseWebDriverIT.java Modified: trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/PollenBaseWebDriverIT.java =================================================================== --- trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/PollenBaseWebDriverIT.java 2012-06-14 16:16:34 UTC (rev 3476) +++ trunk/pollen-ui-struts2/src/test/java/org/chorem/pollen/ui/its/PollenBaseWebDriverIT.java 2012-06-15 07:29:58 UTC (rev 3477) @@ -40,7 +40,6 @@ import org.openqa.selenium.WebElement; import org.openqa.selenium.chrome.ChromeDriver; import org.openqa.selenium.chrome.ChromeDriverService; -import org.openqa.selenium.firefox.FirefoxDriver; import org.openqa.selenium.htmlunit.HtmlUnitDriver; import org.openqa.selenium.remote.DesiredCapabilities; import org.openqa.selenium.remote.RemoteWebDriver; @@ -291,7 +290,6 @@ } if (HtmlUnitDriver.class == driverType) { - result = new HtmlUnitDriver(); ((HtmlUnitDriver) result).setJavascriptEnabled(true); } return result;

1 0