generated from rl-institut/rli_template
-
Notifications
You must be signed in to change notification settings - Fork 5
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
Features/add tests #357
Draft
MaGering
wants to merge
64
commits into
dev
Choose a base branch
from
features/add_tests
base: dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Features/add tests #357
Changes from 40 commits
Commits
Show all changes
64 commits
Select commit
Hold shift + click to select a range
ba5a1b7
Update poetry lock file
MaGering 1fc5da3
Add draft of pipeline test
MaGering ec73c7b
Update draft of pipeline test - first exit 0 version
MaGering c3ae30f
Introduce loop over rules and add comments
MaGering 34031b5
ba5a1b7b34039bc858f05af19cc76d72370dcc97
MaGering f6099cb
Update test_pipeline.py
MaGering 6771c2d
Merge branch 'dev' into features/add_tests
MaGering 3c08821
Add drafts of tests with pipeline execution of raw data creation
MaGering b512b9f
Add drafts of tests with pipeline execution of resources and table da…
MaGering 8b4401a
Add drafts of tests with pipeline execution of scenario and example r…
MaGering 16d09d3
Restructure test_pipeline.py
MaGering a5b3fb5
Add docstring to install_with_extra function
Stefanie08 b7da7de
Move and rename test_pipeline.py
Stefanie08 5857b01
Adjust path to testing_pipeline.py
Stefanie08 f4136eb
Add docstrings to functions and update comments
Stefanie08 1c1f371
Correct docstrings
Stefanie08 61710ac
Add function get_abs_path_list
Stefanie08 4a9158a
Extract function get_abs_path_list
Stefanie08 8a98908
Add function file_name_extension
Stefanie08 2c1775e
Add functions clean_file and rule_test
Stefanie08 43455c9
Add docstring and comment to function pipeline_file_output_test
Stefanie08 53c5a5a
Add and correct docstring
Stefanie08 573ae42
Add docstring to test_pipeline_resources_tables.py
Stefanie08 c0cafc5
Adjust function file_name_extension
Stefanie08 19c2a40
Add docstring to test_pipeline_raw_data.py
Stefanie08 eb1b1a2
Add docstring for testing_pipeline.py
Stefanie08 d3826d2
Apply black
Stefanie08 861cf09
Apply black and update function pipeline_file_output_test
Stefanie08 e70b34c
Apply black
MaGering dd511dc
Raise Exception in test_raw_dir function if raw data can not be downl…
MaGering 9d0e3a3
Fix test_raw_dir snakemake MissingRule Exception which led to assert …
MaGering 874d8a5
Fix UnboundLocalError of renamed_path
MaGering 0417c98
Raise FileExists error if _original name suffix already exists before…
MaGering 22970da
Download raw data for tests of pipeline and delete it afterwards if i…
MaGering 2602880
Add logging for test_raw_dir
MaGering 606062b
Merge branch 'features/modify_tests' into features/add_tests
MaGering 347199b
Convert raw_dir_rule to list because of consistency reasons
MaGering 3997415
Update result rules to be testes in test_pipeline_results
MaGering ed60f78
Add snakemake rule names in comments
MaGering b4feaf2
Add rule names
Stefanie08 4de25ed
Add function test_pipeline_folders to test_pipeline_results.py and ad…
Stefanie08 9189269
Correct variable
Stefanie08 1541b0c
Add raw data check
Stefanie08 f166b25
Move and rename function test_pipeline_folders
Stefanie08 98b4bf2
Add cbc to github checks pre-dependencies
MaGering 7e0d82d
Fix the accidental change in commit f166b25
MaGering 6824ab7
Apply changes to docstrings and functions from review
Stefanie08 abfa330
Update docstring in test_pipeline_raw_data.py
Stefanie08 4f93e74
Adapt docstrings
MaGering c7b53bd
Adapt docstrings
Stefanie08 cbd165c
Resolve merge conflicts
Stefanie08 6bd0463
Apply black
Stefanie08 1d60177
Update docstring
Stefanie08 f7d6cef
Add check for '_original' file existence in function pipeline_file_o…
Stefanie08 7a2ba06
Add docstrings
Stefanie08 030f86d
Update code for function pipeline_folder_output_test
Stefanie08 70ea3de
Update function pipeline_file_output_test
Stefanie08 5ed0d3b
Update docstrings in module testing_pipeline.py
Stefanie08 37753e6
Make code more robust
Stefanie08 6dae213
Add check for files in function pipeline_folder_output_test
Stefanie08 0a55eb6
Rename function pipeline_folder_output_test
Stefanie08 dc762e2
Replace renamed function pipeline_output_test in modules
Stefanie08 ca75bbd
Update function pipeline_file_output_test
Stefanie08 5e3eb48
Delete function pipeline_file_output_test
Stefanie08 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
Oops, something went wrong.
Oops, something went wrong.
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.
Do you know why these and following changed?