This may be my opinion, but I believe InventoryPickupItemEvent should be cancelled when the item is not picked up. I do not see any other way to verify that the inventory did indeed pick up the item.

Currently if a hopper tries to pick up an item but has no space for it, the item will not be picked up and InventoryPickupItemEvent will fire without being cancelled.