From 6e5ccc14567cd469f1806e3f098d26fc120b139f Mon Sep 17 00:00:00 2001 From: Xinyuan Lin Date: Wed, 12 Aug 2026 23:42:12 -0700 Subject: [PATCH] fix(computing-unit): repair the owner-avatar accessor in the spec main does not compile. ComputingUnitManagingResourceSpec asserts on DashboardWorkflowComputingUnit.ownerGoogleAvatar, but that field is named ownerAvatar - so ComputingUnitManagingService/Test fails with two "value ownerGoogleAvatar is not a member" errors, which takes down every amber and amber-integration job. Two PRs raced to produce it: #7563 renamed the field to ownerAvatar while #7580 added assertions written against the old name, and each was green against its own base. Renames the two accessor calls. Nothing else changes; the spec's 31 tests pass, so the assertions were correct about the value all along. --- .../service/resource/ComputingUnitManagingResourceSpec.scala | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/computing-unit-managing-service/src/test/scala/org/apache/texera/service/resource/ComputingUnitManagingResourceSpec.scala b/computing-unit-managing-service/src/test/scala/org/apache/texera/service/resource/ComputingUnitManagingResourceSpec.scala index 5b7c64104ad..4cbb0e0781c 100644 --- a/computing-unit-managing-service/src/test/scala/org/apache/texera/service/resource/ComputingUnitManagingResourceSpec.scala +++ b/computing-unit-managing-service/src/test/scala/org/apache/texera/service/resource/ComputingUnitManagingResourceSpec.scala @@ -176,7 +176,7 @@ class ComputingUnitManagingResourceSpec info.isOwner shouldBe true info.accessPrivilege shouldBe PrivilegeEnum.WRITE info.ownerName shouldBe "owner" - info.ownerGoogleAvatar shouldBe "owner-avatar" + info.ownerAvatar shouldBe "owner-avatar" } it should "report READ access for a grantee holding READ access" in { @@ -282,7 +282,7 @@ class ComputingUnitManagingResourceSpec created.accessPrivilege shouldBe PrivilegeEnum.WRITE created.ownerName shouldBe "creator" // makeUser sets no avatar, so the owner-avatar lookup resolves to null - created.ownerGoogleAvatar shouldBe null + created.ownerAvatar shouldBe null // The unit is persisted, not just echoed back unitName(cuid) shouldBe "local-cu"