Page MenuHomePhabricator

[easy] Initialize self.client early in ManagedGrpcServerRepositoryLocationHandle so that __del__ won't raise an error if you interrupt a test
ClosedPublic

Authored by dgibson on Mon, Nov 16, 2:35 AM.

Details

Summary

This just eliminates a spammy "No property self.client" defined message if you interrupt a test and it calls del on an object that hasn't been created yet.

Test Plan

BK, run pytest on core_tests and interrupt partway through, no weird "missing property" error messages

Diff Detail

Repository
R1 dagster
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.