This is an automated email from the ASF dual-hosted git repository.

panjuan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
     new 859d7c78407 Fix unit test (#25178)
859d7c78407 is described below

commit 859d7c78407e7489e2f1947867a25ced3c445b05
Author: ChenJiaHao <[email protected]>
AuthorDate: Sat Apr 15 12:32:05 2023 +0800

    Fix unit test (#25178)
---
 .../handler/distsql/ral/queryable/ShowDistVariablesExecutorTest.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/proxy/backend/core/src/test/java/org/apache/shardingsphere/proxy/backend/handler/distsql/ral/queryable/ShowDistVariablesExecutorTest.java
 
b/proxy/backend/core/src/test/java/org/apache/shardingsphere/proxy/backend/handler/distsql/ral/queryable/ShowDistVariablesExecutorTest.java
index bcef82ce685..f3a5bbbc549 100644
--- 
a/proxy/backend/core/src/test/java/org/apache/shardingsphere/proxy/backend/handler/distsql/ral/queryable/ShowDistVariablesExecutorTest.java
+++ 
b/proxy/backend/core/src/test/java/org/apache/shardingsphere/proxy/backend/handler/distsql/ral/queryable/ShowDistVariablesExecutorTest.java
@@ -61,12 +61,12 @@ class ShowDistVariablesExecutorTest {
         when(metaData.getProps()).thenReturn(new 
ConfigurationProperties(PropertiesBuilder.build(new 
Property("system_log_level", "INFO"))));
         when(metaData.getInternalProps()).thenReturn(new 
InternalConfigurationProperties(PropertiesBuilder.build(new 
Property("proxy-meta-data-collector-enabled", Boolean.FALSE.toString()))));
         when(metaData.getGlobalRuleMetaData()).thenReturn(new 
ShardingSphereRuleMetaData(Collections.singleton(new LoggingRule(new 
DefaultLoggingRuleConfigurationBuilder().build()))));
+        System.setProperty("AGENT_PLUGINS_ENABLED", "true");
         ShowDistVariablesExecutor executor = new ShowDistVariablesExecutor();
         Collection<LocalDataQueryResultRow> actual = 
executor.getRows(metaData, connectionSession, 
mock(ShowDistVariablesStatement.class));
         assertThat(actual.size(), is(23));
         LocalDataQueryResultRow row = actual.iterator().next();
         assertThat(row.getCell(1), is("agent_plugins_enabled"));
-        // TODO jiahao fix it, mvn install maeby is false
-        // assertThat(row.getCell(2), is("true"));
+        assertThat(row.getCell(2), is("true"));
     }
 }

Reply via email to