Skip to content

chore: Refactor for Docker support - #36

Closed
j4n wants to merge 3 commits into
j4n/source-ref-sha-supportfrom
j4n/docker-driver-prereqs
Closed

chore: Refactor for Docker support#36
j4n wants to merge 3 commits into
j4n/source-ref-sha-supportfrom
j4n/docker-driver-prereqs

Conversation

@j4n

@j4n j4n commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

More refactoring in preparation of the Docker support

refactor(driver_cmdeploy): extract shared pytest runner, restore on_init_relay

Extract cmdeploy test logic to run_cmdeploy_pytest() standalone function so any cmdeploy-based driver (CmdeployDriver and DockerDriver) can call it.

refactor(driver_cmdeploy): extract TEST_INI_OVERRIDES and make_ini_script

Both the cmdeploy and docker drivers need to generate chatmail.ini with relaxed rate limits for testing. Extract the overrides dict and the Python snippet builder into shared helpers.

refactor(container): add bash_get() and bash_do() helpers

Two wrappers for Container.bash() with and without failure checking.

@j4n
j4n force-pushed the j4n/docker-driver-prereqs branch from 66ad66d to e706ea1 Compare August 3, 2026 15:00
@j4n j4n self-assigned this Aug 3, 2026
@j4n
j4n force-pushed the j4n/source-ref-sha-support branch from 19a2e2a to e1c990e Compare August 5, 2026 10:32
@j4n
j4n force-pushed the j4n/docker-driver-prereqs branch from e706ea1 to 68f7425 Compare August 5, 2026 10:32
@j4n
j4n force-pushed the j4n/source-ref-sha-support branch 2 times, most recently from c1c5884 to 9f7ecb5 Compare August 5, 2026 11:57
@j4n
j4n force-pushed the j4n/docker-driver-prereqs branch from 68f7425 to c652452 Compare August 5, 2026 11:57
@j4n
j4n force-pushed the j4n/source-ref-sha-support branch from 9f7ecb5 to f3cac4b Compare August 6, 2026 06:07
@j4n
j4n force-pushed the j4n/docker-driver-prereqs branch from c652452 to 05c62e8 Compare August 6, 2026 06:07
@j4n
j4n force-pushed the j4n/source-ref-sha-support branch from f3cac4b to ee9d4ba Compare August 6, 2026 11:23
@j4n
j4n force-pushed the j4n/docker-driver-prereqs branch from 05c62e8 to 628634a Compare August 6, 2026 11:23
@j4n
j4n force-pushed the j4n/source-ref-sha-support branch from ee9d4ba to 5bb1716 Compare August 6, 2026 11:35
@j4n
j4n force-pushed the j4n/docker-driver-prereqs branch from 628634a to eaf84ed Compare August 6, 2026 11:35
@j4n
j4n force-pushed the j4n/source-ref-sha-support branch from 5bb1716 to b1969ca Compare August 11, 2026 10:39
@j4n
j4n force-pushed the j4n/docker-driver-prereqs branch from eaf84ed to 0836f27 Compare August 11, 2026 10:39
@hpk42
hpk42 force-pushed the j4n/source-ref-sha-support branch from b1969ca to bce82b0 Compare August 11, 2026 12:36
@hpk42
hpk42 force-pushed the j4n/docker-driver-prereqs branch from 0836f27 to 553a878 Compare August 11, 2026 12:36
@hpk42
hpk42 force-pushed the j4n/source-ref-sha-support branch from bce82b0 to 65db096 Compare August 11, 2026 12:53
@hpk42
hpk42 force-pushed the j4n/docker-driver-prereqs branch from 553a878 to d972f27 Compare August 11, 2026 12:53
j4n added 3 commits August 12, 2026 08:43
…nit_relay

Extract cmdeploy test logic to run_cmdeploy_pytest() standalone function so
any cmdeploy-based driver (CmdeployDriver and DockerDriver) can call it.
…ript

Both the cmdeploy and docker drivers need to generate chatmail.ini with
relaxed rate limits for testing. Extract the overrides dict and the
Python snippet builder into shared helpers.
Two wrappers for Container.bash() with and without failure checking.
@j4n
j4n force-pushed the j4n/source-ref-sha-support branch from 65db096 to aa78a29 Compare August 12, 2026 07:04
@j4n
j4n force-pushed the j4n/docker-driver-prereqs branch from d972f27 to 73fb9df Compare August 12, 2026 07:04
@j4n

j4n commented Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

moved these commits into #37

@j4n j4n closed this Aug 13, 2026
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.

1 participant