Skip to content

Cloudsqlpostgresql#9

Open
Ragini2 wants to merge 2 commits intosawantpritam:cloudsqlpostgresqlfrom
Ragini2:cloudsqlpostgresql
Open

Cloudsqlpostgresql#9
Ragini2 wants to merge 2 commits intosawantpritam:cloudsqlpostgresqlfrom
Ragini2:cloudsqlpostgresql

Conversation

@Ragini2
Copy link

@Ragini2 Ragini2 commented Dec 23, 2021

Pr for ClouSqldPsql

Copy link
Owner

@sawantpritam sawantpritam left a comment

Choose a reason for hiding this comment

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

@Ragini2 Please work on review comments.


@CLDPGSQL @TC-Invalid-TestData-for-ReferenceName&ConnectionName
@cloudSQLPostgreSQL
Scenario: User is able to Open and enter invalid test data for Reference Name & Connection Name
Copy link
Owner

Choose a reason for hiding this comment

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

What are you validating in this scenarios ?

Copy link
Author

Choose a reason for hiding this comment

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

removed scenario as not valid

Copy link
Author

Choose a reason for hiding this comment

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

changed scenario name

@@ -38,7 +38,3 @@ Feature: CloudSQLPostGreSQL Sink Design Time
Then Verify Connection Name with private instance type
Copy link
Owner

Choose a reason for hiding this comment

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

are you validating any error message ?

Copy link
Author

Choose a reason for hiding this comment

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

added error validation

Then Enter Table Name "cloudPsqlTableName" and Connection Name
Then Validate Connector properties
Then Enter Driver Name with Invalid value
Then Verify Driver Name field with Invalid value entered
Copy link
Owner

Choose a reason for hiding this comment

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

Verify driver name field error for invalid value

Copy link
Author

Choose a reason for hiding this comment

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

done

@@ -15,23 +15,23 @@ Feature: CloudSQLPostGreSQL Source Design Time
| importQuery |
Copy link
Owner

Choose a reason for hiding this comment

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

alignment of | is not proper

Copy link
Author

Choose a reason for hiding this comment

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

done

Then Enter Connection Name and Import Query "cloudPsqlImportQuery"
Then Validate Connector properties
Then Enter Driver Name with Invalid value
Then Verify Driver Name field with Invalid value entered
Copy link
Owner

Choose a reason for hiding this comment

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

Same comment as Sink validation

Copy link
Author

Choose a reason for hiding this comment

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

done

}

@Then("Capture output schema")
public void captureOutputSchema() {
Copy link
Owner

Choose a reason for hiding this comment

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

Schema validation needs to be changed

Copy link
Author

Choose a reason for hiding this comment

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

done

Then Preview and run the pipeline
Then Verify the preview of pipeline is "success"
Then Click on PreviewData for cloudSQLPostgreSQL
Then Verify Preview output schema is not null
Copy link
Owner

Choose a reason for hiding this comment

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

Schema validation needs to be changed

Copy link
Author

Choose a reason for hiding this comment

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

Pending to change column

}

@Then("Verify Preview output schema is not null")
public void verifyPreviewOutputSchemaIsNotNull() {
Copy link
Owner

Choose a reason for hiding this comment

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

Schema validation needs to be changed

Copy link
Author

Choose a reason for hiding this comment

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

pending

CdfCloudSqlPostGreSqlActions.enterDatabaseName(CdapUtils.pluginProp("pSqlDatabaseName"));
CdfCloudSqlPostGreSqlActions.enterConnectionName(CdapUtils.pluginProp("pSqlConnectionNameInvalid"));
CdfCloudSqlPostGreSqlActions.enterTableName(CdapUtils.pluginProp("pSqlTableNameCS"));
CdfCloudSqlPostGreSqlActions.enterReferenceName(CdapUtils.pluginProp("cloudPsqlReferenceNameInvalid"));
Copy link
Owner

Choose a reason for hiding this comment

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

CdapUtils -> E2ETestUtils

Copy link
Author

Choose a reason for hiding this comment

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

done

ERROR_MESSAGE_SPLIT_COLUMN_NAME=Split-By Field Name must be specified if Number of Splits is not set to 1.
ERROR_MESSAGE_NUMBER_OF_SPLITS_NAME=Invalid value for numSplits '0'. Must be at least 1.
ERROR_MESSAGE_NUMBER_OF_BOUNDING_QUERY=Bounding Query must be specified if Number of Splits is not set to 1. No newline at end of file
ERROR_MESSAGE_NUMBER_OF_BOUNDING_QUERY=Bounding Query must be specified if Number of Splits is not set to 1.
Copy link
Owner

Choose a reason for hiding this comment

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

Add in E2EContants file

Copy link
Author

Choose a reason for hiding this comment

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

removed as not using

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants