-
Notifications
You must be signed in to change notification settings - Fork 14
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.
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
#3260 - CAS Integration 3A - Create new Supplier and Site - Name Fix And Error Log Improvement #3843
#3260 - CAS Integration 3A - Create new Supplier and Site - Name Fix And Error Log Improvement #3843
Conversation
sources/packages/backend/libs/integrations/src/cas/_tests_/unit/cas-test.utils.ts
Dismissed
Show dismissed
Hide dismissed
Quality Gate passedIssues Measures |
supplierData.firstName, | ||
supplierData.lastName, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
const formattedName = `${lastName}, ${firstName}` | ||
.substring(0, CAS_SUPPLIER_NAME_MAX_LENGTH) | ||
.trim(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
* Initializes the service under test. | ||
* @returns array of mocked services. | ||
*/ | ||
export async function initializeService(): Promise< |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
!!error.response?.data[CAS_RETURNED_MESSAGES] | ||
) { | ||
// Checking for bad request errors for better logging while the | ||
// specific ticket to handle exception is pending. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, nice work @andrewsignori-aot
Sample logged error (Invalid SIN).