[ 
https://issues.apache.org/jira/browse/ARTEMIS-4398?focusedWorklogId=878788&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-878788
 ]

ASF GitHub Bot logged work on ARTEMIS-4398:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 29/Aug/23 08:11
            Start Date: 29/Aug/23 08:11
    Worklog Time Spent: 10m 
      Work Description: andytaylor commented on code in PR #4587:
URL: https://github.com/apache/activemq-artemis/pull/4587#discussion_r1308388676


##########
artemis-server/src/test/java/org/apache/activemq/artemis/core/config/impl/ConfigurationImplTest.java:
##########
@@ -1554,6 +1555,48 @@ public void testSystemPropKeyReplacement() throws 
Exception {
       }
    }
 
+   @Test
+   public void testDatabaseStoreConfigurationProps() throws Exception {
+      ConfigurationImpl configuration = new ConfigurationImpl();
+      Properties insertionOrderedProperties = new 
ConfigurationImpl.InsertionOrderedProperties();
+      insertionOrderedProperties.put("storeConfiguration", "DATABASE");
+      
insertionOrderedProperties.put("storeConfiguration.largeMessageTableName", 
"lmtn");
+      insertionOrderedProperties.put("storeConfiguration.messageTableName", 
"mtn");
+      insertionOrderedProperties.put("storeConfiguration.bindingsTableName", 
"btn");
+      insertionOrderedProperties.put("storeConfiguration.dataSourceClassName", 
"dscn");
+      
insertionOrderedProperties.put("storeConfiguration.nodeManagerStoreTableName", 
"nmtn");
+      insertionOrderedProperties.put("storeConfiguration.pageStoreTableName", 
"pstn");
+      insertionOrderedProperties.put("storeConfiguration.jdbcAllowedTimeDiff", 
123);
+      insertionOrderedProperties.put("storeConfiguration.jdbcConnectionUrl", 
"url");
+      insertionOrderedProperties.put("storeConfiguration.jdbcDriverClassName", 
"dcn");
+      
insertionOrderedProperties.put("storeConfiguration.jdbcJournalSyncPeriodMillis",
 456);
+      
insertionOrderedProperties.put("storeConfiguration.jdbcLockAcquisitionTimeoutMillis",
 789);
+      
insertionOrderedProperties.put("storeConfiguration.jdbcLockExpirationMillis", 
321);
+      
insertionOrderedProperties.put("storeConfiguration.jdbcLockRenewPeriodMillis", 
654);
+      insertionOrderedProperties.put("storeConfiguration.jdbcNetworkTimeout", 
987);
+      insertionOrderedProperties.put("storeConfiguration.jdbcPassword", 
"pass");
+      insertionOrderedProperties.put("storeConfiguration.jdbcUser", "user");
+      configuration.parsePrefixedProperties(insertionOrderedProperties, null);
+      Assert.assertTrue(configuration.getStoreConfiguration() instanceof 
DatabaseStorageConfiguration);
+      DatabaseStorageConfiguration dsc = (DatabaseStorageConfiguration) 
configuration.getStoreConfiguration();
+      Assert.assertEquals(dsc.getLargeMessageTableName(), "lmtn");
+      Assert.assertEquals(dsc.getMessageTableName(), "mtn");
+      Assert.assertEquals(dsc.getBindingsTableName(), "btn");
+      Assert.assertEquals(dsc.getDataSourceClassName(), "dscn");
+      Assert.assertEquals(dsc.getJdbcAllowedTimeDiff(), 123);
+      Assert.assertEquals(dsc.getJdbcConnectionUrl(), "url");
+      Assert.assertEquals(dsc.getJdbcDriverClassName(), "dcn");
+      Assert.assertEquals(dsc.getJdbcJournalSyncPeriodMillis(), 456);
+      Assert.assertEquals(dsc.getJdbcLockAcquisitionTimeoutMillis(), 789);
+      Assert.assertEquals(dsc.getJdbcLockExpirationMillis(), 321);
+      Assert.assertEquals(dsc.getJdbcLockRenewPeriodMillis(), 654);
+      Assert.assertEquals(dsc.getJdbcNetworkTimeout(), 987);
+      Assert.assertEquals(dsc.getJdbcPassword(), "pass");
+      Assert.assertEquals(dsc.getJdbcUser(), "user");
+      Assert.assertEquals(dsc.getNodeManagerStoreTableName(), "nmtn");
+      Assert.assertEquals(dsc.getPageStoreTableName(), "pstn");
+   }
+
    @Test

Review Comment:
   Negative test added as well





Issue Time Tracking
-------------------

    Worklog Id:     (was: 878788)
    Time Spent: 40m  (was: 0.5h)

> Support configuring Database with Broker Properties
> ---------------------------------------------------
>
>                 Key: ARTEMIS-4398
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-4398
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>            Reporter: Andy Taylor
>            Assignee: Andy Taylor
>            Priority: Major
>          Time Spent: 40m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to