-
Notifications
You must be signed in to change notification settings - Fork 0
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
Idva5 1269/test structure of csv file #6
base: main
Are you sure you want to change the base?
Conversation
Created Service to handle getting the file from file-transfer-service and retrying the download if the file is still doing AV scan
Added unit test for get request
…-file' into IDVA5-1269/test-structure-of-csv-file
looks good so far. 👍 |
…-file' into IDVA5-1269/test-structure-of-csv-file
|
||
|
||
private Boolean isValidFieldHeaders(CSVRecord record) { | ||
List<String> actualHeaders = record.toList(); |
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.
Will the order matter? If not then we might have to get the index of column dynamically based on the name of the column
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. Good work 👍
Short description outlining key changes/additions
Adding validation for each of the cells in the CSV file
JIRA Ticket Number
IDVA5-1269
Type of change
Pull request checklist
An exhaustive list of peer review checks can be read here