Add a switch to submit_cran() in release()

[maelle]

Like an environment variable JUST_EXPERIMENTING or so. :sweat_smile:

[krlmlr]

This is very relevant in the broader context of testability. I think we need to create internal APIs for everything that is external:

• GitHub (#267)
• CRAN

For testing we would switch to mock implementations of these external components.

[maelle]

Noting that for GitHub PR/issue API data, we have mock files for tests.

[maelle]

What kind of mocking are you envisioning for testing auto.R? Mock files and environment variables to not run some code, or something like https://webfakes.r-lib.org/?

[maelle]

I think I'm seeing some incompatibility between httptest::with_mock_dir() and with_repo() :thinking:

[maelle]

https://github.com/nealrichardson/httptest/issues/74

(good to know if we decide to use httptest when testing bigger parts of the code than what we've done currently -- which is why we hadn't encountered this problem)

[krlmlr]

This is about a specific implementation detail, we still want to think about testing. OTOH, the automated release process is "tested" every time somebody uses it.

Let's revisit this when we make a bigger attempt at testing "everything".