test: run common tests file

Author: AMythicDev

build.zig

@@ -74,7 +74,7 @@ fn addTestRunner(b: *std.Build, target: std.Build.ResolvedTarget, optimize: std.
     const run_zig_tests = b.addRunArtifact(zig_tests);
 
     const common_tests = b.addTest(.{
-        .root_source_file = b.path("src/common/root.zig"),
+        .root_source_file = b.path("src/common/tests.zig"),
         .target = target,
         .optimize = optimize,
     });

src/common/overrides.zig

@@ -90,7 +90,7 @@ pub fn write_overrides(overrides: OverrideMap, cp: CommonPaths) !void {
     try cp.overrides.setEndPos(0);
     var file_writer = std.io.bufferedWriter(cp.overrides.writer());
 
-    try json.stringify(json.Value{ .object = overrides.backing_map }, .{ .whitespace = .indent_4 }, file_writer.writer());
+    try json.stringify(json.Value{ .object = overrides.backing_map }, .{ .whitespace = .indent_2 }, file_writer.writer());
     _ = try file_writer.write("\n");
     try file_writer.flush();
 }

src/common/root.zig

@@ -75,7 +75,7 @@ pub const Release = struct {
         }
     }
 
-    pub fn releasefromVersion(v: []const u8) RelError!Self {
+    pub fn releaseFromVersion(v: []const u8) RelError!Self {
         var rel: Release = undefined;
         if (streql(v, "master"))
             rel = Self{ .spec = .Master }
@@ -113,16 +113,12 @@ pub fn target_name() []const u8 {
     return @tagName(default_arch) ++ "-" ++ @tagName(default_os);
 }
 
-pub fn streql(cmd: []const u8, key: []const u8) bool {
-    return std.mem.eql(u8, cmd, key);
+pub fn streql(lhs: []const u8, rhs: []const u8) bool {
+    return std.mem.eql(u8, lhs, rhs);
 }
 
 pub fn release_name(alloc: Allocator, rel: Release) ![]const u8 {
     const release_string = rel.releaseName();
     const dw_target = comptime target_name();
     return try std.mem.concat(alloc, u8, &[_][]const u8{ "zig-" ++ dw_target ++ "-", release_string });
 }
-
-test {
-    std.testing.refAllDecls(@This());
-}

src/common/tests.zig

@@ -18,6 +18,7 @@ fn make_mock_releases(alloc: Allocator) !std.StringArrayHashMap(json.Value) {
     _ = try releases_map.fetchPut("0.10.1", .null);
     _ = try releases_map.fetchPut("0.11.0", .null);
     _ = try releases_map.fetchPut("0.12.0", .null);
+    _ = try releases_map.fetchPut("0.13.0", .null);
     _ = try releases_map.fetchPut("master", .null);
 
     return releases_map;
@@ -45,7 +46,7 @@ test "Resolve patch version with no patch releases" {
     defer releases_map.deinit();
     const releases = json.Value{ .object = releases_map };
 
-    var rel = try Release.releasefromVersion("0.12");
+    var rel = try Release.releaseFromVersion("0.12");
     try rel.resolve(releases);
 
     const v = try rel.actualVersion(alloc);
@@ -60,7 +61,7 @@ test "Resolve patch version with patch releases" {
     defer releases_map.deinit();
     const releases = json.Value{ .object = releases_map };
 
-    var rel = try Release.releasefromVersion("0.10");
+    var rel = try Release.releaseFromVersion("0.10");
     try rel.resolve(releases);
 
     const v = try rel.actualVersion(alloc);
@@ -75,10 +76,10 @@ test "Resolve stable" {
     defer releases_map.deinit();
     const releases = json.Value{ .object = releases_map };
 
-    var rel = try Release.releasefromVersion("stable");
+    var rel = try Release.releaseFromVersion("stable");
     try rel.resolve(releases);
 
     const v = try rel.actualVersion(alloc);
     defer alloc.free(v);
-    try std.testing.expectEqualStrings("0.12.0", v);
+    try std.testing.expectEqualStrings("0.13.0", v);
 }