Merge pull request #7496 from donny-wong/release_2.7.0

Release 2.7.0

Author: donny-wong

.dockerfiles/entrypoint-dev-deps-updater.sh

@@ -0,0 +1,45 @@
+#!/usr/bin/env bash
+# This script updates all MarkUs non-system dependencies
+
+# Install bundle gems
+printf "[MarkUs] Checking Ruby dependencies..."
+if ! bundle check &> /dev/null; then
+  printf "\n[MarkUs] Not all Ruby dependencies are installed. Running bundle install:\n"
+  bundle install
+else
+  printf " \e[32m✔\e[0m \n"
+fi
+
+# Install node packages
+printf "[MarkUs] Checking Javascript dependencies..."
+if ! npm list &> /dev/null; then
+  printf "\n[MarkUs] Not all Javascript dependencies are installed. Running npm install:\n"
+  npm install
+else
+  printf " \e[32m✔\e[0m \n"
+fi
+
+# Install Python packages
+[ -f ./venv/bin/python3 ] || python3 -m venv ./venv
+./venv/bin/python3 -m pip install -q --upgrade pip
+printf "[MarkUs] Running pip install -q -r requirements-jupyter.txt..."
+if ./venv/bin/python3 -m pip install -q -r requirements-jupyter.txt; then
+  printf " \e[32m✔\e[0m \n"
+fi
+printf "[MarkUs] Running pip install -q -r requirements-scanner.txt..."
+if ./venv/bin/python3 -m pip install -q -r requirements-scanner.txt; then
+  printf " \e[32m✔\e[0m \n"
+fi
+printf "[MarkUs] Running pip install -q -r requirements-qr.txt..."
+if ./venv/bin/python3 -m pip install -q -r requirements-qr.txt; then
+  printf " \e[32m✔\e[0m \n"
+fi
+
+# Install chromium (for nbconvert webpdf conversion)
+printf "[MarkUs] Running playwright install chromium..."
+if ./venv/bin/python3 -m playwright install chromium; then
+  printf " \e[32m✔\e[0m \n"
+fi
+
+# Execute the provided command
+exec "$@"

.dockerfiles/entrypoint-dev-rails.sh

@@ -5,22 +5,6 @@ if [ -z "${LD_PRELOAD+x}" ] && [ -f /usr/lib/*/libjemalloc.so.2 ]; then
   export LD_PRELOAD="$(echo /usr/lib/*/libjemalloc.so.2)"
 fi
 
-# install bundle gems if not up to date with the Gemfile.lock file
-bundle check 2>/dev/null || bundle install
-
-# install node packages
-npm list &> /dev/null || npm ci
-
-# install python packages
-[ -f ./venv/bin/python3 ] || python3 -m venv ./venv
-./venv/bin/python3 -m pip install --upgrade pip > /dev/null
-./venv/bin/python3 -m pip install -r requirements-jupyter.txt
-./venv/bin/python3 -m pip install -r requirements-scanner.txt
-./venv/bin/python3 -m pip install -r requirements-qr.txt
-
-# install chromium (for nbconvert webpdf conversion)
-./venv/bin/python3 -m playwright install chromium
-
 # setup the database (checks for db existence first)
 until pg_isready -q; do
   echo "waiting for database to start up"

.dockerfiles/entrypoint-dev-wait-for-install.sh

@@ -39,6 +39,8 @@ jobs:
       RAILS_ENV: test
       NODE_ENV: test
       RSPEC_RENDER_VIEWS: true
+    permissions:
+      contents: read
     steps:
       - name: Checkout repo
         uses: actions/checkout@v4
@@ -73,6 +75,13 @@ jobs:
           key: ${{ runner.os }}-pip-${{ hashFiles('requirements-jupyter.txt') }}-${{ hashFiles('requirements-scanner.txt') }}-${{ hashFiles('requirements-qr.txt') }}
           restore-keys: |
             ${{ runner.os }}-pip-
