• 
      

    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)