Fix failing schema compare integration tests (#15500)

* fix failing schema compare integration tests

* comment out check and add TODO
This commit is contained in:
Kim Santiago
2021-05-17 17:27:03 -07:00
committed by GitHub
parent e0d8b67fe4
commit 08ae25d7ba

View File

@@ -492,8 +492,9 @@ function assertIncludeExcludeResult(result: mssql.SchemaCompareIncludeExcludeRes
function assertSchemaCompareResult(schemaCompareResult: mssql.SchemaCompareResult, operationId: string, expectedDifferenceCount: number, expectedIfEqual: boolean = false): void {
assert(schemaCompareResult.areEqual === expectedIfEqual, `Expected: the schemas equivalency to be ${expectedIfEqual} Actual: ${schemaCompareResult.areEqual}`);
assert(schemaCompareResult.errorMessage === null, `Expected: there should be no error. Actual Error message: "${schemaCompareResult.errorMessage}"`);
assert(schemaCompareResult.success === true, `Expected: success in schema compare. Actual: Failure`);
// TODO: uncomment this after fix to only return errors, not warnings, gets checking into sqltoolsservice
// assert(schemaCompareResult.errorMessage === null, `Expected: there should be no error for comparison. Actual Error message: "${schemaCompareResult.errorMessage}"`);
assert(schemaCompareResult.differences.length === expectedDifferenceCount, `Expected: ${expectedDifferenceCount} differences. Actual differences: "${schemaCompareResult.differences.length}"`);
assert(schemaCompareResult.operationId === operationId, `Operation Id Expected to be same as passed. Expected : ${operationId}, Actual ${schemaCompareResult.operationId}`);
}