[jboss-svn-commits] JBL Code SVN: r35775 - in labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src: test/java/org/drools/guvnor/server and 1 other directories.

jboss-svn-commits at lists.jboss.org jboss-svn-commits at lists.jboss.org
Wed Oct 27 14:53:12 EDT 2010


Author: kurt.stam at jboss.com
Date: 2010-10-27 14:53:10 -0400 (Wed, 27 Oct 2010)
New Revision: 35775

Modified:
   labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/main/java/org/drools/guvnor/server/util/TestEnvironmentSessionHelper.java
   labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/test/java/org/drools/guvnor/server/ServiceImplementationTest.java
   labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/test/resources/modeshape/roles.properties
   labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/test/resources/modeshape/users.properties
Log:
BRMS-405 removing System.out, using logging instead

Modified: labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/main/java/org/drools/guvnor/server/util/TestEnvironmentSessionHelper.java
===================================================================
--- labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/main/java/org/drools/guvnor/server/util/TestEnvironmentSessionHelper.java	2010-10-27 18:07:06 UTC (rev 35774)
+++ labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/main/java/org/drools/guvnor/server/util/TestEnvironmentSessionHelper.java	2010-10-27 18:53:10 UTC (rev 35775)
@@ -47,6 +47,8 @@
 import org.drools.repository.RepositorySessionUtil;
 import org.drools.repository.RulesRepositoryAdministrator;
 import org.drools.repository.RulesRepositoryConfigurator;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 /**
  * This is only to be used for testing, eg in hosted mode, or unit tests.
@@ -57,6 +59,7 @@
 public class TestEnvironmentSessionHelper {
 
 
+	public static final Logger log = LoggerFactory.getLogger(TestEnvironmentSessionHelper.class);
     public static Repository repository;
 
 
@@ -70,9 +73,9 @@
 
 	            if (erase) {
 	                File repoDir = new File("repository");
-	                System.out.println("DELETE test repo dir: " + repoDir.getAbsolutePath());
+	                log.info("DELETE test repo dir: " + repoDir.getAbsolutePath());
 	                RepositorySessionUtil.deleteDir( repoDir );
-	                System.out.println("TEST repo dir deleted.");
+	                log.info("TEST repo dir deleted.");
 	            }
 
 	            RulesRepositoryConfigurator config = RulesRepositoryConfigurator.getInstance(null);

Modified: labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/test/java/org/drools/guvnor/server/ServiceImplementationTest.java
===================================================================
--- labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/test/java/org/drools/guvnor/server/ServiceImplementationTest.java	2010-10-27 18:07:06 UTC (rev 35774)
+++ labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/test/java/org/drools/guvnor/server/ServiceImplementationTest.java	2010-10-27 18:53:10 UTC (rev 35775)
@@ -230,7 +230,7 @@
 
     }
 
-	@Test @Ignore
+	@Test
     public void testCategory() throws Exception {
 
         RepositoryService impl = getService();
@@ -262,14 +262,14 @@
 
     }
 
-	@Test @Ignore
+	@Test
     public void testCleanHTML() {
         ServiceImplementation impl = new ServiceImplementation();
         assertEquals( "<script>",
                       impl.cleanHTML( "<script>" ) );
     }
 
-	@Test @Ignore
+	@Test
     public void testDeleteUnversionedRule() throws Exception {
         ServiceImplementation impl = getService();
 
@@ -311,7 +311,7 @@
         }
     }
 
-	@Test @Ignore
+	@Test
     public void testAddRuleAndListPackages() throws Exception {
         // ServiceImpl impl = new ServiceImpl(new
         // RulesRepository(SessionHelper.getSession()));
@@ -365,7 +365,7 @@
         assertTrue( dtItem.getBinaryContentAttachmentFileName().endsWith( ".xls" ) );
     }
 
-	@Test @Ignore
+	@Test
     public void testAttemptDupeRule() throws Exception {
         ServiceImplementation impl = getService();
         CategoryItem cat = impl.repository.loadCategory( "/" );
@@ -391,7 +391,7 @@
 
     }
 
-	@Test @Ignore
+	@Test
     public void testCreateNewRule() throws Exception {
         ServiceImplementation impl = getService();
         impl.repository.createPackage( "testCreateNewRule",
@@ -413,7 +413,7 @@
                       "an initial desc" );
     }
 	
-	@Test @Ignore
+	@Test
     public void testCreateLinkedAssetItem() throws Exception {
         ServiceImplementation impl = getService();
         @SuppressWarnings("unused")
@@ -499,7 +499,7 @@
         assertEquals( 1, size );
     }
 
-	@Test @Ignore
+	@Test
     public void testLinkedAssetItemHistoryRelated() throws Exception {
         ServiceImplementation impl = getService();
         @SuppressWarnings("unused")
@@ -598,7 +598,7 @@
                       newHead.metaData.checkinComment );
     }
 
-	@Test @Ignore
+	@Test
     //path name contains Apostrophe is no longer a problem with jackrabbit 2.0
     public void testCreateNewRuleContainsApostrophe() throws Exception {
         ServiceImplementation impl = getService();
@@ -628,7 +628,7 @@
     
     }
 
-	@Test @Ignore
+	@Test
     public void testRuleTableLoad() throws Exception {
         ServiceImplementation impl = getService();
         TableConfig conf = impl.loadTableConfig( AssetItemGrid.RULE_LIST_TABLE_ID );
@@ -668,7 +668,7 @@
         assertTrue( result.data[0].values[0].startsWith( "testRuleTableLoad" ) );
     }
 
-	@Test @Ignore
+	@Test
     public void testDateFormatting() throws Exception {
         Calendar cal = Calendar.getInstance();
         TableDisplayHandler handler = new TableDisplayHandler( AssetItemGrid.RULE_LIST_TABLE_ID );
@@ -678,7 +678,7 @@
         assertTrue( fmt.length() > 8 );
     }
 
-	@Test @Ignore
+	@Test
     public void testLoadRuleAsset() throws Exception {
         ServiceImplementation impl = getService();
         impl.repository.createPackage( "testLoadRuleAsset",
@@ -922,7 +922,7 @@
                       ib.loadRecentEdited().size() );
     }
 
-	@Test @Ignore
+	@Test
     public void testLoadAssetHistoryAndRestore() throws Exception {
 		
         ServiceImplementation impl = getService();
@@ -986,7 +986,7 @@
 
     }
 
-	@Test @Ignore
+	@Test
     public void testCheckin() throws Exception {
         ServiceImplementation serv = getService();
 
@@ -1084,7 +1084,7 @@
 
     }
 
-	@Test @Ignore
+	@Test
     public void testArchivePackage() throws Exception {
         ServiceImplementation impl = getService();
 
@@ -1120,7 +1120,7 @@
                       arch.length );
     }
 
-	@Test @Ignore
+	@Test
     public void testCreatePackage() throws Exception {
         ServiceImplementation impl = getService();
         PackageConfigData[] pkgs = impl.listPackages();
@@ -1156,7 +1156,7 @@
         }
     }
 
-	@Test @Ignore
+	@Test
     public void testLoadPackageConfig() throws Exception {
         ServiceImplementation impl = getService();
         PackageItem it = impl.repository.loadDefaultPackage();
@@ -1201,7 +1201,7 @@
                       data.checkinComment );
     }
 
-	@Test @Ignore
+	@Test
     public void testArchiveAndUnarchivePackageAndHeader() throws Exception {
         ServiceImplementation impl = getService();
         String uuid = impl.createPackage( "testArchiveAndUnarchivePackageAndHeader",
@@ -1245,7 +1245,7 @@
 
     }
 
-	@Test @Ignore
+	@Test
     public void testPackageConfSave() throws Exception {
         RepositoryService impl = getService();
         String uuid = impl.createPackage( "testPackageConfSave",
@@ -1281,7 +1281,7 @@
         assertFalse( res.hasErrors );
     }
 
-	@Test @Ignore
+	@Test
     public void testListByFormat() throws Exception {
         RepositoryService impl = getService();
         String cat = "testListByFormat";
@@ -1402,7 +1402,7 @@
                       result.data.length );
     }
 
-	@Test @Ignore
+	@Test
     public void testListUnregisteredAssetFormats() throws Exception {
         ServiceImplementation impl = getService();
         PackageItem pkg = impl.repository.createPackage( "testListUnregisteredAssetFormats",
@@ -1426,7 +1426,7 @@
                       res.data.length );
     }
 
-	@Test @Ignore
+	@Test
     public void testQuickFind() throws Exception {
         RepositoryService impl = getService();
         String cat = "testQuickFind";
@@ -1478,7 +1478,7 @@
 
     }
 
-	@Test @Ignore
+	@Test
     public void testSearchText() throws Exception {
         ServiceImplementation impl = getService();
         String cat = "testTextSearch";
@@ -1501,7 +1501,7 @@
                       res.data.length );
     }
 
-	@Test @Ignore
+	@Test
     public void testSearchMetaData() throws Exception {
         ServiceImplementation impl = getService();
         PackageItem pkg = impl.repository.createPackage( "testMetaDataSearch",
@@ -1538,7 +1538,7 @@
         return new String[]{s};
     }
 
-    @Test @Ignore
+    @Test
     public void testStatus() throws Exception {
         RepositoryService impl = getService();
         String uuid = impl.createState( "testStatus1" );
@@ -1611,7 +1611,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testMovePackage() throws Exception {
         RepositoryService impl = getService();
         String[] cats = impl.loadChildCategories( "/" );
@@ -1664,7 +1664,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testCopyAsset() throws Exception {
         RepositoryService impl = getService();
         impl.createCategory( "/",
@@ -1689,7 +1689,7 @@
                       asset.metaData.name );
     }
 
-    @Test @Ignore
+    @Test
     public void testSnapshot() throws Exception {
         RepositoryService impl = getService();
         impl.createCategory( "/",
@@ -1759,7 +1759,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testSnapshotRebuild() throws Exception {
 
         ServiceImplementation impl = getService();
@@ -1828,7 +1828,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testPackageRebuild() throws Exception {
 
         ServiceImplementation impl = getService();
@@ -1863,7 +1863,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testRemoveCategory() throws Exception {
 
         RepositoryService impl = getService();
@@ -1879,7 +1879,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testRemoveAsset() throws Exception {
         RepositoryService impl = getService();
         String cat = "testRemoveAsset";
@@ -1931,7 +1931,7 @@
                       res.data.length );
     }
 
-    @Test @Ignore
+    @Test
     public void testRemovePackage() throws Exception {
         ServiceImplementation impl = getService();
         int n = impl.listPackages().length;
@@ -1944,7 +1944,7 @@
                       impl.listPackages().length );
     }
 
-    @Test @Ignore
+    @Test
     public void testArchiveAsset() throws Exception {
         RepositoryService impl = getService();
         String cat = "testArchiveAsset";
@@ -2018,7 +2018,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testArchiveAssetWhenParentPackageArchived() throws Exception {
         ServiceImplementation impl = getService();
         String packageName = "testArchiveAssetWhenParentPackageArchived";
@@ -2100,7 +2100,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testLoadSuggestionCompletionEngine() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -2141,7 +2141,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testDiscussion() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -2206,7 +2206,7 @@
                       d_.size() );
     }
 
-    @Test @Ignore
+    @Test
     /**
      * This will test creating a package, check it compiles, and can exectute
      * rules, then take a snapshot, and check that it reports errors.
@@ -2284,7 +2284,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     /**
      * This will test creating a package with a BRL rule, check it compiles, and
      * can exectute rules, then take a snapshot, and check that it reports
@@ -2419,7 +2419,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     /**
      * this loads up a precompile binary package. If this fails, then it means
      * it needs to be updated. It gets the package form the BRL example above.
@@ -2436,7 +2436,7 @@
                       p.getAge() );
     }
 
-    @Test @Ignore
+    @Test
     public void testSuggestionCompletionLoading() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -2455,7 +2455,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testPackageSource() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -2512,7 +2512,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testAssetSource() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -2574,7 +2574,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testBuildAssetWithError() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -2608,7 +2608,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testBuildAsset() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -2679,7 +2679,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testBuildAssetMultipleFunctionsCallingEachOther() throws Exception {
 
         ServiceImplementation impl = getService();
@@ -2721,7 +2721,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testBuildAssetBRXMLAndCopy() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -2817,7 +2817,7 @@
         return result;
     }
 
-    @Test @Ignore
+    @Test
     public void testBuildAssetWithPackageConfigError() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -2862,7 +2862,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testRuleNameList() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -2903,7 +2903,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     /**
      * This idea of this is to not compile packages more then we have to.
      */
