Skip to content

test: enable GitHub label sync wrapper test#1268

Open
ccai40359-wq wants to merge 1 commit into
causify-ai:masterfrom
ccai40359-wq:fix/helpers-task-753-sync-label-test
Open

test: enable GitHub label sync wrapper test#1268
ccai40359-wq wants to merge 1 commit into
causify-ai:masterfrom
ccai40359-wq:fix/helpers-task-753-sync-label-test

Conversation

@ccai40359-wq

Copy link
Copy Markdown

Summary

  • Enable Test_sync_gh_issue_labels1.test1 by fixing the missing input_label setup.
  • Mock the outer docker wrapper so the test does not require a running Docker daemon.
  • Exercise the dockerized implementation directly to verify that --backup writes the expected label backup file.

Verification

  • python -m pytest dev_scripts_helpers/github/test/test_sync_gh_issue_labels.py::Test_sync_gh_issue_labels1::test1 dev_scripts_helpers/github/test/test_dockerized_sync_gh_issue_labels.py::TestLabel1::test_save_label dev_scripts_helpers/github/test/test_dockerized_sync_gh_issue_labels.py::TestLabel1::test_load_label -q -rs → 3 passed
  • git diff --check → passed
  • Static added-line security scan → security_findings_count 0

Closes #753

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Broken test Test_sync_gh_issue_labels1

2 participants