Skip to content

test: remove trivial tests

Tomas Vik (OOO back on 2024-09-23) requested to merge tv/2023-07/remote-trivial-tests into main

This is a follow-up to the discussion !886 (comment 1464739668)

This MR suggest removing tests that have low value because they copy production code too closely: there is not enough logic to test, and that any refactoring of the production code will imply an identical change to the test. That (IMO) removes the value of the test.

Here is an extreme example of what I mean by "copying production code":

// prod code
const HELLO = "Hello World";

//test
expect(HELLO).toBe("Hello World");

Merge request reports