@yurishkuro As far I can understand this is the meaning of removing conversions. I maybe wrong but after doing this for other backends we can refactoring E2E tests. I aim to remove this type of conversion:

sorry I don't follow the purpose of this, please elaborate.

sorry I don't follow the purpose of this, please elaborate.

@yurishkuro So the second check point of the issue says this Remove unnecessary downgrades done in the tests themselves, leave the downgrades to the storage implementations (when they don't yet support v2 API). As far I could understand is we need to get rid of the downgrades which we are doing in the integration tests. We have to give the responsibility of downgrading to the storage backend (for those for which v2 API is not yet implemented). Am I understanding right? This PR tries to do this!

@yurishkuro I have a doubt, if we will refactor the integration tests to use the v2 factory directly, will it be ok if we will not be testing the flags in e2e tests as v2 factory is directly initialized by config not by flags!

if we will refactor the integration tests to use the v2 factory directly, will it be ok if we will not be testing the flags in e2e tests as v2 factory is directly initialized by config not by flags!

the flags have their own unit tests, the e2e tests use flags not because we need to test them but because there was no other way to initialize v1 factories. Since we can use the configs now we can just populate the config struct instead of flags.

Codecov Report

Attention: Patch coverage is 95.52239% with 3 lines in your changes missing coverage. Please review.

Project coverage is 96.17%. Comparing base (cc7ea8d) to head (b82010c).
Report is 3 commits behind head on main.

@@            Coverage Diff             @@
##             main    #7228      +/-   ##
==========================================
- Coverage   96.20%   96.17%   -0.04%     
==========================================
  Files         370      372       +2     
  Lines       22216    22259      +43     
==========================================
+ Hits        21374    21407      +33     
- Misses        630      638       +8     
- Partials      212      214       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.