Fix up compatibility issues with RBTools 6+

Review Request #14306 — Created Jan. 24, 2025 and submitted

Information

ReviewBot
master

Reviewers

With the API changes in RBTools 6, a few of the things we were doing for
mocking with unit tests were failing. There were two types of errors
being hit:

  1. We now handle NotImplementedError for cases where an API payload
    does not contain a link, instead of just assuming that the method
    won't exist on the class. This is necessary because we now have API
    stubs present on the resource classes.
  2. Always pass in a payload and token for ListResource instantiation.

Ran unit tests.

Summary ID
Fix up compatibility issues with RBTools 6+
With the API changes in RBTools 6, a few of the things we were doing for mocking with unit tests were failing. There were two types of errors being hit: 1. We now handle `NotImplementedError` for cases where an API payload does not contain a link, instead of just assuming that the method won't exist on the class. This is necessary because we now have API stubs present on the resource classes. 2. Always pass in a payload and token for ListResource instantiation. Testing Done: Ran unit tests.
wutktnmqyztqtoltwnymmzpmsmusynkv
chipx86
  1. Ship It!
  2. 
      
david
Review request changed
Status:
Completed
Change Summary:
Pushed to master (06819c7)
Loading...