chore: fix rpc test not collected

[ryanking13]

I noticed that python-rpc/worker.py tests were not running at all because it had a global test function which we deprecated.

This migrates it to test method under the WorkerEntrypoint class.

[ask-bonk]

I'm Bonk, and I've done a quick review of your PR.

This PR migrates the RPC test from a deprecated global test handler to a test method on the Default(WorkerEntrypoint) class, and fixes assert_equal to accept dict subclasses.

Issues

1. Medium: Two test cases silently dropped. The old code tested that passing a lambda and a named def my_func to env.PythonRpc.identity() raises TypeError. These were removed without explanation. If they were dropped intentionally (e.g. the def inside with block was problematic), it would be good to note why — otherwise they should be kept.

[ask-bonk]

I'm Bonk, and I've done a quick review of your PR.

This PR migrates the RPC test from a deprecated global test handler to a test method on the Default(WorkerEntrypoint) class, and fixes assert_equal to accept dict subclasses.

I found one actionable issue and posted a review comment on the PR:

1. Medium: Two test cases silently dropped. The old code verified that passing a lambda and a named function (def my_func) to env.PythonRpc.identity() raises TypeError. These were removed in the migration without explanation. I posted a suggestion to restore at least the lambda test case at worker.py:293.

The rest of the changes (structural migration to class method, handler import removal, js_map[1] -> js_map["1"] key fix, assert_equal relaxation for dict subclasses) all look correct.

github run