@@ -2965,7 +2965,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testRunScenario() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -3079,7 +3079,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testRunScenarioWithGeneratedBeans() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -3137,7 +3137,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testRunPackageScenariosWithDeclaredFacts() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -3250,7 +3250,7 @@
                       s2.scenarioName );
     }
 
-    @Test @Ignore
+    @Test
     public void testRunScenarioWithJar() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -3321,7 +3321,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testRunScenarioWithJarThatHasSourceFiles() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -3384,7 +3384,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testRunPackageScenarios() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -3507,7 +3507,7 @@
                       s2.scenarioName );
     }
 
-    @Test @Ignore
+    @Test
     public void testListFactTypesAvailableInPackage() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -3544,7 +3544,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testGuidedDTExecute() throws Exception {
         ServiceImplementation impl = getService();
         RulesRepository repo = impl.repository;
@@ -3622,7 +3622,7 @@
                       p.getLikes() );
     }
 
-    @Test @Ignore
+    @Test
     public void testPackageNameSorting() {
         PackageConfigData c1 = new PackageConfigData( "org.foo" );
         PackageConfigData c2 = new PackageConfigData( "org.foo.bar" );
@@ -3638,7 +3638,7 @@
                       ls.get( 1 ) );
     }
 
-    @Test @Ignore
+    @Test
     public void testLoadDropDown() throws Exception {
         ServiceImplementation serv = new ServiceImplementation();
         String[] pairs = new String[]{"f1=x", "f2=2"};
@@ -3655,7 +3655,7 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testLoadDropDownNoValuePairs() throws Exception {
         ServiceImplementation serv = new ServiceImplementation();
         String[] pairs = new String[]{null};
@@ -3668,14 +3668,14 @@
 
     }
 
-    @Test @Ignore
+    @Test
     public void testListUserPermisisons() throws Exception {
         ServiceImplementation serv = getService();
         Map<String, List<String>> r = serv.listUserPermissions();
         assertNotNull( r );
     }
 
-    @Test @Ignore
+    @Test
     public void testManageUserPermissions() throws Exception {
         ServiceImplementation serv = getService();
         Map<String, List<String>> perms = new HashMap<String, List<String>>();
@@ -3687,7 +3687,7 @@
                       perms_.size() );
     }
 
