aglinxinyuan commented on PR #7631: URL: https://github.com/apache/texera/pull/7631#issuecomment-5276944653
The red amber jobs here are not from this PR — `main` does not currently compile. `ComputingUnitManagingResourceSpec` asserts on `DashboardWorkflowComputingUnit.ownerGoogleAvatar`, but the field is named `ownerAvatar`, so `ComputingUnitManagingService / Test / compileIncremental` fails with two "value ownerGoogleAvatar is not a member" errors. The amber jobs build the whole test tree, so this takes them down on every open PR regardless of what it touches. This PR only adds four tests to `GmailResourceSpec`. Two PRs raced: #7563 renamed the field, #7580 added assertions against the old name, and each was green against its own base. Verified by stashing a fix on an otherwise-clean `main`: unpatched it exits 1 with exactly those two errors, patched it compiles and the spec's 31 tests pass. Fix is up as #7633 (two-line rename). Once that lands I will rebase this and CI should go green. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
