Skip to content

fix mixed_dml test failures #938 #965

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
manuerwin wants to merge 4 commits into
base: main
Choose a base branch
from
Open

fix mixed_dml test failures #938 #965

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
b938135
fix (partial) mixed_dml test failures #938
manuerwin Mar 19, 2026
68b514c
also fix MIXED_DML for it_should_set_logger_scenario_owner_as_log_own…
manuerwin Mar 20, 2026
690c7af
actually fix MIXED_DML test failure #938
manuerwin Mar 20, 2026
b93853c
Merge branch 'main' into bugfix/partial_fix_938_mixed_dml_test_failures
manuerwin Apr 5, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 11 additions & 8 deletions nebula-logger/core/tests/log-management/classes/LogEntryEventHandler_Tests.cls
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -655,16 +655,19 @@ private class LogEntryEventHandler_Tests {
Schema.User loggerScenarioOwner = LoggerMockDataCreator.createUser();
insert loggerScenarioOwner;
LoggerScenario__c loggerScenario = new LoggerScenario__c(Name = 'Some Scenario', OwnerId = loggerScenarioOwner.Id, UniqueId__c = 'Some Scenario');
insert loggerScenario;
LogEntryEvent__e logEntryEvent = createLogEntryEvent();
logEntryEvent.TransactionScenario__c = loggerScenario.UniqueId__c;
LoggerScenarioRule.setMock(
new LoggerScenarioRule__mdt(IsEnabled__c = true, IsLogAssignmentEnabled__c = String.valueOf(true), Scenario__c = logEntryEvent.TransactionScenario__c)
);

LoggerMockDataStore.getEventBus().publishRecord(logEntryEvent);
LoggerMockDataStore.getEventBus().deliver(new LogEntryEventHandler());
Schema.User currentUser = new Schema.User(Id = System.UserInfo.getUserId(), ProfileId = System.UserInfo.getProfileId());
System.runAs(currentUser) {
insert loggerScenario;
LogEntryEvent__e logEntryEvent = createLogEntryEvent();
logEntryEvent.TransactionScenario__c = loggerScenario.UniqueId__c;
LoggerScenarioRule.setMock(
new LoggerScenarioRule__mdt(IsEnabled__c = true, IsLogAssignmentEnabled__c = String.valueOf(true), Scenario__c = logEntryEvent.TransactionScenario__c)
);

LoggerMockDataStore.getEventBus().publishRecord(logEntryEvent);
LoggerMockDataStore.getEventBus().deliver(new LogEntryEventHandler());
}
Log__c log = getLog();
System.Assert.areEqual(loggerScenario.OwnerId, log.OwnerId);
}
Expand Down
14 changes: 10 additions & 4 deletions nebula-logger/core/tests/log-management/classes/LogHandler_Tests.cls
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -239,10 +239,13 @@ private class LogHandler_Tests {
insert expectedLogOwnerUser;
LoggerSettings__c currentUserSettings = Logger.getUserSettings(currentUser);
currentUserSettings.DefaultLogOwner__c = expectedLogOwnerUser.Id;
insert currentUserSettings;
Log__c log = new Log__c(LoggedBy__c = currentUser.Id, OwnerId = currentUser.Id, TransactionId__c = '1234');

LoggerDataStore.getDatabase().insertRecord(log);
System.runAs(currentuser) {
insert currentUserSettings;

LoggerDataStore.getDatabase().insertRecord(log);
}

System.Assert.areEqual(
2,
Expand All @@ -261,10 +264,13 @@ private class LogHandler_Tests {
insert expectedLogOwnerUser;
LoggerSettings__c currentUserSettings = Logger.getUserSettings(currentUser);
currentUserSettings.DefaultLogOwner__c = expectedLogOwnerUser.Username;
insert currentUserSettings;
Log__c log = new Log__c(LoggedBy__c = currentUser.Id, OwnerId = currentUser.Id, TransactionId__c = '1234');

LoggerDataStore.getDatabase().insertRecord(log);
System.runAs(currentuser) {
insert currentUserSettings;

LoggerDataStore.getDatabase().insertRecord(log);
}

System.Assert.areEqual(
2,
Expand Down