Skip to content

Commit 0512b43

Browse files
jcarlson212edoakes
authored andcommitted
Fix/ray serve test state api usage (ray-project#56948)
## Why are these changes needed? Fixes a unit test that was broken but not running in CI. ## Related issue number Fixes ray-project#53478 ## Checks - [y] I've signed off every commit(by using the -s flag, i.e., `git commit -s`) in this PR. - [y] I've run `scripts/format.sh` to lint the changes in this PR. - [y] I've included any doc changes needed for https://docs.ray.io/en/master/. - [ ] I've added any new APIs to the API Reference. For example, if I added a method in Tune, I've added it in `doc/source/tune/api/` under the corresponding `.rst` file. - [y] I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/. - Testing Strategy - [y] Unit tests - [ ] Release tests - [ ] This PR is not tested :( --------- Signed-off-by: Jason <jcarlson212@gmail.com> Co-authored-by: Edward Oakes <ed.nmi.oakes@gmail.com>
1 parent cbe6bec commit 0512b43

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

python/ray/serve/tests/test_metrics.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1082,7 +1082,7 @@ def f():
10821082

10831083
serve.run(f.bind(), name="app")
10841084
actors = list_actors(filters=[("state", "=", "ALIVE")])
1085-
class_names = {actor["class_name"] for actor in actors}
1085+
class_names = {actor.class_name for actor in actors}
10861086
assert class_names.issuperset(
10871087
{"ServeController", "ProxyActor", "ServeReplica:app:f"}
10881088
)

0 commit comments

Comments
 (0)