search

zopefoundation / zope.app.form

The Original Zope 3 Form Framework
Other
0 stars 4 forks source link

Update tests to run with `multipart >= 1`. #14

Open icemac opened 1 day ago

icemac commented 1 day ago

Fixes #13.

d-maurer commented 1 day ago

Michael Howitz wrote at 2024-10-16 23:47 -0700:

Fixes #13. You can view, comment on, or merge this pull request online at:

Looks it gets a habit that we work concurrently on fixes.

I assume that my #15 is insignificantly better than your fix.

d-maurer commented 1 day ago

@.*** wrote at 2024-10-17 09:30 +0200:

Michael Howitz wrote at 2024-10-16 23:47 -0700: ... I assume that my #15 is insignificantly better than your fix.

But almost surely, the multipart change should get handled in zope.app.wsgi.testlayer (and not in zope.app.form at all). This way, other tests could profit as well.

icemac commented 9 hours ago

@d-maurer I moved your fix from #15 to https://github.com/zopefoundation/zope.app.wsgi/pull/29. Would be nice if you could review.