-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat(api): add method read_study_api
#22
Merged
Merged
Changes from 31 commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
bcdf961
added read_thermal_clusters/storages/renewables methods and realized …
mehdiwahada d31bba1
added read_thermal_clusters/storages/renewables methods and realized …
mehdiwahada e3c5030
added read_thermal_clusters/storages/renewables methods and realized …
mehdiwahada e06f118
added read_thermal_clusters/storages/renewables methods and realized …
mehdiwahada d581ddb
added read_thermal_clusters/storages/renewables methods and realized …
mehdiwahada 2ab7da7
added read_thermal_clusters/storages/renewables methods and realized …
mehdiwahada 6906b17
feat(gh): add release gh action (#14)
MartinBelthle 603cfe7
feat(ci): use tox (#12)
MartinBelthle ab74da8
chore(release): prepare first release (#15)
MartinBelthle 5b8d269
v0.1.1 (#16)
MartinBelthle 877f53a
integrated integration tests for the reading clusters methods
mehdiwahada 838ae57
Merge branch 'main' into features/add_clusters
mehdiwahada bb9c9e4
integrated integration tests for the reading clusters methods
mehdiwahada 2dd6b7a
integrated integration tests for the reading clusters methods
mehdiwahada 483e4d6
integrated integration tests for the reading clusters methods
mehdiwahada d2f4fd9
integrated integration tests for the reading clusters methods
mehdiwahada 6ef393c
adding hydro data (unit and integration testing)
mehdiwahada ed5aa63
adding hydro data (unit and integration testing)
mehdiwahada 4fcd430
Merge branch 'main' into features/add_hydro_data
mehdiwahada 0e16576
adding hydro data (unit and integration testing)
mehdiwahada 8238314
adding hydro data (unit and integration testing)
mehdiwahada 41821a5
fix(api): correcting import problems
mehdiwahada e402e5b
fix(api): correcting import problems
mehdiwahada 671336f
fix(api): correcting import problems
mehdiwahada b56994a
fix(api): correcting import problems
mehdiwahada d0b8052
feat(api): adding read_study method and unit testing
mehdiwahada af39424
feat(api): adding read_study method and unit testing
mehdiwahada 0fd3de8
Merge branch 'main' into feat/read_study_api
mehdiwahada e21278d
feat(api): adding read_study method, unit testing and integration tes…
mehdiwahada 2bdb548
Merge branch 'main' into feat/read_study_api
MartinBelthle a7e28fe
Merge remote-tracking branch 'origin/feat/read_study_api' into feat/r…
mehdiwahada 91d01da
feat(api): adding read_study method, unit testing and integration tes…
mehdiwahada 4c2d927
Merge branch 'main' into feat/read_study_api
MartinBelthle a729713
feat(api): adding read_study method, unit testing and integration tes…
mehdiwahada c4767cf
Merge remote-tracking branch 'origin/feat/read_study_api' into feat/r…
mehdiwahada 7d35ba7
feat(api): adding read_study method, unit testing and integration tes…
mehdiwahada e605d28
Merge branch 'main' into feat/read_study_api
MartinBelthle 4b4edd4
feat(api): adding read_study method, unit testing and integration tes…
mehdiwahada f6f1a61
Merge remote-tracking branch 'origin/feat/read_study_api' into feat/r…
mehdiwahada e140264
feat(api): adding read_study method, unit testing and integration tes…
mehdiwahada 95b08fc
feat(api): adding read_study method, unit testing and integration tes…
mehdiwahada 78c93c1
feat(api): adding read_study method, unit testing and integration tes…
mehdiwahada File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -30,7 +30,7 @@ | |
from antares.model.settings.general import GeneralParameters, Mode | ||
from antares.model.settings.study_settings import PlaylistParameters, StudySettings | ||
from antares.model.st_storage import STStorageGroup, STStorageMatrixName, STStorageProperties | ||
from antares.model.study import create_study_api | ||
from antares.model.study import create_study_api, read_study_api | ||
from antares.model.thermal import ThermalClusterGroup, ThermalClusterProperties | ||
|
||
from tests.integration.antares_web_desktop import AntaresWebDesktop | ||
|
@@ -51,6 +51,12 @@ def test_creation_lifecycle(self, antares_web: AntaresWebDesktop): | |
|
||
study = create_study_api("antares-craft-test", "880", api_config) | ||
|
||
actual_study = read_study_api(api_config, study.service.study_id) | ||
|
||
assert study.service.study_id == actual_study.service.study_id | ||
assert study.name == actual_study.name | ||
assert study.version == actual_study.version | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You should also test that the 2 studies contain the same areas name. And the same settings (just test some values if the == doesn't work) |
||
|
||
# tests area creation with default values | ||
area_name = "FR" | ||
area_fr = study.create_area(area_name) | ||
|
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you remove id ?