Skip to content

Commit

Permalink
test: remove the requirement for a Layout file
Browse files Browse the repository at this point in the history
Previously we required a layout file for any tablet with pad
buttons. This somewhat prevents us from adding tablet files for unknown
tablets where we can get most of the information from external sources,
e.g. see linuxwacom#659. As a result we're missing support for these tablets
altogether when we could have at least partial support.

Let's drop this requirement in our tests, users who want their tablet
working properly can submit layout files.
  • Loading branch information
whot committed Apr 21, 2024
1 parent 6b340d6 commit dbac192
Showing 1 changed file with 0 additions and 14 deletions.
14 changes: 0 additions & 14 deletions test/test-tablet-svg-validity.c
Original file line number Diff line number Diff line change
Expand Up @@ -233,19 +233,6 @@ struct fixture {
xmlNodePtr root;
};

static void
test_filename(struct fixture *f, gconstpointer data)
{
const WacomDevice *device = data;
const char *filename;

filename = libwacom_get_layout_filename(device);
if (libwacom_get_num_buttons(device) > 0) {
g_assert_nonnull(filename);
g_assert_cmpstr(filename, !=, "");
}
}

static void
test_svg(struct fixture *f, gconstpointer data)
{
Expand Down Expand Up @@ -368,7 +355,6 @@ static void setup_tests(WacomDevice *device)
if (g_str_equal(name, "Generic"))
return;

add_test(device, test_filename);
if (!libwacom_get_layout_filename(device))
return;

Expand Down

0 comments on commit dbac192

Please sign in to comment.