Commit 8f00bc9d23

Jakub Konka <kubkon@jakubkonka.com>
2022-06-25 10:57:56
link-tests: put macho search strategy tests into one test case
1 parent dfdb807
Changed files (7)
test
link
macho
search_paths_first
search_strategy
test/link/macho/search_paths_first/a.c
@@ -1,7 +0,0 @@
-#include <stdio.h>
-
-char world[] = "world";
-
-char* hello() {
-  return "Hello";
-}
test/link/macho/search_paths_first/build.zig
@@ -1,41 +0,0 @@
-const std = @import("std");
-const Builder = std.build.Builder;
-
-pub fn build(b: *Builder) void {
-    const mode = b.standardReleaseOptions();
-
-    const test_step = b.step("test", "Test");
-    test_step.dependOn(b.getInstallStep());
-
-    const static = b.addStaticLibrary("a", null);
-    static.setBuildMode(mode);
-    static.addCSourceFile("a.c", &.{});
-    static.linkLibC();
-    static.override_dest_dir = std.build.InstallDir{
-        .custom = "static",
-    };
-    static.install();
-
-    const dylib = b.addSharedLibrary("a", null, b.version(1, 0, 0));
-    dylib.setBuildMode(mode);
-    dylib.addCSourceFile("a.c", &.{});
-    dylib.linkLibC();
-    dylib.override_dest_dir = std.build.InstallDir{
-        .custom = "dynamic",
-    };
-    dylib.install();
-
-    const exe = b.addExecutable("main", null);
-    exe.setBuildMode(mode);
-    exe.addCSourceFile("main.c", &.{});
-    exe.linkSystemLibraryName("a");
-    exe.linkLibC();
-    exe.addLibraryPath(b.pathFromRoot("zig-out/static"));
-    exe.addLibraryPath(b.pathFromRoot("zig-out/dynamic"));
-    exe.search_strategy = .paths_first;
-
-    const run = exe.run();
-    run.cwd = b.pathFromRoot(".");
-    run.expectStdOutEqual("Hello world");
-    test_step.dependOn(&run.step);
-}
test/link/macho/search_paths_first/main.c
@@ -1,9 +0,0 @@
-#include <stdio.h>
-
-char* hello();
-extern char world[];
-
-int main() {
-  printf("%s %s", hello(), world);
-  return 0;
-}
test/link/macho/search_dylibs_first/a.c โ†’ test/link/macho/search_strategy/a.c
File renamed without changes
test/link/macho/search_dylibs_first/build.zig โ†’ test/link/macho/search_strategy/build.zig
@@ -1,5 +1,6 @@
 const std = @import("std");
 const Builder = std.build.Builder;
+const LibExeObjectStep = std.build.LibExeObjStep;
 
 pub fn build(b: *Builder) void {
     const mode = b.standardReleaseOptions();
@@ -7,6 +8,36 @@ pub fn build(b: *Builder) void {
     const test_step = b.step("test", "Test");
     test_step.dependOn(b.getInstallStep());
 
+    {
+        // -search_dylibs_first
+        const exe = createScenario(b, mode);
+        exe.search_strategy = .dylibs_first;
+
+        const check = exe.checkObject(.macho);
+        check.checkStart("cmd LOAD_DYLIB");
+        check.checkNext("name @rpath/liba.dylib");
+
+        test_step.dependOn(&check.step);
+
+        const run = exe.run();
+        run.cwd = b.pathFromRoot(".");
+        run.expectStdOutEqual("Hello world");
+        test_step.dependOn(&run.step);
+    }
+
+    {
+        // -search_paths_first
+        const exe = createScenario(b, mode);
+        exe.search_strategy = .paths_first;
+
+        const run = exe.run();
+        run.cwd = b.pathFromRoot(".");
+        run.expectStdOutEqual("Hello world");
+        test_step.dependOn(&run.step);
+    }
+}
+
+fn createScenario(b: *Builder, mode: std.builtin.Mode) *LibExeObjectStep {
     const static = b.addStaticLibrary("a", null);
     static.setBuildMode(mode);
     static.addCSourceFile("a.c", &.{});
@@ -33,16 +64,5 @@ pub fn build(b: *Builder) void {
     exe.addLibraryPath(b.pathFromRoot("zig-out/static"));
     exe.addLibraryPath(b.pathFromRoot("zig-out/dynamic"));
     exe.addRPath(b.pathFromRoot("zig-out/dynamic"));
-    exe.search_strategy = .dylibs_first;
-
-    const check = exe.checkObject(.macho);
-    check.checkStart("cmd LOAD_DYLIB");
-    check.checkNext("name @rpath/liba.dylib");
-
-    test_step.dependOn(&check.step);
-
-    const run = exe.run();
-    run.cwd = b.pathFromRoot(".");
-    run.expectStdOutEqual("Hello world");
-    test_step.dependOn(&run.step);
+    return exe;
 }
test/link/macho/search_dylibs_first/main.c โ†’ test/link/macho/search_strategy/main.c
File renamed without changes
test/link.zig
@@ -61,11 +61,7 @@ pub fn addCases(cases: *tests.StandaloneContext) void {
             .build_modes = true,
         });
 
-        cases.addBuildFile("test/link/macho/search_paths_first/build.zig", .{
-            .build_modes = true,
-        });
-
-        cases.addBuildFile("test/link/macho/search_dylibs_first/build.zig", .{
+        cases.addBuildFile("test/link/macho/search_strategy/build.zig", .{
             .build_modes = true,
         });
     }