+      - name: Cache playwright's installation of Chromium
+        uses: actions/cache@v4
+        with:
+          path: ~/.cache/ms-playwright
+          key: ${{ runner.os }}-playwright-${{ hashFiles('requirements-jupyter.txt') }}
+          restore-keys: |
+            ${{ runner.os }}-playwright-
       - name: Install python packages and playwright dependencies
         run: |
           python3.10 -m venv venv
@@ -124,6 +133,8 @@ jobs:
     needs: test
     if: github.event.pull_request.draft == false
     runs-on: ubuntu-22.04
+    permissions:
+      pull-requests: write
     steps:
       - name: Coveralls Finished
         uses: coverallsapp/github-action@v2

.dockerfiles/entrypoint-dev-webpack.sh

@@ -16,20 +16,20 @@ repos:
               )$
         -   id: end-of-file-fixer
         -   id: trailing-whitespace
-  -   repo: https://github.com/pre-commit/mirrors-prettier
-      rev: v3.1.0
+  -   repo: https://github.com/rbubley/mirrors-prettier
+      rev: v3.5.3
       hooks:
         - id: prettier
           types_or: [javascript, jsx, css, scss, html]
   -   repo: https://github.com/thibaudcolas/pre-commit-stylelint
-      rev: v16.10.0
+      rev: v16.18.0
       hooks:
         - id: stylelint
           additional_dependencies: [
-              "stylelint@16.3.1",
+              "stylelint@16.11.0",
               "[email protected]",
-              "stylelint-config-sass-guidelines@11.1.0",
-              "[email protected].38",
+              "stylelint-config-sass-guidelines@12.1.0",
+              "[email protected].49",
             ]
           args: ["--fix"]
           types_or: ["css", "scss"]
@@ -39,7 +39,7 @@ repos:
               app/assets/stylesheets/common/_reset.scss
             )$
   - repo: https://github.com/rubocop/rubocop
-    rev: v1.68.0
+    rev: v1.75.2
     hooks:
       - id: rubocop
         args: ["--autocorrect"]
@@ -52,10 +52,10 @@ repos:
             Vagrantfile
           )$
         additional_dependencies:
-          - rubocop-rails:2.26.1
-          - rubocop-performance:1.22.1
+          - rubocop-rails:2.27.0
+          - rubocop-performance:1.23.0
           - rubocop-factory_bot:2.26.1
-          - rubocop-rspec:3.0.5
+          - rubocop-rspec:3.2.0
           - rubocop-rspec_rails:2.30.0
           - rubocop-capybara:2.21.0
 

.github/workflows/test_ci.yml

@@ -1,5 +1,139 @@
 # Changelog
 
