🌱 Re-enable skipped e2e tests. Switch to smaller code review repo.

[spencerschrock]

What kind of change does this PR introduce?

Follow up to #3130

• PR title follows the guidelines defined in our pull request documentation

What is the current behavior?

What is the new behavior (if this is a feature change)?**

• Re-enables skipped tests

Reduced attestor e2e test times by ~50%

• Switches code-review attestor tests to a smaller repo in ossf-tests, instead of using ossf/scorecard
• trims un-needed policies from tests that aren't using them (code review tests don't need binary artifact policies, etc)

• Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes

Fixes #3129

Special notes for your reviewer

It's hard to see changes with everything being uncommented. So a better comparison would be to the version before the tests were disabled (just scroll to the right file):
d961dda...ossf:scorecard:a43601cc408a60f96e5a6e3d7e012a84c13a5239#diff-1eb6d0cf3162bc067afd6a66f7e7380fef3bab5185a2faa320edbe9ea2282453

Does this PR introduce a user-facing change?

For user-facing changes, please add a concise, human-readable release note to
the release-note

(In particular, describe what changes users might need to make in their
application as a result of this pull request.)

NONE

[codecov]

Codecov Report

Merging #3144 (a43601c) into main (0d84edf) will decrease coverage by 0.52%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3144      +/-   ##
==========================================
- Coverage   65.02%   64.50%   -0.52%     
==========================================
  Files         166      166              
  Lines       12402    12402              
==========================================
- Hits         8064     8000      -64     
- Misses       3887     3956      +69     
+ Partials      451      446       -5