Skip to content

Comments

Fix/issue 440 create compreh tests#441

Open
zljubisic wants to merge 4 commits intoEnergieID:masterfrom
zljubisic:fix/issue-440-create-compreh-tests
Open

Fix/issue 440 create compreh tests#441
zljubisic wants to merge 4 commits intoEnergieID:masterfrom
zljubisic:fix/issue-440-create-compreh-tests

Conversation

@zljubisic
Copy link

@zljubisic zljubisic commented Sep 27, 2025

I have added few tests to improve testing.
Feel free to add or change anything.
Closes: #440
@fboerman can you please check?

@Shohail-Ismail
Copy link

Shohail-Ismail commented Jan 20, 2026

I was going to suggest some more tests for the repo (after adding them for a PR using EntsoePandasClient over at OpenClimateFix), before a search brought me here - where I can see that the proposed suite of tests is already quite comprehensive.

Therefore, @zljubisic I'd appreciate it if you could provide an update on the status of this PR, so I can open an Issue to add my tests if needed. Thanks.

@zljubisic
Copy link
Author

Hi @Shohail-Ismail. I have created these tests long time ago. At that time some other guy also created PR with some tests.
I believe that tests from both PRs could be integrated and merged to the main branch.
If you want me to do something with this please let me know.

@Shohail-Ismail
Copy link

Shohail-Ismail commented Jan 21, 2026

Thanks @zljubisic, I've opened an Issue related to the specific tests I wanted to add - will let you know if there's anything further related to your tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create more comprehensive tests

2 participants