diff options
author | 2021-01-29 10:33:36 -0800 | |
---|---|---|
committer | 2021-01-29 10:33:36 -0800 | |
commit | 4c4b137a70a13dd1fe99cb06630000d16188609d (patch) | |
tree | abd5bbda6a285337b829786335212fe055ee7452 /infra/cifuzz/config_utils_test.py | |
parent | 5740711ba12b87f9cbb04f0cfbf9dd01b787b8ca (diff) |
[cifuzz][external] Use ssh_url and fix affected fuzzers (#5072)
1. Use ssh_url.
This only affects external (non-oss-fuzz) users.
Since there are none, it doesn't affect anyone.
Even if it did, exploitation would require owning the network
Github actions runs on.
This is to prevent MITM attacks.
2. Affected fuzzers bug:
We accidentally were skipping the remove unaffected functionality.
Diffstat (limited to 'infra/cifuzz/config_utils_test.py')
-rw-r--r-- | infra/cifuzz/config_utils_test.py | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/infra/cifuzz/config_utils_test.py b/infra/cifuzz/config_utils_test.py index 12daf7ee..71e7450f 100644 --- a/infra/cifuzz/config_utils_test.py +++ b/infra/cifuzz/config_utils_test.py @@ -39,3 +39,14 @@ class BuildFuzzersConfigTest(unittest.TestCase): """Tests that base_ref is set properly.""" expected_base_ref = 'expected_base_ref' os.environ['GITHUB_BASE_REF'] = expected_base_ref + config = self._create_config() + self.assertEqual(config.base_ref, expected_base_ref) + + def test_keep_unaffected_defaults_to_false(self): + """Tests that keep_unaffected_fuzz_targets defaults to false.""" + config = self._create_config() + self.assertFalse(config.keep_unaffected_fuzz_targets) + + +if __name__ == '__main__': + unittest.main() |