Ensure fetch method is always uppercase

[npezza93]

Fixes #1100

In 7.3 this was always upper cased but in v8 it is not. This breaks forms that use formmethod on buttons. On chrome and firefox puma returns a 400 but once the method is uppercased it works.

turbo/src/http/fetch_request.ts

Line 143 in 4593d06

method: FetchMethod[this.method].toUpperCase(),

[npezza93]

@afcapel Are there flaky tests in the suite or are these legit failures?

[afcapel]

The autofocus tests are a bit flaky, see #1154

[npezza93]

On main, locally the loading_test tests is failing for me as well.

[rainerborene]

I am using this workaround until this PR gets merged.

document.addEventListener("turbo:before-fetch-request", ({ detail: { fetchOptions } }) => {
  fetchOptions.method = fetchOptions.method.toUpperCase()
})

[npezza93]

@brunoprietog Finally figured out the issue with tests here. This is ready to go now

[brunoprietog]

Nice! Thanks @npezza93!