Skip to content

[PLUGIN-1952] Add column details in logging in case of invalid data#147

Open
Ashutosh-Jarvis wants to merge 3 commits intodata-integrations:developfrom
Ashutosh-Jarvis:feature/unsupported-field-fix
Open

[PLUGIN-1952] Add column details in logging in case of invalid data#147
Ashutosh-Jarvis wants to merge 3 commits intodata-integrations:developfrom
Ashutosh-Jarvis:feature/unsupported-field-fix

Conversation

@Ashutosh-Jarvis
Copy link
Copy Markdown

@Ashutosh-Jarvis Ashutosh-Jarvis commented Mar 23, 2026

PLUGIN-1952 Add column details in logging in case of invalid data

In case of invalid data mention the column name along with wrong data.

@Ashutosh-Jarvis Ashutosh-Jarvis changed the title Feature/unsupported field fix Unsupported data exception fix Mar 23, 2026
@vikasrathee-cs vikasrathee-cs self-requested a review March 23, 2026 12:45

@Test
public void testConvertToDoubleValue() throws ParseException {
Assert.assertEquals(42.0, ServiceNowRecordConverter.convertToDoubleValue("42"), 0.0);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

write a test case to compare error message

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working on it

Copy link
Copy Markdown
Contributor

@itsankit-google itsankit-google left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Please add a JIRA to the PR.

  • Also add proper description:

    • It should be able to answer What, Why and Testing details.

@vikasrathee-cs vikasrathee-cs changed the title Unsupported data exception fix [PLUGIN-1952] Add column details in logging in case of invalid data Mar 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants