Added tests.

This commit is contained in:
Alexandros Naskos 2020-05-27 21:58:35 +03:00
parent 406e400f0c
commit 6f8fbe1ccb
4 changed files with 118 additions and 4 deletions

View File

@ -28,7 +28,11 @@ jobs:
FILE=zig-cache/bin/zls FILE=zig-cache/bin/zls
if test -f "$FILE"; then if test -f "$FILE"; then
exit 0 if zig test; then
exit 0
else
exit 1
fi
else else
exit 1 exit 1
fi fi

View File

@ -131,7 +131,15 @@ pub fn build(b: *std.build.Builder) !void {
const run_step = b.step("run", "Run the app"); const run_step = b.step("run", "Run the app");
run_step.dependOn(&run_cmd.step); run_step.dependOn(&run_cmd.step);
// const configure_step = std.build.Step.init("config", b.allocator, config);
const configure_step = b.step("config", "Configure zls"); const configure_step = b.step("config", "Configure zls");
configure_step.makeFn = config; configure_step.makeFn = config;
const test_step = b.step("test", "Run all the tests");
test_step.dependOn(&exe.step);
var unit_tests = b.addTest("tests/unit_tests.zig");
unit_tests.addPackage(.{ .name = "analysis", .path = "src/analysis.zig" });
unit_tests.addPackage(.{ .name = "types", .path = "src/types.zig" });
unit_tests.setBuildMode(.Debug);
test_step.dependOn(&unit_tests.step);
} }

View File

@ -910,7 +910,7 @@ pub fn getImportStr(tree: *ast.Tree, source_index: usize) ?[]const u8 {
const types = @import("types.zig"); const types = @import("types.zig");
pub const SourceRange = std.zig.Token.Loc; pub const SourceRange = std.zig.Token.Loc;
const PositionContext = union(enum) { pub const PositionContext = union(enum) {
builtin: SourceRange, builtin: SourceRange,
comment, comment,
string_literal: SourceRange, string_literal: SourceRange,
@ -920,7 +920,7 @@ const PositionContext = union(enum) {
other, other,
empty, empty,
fn range(self: PositionContext) ?SourceRange { pub fn range(self: PositionContext) ?SourceRange {
return switch (self) { return switch (self) {
.builtin => |r| r, .builtin => |r| r,
.comment => null, .comment => null,

102
tests/unit_tests.zig Normal file
View File

@ -0,0 +1,102 @@
const analysis = @import("analysis");
const types = @import("types");
const std = @import("std");
const allocator = std.testing.allocator;
fn makeDocument(uri: []const u8, text: []const u8) !types.TextDocument {
const mem = try allocator.alloc(u8, text.len);
std.mem.copy(u8, mem, text);
return types.TextDocument{
.uri = uri,
.mem = mem,
.text = mem[0..],
};
}
fn freeDocument(doc: types.TextDocument) void {
allocator.free(doc.text);
}
fn makeUnnamedDocument(text: []const u8) !types.TextDocument {
return try makeDocument("test", text);
}
fn testContext(comptime line: []const u8, comptime tag: var, comptime range: ?[]const u8) !void {
const cursor_idx = comptime std.mem.indexOf(u8, line, "<cursor>").?;
const final_line = line[0..cursor_idx] ++ line[cursor_idx + "<cursor>".len ..];
const doc = try makeUnnamedDocument(final_line);
defer freeDocument(doc);
const ctx = try analysis.documentPositionContext(allocator, doc, types.Position{
.line = 0,
.character = @intCast(i64, cursor_idx - 1),
});
if (std.meta.activeTag(ctx) != tag) {
std.debug.warn("Expected tag {}, got {}\n", .{ tag, std.meta.activeTag(ctx) });
return error.DifferentTag;
}
if (ctx.range()) |ctx_range| {
if (range == null) {
std.debug.warn("Expected null range, got `{}`\n", .{
doc.text[ctx_range.start..ctx_range.end],
});
} else {
const range_start = comptime std.mem.indexOf(u8, final_line, range.?).?;
const range_end = range_start + range.?.len;
if (range_start != ctx_range.start or range_end != ctx_range.end) {
std.debug.warn("Expected range `{}` ({}..{}), got `{}` ({}..{})\n", .{
doc.text[range_start..range_end], range_start, range_end,
doc.text[ctx_range.start..ctx_range.end], ctx_range.start, ctx_range.end,
});
return error.DifferentRange;
}
}
} else if (range != null) {
std.debug.warn("Unexpected null range\n", .{});
return error.DifferentRange;
}
}
test "documentPositionContext" {
try testContext(
\\const this_var = id<cursor>entifier;
,
.var_access,
"id",
);
try testContext(
\\if (displ.*.?.c.*[0].<cursor>@"a" == foo) {
,
.field_access,
"displ.*.?.c.*[0].",
);
try testContext(
\\const arr = std.ArrayList(SomeStruct(a, b, c, d)).in<cursor>it(allocator);
,
.field_access,
"std.ArrayList(SomeStruct(a, b, c, d)).in",
);
try testContext(
\\try erroringFn(the_first[arg], second[a..<cursor>]);
,
.empty,
null,
);
try testContext(
\\ fn add(lhf: lself, rhs: rself) !Se<cursor> {
,
.var_access,
"Se",
);
}