Test seeders representation

[stalniy]

Summary

This RFC proposes to replace class based seeders with function based one

Motivation

Function is a simpler primitive for data generation than a class, so it's just less to write and easier to use. Having a class with static methods doesn't give us any benefits in comparison to a simple function.

Example

Class based approach

export class DenomSeeder {
  static create(): Denom {
    return faker.helpers.arrayElement(["uakt", "ibc/170C677610AC31DF0904FFE09CD3B5C657492170E7E52372E48756B71E56F2F1"]);
  }
}

Function based approach

export function createDenom(): Denom {
  return faker.helpers.arrayElement(["uakt", "ibc/170C677610AC31DF0904FFE09CD3B5C657492170E7E52372E48756B71E56F2F1"]);
}

Main benefits

• less to write

Drawbacks

• refactoring of existing seeders. This can be done on the way, if a developer touches particular seeder class, he needs to rewrite it to function based

Additional notes

In reality this change doesn't bring much value and just pure optimization of used primitives.