-
-
Notifications
You must be signed in to change notification settings - Fork 171
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Added query selector classes, fixed some recurring bugs (#314)
* Added new selector classes LoggerEngineDataSelector and LogManagementDataSelector to centralize (almost) all queries * Added LoggerCache class for centralizing transaction caching * Added LogViewerController.cls to use within logViewer LWC and removed Logger.getLog() * Eliminated layer-specific test suites within core - as of v4.7.1, the tests are super fast anyway, so the LoggerCore test suite covers everything + makes maintenance easier * Deleted some deprecated LoggerParameter__mdt records - these were deprecated in v4.7.1, but I forgot to remove them from the repo * Possibly fixed an issue discussed in #310 by reverting to only using a UUID for the transaction ID (instead of using System.Request.getCurrent().getRequestId()) * Hopefully fixed #276 (again) by moving when an email is added to the constant SENT_EMAILS Scratch orgs also can't have deliverability disabled (very frustrating), so I also separately changed email deliverability to 'System Email Only' in the pkg demo org that's used in the pipeline, which should help ensure everything is working going forward 🤞 * Added a missing CMDT record for the async failures plugin, created a new package version * Added missing access to the class LoggerSObjectMetadata in perm sets LoggerAdmin and LoggerLogViewer, moved LoggerLogCreator perm set back to being part of the logger-engine layer * Fixed #313 by switching to using Logger's txn ID + an incremented counter for dynamically creating unique usernames during tests * Fixed #219 (again) - it seems like the original solution recently stopped working due to a change in some orgs of the value used for the Guester User LIcense field on the UserLicense object. I've switched to instead using the User Type (UserInfo.getUserType()), which seems to be a more stable option that's less likely to change - I tried to find an option that didn't rely on a hardcoded String, but so far, have not found any other way to determine it * Fixed a test that failed when running in a namespaced org * Enabled running of tests in the pkg demo org within the pipeline, tweaked a few tests that failed only in the pkg demo org
- Loading branch information
Showing
68 changed files
with
1,988 additions
and
547 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -323,6 +323,9 @@ jobs: | |
- name: 'Create & Install Package Version' | ||
run: npx pwsh ./scripts/build/create-and-install-package-version.ps1 -targetpackagealias '"Nebula Logger - Core"' -targetreadme ./README.md -targetusername nebula-logger-package-demo | ||
|
||
- name: 'Run Apex Tests' | ||
run: npm run test:apex:without-coverage -- --targetusername nebula-logger-package-demo | ||
|
||
- name: 'Commit New Package Version' | ||
run: | | ||
git config --local user.email "[email protected]" | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.