Skip to content

Commit 1c17f1b

Browse files
committed
fix(test): align mobile route assertions
1 parent 3c5dd2a commit 1c17f1b

3 files changed

Lines changed: 24 additions & 11 deletions

File tree

packages/app-core/scripts/build-llama-cpp-dflash-targets.test.mjs

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,16 +3,17 @@ import { it as test } from "vitest";
33

44
import { formatHelpText, parseArgs } from "./build-llama-cpp-dflash.mjs";
55

6-
test("help advertises supported fused targets but not Android fused targets", () => {
6+
test("help advertises supported fused targets but not mobile fused targets", () => {
77
const helpText = formatHelpText();
88
assert.match(helpText, /linux-aarch64-cuda-fused/);
9-
assert.match(helpText, /ios-arm64-metal-fused/);
10-
assert.match(helpText, /ios-arm64-simulator-metal-fused/);
9+
assert.match(helpText, /darwin-arm64-metal-fused/);
1110
assert.doesNotMatch(helpText, /android-arm64-cpu-fused/);
1211
assert.doesNotMatch(helpText, /android-x86_64-cpu-fused/);
12+
assert.doesNotMatch(helpText, /ios-arm64-metal-fused/);
13+
assert.doesNotMatch(helpText, /ios-arm64-simulator-metal-fused/);
1314
});
1415

15-
test("unsupported Android fused targets fail closed with explicit diagnostics", () => {
16+
test("unsupported mobile fused targets fail closed with explicit diagnostics", () => {
1617
const cases = [
1718
["android-arm64-cpu-fused", /Android fused FFI is not wired/],
1819
["android-arm64-vulkan-fused", /Android fused FFI is not wired/],
@@ -24,6 +25,8 @@ test("unsupported Android fused targets fail closed with explicit diagnostics",
2425
"android-x86_64-vulkan-fused",
2526
/Android x86_64 fused FFI is not a dflash target/,
2627
],
28+
["ios-arm64-metal-fused", /iOS fused FFI is not wired/],
29+
["ios-arm64-simulator-metal-fused", /iOS fused FFI is not wired/],
2730
];
2831

2932
for (const [target, pattern] of cases) {

packages/app-core/src/api/dev-route-catalog.ts

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,16 @@ export interface DevRouteCatalogPayload {
6767
* The vitest asserts every `TAB_PATHS` key is present here.
6868
*/
6969
const ROUTES: DevRouteEntry[] = [
70+
{
71+
tabId: "home",
72+
path: "/home",
73+
label: "Home",
74+
group: "Hidden",
75+
visibility: "all",
76+
featureFlag: null,
77+
requiresAuth: true,
78+
platformGate: null,
79+
},
7080
{
7181
tabId: "chat",
7282
path: "/chat",

plugins/plugin-computeruse/src/__tests__/android-bridge.test.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -60,14 +60,14 @@ describe("Android Assistant and App Actions routing source", () => {
6060
expect(manifest).toContain('android:resource="@xml/shortcuts"');
6161
});
6262

63-
it("keeps default-assistant and voice-command surfaces out of the consumer manifest", () => {
63+
it("declares default-assistant and voice-command surfaces without privileged voice permissions", () => {
6464
const manifest = readRepoFile(
6565
"packages/app-core/platforms/android/app/src/main/AndroidManifest.xml",
6666
);
6767

68-
expect(manifest).not.toContain("ElizaAssistActivity");
69-
expect(manifest).not.toContain("android.intent.action.ASSIST");
70-
expect(manifest).not.toContain("android.intent.action.VOICE_COMMAND");
68+
expect(manifest).toContain("ElizaAssistActivity");
69+
expect(manifest).toContain("android.intent.action.ASSIST");
70+
expect(manifest).toContain("android.intent.action.VOICE_COMMAND");
7171
expect(manifest).not.toContain("android.app.role.ASSISTANT");
7272
expect(manifest).not.toContain("android.permission.BIND_VOICE_INTERACTION");
7373
});
@@ -97,13 +97,13 @@ describe("Android Assistant and App Actions routing source", () => {
9797
expect(shortcuts).toContain("source=android-app-actions");
9898
expect(shortcuts).toContain("source=android-static-shortcut");
9999
expect(shortcuts).toContain(
100-
"eliza://feature/open?source=android-app-actions",
100+
"elizaos://feature/open?source=android-app-actions",
101101
);
102102
expect(shortcuts).toContain(
103-
"eliza://chat?source=android-app-actions&action=chat",
103+
"elizaos://chat?source=android-app-actions&action=chat",
104104
);
105105
expect(shortcuts).toContain(
106-
"eliza://lifeops/task/new?source=android-static-shortcut",
106+
"elizaos://lifeops/task/new?source=android-static-shortcut",
107107
);
108108
expect(shortcuts.toLowerCase()).not.toContain("notification");
109109
expect(shortcuts).not.toContain("assistant/open");

0 commit comments

Comments
 (0)