diff --git a/src/volcano.c b/src/volcano.c
index d035d4a23..9bcf62ff4 100644
--- a/src/volcano.c
+++ b/src/volcano.c
@@ -269,6 +269,7 @@ void volcano_update(void)
                 rsetterrain(r, T_VOLCANO);
             }
             else {
+                // TODO: this code path is inactive. are we only keeping it for old data? fix data instead.
                 if (rng_int() % 100 < 12) {
                     ADDMSG(&r->msgs, msg_message("volcanostopsmoke", "region", r));
                     rsetterrain(r, T_VOLCANO);
diff --git a/src/volcano.test.c b/src/volcano.test.c
index c53b39b69..91411b660 100644
--- a/src/volcano.test.c
+++ b/src/volcano.test.c
@@ -4,6 +4,11 @@
 
 #include <kernel/region.h>
 #include <kernel/terrain.h>
+
+#include <util/attrib.h>
+
+#include <attributes/reduceproduction.h>
+
 #include <CuTest.h>
 
 static void test_volcano_update(CuTest *tc) {
@@ -14,6 +19,7 @@ static void test_volcano_update(CuTest *tc) {
     t_active = test_create_terrain("activevolcano", LAND_REGION);
     test_cleanup();
     r = test_create_region(0, 0, t_active);
+    a_add(&r->attribs, make_reduceproduction(25, 10));
     volcano_update();
     CuAssertPtrEquals(tc, (void *)t_volcano, (void *)r->terrain);
     test_cleanup();