HelpersTask629_Add_tests_for_dockerized_executable_sync_gh_issue_labels #649

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.

Sign up for GitHub

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

Jump to bottom

Merged

heanhsok merged 25 commits into master from HelpersTask629_Add_tests_for_dockerized_executable_sync_gh_issue_labels

May 13, 2025

Collaborator

sandeepthalapanane commented May 2, 2025

Addressing #629

Implemented tests to check the Docker flow sequence for the Docker executable and also for the functionality class.

sandeepthalapanane added 4 commits

April 30, 2025 15:22


          HelpersTask629: Work in progress

67e33e6

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅


          HelpersTask629: Work in progress

599a565

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅


          HelpersTask629: Work in progress

6729dfa

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅


          HelpersTask629: Added test for sync_gh_issue_labels

e5e758f

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

sandeepthalapanane self-assigned this

sandeepthalapanane added 2 commits

May 2, 2025 17:59


          HelpersTask629: Modify docstring

d5e7e33

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅


          HelpersTask629: Remove logging which was added for debugging

a51629e

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

sandeepthalapanane requested review from heanhsok, sonniki, gpsaggese and samarth9008

May 2, 2025 22:01

sandeepthalapanane and others added 3 commits

May 2, 2025 18:02


          Merge remote-tracking branch 'origin/master' into HelpersTask629_Add_…

d927547

…tests_for_dockerized_executable_sync_gh_issue_labels

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅


          HelpersTask629: Modify test yaml file

4a96eef

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅


          Merge branch 'master' into HelpersTask629_Add_tests_for_dockerized_ex…

a3de9c5

…ecutable_sync_gh_issue_labels

gpsaggese reviewed

View reviewed changes

dev_scripts_helpers/github/test/test_dockerized_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

dev_scripts_helpers/github/test/test_dockerized_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

dev_scripts_helpers/github/test/test_dockerized_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

dev_scripts_helpers/github/test/test_dockerized_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

dev_scripts_helpers/github/test/test_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

dev_scripts_helpers/github/test/test_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

gpsaggese mentioned this pull request

Document our guidelines for mocking #655

Open

sandeepthalapanane added 7 commits

May 5, 2025 14:02


          Merge remote-tracking branch 'origin/master' into HelpersTask629_Add_…

605373c

…tests_for_dockerized_executable_sync_gh_issue_labels

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅


          HelpersTask629: Modify test to mock GH only

e7c27e4

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅


          HelpersTask629: Modify dockerized test to slow

219fdc1

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅


          HelpersTask629: Skip the test

16fc945

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅


          Merge master

fe9ecc7

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅


          HelpersTask629: Remove renamed files

a57d627

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅


          HelpersTask629: Modify test_sync_gh_issue_labels1 docstring

89278b2

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

sandeepthalapanane requested a review from gpsaggese

May 6, 2025 01:44

heanhsok requested changes

View reviewed changes

dev_scripts_helpers/github/test/test_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

dev_scripts_helpers/github/test/test_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

dev_scripts_helpers/github/test/test_sync_gh_issue_labels.py Show resolved Hide resolved

dev_scripts_helpers/github/test/test_dockerized_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

dev_scripts_helpers/github/test/test_dockerized_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

dev_scripts_helpers/github/test/test_dockerized_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

dev_scripts_helpers/github/test/test_dockerized_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

dev_scripts_helpers/github/test/test_sync_gh_issue_labels.py Show resolved Hide resolved

sandeepthalapanane added 2 commits

May 6, 2025 23:15


          HelpersTask629: Add GitHub Mocks

47a234d

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅


          Merge remote-tracking branch 'origin/master' into HelpersTask629_Add_…

0f88027

…tests_for_dockerized_executable_sync_gh_issue_labels

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

sandeepthalapanane requested a review from heanhsok

May 7, 2025 03:18


          HelpersTask629: Modify input arguments

aa5dd4b

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

heanhsok requested changes

View reviewed changes

Contributor

heanhsok left a comment

Nice work! Just a couple of small nits.

dev_scripts_helpers/github/test/test_dockerized_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

dev_scripts_helpers/github/test/test_dockerized_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

dev_scripts_helpers/github/test/test_dockerized_sync_gh_issue_labels.py Show resolved Hide resolved

dev_scripts_helpers/github/test/test_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

dev_scripts_helpers/github/test/test_sync_gh_issue_labels.py Show resolved Hide resolved

dev_scripts_helpers/github/test/test_sync_gh_issue_labels.py Show resolved Hide resolved

dev_scripts_helpers/github/test/test_dockerized_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

gpsaggese reviewed

View reviewed changes

Contributor

gpsaggese left a comment

Nicely done!


          Merge remote-tracking branch 'origin/master' into HelpersTask629_Add_…

8618ac4

…tests_for_dockerized_executable_sync_gh_issue_labels

No pre-commit output found.

gpsaggese reviewed

View reviewed changes

Contributor

gpsaggese left a comment

Good work. Let's do the improvements suggested by @heanhsok and we can merge and be done

dev_scripts_helpers/github/test/test_dockerized_sync_gh_issue_labels.py Outdated Show resolved Hide resolved

dev_scripts_helpers/github/test/test_sync_gh_issue_labels.py Show resolved Hide resolved

sandeepthalapanane added 3 commits

May 12, 2025 13:50


          HelpersTask629: Fix the issues raised in PR

c631009

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅


          Merge remote-tracking branch 'origin/master' into HelpersTask629_Add_…

5e288c5

…tests_for_dockerized_executable_sync_gh_issue_labels

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅


          Merge remote-tracking branch 'origin/master' into HelpersTask629_Add_…

0ee04e3

…tests_for_dockerized_executable_sync_gh_issue_labels

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

sandeepthalapanane requested review from heanhsok and gpsaggese

May 12, 2025 17:58

Contributor

heanhsok commented May 12, 2025

LGTM


          HelpersTask629: Add a TODO

bdcf03e

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

Collaborator Author

sandeepthalapanane commented May 13, 2025

@heanhsok I've completed all the changes. Please review the changes.

Contributor

heanhsok commented May 13, 2025

LG merging


          Merge branch 'master' into HelpersTask629_Add_tests_for_dockerized_ex…

871a04c

…ecutable_sync_gh_issue_labels

heanhsok enabled auto-merge (squash)

May 13, 2025 16:00

heanhsok approved these changes

View reviewed changes

heanhsok merged commit cce0a13 into master

4 checks passed

heanhsok deleted the HelpersTask629_Add_tests_for_dockerized_executable_sync_gh_issue_labels branch

May 13, 2025 16:10

heanhsok mentioned this pull request

Add tests for dockerized executable sync_gh_issue_labels #629

Closed

sandeepthalapanane mentioned this pull request

Broken test Test_sync_gh_issue_labels1 #753

Open

heanhsok added a commit to heanhsok/helpers that referenced this pull request


          HelpersTask629_Add_tests_for_dockerized_executable_sync_gh_issue_labe…

3b09aaf

…ls (causify-ai#649)

* HelpersTask629: Work in progress

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Work in progress

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Work in progress

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Added test for sync_gh_issue_labels

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Modify docstring

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Remove logging which was added for debugging

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Modify test yaml file

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Modify test to mock GH only

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Modify dockerized test to slow

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Skip the test

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Remove renamed files

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Modify test_sync_gh_issue_labels1 docstring

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Add GitHub Mocks

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Modify input arguments

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Fix the issues raised in PR

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Add a TODO

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

---------

Co-authored-by: Heanh Sok <[email protected]>

heanhsok added a commit to heanhsok/helpers that referenced this pull request


          HelpersTask629_Add_tests_for_dockerized_executable_sync_gh_issue_labe…

3f10c29

…ls (causify-ai#649)

* HelpersTask629: Work in progress

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Work in progress

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Work in progress

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Added test for sync_gh_issue_labels

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Modify docstring

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Remove logging which was added for debugging

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Modify test yaml file

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Modify test to mock GH only

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Modify dockerized test to slow

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Skip the test

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Remove renamed files

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Modify test_sync_gh_issue_labels1 docstring

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Add GitHub Mocks

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Modify input arguments

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Fix the issues raised in PR

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

* HelpersTask629: Add a TODO

Pre-commit checks:
- 'check_master' passed
- 'check_author' passed
- 'check_file_size' passed
- 'check_python_compile' passed
- 'check_gitleaks' passed
All checks passed ✅

---------

Co-authored-by: Heanh Sok <[email protected]>

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

Reviewers

aangelo9 aangelo9 left review comments

heanhsok heanhsok approved these changes

sonniki Awaiting requested review from sonniki

samarth9008 Awaiting requested review from samarth9008

gpsaggese Awaiting requested review from gpsaggese

Labels

None yet