fix: only add cache to manifest if set

also add a new test for the order of internal functions

node/__snapshots__/declaration.test.ts.snap

@@ -0,0 +1,38 @@
+// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
+
+exports[`Deploy config takes precedence over user config 1`] = `
+[
+  {
+    "function": "user-a",
+    "path": "/path1",
+  },
+  {
+    "function": "user-b",
+    "path": "/path2",
+  },
+  {
+    "function": "framework-a",
+    "path": "/path1",
+  },
+  {
+    "function": "framework-b",
+    "path": "/path2",
+  },
+  {
+    "function": "user-c",
+    "path": "/path1",
+  },
+  {
+    "function": "user-c",
+    "path": "/path2",
+  },
+  {
+    "function": "framework-c",
+    "path": "/path1",
+  },
+  {
+    "function": "framework-c",
+    "path": "/path2",
+  },
+]
+`;

node/declaration.test.ts

@@ -3,9 +3,27 @@ import { test, expect } from 'vitest'
 import { FunctionConfig } from './config.js'
 import { Declaration, mergeDeclarations } from './declaration.js'
 
-// TODO: Add tests with the deploy config.
 const deployConfigDeclarations: Declaration[] = []
 
+test('Deploy config takes precedence over user config', () => {
+  const deployConfigDeclarations: Declaration[] = [
+    { function: 'framework-a', path: '/path1' },
+    { function: 'framework-b', path: '/path2' },
+  ]
+
+  const tomlConfig: Declaration[] = [
+    { function: 'user-a', path: '/path1' },
+    { function: 'user-b', path: '/path2' },
+  ]
+
+  const funcConfig = {
+    'user-c': { path: ['/path1', '/path2'] },
+    'framework-c': { path: ['/path1', '/path2'] },
+  } as Record<string, FunctionConfig>
+
+  expect(mergeDeclarations(tomlConfig, funcConfig, deployConfigDeclarations)).toMatchSnapshot()
+})
+
 test('In-source config takes precedence over netlify.toml config', () => {
   const tomlConfig: Declaration[] = [
     { function: 'geolocation', path: '/geo', cache: 'off' },

node/declaration.ts

@@ -67,7 +67,11 @@ export const mergeDeclarations = (
       const paths = Array.isArray(path) ? path : [path]
 
       paths.forEach((singlePath) => {
-        declarations.push({ cache, function: name, path: singlePath })
+        const declaration: Declaration = { function: name, path: singlePath }
+        if (cache) {
+          declaration.cache = cache
+        }
+        declarations.push(declaration)
       })
     }
   }