Skip to content

rename test file - #193

Merged
pawelru merged 3 commits into
mainfrom
rename_test_file
Jan 22, 2024
Merged

rename test file#193
pawelru merged 3 commits into
mainfrom
rename_test_file

Conversation

@pawelru

@pawelru pawelru commented Jan 22, 2024

Copy link
Copy Markdown
Contributor

rename test file for consistency across all other test files

@pawelru pawelru added the core label Jan 22, 2024
@pawelru
pawelru enabled auto-merge (squash) January 22, 2024 11:59
@github-actions

github-actions Bot commented Jan 22, 2024

Copy link
Copy Markdown
Contributor

badge

Code Coverage Summary

Filename                 Stmts    Miss  Cover    Missing
---------------------  -------  ------  -------  ---------
R/qenv-concat.R             10       0  100.00%
R/qenv-constructor.R        16      13  18.75%   54-93
R/qenv-eval_code.R          52       2  96.15%   100, 109
R/qenv-get_code.R           20       1  95.00%   38
R/qenv-get_env.R             3       3  0.00%    23-31
R/qenv-get_var.R            19       0  100.00%
R/qenv-get_warnings.R       24       0  100.00%
R/qenv-join.R               46       0  100.00%
R/qenv-show.R                1       1  0.00%    19
R/qenv-within.R              8       0  100.00%
R/utils.R                   10       0  100.00%
TOTAL                      209      20  90.43%

Diff against main

Filename      Stmts    Miss  Cover
----------  -------  ------  --------
TOTAL             0       0  +100.00%

Results for commit: be0e8b9

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions

github-actions Bot commented Jan 22, 2024

Copy link
Copy Markdown
Contributor

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
qenv-within 💀 $0.17$ $-0.17$ $-14$ $0$ $0$ $0$
qenv_within 👶 $+0.05$ $+14$ $0$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
qenv-within 💀 $0.01$ $-0.01$ external_values_are_not_taken_from_calling_frame
qenv-within 💀 $0.00$ $-0.00$ external_values_can_be_injected_into_expressions_through_..._
qenv-within 💀 $0.03$ $-0.03$ simple_and_compound_expressions_are_evaluated
qenv-within 💀 $0.10$ $-0.10$ styling_of_input_code_does_not_impact_evaluation_results
qenv-within 💀 $0.01$ $-0.01$ within.qenv_renturns_a_qenv_where_env_is_a_deep_copy_of_that_in_data_
qenv-within 💀 $0.01$ $-0.01$ within.qenv_renturns_qenv.error_even_if_evaluation_raises_error
qenv-within 💀 $0.00$ $-0.00$ within_run_on_qenv.error_returns_the_qenv.error_as_is
qenv_within 👶 $+0.01$ external_values_are_not_taken_from_calling_frame
qenv_within 👶 $+0.00$ external_values_can_be_injected_into_expressions_through_..._
qenv_within 👶 $+0.01$ simple_and_compound_expressions_are_evaluated
qenv_within 👶 $+0.01$ styling_of_input_code_does_not_impact_evaluation_results
qenv_within 👶 $+0.01$ within.qenv_renturns_a_qenv_where_env_is_a_deep_copy_of_that_in_data_
qenv_within 👶 $+0.01$ within.qenv_renturns_qenv.error_even_if_evaluation_raises_error
qenv_within 👶 $+0.00$ within_run_on_qenv.error_returns_the_qenv.error_as_is

Results for commit 87c9043

♻️ This comment has been updated with latest results.

@github-actions

Copy link
Copy Markdown
Contributor

Unit Tests Summary

  1 files    9 suites   0s ⏱️
 62 tests  62 ✅ 0 💤 0 ❌
134 runs  134 ✅ 0 💤 0 ❌

Results for commit be0e8b9.

@averissimo averissimo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch! LGTM

@pawelru
pawelru merged commit b4b303e into main Jan 22, 2024
@pawelru
pawelru deleted the rename_test_file branch January 22, 2024 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants