September 2009 - Isis-fish-commits - mailman.cloud.codelutin.com

r2574 - isis-fish/trunk/src/test/java/fr/ifremer/isisfish/util/ssh
by chatellier＠users.labs.libre-entreprise.org 02 Sep '09

02 Sep '09

Author: chatellier Date: 2009-09-02 13:28:25 +0000 (Wed, 02 Sep 2009) New Revision: 2574 Modified: isis-fish/trunk/src/test/java/fr/ifremer/isisfish/util/ssh/SSHAgentTest.java Log: Fix junit package Modified: isis-fish/trunk/src/test/java/fr/ifremer/isisfish/util/ssh/SSHAgentTest.java =================================================================== --- isis-fish/trunk/src/test/java/fr/ifremer/isisfish/util/ssh/SSHAgentTest.java 2009-09-02 12:29:08 UTC (rev 2573) +++ isis-fish/trunk/src/test/java/fr/ifremer/isisfish/util/ssh/SSHAgentTest.java 2009-09-02 13:28:25 UTC (rev 2574) @@ -22,8 +22,7 @@ import java.net.URISyntaxException; import java.net.URL; -import junit.framework.Assert; - +import org.junit.Assert; import org.junit.BeforeClass; import org.junit.Test;

1 0

r2573 - in isis-fish/trunk/src/test: java/fr/ifremer/isisfish/util/ssh resources resources/fr/ifremer/isisfish resources/fr/ifremer/isisfish/util resources/fr/ifremer/isisfish/util/ssh
by chatellier＠users.labs.libre-entreprise.org 02 Sep '09

02 Sep '09

Author: chatellier Date: 2009-09-02 12:29:08 +0000 (Wed, 02 Sep 2009) New Revision: 2573 Added: isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/util/ isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/util/ssh/ isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/util/ssh/isistestkey isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/util/ssh/isistestkey.pub isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/util/ssh/isistestkey.txt Removed: isis-fish/trunk/src/test/resources/ssh/ Modified: isis-fish/trunk/src/test/java/fr/ifremer/isisfish/util/ssh/SSHAgentTest.java Log: Load private key in classpath. Modified: isis-fish/trunk/src/test/java/fr/ifremer/isisfish/util/ssh/SSHAgentTest.java =================================================================== --- isis-fish/trunk/src/test/java/fr/ifremer/isisfish/util/ssh/SSHAgentTest.java 2009-09-02 11:04:38 UTC (rev 2572) +++ isis-fish/trunk/src/test/java/fr/ifremer/isisfish/util/ssh/SSHAgentTest.java 2009-09-02 12:29:08 UTC (rev 2573) @@ -20,6 +20,7 @@ import java.io.File; import java.net.URISyntaxException; +import java.net.URL; import junit.framework.Assert; @@ -51,7 +52,8 @@ */ @BeforeClass public static void init() throws URISyntaxException { - keyFile = new File("src/test/resources/ssh/isistestkey"); + URL keyURL = SSHAgentTest.class.getResource("isistestkey"); + keyFile = new File(keyURL.getPath()); } /** Copied: isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/util/ssh/isistestkey (from rev 2550, isis-fish/trunk/src/test/resources/ssh/isistestkey) =================================================================== --- isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/util/ssh/isistestkey (rev 0) +++ isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/util/ssh/isistestkey 2009-09-02 12:29:08 UTC (rev 2573) @@ -0,0 +1,30 @@ +-----BEGIN RSA PRIVATE KEY----- +Proc-Type: 4,ENCRYPTED +DEK-Info: DES-EDE3-CBC,C78258F97D4516F9 + +31azf6UcEu1zvBpQE/nZuQLR7F97paRe3AvJcfHGlz8bqQd5JP8axRA0RzbKA22+ +C3p8Sk980/oNPDnceR7fGfRWrypzsSaGkV1BsDK2wkcWgpdJIomQVjEqKmNO1jkZ +wKHSE1rAQq1PtJLVoXEwOwpOnH2LRjuiExyIqZePciyMCd/QJ5orU+i4mCZErx3F +1Pdz8VeDLEZRmsfbZh/OKqrZ7z9zRMtpIS9oXgw1sRH303AmU9/2hBflnhFddQFx +d2bjAzAL/5rBIsBuwVJfcfRENWVlJcGJBmeiC8jW7GiBKNv8syPHq3DVg1JbnGWK +0mYRzOcQlj9JOh/C3f2inU2agIJBPnpUqEmhBhXGziBVBwf7ulApjTKyFt2KZ5Tw +aIvrm5LASrUaEDeIx0wCuAyAvgJ0CkcB5BwzTImICqjmKuLnMqV0lUUU4ZmZkDkn +JPwaceaqwHxfHomOhUpESTSrEd6hIoAmBKi++jRWFKzEod1KlK7HL3KNhvnlHyiY +G4i2eIbi7D76iQipynrHtOwdKswqWsz7+YifY5gY/HnJZl4AGGTNG+RXJnoWmMzp +BCXBNbUDRWNZSVgKtcMKBV+0q4REp/EexdpaSNdmmmusMKi/41xNn68pfjGqqNgh +XHHe93lMBdRzwhMbjZzMnGY3mcFuw+BjL3v6L1etz6j/z2mDLGqaWmHbjmWBRxni +y+RdOoX/uUL9hY/aisX7Zm4H6rVNVgxPC7rRPzFu0Oaf5iV3Qbdp5YkMbZCZlQFD +S/g+UOtpx7gYLMWiuqd6CljMgiYVVY/jmMbtL44cKCQbqqfncAmnrGuKMJ/uJR46 +Hw4uSv1i4fZKxb6LhFkQRjkY+yED7a9Mz3yddqeT79DsacM4ctNtnp07LWXaufKt +FXVnBXA0YgjlJDEtDaXvJfXfoV0GjqhogOS+MyYyR0fAhbD+8MOH0wrdCN6ov1PV ++ZdKOgKgTzLg3Aeh+Bkar04W+8YnO4c12IVMe2Iv0IxUZnYPmKBy1UK9PO7qDH3G +GQJrjxcXwFR8CaOzLGNVMvOdLIAzlxrunpTiDOO76EB4uoZCRa4NC9p9aMC0KJ5W +FuSkdpypu3O4JtC+bs2+rjbn9kGzV1rsmTQKs5ZpX7elJZLBVTw6E3it6qCsXv7e +Ak2B6zXd1OiwMYbWdtKGkp70UFprWbha8N46DL2LgSay4OXJLSN8Ha0+Z6vUE8dE +4Qe4oqUGjgE9QnYRpN/sCddJKQMtNjlqkHFmNNo2uIXVZGpueLn5dLNXbOmzI5a2 +R42HIxGMiRbOLc/0X2Dl9S2PU27w2n0woFHI7C1siO8jfHcYrUJWnfTng+/wY0jC +W+qdSsBjjbOzSwalZV7r+xUSjoVe6MemhwYM4Tg7Z/kVGIWt7F2q3GiOHLiYPNdw +9lWQtn8yTZcLUIbwWPWkPZ0vNYUPQcd7z86ysiUtEEZDWVA98fkA8nRWP1qKbO3D +7V9G1YD3KUx8l88kFCKAgt8N+pB9Co8L1P58FioxsXMXbSBlnrf4za80IK5CZ2IR +C2aU6FimsuTnP19cVSj4sUdHr7zYdqJ0JQHlYn/FGb1cp+wPWNvJOw== +-----END RSA PRIVATE KEY----- Copied: isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/util/ssh/isistestkey.pub (from rev 2550, isis-fish/trunk/src/test/resources/ssh/isistestkey.pub) =================================================================== --- isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/util/ssh/isistestkey.pub (rev 0) +++ isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/util/ssh/isistestkey.pub 2009-09-02 12:29:08 UTC (rev 2573) @@ -0,0 +1 @@ +ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAzQdBma9PwFdIx+GZg/bOlgjFINc9+iv5pe7HmD+v+RuJ7Miw7MBnttpB2mg7Q+p1V/2V2h9QX1QiRC78/JQptkQrqItpnifQduwL7p53oLC57DQXdLdg5waeqp6yhifFosTpvkhNmhni8A8GoJpd8YIAdExttOe2vboBB4NBjlj6ujqXBBwIGNQuGJ+yCN9wiitwawzCPtAn7HXfGSjWSH+vxo75DcT6BaLi2Rwni7CjHQZUoNQMK1aTL0Wn5EvqOW/vHaWQ4ARKxuuVOCwlYMaJxk3orWl90ZZuOtEHkwxTeXEnBS2JnHbiVFTj3R63rr23UofSC32tsz3+YiFTFQ== chatellier at genux Copied: isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/util/ssh/isistestkey.txt (from rev 2550, isis-fish/trunk/src/test/resources/ssh/isistestkey.txt) =================================================================== --- isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/util/ssh/isistestkey.txt (rev 0) +++ isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/util/ssh/isistestkey.txt 2009-09-02 12:29:08 UTC (rev 2573) @@ -0,0 +1,3 @@ +Clé ssh pour les tests unitaire de ISIS. + +La passphrase est "isispassphrase".

1 0

r2572 - in isis-fish/trunk/src: main/java/fr/ifremer/isisfish/datastore test/java/fr/ifremer/isisfish/datastore test/resources test/resources/fr test/resources/fr/ifremer test/resources/fr/ifremer/isisfish test/resources/fr/ifremer/isisfish/datastore
by chatellier＠users.labs.libre-entreprise.org 02 Sep '09

02 Sep '09

Author: chatellier Date: 2009-09-02 11:04:38 +0000 (Wed, 02 Sep 2009) New Revision: 2572 Added: isis-fish/trunk/src/test/resources/fr/ isis-fish/trunk/src/test/resources/fr/ifremer/ isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/ isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/datastore/ isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/datastore/exception.information isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/datastore/information Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulationInformation.java isis-fish/trunk/src/test/java/fr/ifremer/isisfish/datastore/SimulationInformationTest.java Log: Ajout de test sur un format de fichier existant. Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulationInformation.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulationInformation.java 2009-09-02 10:42:23 UTC (rev 2571) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulationInformation.java 2009-09-02 11:04:38 UTC (rev 2572) @@ -41,7 +41,7 @@ * Cette classe permet de conserver des informations sur le deroulement d'une * simulation. La plupart des informations sont automatiquement renseignees, * mais l'utilisateur peut lui aussi ajouter des informations avec la methode - * {@link #addInformation(String)} + * {@link #addInformation(String)}. * * @author poussin * @version $Revision$ @@ -307,7 +307,7 @@ * * @return other information */ - public String getInformation() { + protected String getInformation() { String result = info.getProperty(OTHER_INFO); if (result == null) { result = ""; @@ -315,7 +315,12 @@ return result; } - public void addInformation(String value) { - setInfo(OTHER_INFO, getInformation() + value + "\n"); + /** + * Add additional simulation information. + * + * @param info new info + */ + public void addInformation(String info) { + setInfo(OTHER_INFO, getInformation() + info + "\n"); } } Modified: isis-fish/trunk/src/test/java/fr/ifremer/isisfish/datastore/SimulationInformationTest.java =================================================================== --- isis-fish/trunk/src/test/java/fr/ifremer/isisfish/datastore/SimulationInformationTest.java 2009-09-02 10:42:23 UTC (rev 2571) +++ isis-fish/trunk/src/test/java/fr/ifremer/isisfish/datastore/SimulationInformationTest.java 2009-09-02 11:04:38 UTC (rev 2572) @@ -20,6 +20,7 @@ import java.io.File; import java.io.IOException; +import java.net.URL; import java.text.DateFormat; import java.text.ParseException; import java.text.SimpleDateFormat; @@ -62,6 +63,47 @@ } /** + * Test que le format du fichier de simulation ne change pas. + * + * @throws ParseException + */ + @Test + public void testExistingSimulationFile() throws ParseException { + URL informationURL = SimulationInformationTest.class.getResource("information"); + File file = new File(informationURL.getPath()); + + Assert.assertTrue(file.exists()); + + // Do not use same format as {@link SimulationInformation#dateFormat} + DateFormat df = new SimpleDateFormat("yyyy/dd/MM hh:mm:ss"); + + SimulationInformation info = new SimulationInformation(file); + Assert.assertEquals(df.parse("2009/31/08 14:44:57"), info.getSimulationStart()); + Assert.assertEquals(df.parse("2009/31/08 14:48:00"), info.getSimulationEnd()); + Assert.assertEquals(1482, info.getExportTime("VesselMargin.java")); + Assert.assertFalse(info.hasError()); + Assert.assertTrue(info.getOptimizationUsage().indexOf("Cache used") > 0); + Assert.assertNull(info.getStatistic()); + } + + /** + * Test que le format du fichier de simulation ne change pas. + * + * Cas d'un fichier avec une erreur. + */ + @Test + public void testExistingSimulationFileException() { + URL informationURL = SimulationInformationTest.class.getResource("exception.information"); + File file = new File(informationURL.getPath()); + + Assert.assertTrue(file.exists()); + + SimulationInformation info = new SimulationInformation(file); + Assert.assertTrue(info.hasError()); + Assert.assertTrue(info.getException().indexOf("TopiaException") > 0); + } + + /** * Test simulation date storage. * * @throws IOException Added: isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/datastore/exception.information =================================================================== --- isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/datastore/exception.information (rev 0) +++ isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/datastore/exception.information 2009-09-02 11:04:38 UTC (rev 2572) @@ -0,0 +1,4 @@ +#Simulation Information +#Mon Aug 31 16:02:56 CEST 2009 +exception=org.codelutin.topia.TopiaException\: Error during commit\n\tat org.codelutin.topia.framework.TopiaContextImpl.commitTransaction(TopiaContextImpl.java\:690)\n\tat simulators.DefaultSimulator.simulate(DefaultSimulator.java\:316)\n\tat fr.ifremer.isisfish.simulator.SimulationHelper.localSimulateSameThread(SimulationHelper.java\:475)\n\tat fr.ifremer.isisfish.simulator.SimulationHelper$SimThread.run(SimulationHelper.java\:385)\nCaused by\: org.hibernate.exception.GenericJDBCException\: could not insert\: [fr.ifremer.isisfish.entities.ResultImpl]\n\tat org.hibernate.exception.SQLStateConverter.handledNonSpecificException(SQLStateConverter.java\:103)\n\tat org.hibernate.exception.SQLStateConverter.convert(SQLStateConverter.java\:91)\n\tat org.hibernate.exception.JDBCExceptionHelper.convert(JDBCExceptionHelper.java\:43)\n\tat org.hibernate.persister.entity.AbstractEntityPersister.insert(AbstractEntityPersister.java\:2267)\n\tat org.hibernate.persister.entity.AbstractEntityPersister.insert(AbstractEntityPersister.java\:2660)\n\tat org.hibernate.action.EntityInsertAction.execute(EntityInsertAction.java\:56)\n\tat org.hibernate.engine.ActionQueue.execute(ActionQueue.java\:250)\n\tat org.hibernate.engine.ActionQueue.executeActions(ActionQueue.java\:234)\n\tat org.hibernate.engine.ActionQueue.executeActions(ActionQueue.java\:141)\n\tat org.hibernate.event.def.AbstractFlushingEventListener.performExecutions(AbstractFlushingEventListener.java\:298)\n\tat org.hibernate.event.def.DefaultFlushEventListener.onFlush(DefaultFlushEventListener.java\:27)\n\tat org.hibernate.impl.SessionImpl.flush(SessionImpl.java\:1000)\n\tat org.codelutin.topia.framework.TopiaContextImpl.commitTransaction(TopiaContextImpl.java\:675)\n\t... 3 more\nCaused by\: org.h2.jdbc.JdbcSQLException\: Database is already closed (to disable automatic closing at VM shutdown, add ";DB_CLOSE_ON_EXIT\=FALSE" to the db URL) [90121-60]\n\tat org.h2.message.Message.getSQLException(Message.java\:84)\n\tat org.h2.message.Message.getSQLException(Message.java\:88)\n\tat org.h2.message.Message.getSQLException(Message.java\:66)\n\tat org.h2.message.Message.getSQLException(Message.java\:109)\n\tat org.h2.jdbc.JdbcConnection.checkClosed(JdbcConnection.java\:1174)\n\tat org.h2.jdbc.JdbcConnection.prepareStatement(JdbcConnection.java\:163)\n\tat org.hibernate.jdbc.AbstractBatcher.getPreparedStatement(AbstractBatcher.java\:505)\n\tat org.hibernate.jdbc.AbstractBatcher.prepareStatement(AbstractBatcher.java\:94)\n\tat org.hibernate.jdbc.AbstractBatcher.prepareStatement(AbstractBatcher.java\:87)\n\tat org.hibernate.jdbc.AbstractBatcher.prepareBatchStatement(AbstractBatcher.java\:222)\n\tat org.hibernate.persister.entity.AbstractEntityPersister.insert(AbstractEntityPersister.java\:2224)\n\t... 12 more\n +simulationStart=2009.08.31 15\:37\:17 Added: isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/datastore/information =================================================================== --- isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/datastore/information (rev 0) +++ isis-fish/trunk/src/test/resources/fr/ifremer/isisfish/datastore/information 2009-09-02 11:04:38 UTC (rev 2572) @@ -0,0 +1,19 @@ +#Simulation Information +#Mon Aug 31 14:48:01 CEST 2009 +exportTime.VesselMargin.java=1482 +optimizationUsage=--- Cache Statistiques ---\nTotal call\: 915765\nCache used\: 527279\nCache usage\: 57%\n--------------------\n +exportTime.EffortsMetier.java=1783 +exportTime.ZonesDefinition.java=29 +exportTime.RejetsPoids.java=2 +exportTime.OwnerMargin.java=275 +exportTime.CapturesPoids.java=1645 +exportTime.RejetsNombre.java=98 +simulationStart=2009.08.31 14\:44\:57 +exportTime.CapturesNombre.java=368 +exportTime.RegionDefinition.java=2 +exportTime.CellsDefinition.java=11 +exportTime.Biomasses.java=1558 +exportTime.NonActivite.java=273 +simulationEnd=2009.08.31 14\:48\:00 +exportTime.All\ export=7839 +exportTime.Abundances.java=305

1 0

r2571 - isis-fish/trunk/src/test/java/fr/ifremer/isisfish/datastore
by chatellier＠users.labs.libre-entreprise.org 02 Sep '09

02 Sep '09

Author: chatellier Date: 2009-09-02 10:42:23 +0000 (Wed, 02 Sep 2009) New Revision: 2571 Added: isis-fish/trunk/src/test/java/fr/ifremer/isisfish/datastore/SimulationInformationTest.java Log: Add tests about simulation information Added: isis-fish/trunk/src/test/java/fr/ifremer/isisfish/datastore/SimulationInformationTest.java =================================================================== --- isis-fish/trunk/src/test/java/fr/ifremer/isisfish/datastore/SimulationInformationTest.java (rev 0) +++ isis-fish/trunk/src/test/java/fr/ifremer/isisfish/datastore/SimulationInformationTest.java 2009-09-02 10:42:23 UTC (rev 2571) @@ -0,0 +1,140 @@ +/* *##% + * Copyright (C) 2009 Ifremer, Code Lutin + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License + * as published by the Free Software Foundation; either version 2 + * 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 General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. + *##%*/ + +package fr.ifremer.isisfish.datastore; + +import java.io.File; +import java.io.IOException; +import java.text.DateFormat; +import java.text.ParseException; +import java.text.SimpleDateFormat; +import java.util.Date; + +import junit.framework.Assert; + +import org.junit.Test; + +/** + * Test for {@link SimulationInformation} class. + * + * No need for isolated directory (system directory not used). + * + * @author chatellier + * @version $Revision$ + * + * Last update : $Date$ + * By : $Author$ + */ +public class SimulationInformationTest { + + /** + * Test simulation information with no data. + * + * @throws IOException + */ + @Test + public void testSimulationEmpty() throws IOException { + + File file = File.createTempFile("information-empty", ".txt"); + file.deleteOnExit(); + + SimulationInformation info = new SimulationInformation(file); + Assert.assertEquals(new Date(0), info.getSimulationStart()); + Assert.assertEquals(new Date(0), info.getSimulationEnd()); + Assert.assertFalse(info.hasError()); + Assert.assertNull(info.getException()); + Assert.assertEquals(0, info.getExportTime("test")); + } + + /** + * Test simulation date storage. + * + * @throws IOException + * @throws ParseException + */ + @Test + public void testSimulationDate() throws IOException, ParseException { + + // Do not use same format as {@link SimulationInformation#dateFormat} + DateFormat df = new SimpleDateFormat("hh:mm:ss a"); + + File file = File.createTempFile("information-date", ".txt"); + file.deleteOnExit(); + + SimulationInformation info = new SimulationInformation(file); + + Date d1 = df.parse("3:30:32 pm"); + Date d2 = df.parse("3:50:47 pm"); + info.setSimulationStart(d1); + info.setSimulationEnd(d2); + + info.store(); + + SimulationInformation testInfo = new SimulationInformation(file); + Assert.assertEquals(d1, testInfo.getSimulationStart()); + Assert.assertEquals(d2, testInfo.getSimulationEnd()); + } + + /** + * Test simulation exception storage. + * + * @throws IOException + */ + @Test + public void testSimulationException() throws IOException { + File file = File.createTempFile("information-exception", ".txt"); + file.deleteOnExit(); + + SimulationInformation info = new SimulationInformation(file); + Assert.assertFalse(info.hasError()); + + Exception e = new Exception("Oula la, ya eu une exception super grave"); + info.setException(e); + info.store(); + + SimulationInformation testInfo = new SimulationInformation(file); + Assert.assertTrue(testInfo.hasError()); + Assert.assertTrue(testInfo.getException().indexOf("grave") > 0); + + } + + /** + * Test simulation export time. + * @throws IOException + */ + @Test + public void testSimulationExportTime() throws IOException { + File file = File.createTempFile("information-export", ".txt"); + file.deleteOnExit(); + + SimulationInformation info = new SimulationInformation(file); + info.addExportTime("export1", 30); + info.addExportTime("export2", 40); + info.addExportTime("export3", 50); + info.addExportTime("export4", 60); + + info.store(); + + SimulationInformation testInfo = new SimulationInformation(file); + Assert.assertFalse(testInfo.hasError()); + Assert.assertEquals(30, testInfo.getExportTime("export1")); + Assert.assertEquals(40, testInfo.getExportTime("export2")); + Assert.assertEquals(50, testInfo.getExportTime("export3")); + Assert.assertEquals(60, testInfo.getExportTime("export4")); + } +} Property changes on: isis-fish/trunk/src/test/java/fr/ifremer/isisfish/datastore/SimulationInformationTest.java ___________________________________________________________________ Added: svn:keywords + "Author Date Id Revision HeadURL"

1 0

r2570 - isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore
by chatellier＠users.labs.libre-entreprise.org 02 Sep '09

02 Sep '09

Author: chatellier Date: 2009-09-02 10:39:54 +0000 (Wed, 02 Sep 2009) New Revision: 2570 Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulationInformation.java Log: Rename "getInfomation" > "getInformation" Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulationInformation.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulationInformation.java 2009-09-02 09:13:35 UTC (rev 2569) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/SimulationInformation.java 2009-09-02 10:39:54 UTC (rev 2570) @@ -51,24 +51,32 @@ */ public class SimulationInformation { - /** to use log facility, just put in your code: log.info(\"...\"); */ - static private Log log = LogFactory.getLog(SimulationInformation.class); + /** Class logger. */ + private static Log log = LogFactory.getLog(SimulationInformation.class); - static private final String START_SIMULATION = "simulationStart"; - static private final String END_SIMULATION = "simulationEnd"; - static private final String EXPORT_TIME = "exportTime"; - static private final String OTHER_INFO = "otherInfo"; - static private final String STATISTIC = "statistic"; - static private final String OPTIMIZATION_USAGE = "optimizationUsage"; - static private final String SIMULATION_EXCEPTION = "exception"; + private static final String START_SIMULATION = "simulationStart"; + private static final String END_SIMULATION = "simulationEnd"; + private static final String EXPORT_TIME = "exportTime"; + private static final String RULE_TIME = "ruleTime"; + private static final String OTHER_INFO = "otherInfo"; + private static final String STATISTIC = "statistic"; + private static final String OPTIMIZATION_USAGE = "optimizationUsage"; + private static final String SIMULATION_EXCEPTION = "exception"; - static private final SimpleDateFormat dateFormat = new SimpleDateFormat("yyyy.MM.dd HH:mm:ss"); + private static final String RULE_TIME_INIT = "init"; + private static final String RULE_TIME_PRE = "pre"; + private static final String RULE_TIME_POST = "post"; + private static final SimpleDateFormat dateFormat = new SimpleDateFormat( + "yyyy.MM.dd HH:mm:ss"); + protected Properties info = new Properties(); protected File file = null; - + /** + * Constructor. * + * @param file simulation information output file */ public SimulationInformation(File file) { this.file = file; @@ -79,32 +87,38 @@ reader.close(); } catch (IOException eee) { if (log.isWarnEnabled()) { - log.warn(_("isisfish.error.read.simulation", file.getPath()), eee); + log.warn(_("isisfish.error.read.simulation", file + .getPath()), eee); } } } } - + /* * @see java.lang.Object#toString() */ @Override public String toString() { String result = "Simulation Information:\n"; - result += "Start: " + dateFormat.format(getSimulationStart()) + " End: " + dateFormat.format(getSimulationEnd()) + "\n"; + result += "Start: " + dateFormat.format(getSimulationStart()) + + " End: " + dateFormat.format(getSimulationEnd()) + "\n"; Map<String, Long> exportTime = getExportTime(); if (exportTime.size() > 0) { result += "Export time:\n"; for (Map.Entry<String, Long> entry : exportTime.entrySet()) { - result += "\t" + entry.getKey() + " : " + DurationFormatUtils.formatDuration(entry.getValue(), "s'.'S") + "\n"; + result += "\t" + + entry.getKey() + + " : " + + DurationFormatUtils.formatDuration(entry.getValue(), + "s'.'S") + "\n"; } } - - String info = getInfomation(); + + String info = getInformation(); if (info != null && !"".equals(info)) { result += "Information:\n" + info + "\n"; } - + String v = getStatistic(); if (v != null) { result += "Statistic:\n" + v + "\n"; @@ -120,7 +134,7 @@ return result; } - + protected void store() { try { FileWriter writer = new FileWriter(file); @@ -128,19 +142,21 @@ writer.close(); } catch (IOException eee) { if (log.isWarnEnabled()) { - log.warn(_("isisfish.error.write.simulation", file.getPath()), eee); + log.warn(_("isisfish.error.write.simulation", file.getPath()), + eee); } } } - + protected void setInfo(String key, String value) { info.setProperty(key, value); store(); } - + /** - * Get the date of simulation start - * @return + * Get the date of simulation start. + * + * @return simulation start date */ public Date getSimulationStart() { String d = info.getProperty(START_SIMULATION); @@ -159,11 +175,11 @@ } return result; } - + public void setSimulationStart(Date date) { setInfo(START_SIMULATION, dateFormat.format(date)); } - + /** * Get the date of simulation start * @return @@ -185,11 +201,11 @@ } return result; } - + public void setSimulationEnd(Date date) { setInfo(END_SIMULATION, dateFormat.format(date)); } - + /** * get all export time in map * @return @@ -218,7 +234,7 @@ } return result; } - + public long getExportTime(String exportName) { String t = info.getProperty(EXPORT_TIME + "." + exportName); long result = 0; @@ -233,56 +249,73 @@ } return result; } - + public void addExportTime(String exportName, long time) { setInfo(EXPORT_TIME + "." + exportName, String.valueOf(time)); } - + public String getStatistic() { String result = info.getProperty(STATISTIC); return result; - } - + } + public void setStatistic(String v) { setInfo(STATISTIC, v); } - + public String getOptimizationUsage() { String result = info.getProperty(OPTIMIZATION_USAGE); return result; } - + public void setOptimizationUsage(String v) { - setInfo(OPTIMIZATION_USAGE, v); + setInfo(OPTIMIZATION_USAGE, v); } - + public String getException() { String result = info.getProperty(SIMULATION_EXCEPTION); return result; } - + public void setException(Throwable eee) { StringWriter w = new StringWriter(); PrintWriter pw = new PrintWriter(w); eee.printStackTrace(pw); String v = w.getBuffer().toString(); - setInfo(SIMULATION_EXCEPTION, v); + setInfo(SIMULATION_EXCEPTION, v); } - + + /** + * Return {@code true} if an exception has been set. + * + * @return {@code true} if there is an exception + */ public boolean hasError() { boolean result = getException() != null && getException().length() > 0; return result; } - + + /** + * @deprecated since 3.2.0.5, use {@link #getInformation()} instead + */ public String getInfomation() { + return getInformation(); + } + + /** + * Get other information. + * + * @return other information + */ + public String getInformation() { String result = info.getProperty(OTHER_INFO); if (result == null) { result = ""; } return result; } - + public void addInformation(String value) { - setInfo(OTHER_INFO, getInfomation() + value + "\n"); + setInfo(OTHER_INFO, getInformation() + value + "\n"); } }

1 0

r2569 - isis-fish/trunk/src/test/java/fr/ifremer/isisfish
by chatellier＠users.labs.libre-entreprise.org 02 Sep '09

02 Sep '09

Author: chatellier Date: 2009-09-02 09:13:35 +0000 (Wed, 02 Sep 2009) New Revision: 2569 Modified: isis-fish/trunk/src/test/java/fr/ifremer/isisfish/AbstractIsisFishTest.java Log: Add doc about abstract test framework. Modified: isis-fish/trunk/src/test/java/fr/ifremer/isisfish/AbstractIsisFishTest.java =================================================================== --- isis-fish/trunk/src/test/java/fr/ifremer/isisfish/AbstractIsisFishTest.java 2009-09-02 08:58:52 UTC (rev 2568) +++ isis-fish/trunk/src/test/java/fr/ifremer/isisfish/AbstractIsisFishTest.java 2009-09-02 09:13:35 UTC (rev 2569) @@ -32,9 +32,12 @@ import freemarker.template.Configuration; /** - * Abtract test case for isis fish. + * Abstract test case for isis fish. * * Contains BeforeClass and AfterClass method to proper init cases. + * + * Each test is done in a isolated isis-database-3 directory, and user.home + * is set to this directory. * * @author chatellier * @version $Revision: 1.0 $

1 0

r2568 - isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule
by chatellier＠users.labs.libre-entreprise.org 02 Sep '09

02 Sep '09

Author: chatellier Date: 2009-09-02 08:58:52 +0000 (Wed, 02 Sep 2009) New Revision: 2568 Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule/AbstractRule.java Log: Update rule documentation. Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule/AbstractRule.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule/AbstractRule.java 2009-09-02 08:55:01 UTC (rev 2567) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule/AbstractRule.java 2009-09-02 08:58:52 UTC (rev 2568) @@ -23,9 +23,13 @@ import java.util.Map; /** - * Les parametres d'une regle sont tous les attributs prefixé de param, mais - * ce prefix n'apparait que dans le code source. + * Class abstraite d'une regles comprenant le code commun à toutes les regles. * + * Permet de : + * <ul> + * <li>Gerer les valeurs entre les regles + * </ul> + * * Created: 12 janv. 2006 17:12:51 * * @author poussin

1 0

r2567 - isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule
by chatellier＠users.labs.libre-entreprise.org 02 Sep '09

02 Sep '09

Author: chatellier Date: 2009-09-02 08:55:01 +0000 (Wed, 02 Sep 2009) New Revision: 2567 Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule/AbstractRule.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule/Rule.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule/RuleHelper.java Log: Update rule documentation. Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule/AbstractRule.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule/AbstractRule.java 2009-09-02 08:23:42 UTC (rev 2566) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule/AbstractRule.java 2009-09-02 08:55:01 UTC (rev 2567) @@ -36,13 +36,13 @@ */ public abstract class AbstractRule implements Rule { - protected transient Map<String, Object> values = new HashMap<String, Object>(); - + protected transient Map<String, Object> values = new HashMap<String, Object>(); + /* * @see fr.ifremer.isisfish.rule.Rule#getValue(java.lang.String) */ @Override - public Object getValue(String name) { + public Object getValue(String name) { return values.get(name); } @@ -53,7 +53,4 @@ public void setValue(String name, Object value) { values.put(name, value); } - } - - Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule/Rule.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule/Rule.java 2009-09-02 08:23:42 UTC (rev 2566) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule/Rule.java 2009-09-02 08:55:01 UTC (rev 2567) @@ -23,7 +23,6 @@ import fr.ifremer.isisfish.simulator.SimulationContext; import fr.ifremer.isisfish.types.Date; - /** * Interface d'une regle. setValue et getValue permet de stocker des * informations liées a l'instance de la règle. @@ -37,7 +36,7 @@ * <li> une chaine String * <li> un objet Date ({@link fr.ifremer.isisfish.types.Date}) * <li> un objet Month ({@link fr.ifremer.isisfish.types.Month}) - * <li> Le type d'une entité ({@link fr.ifremer.isisfish.entities}) + * <li> Le type d'une entité (@see fr.ifremer.isisfish.entities) * * <p> Pour supporter d'autre type il faut créer des converters commons-beans * //@link fr.ifremer.isisfish.simulator.SimulationParameter#ruleParametertoProperties(Rule) @@ -51,70 +50,85 @@ * by : $Author$ */ public interface Rule { - + /** + * Return necessary results names for rule execution. + * * @return the necessaryResult name */ public String[] getNecessaryResult(); - + /** * Permet d'afficher a l'utilisateur une aide sur la regle. + * * @return L'aide ou la description de la regle * @throws Exception */ public String getDescription() throws Exception; - + /** * Appelé au démarrage de la simulation, cette méthode permet d'initialiser - * des valeurs + * la règle. + * * @param context La simulation pour lequel on utilise cette regle * @throws Exception */ public void init(SimulationContext context) throws Exception; /** - * La condition qui doit etre vrai pour faire les actions + * La condition qui doit etre vrai pour faire les actions. + * + * Si la confition return vrai alors les methodes + * {@link #preAction(SimulationContext, Date, Metier)} et + * {@link #postAction(SimulationContext, Date, Metier)} seront appelée. + * * @param context La simulation pour lequel on utilise cette regle * @param date la date courante * @param metier le metier * @return vrai si on souhaite que les actions soit faites * @throws Exception s'il y a une erreur, met fin a la simulation */ - public boolean condition(SimulationContext context, Date date, Metier metier) throws Exception; + public boolean condition(SimulationContext context, Date date, Metier metier) + throws Exception; /** * Si la condition est vrai alors cette action est executée avant le pas * de temps de la simulation. + * * @param context La simulation pour lequel on utilise cette regle * @param date la date courant * @param metier le metier * @throws Exception s'il y a une erreur, met fin a la simulation */ - public void preAction(SimulationContext context, Date date, Metier metier) throws Exception; + public void preAction(SimulationContext context, Date date, Metier metier) + throws Exception; /** * Si la condition est vrai alors cette action est executée apres le pas * de temps de la simulation. + * * @param context La simulation pour lequel on utilise cette regle * @param date la date courante * @param metier le metier * @throws Exception s'il y a une erreur, met fin a la simulation */ - public void postAction(SimulationContext context, Date date, Metier metier) throws Exception; + public void postAction(SimulationContext context, Date date, Metier metier) + throws Exception; /** - * Permet de recuperer une valeur prealablement stocker avec un setValue + * Permet de recuperer une valeur prealablement stockée avec un + * {@link #setValue(String, Object)}. + * * @param name le nom de la valeur souhaitée * @return la valeur ou null si aucune valeur ne porte se nom */ public Object getValue(String name); - + /** - * Permet de stocker une valeur en fonction d'une cle + * Permet de stocker une valeur en fonction d'une cle. + * * @param name le nom de la valeur * @param value la valeur */ public void setValue(String name, Object value); } - - Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule/RuleHelper.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule/RuleHelper.java 2009-09-02 08:23:42 UTC (rev 2566) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/rule/RuleHelper.java 2009-09-02 08:55:01 UTC (rev 2567) @@ -32,10 +32,14 @@ import fr.ifremer.isisfish.datastore.RuleStorage; import fr.ifremer.isisfish.util.ConverterUtil; - /** - * RuleHelper. + * Helper pour effectuer des manipulations sur les règles. * + * Permet de : + * <ul> + * <li>affecter les valeurs aux parametres des règles + * </ul> + * * Created: 6 juin 07 12:03:42 * * @author poussin @@ -46,9 +50,7 @@ */ public class RuleHelper { - /** - * Logger for this class - */ + /** Logger for this class. */ private static final Log log = LogFactory.getLog(RuleHelper.class); /** @@ -67,7 +69,7 @@ public static void populateRule(int ruleIndex, RegionStorage region, Rule rule, Properties props) { populateRule(ruleIndex, region.getStorage(), rule, props); } - + /** * Recupere dans propriétés les valeurs des champs spécifiques à la règle et * met à jour les champs de la règle. @@ -91,17 +93,26 @@ String valueString = "not initialized"; try { valueString = props.getProperty(paramTag + propName); - log.info("DEBUG: about to convert rule param: " + paramTag + propName + " = "+ valueString); - // FIXME il semblerait que notre intance de beanUtils - // soit ecrasé par celle du MatrixType ... !!!! $$$ - ConvertUtilsBean beanUtils = ConverterUtil.getConverter(context); + + if (log.isDebugEnabled()) { + log.debug("About to convert rule param: " + paramTag + propName + " = " + valueString); + } + + // ATTENTION il semblerait que notre intance de beanUtils + // soit ecrasé par celle du MatrixType ... + // a ne pas deplacer avant la boucle + ConvertUtilsBean beanUtils = ConverterUtil + .getConverter(context); Object value = beanUtils.convert(valueString, type); - log.info("DEBUG: set rule param: " + paramTag + propName + " = " +value+"("+ valueString+ ")"); + + if (log.isDebugEnabled()) { + log.debug("Set rule param: " + paramTag + propName + " = " + value + "(" + valueString + ")"); + } RuleStorage.setParameterValue(rule, propName, value); } catch (Exception eee) { if (log.isWarnEnabled()) { log.info("Properties: " + props); - log.warn("Can't reload field " + propName + " for rule " + ruleName + " with value " + valueString , eee); + log.warn("Can't reload field " + propName + " for rule " + ruleName + " with value " + valueString, eee); } } }

1 0

r2566 - isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore
by chatellier＠users.labs.libre-entreprise.org 02 Sep '09

02 Sep '09

Author: chatellier Date: 2009-09-02 08:23:42 +0000 (Wed, 02 Sep 2009) New Revision: 2566 Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/VersionStorage.java Log: Add add and remove methods. Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/VersionStorage.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/VersionStorage.java 2009-09-02 08:15:23 UTC (rev 2565) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/VersionStorage.java 2009-09-02 08:23:42 UTC (rev 2566) @@ -194,7 +194,9 @@ // we can sure remove module directory and root directory files.remove(IsisFish.config.getDatabaseDirectory()); - log.debug("files to add: " + files); + if (log.isDebugEnabled()) { + log.debug("files to add: " + files); + } getVCS().add(files, msg); } @@ -307,7 +309,9 @@ try { result = getVCS().getFileList(directory); } catch (VCSException e) { - log.warn("Error during connection to VCS serveur", e); + if (log.isWarnEnabled()) { + log.warn("Error during connection to VCS server", e); + } result = new ArrayList<String>(); } Collections.sort(result); @@ -321,7 +325,7 @@ * * @param event event to fire */ - protected void fireDataChanged(Event event) { + protected static void fireDataChanged(Event event) { try { changeListeners.fire("dataChanged", event); } catch (Exception e) { @@ -330,4 +334,22 @@ } } } + + /** + * Add listener to be notified on change to storage. + * + * @param listener listener to add + */ + public static void addStorageChangeListener(StorageChangeListener listener) { + changeListeners.add(listener); + } + + /** + * Remove listener for storage change notification. + * + * @param listener listener to remove + */ + public static void removeStorageChangeListener(StorageChangeListener listener) { + changeListeners.remove(listener); + } }

1 0

r2565 - in isis-fish/trunk/src/main/java/fr/ifremer/isisfish: datastore vcs
by chatellier＠users.labs.libre-entreprise.org 02 Sep '09

02 Sep '09

Author: chatellier Date: 2009-09-02 08:15:23 +0000 (Wed, 02 Sep 2009) New Revision: 2565 Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/CodeSourceStorage.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/DataStorage.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/RegionStorage.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/VersionStorage.java isis-fish/trunk/src/main/java/fr/ifremer/isisfish/vcs/VCS.java Log: Fix doc & comments. Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/CodeSourceStorage.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/CodeSourceStorage.java 2009-09-02 07:43:03 UTC (rev 2564) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/CodeSourceStorage.java 2009-09-02 08:15:23 UTC (rev 2565) @@ -47,26 +47,27 @@ public abstract class CodeSourceStorage extends VersionStorage { /** - * Le nom de la classe sans le package + * Le nom de la classe sans le package. */ protected String name = null; /** - * le contenu du fichier + * Le contenu du fichier. */ protected String content = null; /** - * date de derniere mise a jour de content, sert a detecter si content doit etre relu + * Date de derniere mise a jour de content, sert a detecter si content doit etre relu. */ protected long lastContentUpdate = 0; + /** - * la derniere taille connu du content, sert a detecter si content doit etre relu + * La derniere taille connu du content, sert a detecter si content doit etre relu. */ protected long lastContentLength = 0; /** - * Contruit un nouveau rule storage + * Contruit un nouveau rule storage. * * @param rootSrc repertoire racine des sources * @param directory le repertoire ou devrait se trouver la classe. Ce @@ -82,31 +83,34 @@ this.name = name.endsWith(suffix) ? name : name + suffix; } - /* (non-Javadoc) - * @see fr.ifremer.isisfish.datastore.VersionStorage#prepare() - */ + /* + * @see fr.ifremer.isisfish.datastore.VersionStorage#prepare() + */ @Override protected void prepare() { // Il n'y a rien a faire pour les code sources } /** - * Check if script exists - * @return + * Check if script exists. + * + * @return {@code true} if storage file exists */ public boolean exists() { return getFile().exists(); } /** - * @return Returns the name. + * Get storage name. + * + * @return the name. */ public String getName() { return this.name; } /** - * Retourne le contenu du fichier, si le fichier reel est plus recent + * Retourne le contenu du fichier. Si le fichier reel est plus recent * que la derniere lecture, alors il est relu automatiquement. Ce * mecanisme de relecture automatique peut ne pas fonctionner si entre * la derniere lecture de content et la modification directement du fichier @@ -135,7 +139,9 @@ } /** - * @param content The content to set. + * Set file content. + * + * @param content new content * @throws IOException */ public void setContent(String content) throws IOException { @@ -147,14 +153,20 @@ } /** - * permet de forcer la relecture du fichier sur le disque + * Permet de forcer la relecture du fichier sur le disque. */ public void reload() { content = null; } + /** + * Return diff between current file content and VCS file content. + * + * @return diff as string + * @throws VCSException + * @throws IOException + */ public String diff() throws VCSException, IOException { return getVCS().getDiff(getFile()); } - } Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/DataStorage.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/DataStorage.java 2009-09-02 07:43:03 UTC (rev 2564) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/DataStorage.java 2009-09-02 08:15:23 UTC (rev 2565) @@ -53,17 +53,21 @@ */ public abstract class DataStorage extends VersionStorage { - static final public String DATA_BACKUP_FILENAME = "data-backup.sql.gz"; + /** Backup archive filename. */ + public static final String DATA_BACKUP_FILENAME = "data-backup.sql.gz"; /** Logger for this class */ private static final Log log = LogFactory.getLog(DataStorage.class); /** Le nom du storage (nom du dernier répertoire) */ protected String name = null; + /** Le TopiaContext contenant les données */ protected TopiaContext storage = null; + /** Le TopiaContext contenant les données en memoire */ protected TopiaContext memstorage = null; + /** le répertoire contenant les données sur le disque */ private File storageDirectory = null; @@ -327,8 +331,7 @@ /** * Cree un zip. Le zip contient les memes fichiers lorsqu'on utilise le CVS. * - * @param file - * le fichier dans lequel l'export doit se faire + * @param file le fichier dans lequel l'export doit se faire * @return l'argument file * @throws IOException si problème lors de la création du zip */ @@ -357,8 +360,6 @@ } /* - * (non-Javadoc) - * * @see java.lang.Object#finalize() */ @Override Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/RegionStorage.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/RegionStorage.java 2009-09-02 07:43:03 UTC (rev 2564) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/RegionStorage.java 2009-09-02 08:15:23 UTC (rev 2565) @@ -44,7 +44,7 @@ import java.util.Map; /** - * Classe permettant de recuperer les TopiaContext des regions. + * Classe permettant de recuperer les {@link TopiaContext} des regions. * * Created: 17 août 2005 03:44:57 CEST * @@ -56,16 +56,16 @@ */ public class RegionStorage extends DataStorage { // RegionStorage - static final public String REGION_PATH = "regions"; - static final public String MAP_DIRECTORY = "maps"; + public static final String REGION_PATH = "regions"; + public static final String MAP_DIRECTORY = "maps"; /** to use log facility, just put in your code: log.info(\"...\"); */ - static private Log log = LogFactory.getLog(RegionStorage.class); + private static Log log = LogFactory.getLog(RegionStorage.class); @SuppressWarnings("unchecked") - static protected Map<String, RegionStorage> regions = (Map<String, RegionStorage>) new ReferenceMap(); + protected static Map<String, RegionStorage> regions = (Map<String, RegionStorage>) new ReferenceMap(); - /** File to use to store next commit region comment */ + /** File to use to store next commit region comment. */ protected File commentForNextCommitFile = null; /** @@ -75,27 +75,27 @@ * @param name region name * * FIXME constructor is public for hack in - * fr.ifremer.isisfish.simulator.SimulationParameter.fromProperties(File, Properties) + * {@link fr.ifremer.isisfish.simulator.SimulationParameter#fromProperties(File, java.util.Properties)} * set protected when fix hack */ public RegionStorage(File directory, String name) { super(directory, name); commentForNextCommitFile = new File(getDirectory(), "commentForNextCommit.txt"); } - - static public File getRegionDirectory() { + + public static File getRegionDirectory() { File result = IsisFish.config.getDatabaseDirectory(); result = new File(result, REGION_PATH); result.mkdirs(); return result; } - static public File getRegionDirectory(String name) { + public static File getRegionDirectory(String name) { File result = getRegionDirectory(); result = new File(result, name); return result; } - + /** * @return Returns the commentForNextCommit. * @throws IOException if IO problem while reading file @@ -126,9 +126,9 @@ commentForNextCommitFile.renameTo(dest); } - /* (non-Javadoc) - * @see fr.ifremer.isisfish.datastore.DataStorage#isVersionnableAbleFile(java.io.File) - */ + /* + * @see fr.ifremer.isisfish.datastore.DataStorage#isVersionnableAbleFile(java.io.File) + */ @Override protected boolean isVersionnableAbleFile(File file) { boolean result = super.isVersionnableAbleFile(file); @@ -151,9 +151,9 @@ setCommentForNextCommit(comment); } - /* (non-Javadoc) - * @see fr.ifremer.isisfish.datastore.DataStorage#rename(java.lang.String) - */ + /* + * @see fr.ifremer.isisfish.datastore.DataStorage#rename(java.lang.String) + */ @Override public void rename(String toName) throws StorageException { try { @@ -460,4 +460,3 @@ } } // RegionStorage - Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/VersionStorage.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/VersionStorage.java 2009-09-02 07:43:03 UTC (rev 2564) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/datastore/VersionStorage.java 2009-09-02 08:15:23 UTC (rev 2565) @@ -39,7 +39,7 @@ import fr.ifremer.isisfish.vcs.VCSException; /** - * Classe permettant de géré l'interaction avec le cvs + * Classe permettant de géré l'interaction avec le VSC. * * @author poussin * @version $Revision$ @@ -100,7 +100,8 @@ * (ex pour Region) alors vrai meme si tous les fichiers du repertoire * ne sont pas sur le VCSNone * - * @return vrai si deja dans le VCSNone + * @return {@code true} si deja dans le VCS + * @throws VCSException */ public boolean isOnRemote() throws VCSException { return getVCS().isOnRemote(getFile()); @@ -108,11 +109,11 @@ /** * Permet de savoir si un fichier doit etre géré par le vcs ou non. - * L'implantation par defaut exclus les répertoires VCSNone, il faut + * L'implantation par defaut exclus les répertoires VCS, il faut * toujours appeler le super si on surcharge la methode. * * @param file le fichier a tester - * @return <code>true</code> si le fichier est versionné + * @return {@code true} si le fichier est versionné */ protected boolean isVersionnableAbleFile(File file) { @@ -127,14 +128,14 @@ // du storage boolean result = true; - if(getVCS().getLocalRepository().equals(this.file)) { + if (getVCS().getLocalRepository().equals(this.file)) { result = getVCS().isVersionnableAbleFile(file); } return result; } /** - * Donne la liste de tous les fichiers que le VCSNone doit gérer. + * Donne la liste de tous les fichiers que le VCS doit gérer. * Par defaut parcours tous les repertoires et ajouter tous les fichiers * et repertoire trouvé. Il est possible d'exclure des fichiers et/ou * repertoire en surchargeant {@link #isVersionnableAbleFile(File)} @@ -214,7 +215,7 @@ } } //TODO There is a bug to fix ? some files are not deleted! - if (getFile().isDirectory()){ + if (getFile().isDirectory()) { FileUtil.deleteRecursively(getFile()); } else { getFile().delete(); @@ -239,7 +240,6 @@ * a sur le VCS * * @throws VCSException si pb pendant l'op - * */ public void update() throws VCSException { prepare(); @@ -258,9 +258,8 @@ * @param destDir le repertoire parent * @param module le repertoire qui peut etre scripts ou exports * @throws VCSException si pb pendant l'opération - * */ - static public void checkout(File destDir, String module) throws VCSException { + public static void checkout(File destDir, String module) throws VCSException { // Si on utilise pas le bon tag on change de tag VersionNumber tag = IsisConfig.getApiVersion(); if (!getVCS().isTag(tag)) { @@ -278,7 +277,7 @@ * @param directory le répertoire dans lequel vie l'ensemble des storage * @return la liste des noms de toutes les storages disponible en local */ - static public List<String> getStorageNames(File directory) { + public static List<String> getStorageNames(File directory) { List<String> result = new ArrayList<String>(); if (directory.exists()) { @@ -303,7 +302,7 @@ * serveur VCSNone. Si le serveur n'est pas disponible la liste retournée * est vide. */ - static public List<String> getRemoteStorageNames(File directory) { + public static List<String> getRemoteStorageNames(File directory) { List<String> result = null; try { result = getVCS().getFileList(directory); @@ -318,6 +317,8 @@ /** * Fire a change event to all registred listeners. * + * For example, to notify, that a {@link JavaStorage} file has been added... + * * @param event event to fire */ protected void fireDataChanged(Event event) { @@ -330,5 +331,3 @@ } } } - - Modified: isis-fish/trunk/src/main/java/fr/ifremer/isisfish/vcs/VCS.java =================================================================== --- isis-fish/trunk/src/main/java/fr/ifremer/isisfish/vcs/VCS.java 2009-09-02 07:43:03 UTC (rev 2564) +++ isis-fish/trunk/src/main/java/fr/ifremer/isisfish/vcs/VCS.java 2009-09-02 08:15:23 UTC (rev 2565) @@ -1,5 +1,5 @@ /* *##% Isis-Fish - * Copyright (C) 2007 - 2009 CodeLutin + * Copyright (C) 2007 - 2009 Ifremer, CodeLutin * * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU Lesser General Public License as @@ -247,7 +247,7 @@ boolean isUpToDate(File file) throws VCSException; /** - * Check if file can be put in vcs repository, for example when you used + * Check if file can be put in vcs repository. For example when you used * CVS, you must not put CVS file. * <p> * default refused .svn, CVS and ~ filename @@ -255,7 +255,7 @@ * Must be override, this default implementation return true. * * @param file - * @return <tt>true</tt> if file is versionnable + * @return {@code true} if file is versionnable */ boolean isVersionnableAbleFile(File file);

1 0