+## [v2.7.0]
+
+### 🚨 Breaking changes
+
+### ✨ New features and improvements
+
+- Fix front-end content shift when an input element is focused (#7384)
+- Tooltip text for Package requirements file for Python autotester settings (#7383)
+- Pass back tags and overall comments from Autotester to MarkUs (#7387)
+- Render download test result button on assignment summary page only if the assignment has automated testing (#7417)
+- Group test results by Test Groups id (#7422)
+- Display HTML previews of RMarkdown files (#7394)
+- Allow instructors to assign scans to inactive students (#7482)
+- Added members parameter to add_group_api to support explicit member assignment during group creation (#7481)
+- Modified add_group_api to use username as group name for individual assignments (#7481)
+- Improved styling of markdown previews in `MarkdownPreview` component and annotation displays (#7487)
+- Allow creating image annotations from a test run's outputs (#7486)
+- Added an API that collects a single submission (#7494)
+- Enable removal of a student from a course (#7480)
+
+### 🐛 Bug fixes
+
+- Ensures row selection for peer reviewer unassigning has the same validation checks as individual selections (#7274)
+- Ensures mark levels on a rubric criterion are properly scaled when its max mark is updated (#7311)
+- Refactor contributors list in About section to read from markus-contributors.txt (#7374)
+- Optimized the querying of submissions when assigning graders (#7381)
+- Update remote_autotest_settings_id validation to be unique to an autotester (#7393)
+- Fix tag creation failing in tags table (#7426)
+- Ensure tag names are unique within an assignment (#7430)
+- Update flash style to "notice" for "setting up test env" message (#7445)
+- Fixed bug in `ImageViewer` where image width was set to 0 on initial load (#7485)
+- Fixed bug in `ImageViewer` where annotations did not disappear when moving the mouse out of the image (#7485)
+- Ensured annotations appear in results view when in fullscreen mode (#7487)
+- Fixed bug in `ImageViewer` where annotations did not always appear on initial image load (#7487)
+
+### 🔧 Internal changes
+
+- Reorganize Javascript files to separate Sprockets and Webpack compilation (#7345)
+- Replace moment.js dependency with much smaller dayjs (#7346)
+- Refactor `FileViewer`, `TextViewer`, `BinaryViewer` and `ImageViewer` components (#7270)
+- Pin `mistune` Python dependency to preserve compatibility with `nbconvert` (#7371)
+- Cache playwright's chromium installation on GitHub Actions (#7372)
+- Fix broken link to the Vagrant installation guide in `README.md` (#7349)
+- Fix `extra_hosts` configuration in `compose.yaml` (#7375)
+- Add unit tests for `marks_graders_controller` (#7382)
+- Convert front-end tests from enzyme to react testing library; add `@testing-library/user-event` (#7379)
+- Refactor the `Result` component and its children to use React context API (#7380)
+- Implement `contain_message` and `have_message` custom Rspec matchers to check for flash message content (#7386)
+- Update Python version to 3.13 in seed autotest schemas (#7388)
+- Rename jupyter notebook content functions and files to generalize to html content (#7391)
+- Update to React v18 (#7392)
+- Remove unused Javascript files (#7442)
+- Refactor scanned test generation and test splitting from polling to websockets (#7428)
+- Move Docker dependency updates into separate service (#7451)
+- Fixed flaky test due to daylight savings time issue (#7452)
+- Updated Python autotest seed file to illustrate pytest metadata custom markers (#7457)
+- Updated to react-flatpickr v4.0.0 (#7478)
+- Remove `mistune` Python explicit dependency (#7483)
+- Applied pre-commit (specifically Rubocop) changes to use it_behaves_like over include_examples (#7492)
+- Added explicit permissions to GitHub Actions workflow configuration (#7495)
+
+## [v2.6.1]
+
+### ✨ New features and improvements
+
+- Give instructors the ability to delete a TA from the Users Graders Tab (#7304)
+- Added zoom and rotate functionality to PDF viewer (#7306)
+
+### 🐛 Bug fixes
+
+- Ensure we handle JSON parsing exceptions when converting Jupyter Notebooks (#7308)
+- Fixed bug in grading context menu for editing/deleting annotations (#7314)
+- Fixed bug in grading annotations table when deleting annotations (#7314)
+- Ensure correct LTI version of lti_user_id is used on launch (#7335)
+
+## [v2.6.0]
+
+### ✨ New features and improvements
+
+- Fix front-end content shift when an input element is focused (#7384)
+- Tooltip text for Package requirements file for Python autotester settings (#7383)
+- Pass back tags and overall comments from Autotester to MarkUs (#7387)
+- Render download test result button on assignment summary page only if the assignment has automated testing (#7417)
+- Group test results by Test Groups id (#7422)
+- Display HTML previews of RMarkdown files (#7394)
+- Allow instructors to assign scans to inactive students (#7482)
+- Added members parameter to add_group_api to support explicit member assignment during group creation (#7481)
+- Modified add_group_api to use username as group name for individual assignments (#7481)
+- Improved styling of markdown previews in `MarkdownPreview` component and annotation displays (#7487)
+- Allow creating image annotations from a test run's outputs (#7486)
+- Added an API that collects a single submission (#7494)
+- Enable removal of a student from a course (#7480)
+
+### 🐛 Bug fixes
+
+- Ensures row selection for peer reviewer unassigning has the same validation checks as individual selections (#7274)
+- Ensures mark levels on a rubric criterion are properly scaled when its max mark is updated (#7311)
+- Refactor contributors list in About section to read from markus-contributors.txt (#7374)
+- Optimized the querying of submissions when assigning graders (#7381)
+- Update remote_autotest_settings_id validation to be unique to an autotester (#7393)
+- Fix tag creation failing in tags table (#7426)
+- Ensure tag names are unique within an assignment (#7430)
+- Update flash style to "notice" for "setting up test env" message (#7445)
+- Fixed bug in `ImageViewer` where image width was set to 0 on initial load (#7485)
+- Fixed bug in `ImageViewer` where annotations did not disappear when moving the mouse out of the image (#7485)
+- Ensured annotations appear in results view when in fullscreen mode (#7487)
+- Fixed bug in `ImageViewer` where annotations did not always appear on initial image load (#7487)
+
+### 🔧 Internal changes
+
+- Reorganize Javascript files to separate Sprockets and Webpack compilation (#7345)
+- Replace moment.js dependency with much smaller dayjs (#7346)
+- Refactor `FileViewer`, `TextViewer`, `BinaryViewer` and `ImageViewer` components (#7270)
+- Pin `mistune` Python dependency to preserve compatibility with `nbconvert` (#7371)
+- Cache playwright's chromium installation on GitHub Actions (#7372)
+- Fix broken link to the Vagrant installation guide in `README.md` (#7349)
+- Fix `extra_hosts` configuration in `compose.yaml` (#7375)
+- Add unit tests for `marks_graders_controller` (#7382)
+- Convert front-end tests from enzyme to react testing library; add `@testing-library/user-event` (#7379)
+- Refactor the `Result` component and its children to use React context API (#7380)
+- Implement `contain_message` and `have_message` custom Rspec matchers to check for flash message content (#7386)
+- Update Python version to 3.13 in seed autotest schemas (#7388)
+- Rename jupyter notebook content functions and files to generalize to html content (#7391)
+- Update to React v18 (#7392)
+- Remove unused Javascript files (#7442)
+- Refactor scanned test generation and test splitting from polling to websockets (#7428)
+- Move Docker dependency updates into separate service (#7451)
+- Fixed flaky test due to daylight savings time issue (#7452)
+- Updated Python autotest seed file to illustrate pytest metadata custom markers (#7457)
+- Updated to react-flatpickr v4.0.0 (#7478)
+- Remove `mistune` Python explicit dependency (#7483)
+- Applied pre-commit (specifically Rubocop) changes to use it_behaves_like over include_examples (#7492)
+- Added explicit permissions to GitHub Actions workflow configuration (#7495)
+
 ## [v2.6.1]
 
 ### ✨ New features and improvements

.pre-commit-config.yaml

@@ -37,16 +37,16 @@ gem 'histogram'
 # Internationalization
 gem 'i18n'
 gem 'i18n-js'
-gem 'rails-i18n', '~> 7.0.0'
+gem 'rails-i18n', '~> 7.0.10'
 
 # Redis
-gem 'redis', '~> 5.3.0'
+gem 'redis', '~> 5.4.0'
 
 # Exam template requirements
 gem 'combine_pdf'
 gem 'prawn'
 gem 'prawn-qrcode'
-gem 'rmagick', '~> 6.0.1'
+gem 'rmagick', '~> 6.1.1'
 gem 'rtesseract'
 
 # Ruby miscellany
@@ -107,7 +107,7 @@ group :development, :test do
   gem 'capybara'
   gem 'debug', '>= 1.0.0'
   gem 'i18n-tasks', require: false
-  gem 'rspec-rails', '~> 7.0.1'
+  gem 'rspec-rails', '~> 7.1.1'
   gem 'selenium-webdriver'
 end