diff --git a/dGame/dInventory/Item.cpp b/dGame/dInventory/Item.cpp
index eeb75ef47..ef03b62cb 100644
--- a/dGame/dInventory/Item.cpp
+++ b/dGame/dInventory/Item.cpp
@@ -384,7 +384,7 @@ void Item::UseNonEquip(Item* item) {
 			auto* packCompTable = CDClientManager::GetTable<CDPackageComponentTable>();
 			auto packages = packCompTable->Query([=](const CDPackageComponent entry) {return entry.id == static_cast<uint32_t>(packageComponentId); });
 
-			auto success = !packages.empty();
+			success = !packages.empty();
 			if (success) {
 				if (this->GetPreconditionExpression()->Check(playerInventoryComponent->GetParent())) {
 					auto* entityParent = playerInventoryComponent->GetParent();
diff --git a/tests/dGameTests/dComponentsTests/CMakeLists.txt b/tests/dGameTests/dComponentsTests/CMakeLists.txt
index c74ede502..57665c224 100644
--- a/tests/dGameTests/dComponentsTests/CMakeLists.txt
+++ b/tests/dGameTests/dComponentsTests/CMakeLists.txt
@@ -22,6 +22,7 @@ set(DCOMPONENTS_TESTS
 	"ReplicaDecoderTests.cpp"
 	"CapturePropertyTests.cpp"
 	"ItemRemovalTests.cpp"
+	"UseItemResultTests.cpp"
 	"LootMetricsTests.cpp"
 	"SpawnerComponentTests.cpp"
 )
diff --git a/tests/dGameTests/dComponentsTests/UseItemResultTests.cpp b/tests/dGameTests/dComponentsTests/UseItemResultTests.cpp
new file mode 100644
index 000000000..8f3af50c3
--- /dev/null
+++ b/tests/dGameTests/dComponentsTests/UseItemResultTests.cpp
@@ -0,0 +1,67 @@
+#include "GameDependencies.h"
+
+#include "CDClientManager.h"
+#include "CDComponentsRegistryTable.h"
+#include "CDItemComponentTable.h"
+#include "CDLootMatrixTable.h"
+#include "CDPackageComponentTable.h"
+#include "Entity.h"
+#include "InventoryComponent.h"
+#include "InventoryMessages.h"
+#include "Item.h"
+#include "eReplicaComponentType.h"
+
+#include "dGameMessagesTests/GameMessageTestUtils.h"
+
+#include <gtest/gtest.h>
+
+using namespace GameMessageTestUtils;
+
+// Opening a package: the client plays the item's use sound on a successful UseItemResult and a failure sound
+// otherwise; live sent true for packages that opened (183 of the 187 in the captures).
+class UseItemResultTests : public GameDependenciesTest {
+protected:
+	static constexpr LOT PACKAGE = 2998;
+	static constexpr uint32_t PACKAGE_COMPONENT = 97000;
+	static constexpr uint32_t LOOT_MATRIX = 97001;
+
+	std::unique_ptr<Entity> entity;
+	InventoryComponent* inventoryComponent{};
+
+	void SetUp() override {
+		SetUpDependencies();
+		const auto itemComponent = static_cast<uint32_t>(96000 + PACKAGE);
+		auto& registry = CDClientManager::GetEntriesMutable<CDComponentsRegistryTable>();
+		registry.insert_or_assign(static_cast<uint64_t>(info.lot), 0);
+		registry.insert_or_assign(static_cast<uint64_t>(PACKAGE), itemComponent);
+		registry.insert_or_assign(static_cast<uint64_t>(eReplicaComponentType::ITEM) << 32 | static_cast<uint64_t>(PACKAGE), itemComponent);
+		registry.insert_or_assign(static_cast<uint64_t>(eReplicaComponentType::PACKAGE) << 32 | static_cast<uint64_t>(PACKAGE), PACKAGE_COMPONENT);
+		CDItemComponent component{};
+		component.id = itemComponent;
+		CDClientManager::GetEntriesMutable<CDItemComponentTable>().insert_or_assign(itemComponent, component);
+		CDClientManager::GetEntriesMutable<CDPackageComponentTable>().push_back({ PACKAGE_COMPONENT, LOOT_MATRIX, 0 });
+		CDClientManager::GetEntriesMutable<CDLootMatrixTable>()[LOOT_MATRIX] = {}; // an empty package
+
+		entity = std::make_unique<Entity>(1, info);
+		inventoryComponent = entity->AddComponent<InventoryComponent>(-1);
+	}
+
+	void TearDown() override {
+		entity.reset();
+		CDClientManager::GetEntriesMutable<CDPackageComponentTable>().clear();
+		CDClientManager::GetEntriesMutable<CDLootMatrixTable>().erase(LOOT_MATRIX);
+		TearDownDependencies();
+	}
+};
+
+TEST_F(UseItemResultTests, AnOpenedPackageReportsSuccess) {
+	auto* const items = inventoryComponent->GetInventory(eInventoryType::ITEMS);
+	auto* const package = new Item(0x7000, PACKAGE, items, 0, 2, false, {}, LWOOBJID_EMPTY, LWOOBJID_EMPTY, eLootSourceType::NONE);
+
+	const auto results = SentGameMessages<GameMessages::UseItemResult>(Capture([&] { package->UseNonEquip(package); }));
+
+	ASSERT_EQ(results.size(), 1u);
+	EXPECT_EQ(results[0].itemTemplateID, PACKAGE);
+	EXPECT_TRUE(results[0].useItemResult);
+	EXPECT_EQ(package->GetCount(), 1u);
+}