-
Notifications
You must be signed in to change notification settings - Fork 86
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
GCS Additonal e2e tests #1306
GCS Additonal e2e tests #1306
Conversation
7c1cd96
to
7a10c09
Compare
Then Validate the data from GCS Source to GCS Sink with expected csv file and target data in GCS bucket | ||
|
||
@GCS_CSV @GCS_SINK_TEST | ||
Scenario: To verify the pipeline is getting failed from GCS to GCS when Schema is not cleared in GCS source On Single File |
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.
nit: To verify the pipeline....when default schema
is not cleared....
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.
Changed it to To verify the pipeline is getting failed from GCS to GCS when default schema is not cleared in GCS source On Single File
Then Validate the data from GCS Source to GCS Sink with expected json file and target data in GCS bucket | ||
|
||
@GCS_MULTIPLE_FILES_REGEX_TEST @GCS_SINK_TEST | ||
Scenario: To verify the pipeline is getting failed from GCS to GCS On Multiple File with filter regex without using connection |
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.
Are multiple files having different schema here?
Can we add a success scenario as well to understand when multiple files are supported?
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.
Three files were used (vehicle_inventory-part-1, vehicle_inventory-part-2, book) using the same schema in vehicle_inventory-part-1, vehicle_inventory-part-2 and a different schema is used in book, and putting regex filter on top of it.(.+vehicle_inventory.*). Hence, the output schema generated based on same schema.
We have already covered the success situation in this test. Mistakenly, the word failure has been used instead of success.
Changed the scenario outline to :- To verify data is getting transferred from GCS to GCS On Multiple File with filter regex without using connection
@@ -85,3 +85,306 @@ Feature: GCS source - Verification of GCS to GCS Additional Tests successful | |||
Then Open and capture logs | |||
Then Verify the pipeline status is "Succeeded" | |||
Then Validate the data transferred from GCS Source to GCS Sink with Expected avro file and target data in GCS bucket | |||
|
|||
@GCS_CSV @GCS_SINK_TEST @EXISTING_GCS_CONNECTION | |||
Scenario: To verify data is getting transferred from GCS Source to GCS Sink using test Schema Detection On Single File with connection |
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.
nit: To verify data is getting transferred from GCS Source to GCS Sink using Schema Detection On Single File with connection
test
seems unnecessary here.
This comment applies to all similar scenarios.
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.
As per the ITN Class in Single File and Multiple file with regex (useConnection is true) and in Multiple File with different schema (useConnection is not used). So, We are removing a particular tests which has not used useConnection in Multiple file with different schema. If you are suggesting that to remove unnecessary tests, then we can remove those tests.
1)Removing a scenario for multiple file with different schema with connection(True).
2)Removing a scenario for multiple file with regex with connection(False).
3)Removing a scenario for single file with connection(False).
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.
I just suggested to remove test
word from the scenario name not the whole scenario.
Sorry for the confusion.
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.
Removed test word from the scenario.
Then Verify the pipeline status is "Failed" | ||
|
||
@GCS_MULTIPLE_FILES_TEST @GCS_SINK_TEST @EXISTING_GCS_CONNECTION | ||
Scenario: To verify the pipeline is getting failed from GCS Source to GCS Sink On Multiple File with connection |
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.
It is not clear from the scenario title why will the pipeline fail? Is it due to multiple files having different schema?
Then we can rename it as To verify the pipeline is getting failed from GCS Source to GCS Sink On Multiple File having different schemas with connection
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.
Correct, pipeline will fail due to multiple files having different schema.
Renamed it to 'To verify the pipeline is getting failed from GCS Source to GCS Sink On Multiple File having different schemas with connection'.
Thanks for the suggestion.
Then Verify the pipeline status is "Failed" | ||
|
||
@GCS_MULTIPLE_FILES_TEST @GCS_SINK_TEST | ||
Scenario: To verify the pipeline is getting failed from GCS Source to GCS Sink On Multiple File without connection |
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.
similar comment here
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.
Renamed it to 'To verify the pipeline is getting failed from GCS Source to GCS Sink On Multiple File having different schemas without connection'
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.
Please squash commits & rebase so that we can merge.
4789541
to
abc6c4f
Compare
All commits were Squashed, and the code was rebased. When the build is green. I'll notify you when it's ready to be merged. |
No description provided.