@bongos/core 1.24.6 → 1.24.8

package/tests/project_settings_page.mjs

@@ -65,9 +65,11 @@ function buildRouterWithStubs({ timer, categories, reward = WORKING_REWARD }) {
     list: gc.listCategories,
     set: api.projectSettings.set,
     resolveShipMode: api.projectSettings.resolveShipMode,
+    resolveTweak: api.projectSettings.resolveTweakAutopickup,
   };
   api.resolveRotDays = async () => timer;
   api.projectSettings.resolveShipMode = async () => ({ mode: 'auto', source: 'default', chosen: false, invalid: null });
+  api.projectSettings.resolveTweakAutopickup = async () => ({ mode: 'off', cap: 200, rank_caps: {}, source: { mode: 'default', cap: 'default', rank_caps: 'default' }, invalid: [] });
   // reward: WORKING_REWARD (default) | null (port absent) | 'throws' (rewardConfig raises).
   const rewardPort = reward === 'throws'
     ? { rewardConfig: () => { throw new Error('reward boom'); } }
@@ -84,6 +86,7 @@ function buildRouterWithStubs({ timer, categories, reward = WORKING_REWARD }) {
     gc.listCategories = saved.list;
     api.projectSettings.set = saved.set;
     api.projectSettings.resolveShipMode = saved.resolveShipMode;
+    api.projectSettings.resolveTweakAutopickup = saved.resolveTweak;
   };
   // The handler is the LAST layer on the matching route (requireBuilder and the
   // permission gate sit in front); calling it directly is what keeps this a unit test
@@ -103,22 +106,22 @@ const CATS = [
   { id: 2, slug: 'retired-thing', label: 'Retired thing', sort_order: 20, active: false },
 ];
 
-test('one read returns all six fundamentals, each naming where its value came from', async () => {
+test('one read returns all seven fundamentals, each naming where its value came from', async () => {
   const { handlerFor, restore } = buildRouterWithStubs({ timer: TIMER_OK, categories: CATS });
   try {
     const res = fakeRes();
     await handlerFor('get', '/project-settings')({ query: {} }, res);
     const rows = res.payload.rows;
-    assert.equal(rows.length, 6, 'the page draws six rows, so the read returns six');
+    assert.equal(rows.length, 7, 'the page draws seven rows, so the read returns seven');
     assert.deepEqual(rows.map((r) => r.key),
-      ['ship_mode', 'rot_timer', 'work_categories', 'credit_calculation', 'equity_calculation', 'upmarket_percent']);
+      ['ship_mode', 'tweak_round_cap', 'rot_timer', 'work_categories', 'credit_calculation', 'equity_calculation', 'upmarket_percent']);
     // Provenance is the point of the page: every row says where its value came from.
     for (const r of rows) {
       assert.ok(r.reads && r.reads.file, `${r.key} names no file`);
       assert.ok(r.sentence && r.sentence.length > 20, `${r.key} has no readable sentence`);
     }
-    // Three knobs: the criterion's two, plus ship mode (task 1004628).
-    assert.deepEqual(rows.filter((r) => r.editable).map((r) => r.key), ['ship_mode', 'rot_timer', 'work_categories']);
+    // Four knobs: the criterion's two, ship mode (task 1004628) and the monthly tweak-round cap (task 1005003).
+    assert.deepEqual(rows.filter((r) => r.editable).map((r) => r.key), ['ship_mode', 'tweak_round_cap', 'rot_timer', 'work_categories']);
     // A no-store receipt: a cached settings read is wrong the moment a knob turns.
     assert.equal(res.headers['Cache-Control'], 'no-store');
   } finally { restore(); }