Skip to content

Commit fae0768

Browse files
seawindeCopilot
andcommitted
[fix](fe) Convert testCtePrivilegeCheck from JMockit to Mockito
### What problem does this PR solve? Problem Summary: After PR #62221 removed all JMockit usage from the codebase, the testCtePrivilegeCheck method still used JMockit Expectations pattern which causes compilation failure. Convert to Mockito spy pattern consistent with the rest of the test file. ### Release note None ### Check List (For Author) - Test: Unit Test (compilation verified) - Behavior changed: No - Does this need documentation: No Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 4efdbf9 commit fae0768

1 file changed

Lines changed: 4 additions & 7 deletions

File tree

fe/fe-core/src/test/java/org/apache/doris/nereids/privileges/TestCheckPrivileges.java

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -261,13 +261,10 @@ public void testCtePrivilegeCheck() throws Exception {
261261

262262
AccessControllerManager accessManager = Env.getCurrentEnv().getAccessManager();
263263
CatalogAccessController catalogAccessController = accessManager.getAccessControllerOrDefault(catalog);
264-
new Expectations(accessManager) {
265-
{
266-
accessManager.getAccessControllerOrDefault("internal");
267-
minTimes = 0;
268-
result = catalogAccessController;
269-
}
270-
};
264+
AccessControllerManager spyAccessManager = Mockito.spy(accessManager);
265+
Mockito.doReturn(catalogAccessController).when(spyAccessManager)
266+
.getAccessControllerOrDefault("internal");
267+
Deencapsulation.setField(Env.getCurrentEnv(), "accessManager", spyAccessManager);
271268

272269
withPrivileges(privileges, () -> {
273270
// CTE with authorized table should succeed

0 commit comments

Comments
 (0)