-    @Test @Ignore //Ignore
+    @Test //Ignore
     public void testImportSampleRepository() throws Exception {
         ServiceImplementation serv = getService();
         serv.installSampleRepository();
@@ -3701,7 +3701,7 @@
                       res.result );
     }
 
-    @Test @Ignore
+    @Test
     public void testAddCategories() throws Exception {
         ServiceImplementation impl = getService();
         impl.repository.createPackage( "testAddCategoriesPackage",
@@ -3738,7 +3738,7 @@
         assertTrue( dtItem3.getCategorySummary().contains( "testAddCategoriesCat2" ) );
     }
 
-    @Test @Ignore
+    @Test
     public void testSnapshotDiff() throws Exception {
         RepositoryService impl = getService();
 

Modified: labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/test/resources/modeshape/roles.properties
===================================================================
--- labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/test/resources/modeshape/roles.properties	2010-10-27 18:07:06 UTC (rev 35774)
+++ labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/test/resources/modeshape/roles.properties	2010-10-27 18:53:10 UTC (rev 35775)
@@ -2,3 +2,5 @@
 ADMINISTRATOR=admin
 fdd=readwrite
 thirdpartyuser=readwrite
+admin=admin
+mailman=readwrite

Modified: labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/test/resources/modeshape/users.properties
===================================================================
--- labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/test/resources/modeshape/users.properties	2010-10-27 18:07:06 UTC (rev 35774)
+++ labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/test/resources/modeshape/users.properties	2010-10-27 18:53:10 UTC (rev 35775)
@@ -2,3 +2,5 @@
 ADMINISTRATOR=password
 fdd=password
 thirdpartyuser=password
+admin=password
+mailman=password



More information about the jboss-svn-commits mailing list