Skip to content

Testing: Item Movement #10253

@bhollis

Description

@bhollis

There are a number of bugs or at least weird corner cases with how item movement works. I'd love for us to fix them, but the item movement logic is very complex and the chances of any fix breaking something else is high.

To make fixes possible, we should have a robust test suite for item movement. That will allow us to change things without fear of breaking existing behavior, and allow us to reproduce bugs in a test environment and be sure they're fixed.

There are a few steps here:

  1. Create an inventory test fixture that lets us easily configure different scenarios. This can start with my profile export but then create conditions like varying levels of free space in different buckets, different stacks of consumables, etc. It should be simple to set up states that allow us to reproduce bugs.
  2. Set up some sort of mocking for the Bungie.net transfer/lock APIs so we can simulate different outcomes.
  3. Build out a suite of tests for existing item movement scenarios, guided by code coverage and comments to try and capture all the edge cases that are currently handled.
  4. Write tests that reproduce our open bugs and then fix them. See the "Feature: Item Movement" label.

I would love help with this, as I don't feel like I have time to really make progress on it.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions