Repository navigation
Add Plone 6.2 support and tests, fix the remaining Plone 6 incompatibilities - #19
Open
chris-adam wants to merge 11 commits into
Open
chris-adam wants to merge 11 commits into
chris-adam wants to merge 11 commits into
Conversation
collective.monkeypatcher 2.0.1 (Plone 6.1) still declares the pkg_resources "collective" namespace, which hides the PEP 420 collective.upgrade: bin/test failed with "No module named 'collective.upgrade'". horse-with-no-namespace makes both work. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
test-6.2.cfg and requirements-6.2.txt from IMIO/scripts-buildout, Makefile limited to 6.1 and 6.2 (Python 3.13), GitHub Actions tests 6.1 and 6.2, coverage on 6.2 with a .coveragerc for Coveralls. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
51 unit/functional tests (one module per production module, pas.rst doctest kept) and 4 robot scenarios for the only browser entry point, @@collective.upgrade.form. They pin the Plone 4 behaviour: green on Plone 4.3 (master worktree). On Plone 6.2, 7 tests and 1 scenario fail on the code still to port (quickinstaller, reference_catalog, plone.app.folder, @@plone-addsite, log streaming to the WSGI response, Python 3 dict view). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Stream the upgrade log to the WSGI response as bytes (the form showed an empty page). - No more portal_quickinstaller: profile versions decide what is installed, uninstallAddOns uses the installer view. - fixDuplicateUIDs without reference_catalog (plone.uuid), Python 3 sort key fix; CMFEditions folder migration and reference targets cleanup skip when plone.app.folder / Archetypes are missing. - resetProfiles without @@plone-addsite; cleanupBrokenTextIndexes on Python 3. - Version 2.0.0.dev0, Plone 6.2 classifier, obsolete test_suite removed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Coverage Report for CI Build 38566872675Warning No base build found for commit Coverage: 89.825%Details
Uncovered Changes
Coverage RegressionsRequires a base build to compare against. How to fix this → Coverage Stats
💛 - Coveralls |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
chris-adam
marked this pull request as ready for review
October 28, 2026 09:56
rfbrowser init fails on ubuntu 26.04 (not supported by Playwright yet). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Builds on
plone61: adds a Plone 6.2 buildout and CI (6.1 and 6.2), fixes the 6.1 tests broken by the native namespace (horse-with-no-namespace), and ports what was left for Plone 6: the upgrade form's log stream,portal_quickinstaller,reference_catalog,plone.app.folderand@@plone-addsite. It also adds unit tests and robot tests for the upgrade form, which first pinned the Plone 4 behaviour on master.resetProfilescan't re-import the Plone base profile on 6.2: Products.DateRecurringIndex 4.0.0 breaks the catalog step re-import (AttributeError: _since_field), which is a Plone core issue.🤖 Generated with Claude Code