Skip to content

Rename data providers from xyzDataProvider to provideXyz #458

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
oliverklee opened this issue Feb 11, 2024 · 4 comments
Open

Rename data providers from xyzDataProvider to provideXyz #458

oliverklee opened this issue Feb 11, 2024 · 4 comments

Comments

@oliverklee
Copy link
Collaborator

No description provided.

@JakeQZ
Copy link
Collaborator

JakeQZ commented Feb 14, 2024

Also, while on #456, it seemed to make sense to move all the data providers before the tests:

  • Some data providers were used for more than one test, so could not be easily grouped with the relevant test;
  • Data providers are static, tests are not, also suggested grouping data separately from tests (though within the same file)
    • My personal coding style is to have all statics before non-statics within a similar classification (still properties before methods - don't know if that's covered by any standard);
  • For readability, it seems logical to present the data first:
    • This is the data we're going to use for testing;
    • Now this is what were going to use it for.

WDYT?

@oliverklee
Copy link
Collaborator Author

I'm fine with this. I'd prefer to to have the data provider directly above the first test that uses is to reduce scrolling.

@JakeQZ JakeQZ changed the title Rename data providers from xzyDataProvider to providerXyz Rename data providers from xzyDataProvider to provideXyz Feb 15, 2024
@JakeQZ
Copy link
Collaborator

JakeQZ commented Feb 15, 2024

I just spotted the dxsleyia test in the title.

@JakeQZ JakeQZ changed the title Rename data providers from xzyDataProvider to provideXyz Rename data providers from xzyDataProvider to provideXzy Feb 15, 2024
@JakeQZ
Copy link
Collaborator

JakeQZ commented Feb 15, 2024

I'd prefer to to have the data provider directly above the first test that uses is to reduce scrolling.

Agree. Though In an ideal world where classes are small, that wouldn't be necessary :)

@oliverklee oliverklee changed the title Rename data providers from xzyDataProvider to provideXzy Rename data providers from xyzDataProvider to provideXyz Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants