Skip to content
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

Fix/issue151/csv input only postmortem #152

Merged
merged 4 commits into from
Mar 31, 2023

Conversation

roda82
Copy link
Contributor

@roda82 roda82 commented Mar 29, 2023

Closes: #151

Now, the config_validator only enables to use it in post-mortem (stream=false) mode as input.
Add tests for checking this behavior.
Correct some typos in existing tests
@gfieni gfieni added the bug Something isn't working label Mar 30, 2023
…ptation and Integration Tests.

Remove useless fixtures imports in tests.
docs: Add licence information in __init__ file
@codecov
Copy link

codecov bot commented Mar 31, 2023

Codecov Report

Merging #152 (ef221c1) into master (d577f15) will increase coverage by 0.86%.
The diff coverage is 57.89%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #152      +/-   ##
==========================================
+ Coverage   59.82%   60.69%   +0.86%     
==========================================
  Files          85       85              
  Lines        3134     3132       -2     
==========================================
+ Hits         1875     1901      +26     
+ Misses       1259     1231      -28     
Impacted Files Coverage Δ
powerapi/dispatcher/dispatcher_actor.py 66.00% <ø> (-0.67%) ⬇️
...pi/dispatcher/simple/simple_dispatcher_handlers.py 0.00% <0.00%> (ø)
powerapi/actor/actor.py 52.00% <50.00%> (ø)
powerapi/actor/state.py 70.58% <50.00%> (ø)
powerapi/handler/handler.py 48.00% <50.00%> (ø)
powerapi/handler/poison_pill_message_handler.py 33.33% <50.00%> (ø)
powerapi/message.py 79.68% <50.00%> (ø)
powerapi/puller/simple/simple_puller_handlers.py 37.03% <50.00%> (ø)
powerapi/cli/config_validator.py 75.51% <100.00%> (+58.11%) ⬆️
powerapi/formula/__init__.py 100.00% <100.00%> (ø)
... and 1 more

@gfieni gfieni merged commit bad36a5 into master Mar 31, 2023
@gfieni gfieni deleted the fix/issue151/csv_input_only_postmortem branch March 31, 2023 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CVS File as input has to be used only in post-mortem mode
2 participants