tests: five adopt tests are weaker than they read: text on stdin, a stub gh written with write_text, GIT_ALLOW_PROTOCOL not cleared, two guard pairs that mask each other, and one name shape in the naming test #916

Goal: each adopt test named here fails when the thing it guards is removed, on every OS and environment the suite runs in.

Filed by lane openRepoShape-3, from its triage of #457 at the v1.1.1 release commit 5017830. It groups #457's lines L4-54, L1-60, L1-62, L1-K6 and L2-91, one pull request's worth. None is a product bug.

State at 5017830. Read, not run.

• L4-54. split_commit_of feeds git hash-object --stdin a link target with input=target, ... text=True (tests/test_adopt_ignored_shape_files.py:294-295). It is the sibling of the two helpers tests.yml and adopt-project.py: the Windows job installs git-filter-repo 2.47.0, so the adoption test files that skip without it run there, and a source that keeps its own root .gitattributes gets one marked block of /<copy> text eol=lf lines appended to it in the split, so a Windows clone writes the pinned copies at the root with the bytes the pin records; the verification counts that file as the source's bytes followed by exactly that block, and update-shape.py apply --add extends it (#471) #511's NIT-6 moved to bytes. It is harmless today, but it goes against the bytes-on-stdin rule.
• L1-60. stub_gh writes its #! script with Path.write_text (tests/test_adopt_pins.py:112-122), so a CRLF emulation run fails test_with_no_pin_source_plan_reads_the_digest_from_gh.
• L1-62. test_git_is_held_to_the_disk_when_it_asks (tests/test_adopt_submodule_url_answers.py:301) never clears GIT_ALLOW_PROTOCOL, so a caller's environment can mask a missing guard.
• L1-K6. Two pairs of guards each cover the other:
  • _execute_gets_to_write's visibility clause (adopt-project.py:4120) and left = judged | frozenset(unwritten) (:4264);
  • _pushed_in_every_mode's two questions (:4161).
    Lane 1 measured on 9efe95a that reverting either guard of a pair alone fails nothing, or only that guard's own cases. The code is unchanged in shape here; the mutants were not run again.
• L2-91. test_no_root_or_family_name_holds_a_character_no_id_takes asks only North{c}Wind (tests/test_naming_policy.py:252-254). So a policy whose neutral-product or domain-descendant pattern takes _ passes the whole file.

Direction.

• Feed bytes on stdin.
• Write the stub as bytes.
• monkeypatch.delenv("GIT_ALLOW_PROTOCOL", raising=False).
• Give each guard a test that reaches it alone, or keep one guard of each pair.
• Parametrize the naming test over openDo{c}x and Med{c}xScribe as well.

Size: M.

Lane: openreposhape-3 (openRepoShape-3)