Skip to content

Prevent calling as_base_types on class objects in openhtf.util.data. - #1325

Merged
copybara-service[bot] merged 1 commit into
masterfrom
test_963431550
Aug 13, 2026
Merged

Prevent calling as_base_types on class objects in openhtf.util.data.#1325
copybara-service[bot] merged 1 commit into
masterfrom
test_963431550

Conversation

@copybara-service

Copy link
Copy Markdown
Contributor

Prevent calling as_base_types on class objects in openhtf.util.data.

Update openhtf.util.data to check that an object is not a class before calling its as_base_types method. This ensures that class objects themselves are converted to their string representations instead of attempting to call the method on the class. Also add corresponding unit tests.

@google-cla

google-cla Bot commented Aug 12, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@copybara-service
copybara-service Bot force-pushed the test_963431550 branch 2 times, most recently from bfb975e to 189e0e7 Compare August 13, 2026 00:23
Update openhtf.util.data to check that an object is not a class before calling its as_base_types method. This ensures that class objects themselves are converted to their string representations instead of attempting to call the method on the class. Also add corresponding unit tests.

PiperOrigin-RevId: 963749692
@copybara-service
copybara-service Bot merged commit 9bfeea8 into master Aug 13, 2026
10 of 11 checks passed
@copybara-service
copybara-service Bot deleted the test_963431550 branch August 13, 2026 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants