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

test: add testing workflow yaml #194

Merged
merged 2 commits into from
Jan 17, 2024
Merged

test: add testing workflow yaml #194

merged 2 commits into from
Jan 17, 2024

Conversation

yanksyoon
Copy link
Collaborator

Applicable spec: N/A

Overview

In order to test the tmate action workflow in #193, the workflow must exist in the main branch as per the documentation

To trigger the workflow_dispatch event, your workflow must be in the default branch.

Rationale

To allow integration testing with ssh-debug integration.

Juju Events Changes

None.

Module Changes

None.

Library Changes

None.

Checklist

Copy link
Contributor

Test coverage for 7e60326

Name                         Stmts   Miss Branch BrPart  Cover   Missing
------------------------------------------------------------------------
src/charm.py                   458    110    117     26    74%   86-88, 112-113, 117-119, 157-159, 170->182, 180, 215-234, 243-244, 258-260, 261->273, 264->273, 297-301, 332, 412-417, 440, 457-458, 467-490, 508-513, 519-522, 529, 532-533, 549, 553-558, 603->606, 619-620, 622-623, 657-664, 688-689, 701-703, 718-719, 747-748, 750-751, 753-754, 828->830, 891-892, 908, 926-928, 932-933, 937
src/charm_state.py             135     18     18      3    85%   131-138, 220-221, 260-262, 273-274, 280-285, 296-298
src/errors.py                   39      0      0      0   100%
src/event_timer.py              42      8      4      0    74%   94-97, 117-120
src/firewall.py                 43     25     10      0    38%   38-42, 64-67, 75-149
src/github_client.py            68      6     32      3    87%   124, 146, 158-165, 183->216
src/github_metrics.py           14      0      0      0   100%
src/github_type.py              50      0      0      0   100%
src/lxd_type.py                 37      0      2      0   100%
src/metrics.py                  73      2     10      1    96%   61->64, 170-171
src/metrics_type.py              6      0      0      0   100%
src/runner.py                  320     70     96     24    73%   44->46, 125, 252-253, 294-303, 327-332, 337, 357, 361-371, 420->423, 426-428, 435, 449, 459, 463, 465, 480, 506-511, 527-540, 551-590, 595, 633, 680-682, 686, 704, 739, 765, 770-782, 796, 801->803, 806-808
src/runner_logs.py              35      2      6      1    93%   62->61, 66-67
src/runner_manager.py          274     47     90      8    82%   122, 167-169, 182-183, 195-197, 203-208, 212-213, 223-224, 243, 286-287, 331-333, 398, 418-422, 447, 499-502, 538, 645-658, 668, 673-684
src/runner_manager_type.py      33      0      6      0   100%
src/runner_metrics.py          123      8     20      2    93%   147-148, 160, 191-192, 307-311
src/runner_type.py              47      0     10      0   100%
src/shared_fs.py               116     16     20      0    88%   108-109, 133-134, 217-218, 241-242, 254-255, 260-261, 267-268, 292-293
src/utilities.py                68      5     20      7    86%   74->76, 78->84, 91, 121, 135, 173, 226
------------------------------------------------------------------------
TOTAL                         1981    317    461     75    81%

Static code analysis report

Run started:2024-01-17 01:38:38.991955

Test results:
  No issues identified.

Code scanned:
  Total lines of code: 4322
  Total lines skipped (#nosec): 0
  Total potential issues skipped due to specifically being disabled (e.g., #nosec BXXX): 12

Run metrics:
  Total issues (by severity):
  	Undefined: 0
  	Low: 0
  	Medium: 0
  	High: 0
  Total issues (by confidence):
  	Undefined: 0
  	Low: 0
  	Medium: 0
  	High: 0
Files skipped (0):

@yanksyoon yanksyoon merged commit f0df63a into main Jan 17, 2024
44 of 54 checks passed
@yanksyoon yanksyoon deleted the test/ssh_debug_workflow branch January 17, 2024 05:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants