Skip to content

Course-package refresh + quarto/python smoke test (v1.1.4) #33

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

Merged
davidkane9 merged 2 commits into from
Aug 22, 2026
Merged

Course-package refresh + quarto/python smoke test (v1.1.4) #33

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
fcc9fe2
Course-package refresh + quarto/python-chunk smoke test (v1.1.4 prep)
davidkane9 Aug 22, 2026
807ac94
Fix the python smoke test: su resets PATH, hiding the venv from quarto
davidkane9 Aug 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
40 changes: 39 additions & 1 deletion Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -459,7 +459,7 @@ RUN R -q -e 'pak::pkg_install(c("devtools", "pkgdown", "roxygen2", "testthat", "
# cache hit (2026-07-27) and delivered bit-identical bits. Bump this date in
# any release whose purpose is picking up new course-package commits from
# GitHub HEAD; layers above stay cached, this one and everything after rebuild.
ARG COURSE_PKG_REFRESH=2026-08-14
ARG COURSE_PKG_REFRESH=2026-08-22
RUN echo "course-package refresh: ${COURSE_PKG_REFRESH}" \
&& R -q -e 'pak::pkg_install(c( \
"PPBDS/tutorial.helpers", \
Expand Down Expand Up @@ -578,6 +578,44 @@ RUN python -c "import numpy, pandas, matplotlib, seaborn, sklearn, statsmodels,
RUN su rstudio -c "/opt/venv/bin/pip install --no-cache-dir --quiet cowsay" \
&& /opt/venv/bin/python -c "import cowsay; print('rstudio-can-install OK')" \
&& /opt/venv/bin/pip uninstall -y --quiet cowsay

# Smoke test 3: Quarto renders a Python chunk through the jupyter engine,
# driven entirely from the CLI. This is the student-facing Python path now
# that codespace-starter dropped the VS Code Python extensions (2026-08-20,
# to thin the Activity Bar): the claim was that nothing students actually do
# depends on those extensions, and this test is that claim made falsifiable.
# The chunk writes a file rather than checking rendered HTML, so the test
# proves the chunk EXECUTED (with pandas importable) rather than merely that
# quarto emitted a document. Runs as rstudio, like a student. NOT covered,
# and deliberately gone with the extensions: the editor-side .ipynb UI, cell
# run-buttons, and Python IntelliSense.
#
# PATH is passed through explicitly because `su rstudio -c` RESETS it on
# Debian (login.defs), dropping the /opt/venv/bin that this image's ENV puts
# first — Quarto then falls back to the system python3, which has no jupyter
# or yaml, and the render dies with "Jupyter is not available in this Python
# installation" (build 32544605003). Students are unaffected: their terminals
# inherit the container ENV. Same reason the pip test above uses an absolute
# /opt/venv/bin path. The outer shell expands $PATH here, so the literal
# venv-first path is what reaches rstudio.
RUN mkdir -p /tmp/pysmoke \
&& printf '%s\n' \
'---' \
'title: py-engine smoke' \
'format: html' \
'---' \
Comment on lines +603 to +606
'' \
'```{python}' \
'import pandas as pd' \
'open("/tmp/pysmoke/chunk-ran", "w").write(str(int(pd.Series([1, 2, 3]).sum())))' \
'```' \
> /tmp/pysmoke/smoke.qmd \
&& chown -R rstudio /tmp/pysmoke \
&& su rstudio -c "cd /tmp/pysmoke && PATH='$PATH' quarto render smoke.qmd --to html" \
&& test -f /tmp/pysmoke/smoke.html \
&& grep -qx '6' /tmp/pysmoke/chunk-ran \
&& echo "quarto python-chunk render OK" \
&& rm -rf /tmp/pysmoke
# ─────────────────────────────────────────────────────────────────────────────

# ── Observable Framework ─────────────────────────────────────────────────────
Expand Down