ic-mops 2.3.2 → 2.5.0

package/dist/commands/build.js

@@ -15,9 +15,12 @@ export async function build(canisterNames, options) {
     if (canisterNames?.length == 0) {
         cliError("No canisters specified to build");
     }
-    let outputDir = options.outputDir ?? DEFAULT_BUILD_OUTPUT_DIR;
-    let mocPath = await toolchain.bin("moc", { fallback: true });
     let config = readConfig();
+    let configOutputDir = config.build?.outputDir
+        ? resolveConfigPath(config.build.outputDir)
+        : undefined;
+    let outputDir = options.outputDir ?? configOutputDir ?? DEFAULT_BUILD_OUTPUT_DIR;
+    let mocPath = await toolchain.bin("moc", { fallback: true });
     let canisters = resolveCanisterConfigs(config);
     if (!Object.keys(canisters).length) {
         cliError(`No Motoko canisters found in mops.toml configuration`);
@@ -56,19 +59,7 @@ export async function build(canisterNames, options) {
             ...(await sourcesArgs()).flat(),
             ...getGlobalMocArgs(config),
         ];
-        if (config.build?.args) {
-            if (typeof config.build.args === "string") {
-                cliError(`[build] config 'args' should be an array of strings in mops.toml config file`);
-            }
-            args.push(...config.build.args);
-        }
-        if (canister.args) {
-            if (typeof canister.args === "string") {
-                cliError(`Canister config 'args' should be an array of strings for canister ${canisterName}`);
-            }
-            args.push(...canister.args);
-        }
-        args.push(...(options.extraArgs ?? []));
+        args.push(...collectExtraArgs(config, canister, canisterName, options.extraArgs));
         const isPublicCandid = true; // always true for now to reduce corner cases
         const candidVisibility = isPublicCandid ? "icp:public" : "icp:private";
         if (isPublicCandid) {
@@ -142,3 +133,36 @@ export async function build(canisterNames, options) {
     }
     console.log(chalk.green(`\n✓ Built ${Object.keys(filteredCanisters).length} canister${Object.keys(filteredCanisters).length == 1 ? "" : "s"} successfully`));
 }
+const managedFlags = {
+    "-o": "use [build].outputDir in mops.toml or --output flag instead",
+    "-c": "this flag is always set by mops build",
+    "--idl": "this flag is always set by mops build",
+    "--public-metadata": "this flag is managed by mops build",
+};
+function collectExtraArgs(config, canister, canisterName, extraArgs) {
+    const args = [];
+    if (config.build?.args) {
+        if (typeof config.build.args === "string") {
+            cliError(`[build] config 'args' should be an array of strings in mops.toml config file`);
+        }
+        args.push(...config.build.args);
+    }
+    if (canister.args) {
+        if (typeof canister.args === "string") {
+            cliError(`Canister config 'args' should be an array of strings for canister ${canisterName}`);
+        }
+        args.push(...canister.args);
+    }
+    if (extraArgs) {
+        args.push(...extraArgs);
+    }
+    const warned = new Set();
+    for (const arg of args) {
+        const hint = managedFlags[arg];
+        if (hint && !warned.has(arg)) {
+            warned.add(arg);
+            console.warn(chalk.yellow(`Warning: '${arg}' in args for canister ${canisterName} may conflict with mops build — ${hint}`));
+        }
+    }
+    return args;
+}

package/dist/commands/replica.d.ts

@@ -1,8 +1,7 @@
 import { ChildProcessWithoutNullStreams } from "node:child_process";
 import { PassThrough } from "node:stream";
 import { IDL } from "@icp-sdk/core/candid";
-import { PocketIc, PocketIcServer } from "pic-ic";
-import { PocketIc as PocketIcMops, PocketIcServer as PocketIcServerMops } from "pic-js-mops";
+import { type AnyPocketIcServer, type AnyPocketIc } from "../helpers/pocket-ic-client.js";
 type StartOptions = {
     type?: "dfx" | "pocket-ic" | "dfx-pocket-ic";
     dir?: string;
@@ -18,8 +17,8 @@ export declare class Replica {
         actor: any;
         stream: PassThrough;
     }>;
-    pocketIcServer?: PocketIcServer | PocketIcServerMops;
-    pocketIc?: PocketIc | PocketIcMops;
+    pocketIcServer?: AnyPocketIcServer;
+    pocketIc?: AnyPocketIc;
     dfxProcess?: ChildProcessWithoutNullStreams;
     dir: string;
     ttl: number;

package/dist/commands/replica.js

@@ -5,10 +5,8 @@ import fs from "node:fs";
 import { PassThrough } from "node:stream";
 import { spawn as spawnAsync } from "promisify-child-process";
 import { Actor, HttpAgent } from "@icp-sdk/core/agent";
-import { PocketIc, PocketIcServer } from "pic-ic";
-import { PocketIc as PocketIcMops, PocketIcServer as PocketIcServerMops, } from "pic-js-mops";
 import chalk from "chalk";
-import { readConfig } from "../mops.js";
+import { startPocketIc, } from "../helpers/pocket-ic-client.js";
 import { toolchain } from "./toolchain/index.js";
 import { getDfxVersion } from "../helpers/get-dfx-version.js";
 export class Replica {
@@ -74,32 +72,14 @@ export class Replica {
         }
         else {
             let pocketIcBin = await toolchain.bin("pocket-ic");
-            let config = readConfig();
-            if (config.toolchain?.["pocket-ic"] !== "4.0.0" &&
-                !config.toolchain?.["pocket-ic"]?.startsWith("9.")) {
-                console.error("Current Mops CLI only supports pocket-ic 9.x.x and 4.0.0");
-                process.exit(1);
-            }
-            // pocket-ic 9.x.x
-            if (config.toolchain?.["pocket-ic"]?.startsWith("9.")) {
-                this.pocketIcServer = await PocketIcServerMops.start({
-                    showRuntimeLogs: false,
-                    showCanisterLogs: false,
-                    binPath: pocketIcBin,
-                    ttl: this.ttl,
-                });
-                this.pocketIc = await PocketIcMops.create(this.pocketIcServer.getUrl());
-            }
-            // pocket-ic 4.0.0
-            else {
-                this.pocketIcServer = await PocketIcServer.start({
-                    showRuntimeLogs: false,
-                    showCanisterLogs: false,
-                    binPath: pocketIcBin,
-                    ttl: this.ttl,
-                });
-                this.pocketIc = await PocketIc.create(this.pocketIcServer.getUrl());
-            }
+            let pic = await startPocketIc({
+                binPath: pocketIcBin,
+                showRuntimeLogs: false,
+                showCanisterLogs: false,
+                ttl: this.ttl,
+            });
+            this.pocketIcServer = pic.server;
+            this.pocketIc = pic.client;
             // process canister logs
             this._attachCanisterLogHandler(this.pocketIcServer.serverProcess);
         }

package/dist/environments/web/cli.js

@@ -1,4 +1,13 @@
 import * as wasm from "../../wasm/pkg/web/wasm.js";
 import { setWasmBindings } from "../../wasm.js";
+import { readFileSync } from "node:fs";
+import { resolve } from "node:path";
+// Web wasm-pack target requires explicit initialization (unlike the nodejs
+// target which auto-inits on require). Load the CLI's own WASM binary and
+// call initSync before exposing the bindings.
+// In the bundle __dirname is defined as import.meta.dirname and the binary
+// sits next to cli.js; bundle:fix rewrites the path accordingly.
+const wasmBytes = readFileSync(resolve(__dirname, "../../wasm/pkg/web/wasm_bg.wasm"));
+wasm.initSync({ module: wasmBytes });
 setWasmBindings(wasm);
 export * from "../../cli.js";

package/dist/helpers/pocket-ic-client.d.ts

@@ -0,0 +1,9 @@
+import { PocketIc, PocketIcServer } from "pic-ic";
+import { PocketIc as PocketIcModern, PocketIcServer as PocketIcServerModern, type StartServerOptions } from "pic-js-mops";
+export type AnyPocketIcServer = PocketIcServer | PocketIcServerModern;
+export type AnyPocketIc = PocketIc | PocketIcModern;
+export type AnySetupCanister = PocketIc["setupCanister"] & PocketIcModern["setupCanister"];
+export declare function startPocketIc(options: StartServerOptions): Promise<{
+    server: AnyPocketIcServer;
+    client: AnyPocketIc;
+}>;

package/dist/helpers/pocket-ic-client.js

@@ -0,0 +1,18 @@
+import semver from "semver";
+import { PocketIc, PocketIcServer } from "pic-ic";
+import { PocketIc as PocketIcModern, PocketIcServer as PocketIcServerModern, } from "pic-js-mops";
+import { readConfig } from "../mops.js";
+function isLegacy() {
+    let version = readConfig().toolchain?.["pocket-ic"];
+    return !!version && !!semver.valid(version) && semver.lt(version, "9.0.0");
+}
+export async function startPocketIc(options) {
+    if (isLegacy()) {
+        let server = await PocketIcServer.start(options);
+        let client = await PocketIc.create(server.getUrl());
+        return { server, client };
+    }
+    let server = await PocketIcServerModern.start(options);
+    let client = await PocketIcModern.create(server.getUrl());
+    return { server, client };
+}

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ic-mops",
-  "version": "2.3.2",
+  "version": "2.5.0",
   "type": "module",
   "bin": {
     "mops": "bin/mops.js",

package/dist/parallel.js

@@ -1,8 +1,12 @@
 export async function parallel(threads, items, fn) {
-    return new Promise((resolve) => {
+    return new Promise((resolve, reject) => {
         let busyThreads = 0;
+        let failed = false;
         items = items.slice();
         let loop = () => {
+            if (failed) {
+                return;
+            }
             if (!items.length) {
                 if (busyThreads === 0) {
                     resolve();
@@ -16,6 +20,10 @@ export async function parallel(threads, items, fn) {
             fn(items.shift()).then(() => {
                 busyThreads--;
                 loop();
+            }, (err) => {
+                busyThreads--;
+                failed = true;
+                reject(err);
             });
             loop();
         };

package/dist/release-cli.js

@@ -8,8 +8,12 @@ import { sha256 } from "@noble/hashes/sha256";
 import { bytesToHex } from "@noble/hashes/utils";
 import { findChangelogEntry } from "./helpers/find-changelog-entry.js";
 let __dirname = new URL(".", import.meta.url).pathname;
-// build using Docker
-execSync("./build.sh", { stdio: "inherit", cwd: __dirname });
+if (!process.env.SKIP_BUILD) {
+    execSync("./build.sh", { stdio: "inherit", cwd: __dirname });
+}
+else if (!fs.existsSync(path.resolve(__dirname, "bundle/cli.tgz"))) {
+    throw new Error("SKIP_BUILD is set but bundle/cli.tgz does not exist. Run build.sh first.");
+}
 let commitHash = process.env.COMMIT_HASH || execSync("git rev-parse HEAD").toString().trim();
 let version = JSON.parse(fs.readFileSync(path.resolve(__dirname, "package.json"), "utf8")).version;
 let major = semver.parse(version)?.major;

package/dist/tests/build.test.js

@@ -1,22 +1,86 @@
-import { describe, expect, test } from "@jest/globals";
+import { describe, expect, jest, test } from "@jest/globals";
 import { execa } from "execa";
-import { existsSync } from "node:fs";
+import { existsSync, rmSync } from "node:fs";
 import path from "path";
-import { cliSnapshot } from "./helpers";
+import { cli, cliSnapshot } from "./helpers";
 const distBin = path.resolve(import.meta.dirname, "../dist/bin/mops.js");
+function cleanFixture(cwd, ...extras) {
+    rmSync(path.join(cwd, ".mops"), { recursive: true, force: true });
+    for (const p of extras) {
+        rmSync(p, { recursive: true, force: true });
+    }
+}
 describe("build", () => {
+    // Several dfx/pocket-ic builds per test; slow CI (e.g. node 20 matrix) can exceed 60s default.
+    jest.setTimeout(120_000);
     test("ok", async () => {
         const cwd = path.join(import.meta.dirname, "build/success");
-        await cliSnapshot(["build", "--verbose"], { cwd }, 0);
-        await cliSnapshot(["build", "foo"], { cwd }, 0);
-        await cliSnapshot(["build", "bar"], { cwd }, 0);
-        await cliSnapshot(["build", "foo", "bar"], { cwd }, 0);
+        try {
+            await cliSnapshot(["build", "--verbose"], { cwd }, 0);
+            await cliSnapshot(["build", "foo"], { cwd }, 0);
+            await cliSnapshot(["build", "bar"], { cwd }, 0);
+            await cliSnapshot(["build", "foo", "bar"], { cwd }, 0);
+        }
+        finally {
+            cleanFixture(cwd);
+        }
     });
     test("error", async () => {
         const cwd = path.join(import.meta.dirname, "build/error");
-        await cliSnapshot(["build", "foo", "--verbose"], { cwd }, 0);
-        expect((await cliSnapshot(["build", "bar"], { cwd }, 1)).stderr).toMatch("Candid compatibility check failed for canister bar");
-        expect((await cliSnapshot(["build", "foo", "bar"], { cwd }, 1)).stderr).toMatch("Candid compatibility check failed for canister bar");
+        try {
+            await cliSnapshot(["build", "foo", "--verbose"], { cwd }, 0);
+            expect((await cliSnapshot(["build", "bar"], { cwd }, 1)).stderr).toMatch("Candid compatibility check failed for canister bar");
+            expect((await cliSnapshot(["build", "foo", "bar"], { cwd }, 1)).stderr).toMatch("Candid compatibility check failed for canister bar");
+        }
+        finally {
+            cleanFixture(cwd);
+        }
+    });
+    // [build].outputDir in mops.toml should control where build output goes
+    test("custom output path via config outputDir", async () => {
+        const cwd = path.join(import.meta.dirname, "build/custom-output");
+        const customOut = path.join(cwd, "custom-out");
+        const customWasm = path.join(customOut, "main.wasm");
+        const customDid = path.join(customOut, "main.did");
+        const defaultDid = path.join(cwd, ".mops/.build/main.did");
+        try {
+            const result = await cli(["build"], { cwd });
+            expect(result.exitCode).toBe(0);
+            expect(existsSync(customWasm)).toBe(true);
+            expect(existsSync(customDid)).toBe(true);
+            expect(existsSync(defaultDid)).toBe(false);
+        }
+        finally {
+            cleanFixture(cwd, customOut);
+        }
+    });
+    // Regression: --output CLI option was silently ignored due to
+    // Commander storing it as options.output while build() read options.outputDir
+    test("--output CLI option", async () => {
+        const cwd = path.join(import.meta.dirname, "build/success");
+        const outputDir = path.join(cwd, "cli-output-test");
+        try {
+            const result = await cli(["build", "foo", "--output", outputDir], {
+                cwd,
+            });
+            expect(result.exitCode).toBe(0);
+            expect(existsSync(path.join(outputDir, "foo.wasm"))).toBe(true);
+            expect(existsSync(path.join(outputDir, "foo.did"))).toBe(true);
+        }
+        finally {
+            cleanFixture(cwd, outputDir);
+        }
+    });
+    test("warns when args contain managed flags", async () => {
+        const cwd = path.join(import.meta.dirname, "build/success");
+        const artifact = path.join(cwd, "x");
+        const artifactDid = path.join(cwd, "x.did");
+        try {
+            await cliSnapshot(["build", "foo", "--", "-o", "x", "-c", "--idl"], { cwd }, 1);
+        }
+        finally {
+            cleanFixture(cwd, artifact, artifactDid);
+        }
     });
     // Regression: bin/mops.js must route through environments/nodejs/cli.js
     // so that setWasmBindings() is called before any command runs.
@@ -25,12 +89,17 @@ describe("build", () => {
     const hasDistBin = existsSync(distBin);
     (hasDistBin ? test : test.skip)("wasm bindings initialized via dist entry point", async () => {
         const cwd = path.join(import.meta.dirname, "build/success");
-        const result = await execa("node", [distBin, "build", "foo"], {
-            cwd,
-            stdio: "pipe",
-            reject: false,
-        });
-        expect(result.stderr).not.toContain("Wasm bindings have not been set");
-        expect(result.exitCode).toBe(0);
+        try {
+            const result = await execa("node", [distBin, "build", "foo"], {
+                cwd,
+                stdio: "pipe",
+                reject: false,
+            });
+            expect(result.stderr).not.toContain("Wasm bindings have not been set");
+            expect(result.exitCode).toBe(0);
+        }
+        finally {
+            cleanFixture(cwd);
+        }
     });
 });

package/dist/tests/pocket-ic.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/tests/pocket-ic.test.js

@@ -0,0 +1,12 @@
+import { describe, test, expect } from "@jest/globals";
+import path from "path";
+import { cli } from "./helpers";
+describe("pocket-ic", () => {
+    test("runs tests with pocket-ic 12.0.0", async () => {
+        const cwd = path.join(import.meta.dirname, "pocket-ic");
+        const result = await cli(["test", "--reporter", "verbose", "--replica", "pocket-ic"], { cwd });
+        expect(result.stderr).not.toContain("is not supported");
+        expect(result.stderr).not.toContain("only supports pocket-ic 9.x.x and 4.0.0");
+        expect(result.exitCode).toBe(0);
+    });
+});

package/environments/web/cli.ts

@@ -1,5 +1,17 @@
 import * as wasm from "../../wasm/pkg/web/wasm.js";
 import { setWasmBindings } from "../../wasm.js";
+import { readFileSync } from "node:fs";
+import { resolve } from "node:path";
+
+// Web wasm-pack target requires explicit initialization (unlike the nodejs
+// target which auto-inits on require). Load the CLI's own WASM binary and
+// call initSync before exposing the bindings.
+// In the bundle __dirname is defined as import.meta.dirname and the binary
+// sits next to cli.js; bundle:fix rewrites the path accordingly.
+const wasmBytes = readFileSync(
+  resolve(__dirname, "../../wasm/pkg/web/wasm_bg.wasm"),
+);
+wasm.initSync({ module: wasmBytes });
 
 setWasmBindings(wasm);
 

package/helpers/pocket-ic-client.ts

@@ -0,0 +1,32 @@
+import semver from "semver";
+import { PocketIc, PocketIcServer } from "pic-ic";
+import {
+  PocketIc as PocketIcModern,
+  PocketIcServer as PocketIcServerModern,
+  type StartServerOptions,
+} from "pic-js-mops";
+import { readConfig } from "../mops.js";
+
+export type AnyPocketIcServer = PocketIcServer | PocketIcServerModern;
+export type AnyPocketIc = PocketIc | PocketIcModern;
+export type AnySetupCanister = PocketIc["setupCanister"] &
+  PocketIcModern["setupCanister"];
+
+function isLegacy(): boolean {
+  let version = readConfig().toolchain?.["pocket-ic"];
+  return !!version && !!semver.valid(version) && semver.lt(version, "9.0.0");
+}
+
+export async function startPocketIc(
+  options: StartServerOptions,
+): Promise<{ server: AnyPocketIcServer; client: AnyPocketIc }> {
+  if (isLegacy()) {
+    let server = await PocketIcServer.start(options);
+    let client = await PocketIc.create(server.getUrl());
+    return { server, client };
+  }
+
+  let server = await PocketIcServerModern.start(options);
+  let client = await PocketIcModern.create(server.getUrl());
+  return { server, client };
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ic-mops",
-  "version": "2.3.2",
+  "version": "2.5.0",
   "type": "module",
   "bin": {
     "mops": "dist/bin/mops.js",
@@ -36,8 +36,8 @@
     "build:wasm:web": "wasm-pack build wasm --target web --out-dir pkg/web && rm wasm/pkg/web/.gitignore",
     "dist": "npm run build:wasm && tsc && mkdir -p dist/wasm && cp -r wasm/pkg dist/wasm",
     "bundle": "rm -rf ./bundle && bun build ./environments/web/cli.ts --outdir ./bundle --target node --minify --external @napi-rs/lzma --external fsevents --format esm --define '__dirname=import.meta.dirname' && run-s bundle:fix bundle:copy bundle:package-json bundle:tar",
-    "bundle:fix": "rexreplace 'new URL\\(\"\\.\\./templates' 'new URL(\"./templates' bundle/cli.js && rexreplace 'resolve\\(\".*?/xhr-sync-worker\\.js\"\\)' 'resolve(\"./xhr-sync-worker.js\")' bundle/cli.js && rexreplace '\"import.meta.dirname\",\"wasm_bg.wasm\"' 'import.meta.dirname || new URL(\".\", import.meta.url).pathname,\"wasm_bg.wasm\"' bundle/cli.js && rexreplace 'resolve\\(\"import.meta.dirname\",\".*?/default-stylesheet\\.css\"\\)' 'resolve(import.meta.dirname,\"./default-stylesheet.css\")' bundle/cli.js",
-    "bundle:copy": "cp -r commands/bench bundle && cp -r bin declarations templates package.json bundle && cp -r node_modules/prettier-plugin-motoko/wasm/pkg/nodejs/wasm_bg.wasm node_modules/jsdom/lib/jsdom/living/xhr/xhr-sync-worker.js node_modules/jsdom/lib/jsdom/browser/default-stylesheet.css bundle",
+    "bundle:fix": "rexreplace 'new URL\\(\"\\.\\./templates' 'new URL(\"./templates' bundle/cli.js && rexreplace 'resolve\\(\".*?/xhr-sync-worker\\.js\"\\)' 'resolve(\"./xhr-sync-worker.js\")' bundle/cli.js && rexreplace '\"import.meta.dirname\",\"wasm_bg.wasm\"' 'import.meta.dirname || new URL(\".\", import.meta.url).pathname,\"wasm_bg.wasm\"' bundle/cli.js && rexreplace 'resolve\\(\"import.meta.dirname\",\".*?/default-stylesheet\\.css\"\\)' 'resolve(import.meta.dirname,\"./default-stylesheet.css\")' bundle/cli.js && rexreplace '\"import.meta.dirname\",\"../../wasm/pkg/web/wasm_bg.wasm\"' 'import.meta.dirname || new URL(\".\", import.meta.url).pathname,\"mops_wasm_bg.wasm\"' bundle/cli.js",
+    "bundle:copy": "cp -r commands/bench bundle && cp -r bin declarations templates package.json bundle && cp wasm/pkg/web/wasm_bg.wasm bundle/mops_wasm_bg.wasm && cp -r node_modules/prettier-plugin-motoko/wasm/pkg/nodejs/wasm_bg.wasm node_modules/jsdom/lib/jsdom/living/xhr/xhr-sync-worker.js node_modules/jsdom/lib/jsdom/browser/default-stylesheet.css bundle",
     "bundle:package-json": "tsx bundle-package-json.ts",
     "bundle:tar": "rm -f bundle/cli.tgz && touch -t 200101010101 bundle/cli.tgz && find bundle -exec touch -d '1970-01-01 00:00:00' {} + && tar --sort name --exclude bundle/cli.tgz -cvf - bundle | gzip -n > bundle/cli.tgz",
     "copy": "cp -r commands/bench dist/commands && cp -r declarations templates package.json bin dist | true",

package/parallel.ts

@@ -3,11 +3,15 @@ export async function parallel<T>(
   items: T[],
   fn: (item: T) => Promise<void>,
 ) {
-  return new Promise<void>((resolve) => {
+  return new Promise<void>((resolve, reject) => {
     let busyThreads = 0;
+    let failed = false;
     items = items.slice();
 
     let loop = () => {
+      if (failed) {
+        return;
+      }
       if (!items.length) {
         if (busyThreads === 0) {
           resolve();
@@ -18,10 +22,17 @@ export async function parallel<T>(
         return;
       }
       busyThreads++;
-      fn(items.shift() as T).then(() => {
-        busyThreads--;
-        loop();
-      });
+      fn(items.shift() as T).then(
+        () => {
+          busyThreads--;
+          loop();
+        },
+        (err) => {
+          busyThreads--;
+          failed = true;
+          reject(err);
+        },
+      );
       loop();
     };
     loop();

package/release-cli.ts

@@ -11,8 +11,13 @@ import { findChangelogEntry } from "./helpers/find-changelog-entry.js";
 
 let __dirname = new URL(".", import.meta.url).pathname;
 
-// build using Docker
-execSync("./build.sh", { stdio: "inherit", cwd: __dirname });
+if (!process.env.SKIP_BUILD) {
+  execSync("./build.sh", { stdio: "inherit", cwd: __dirname });
+} else if (!fs.existsSync(path.resolve(__dirname, "bundle/cli.tgz"))) {
+  throw new Error(
+    "SKIP_BUILD is set but bundle/cli.tgz does not exist. Run build.sh first.",
+  );
+}
 
 let commitHash =
   process.env.COMMIT_HASH || execSync("git rev-parse HEAD").toString().trim();

package/tests/__snapshots__/build.test.ts.snap

@@ -75,3 +75,15 @@ build canister bar
 ✓ Built 2 canisters successfully",
 }
 `;
+
+exports[`build warns when args contain managed flags 1`] = `
+{
+  "exitCode": 1,
+  "stderr": "Warning: '-o' in args for canister foo may conflict with mops build — use [build].outputDir in mops.toml or --output flag instead
+Warning: '-c' in args for canister foo may conflict with mops build — this flag is always set by mops build
+Warning: '--idl' in args for canister foo may conflict with mops build — this flag is always set by mops build
+Error while compiling canister foo
+ENOENT: no such file or directory, open '.mops/.build/foo.did'",
+  "stdout": "build canister foo",
+}
+`;

package/tests/build/custom-output/dfx.json

@@ -0,0 +1,8 @@
+{
+  "canisters": {
+    "main": {
+      "type": "motoko",
+      "main": "src/Main.mo"
+    }
+  }
+}

package/tests/build/custom-output/mops.toml

@@ -0,0 +1,8 @@
+[dependencies]
+core = "1.0.0"
+
+[canisters.main]
+main = "src/Main.mo"
+
+[build]
+outputDir = "custom-out"

package/tests/build/custom-output/src/Main.mo

@@ -0,0 +1,5 @@
+persistent actor {
+  public func greet(name : Text) : async Text {
+    "Hello, " # name # "!";
+  };
+};