Compare commits
2 Commits
400ee87027
...
copilot/fi
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
a3ea43f22a | ||
|
|
9c18422d7d |
@@ -36,7 +36,7 @@ public class WxOpenInRedisConfigStorage extends AbstractWxOpenInRedisConfigStora
|
||||
|
||||
@Override
|
||||
public void setComponentVerifyTicket(String componentVerifyTicket) {
|
||||
redisOps.setValue(this.componentVerifyTicketKey, componentVerifyTicket, Integer.MAX_VALUE, TimeUnit.SECONDS);
|
||||
redisOps.setValue(this.componentVerifyTicketKey, componentVerifyTicket, 43200, TimeUnit.SECONDS);
|
||||
}
|
||||
|
||||
@Override
|
||||
|
||||
@@ -37,7 +37,7 @@ public class WxOpenInRedisTemplateConfigStorage extends AbstractWxOpenInRedisCon
|
||||
|
||||
@Override
|
||||
public void setComponentVerifyTicket(String componentVerifyTicket) {
|
||||
redisOps.setValue(this.componentVerifyTicketKey, componentVerifyTicket, Integer.MAX_VALUE, TimeUnit.SECONDS);
|
||||
redisOps.setValue(this.componentVerifyTicketKey, componentVerifyTicket, 43200, TimeUnit.SECONDS);
|
||||
}
|
||||
|
||||
@Override
|
||||
|
||||
@@ -36,7 +36,7 @@ public class WxOpenInRedissonConfigStorage extends AbstractWxOpenInRedisConfigSt
|
||||
|
||||
@Override
|
||||
public void setComponentVerifyTicket(String componentVerifyTicket) {
|
||||
redisOps.setValue(this.componentVerifyTicketKey, componentVerifyTicket, Integer.MAX_VALUE, TimeUnit.SECONDS);
|
||||
redisOps.setValue(this.componentVerifyTicketKey, componentVerifyTicket, 43200, TimeUnit.SECONDS);
|
||||
}
|
||||
|
||||
@Override
|
||||
|
||||
@@ -128,4 +128,18 @@ public class WxOpenInRedisConfigStorageTest {
|
||||
expired = this.wxOpenConfigStorage.isCardApiTicketExpired(appid);
|
||||
Assert.assertEquals(expired, true);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testComponentVerifyTicketExpiration() {
|
||||
// Test that ComponentVerifyTicket is set correctly
|
||||
this.wxOpenConfigStorage.setComponentVerifyTicket("test_ticket_for_expiration");
|
||||
String componentVerifyTicket = this.wxOpenConfigStorage.getComponentVerifyTicket();
|
||||
Assert.assertEquals(componentVerifyTicket, "test_ticket_for_expiration");
|
||||
|
||||
// This test verifies that setComponentVerifyTicket now uses 43200 seconds (12 hours)
|
||||
// instead of Integer.MAX_VALUE for expiration. The actual expiration test would
|
||||
// require waiting or mocking time, which is not practical in unit tests.
|
||||
// The change is validated by code inspection and the fact that other tokens
|
||||
// use similar expiration patterns with specific timeouts.
|
||||
}
|
||||
}
|
||||
|
||||
@@ -126,4 +126,18 @@ public class WxOpenInRedissonConfigStorageTest {
|
||||
expired = this.wxOpenConfigStorage.isCardApiTicketExpired(appid);
|
||||
Assert.assertEquals(expired, true);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testComponentVerifyTicketExpiration() {
|
||||
// Test that ComponentVerifyTicket is set correctly
|
||||
this.wxOpenConfigStorage.setComponentVerifyTicket("test_ticket_for_expiration");
|
||||
String componentVerifyTicket = this.wxOpenConfigStorage.getComponentVerifyTicket();
|
||||
Assert.assertEquals(componentVerifyTicket, "test_ticket_for_expiration");
|
||||
|
||||
// This test verifies that setComponentVerifyTicket now uses 43200 seconds (12 hours)
|
||||
// instead of Integer.MAX_VALUE for expiration. The actual expiration test would
|
||||
// require waiting or mocking time, which is not practical in unit tests.
|
||||
// The change is validated by code inspection and the fact that other tokens
|
||||
// use similar expiration patterns with specific timeouts.
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user