mengw15 commented on code in PR #7223:
URL: https://github.com/apache/texera/pull/7223#discussion_r3694813090
##########
amber/src/test/scala/org/apache/texera/web/resource/pythonvirtualenvironment/PveResourceSpec.scala:
##########
@@ -429,4 +429,42 @@ class PveResourceSpec
val resp = new PveResource().deletePveFromDb(-1, sessionUser)
resp.getStatus shouldBe Response.Status.NOT_FOUND.getStatusCode
}
+
+ "PveResource.getSystemPackages" should "wrap the manager's list under a
'system' key" in {
+ val result = new PveResource().getSystemPackages
+ result.keySet.asScala shouldBe Set("system")
+ result.get("system") shouldBe PveManager.getSystemPackages.toList.asJava
+ }
Review Comment:
Confirmed and fixed. `PveManager.systemPackages` is a `lazy val` resolved
via a mocked `pip freeze`, so in isolation this test was the first to force
resolution and hit an unexpected-mock failure (verified: `testOnly … -z "wrap
the manager"` failed before, passes now). Added `expectProcessCalls()` so it's
self-contained regardless of order.
--
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]
