From 04cb57a14bc2cca347c6c9a3c48c8edb94fd16be Mon Sep 17 00:00:00 2001
From: alafanechere <augustin.lafanechere@gmail.com>
Date: Tue, 20 Feb 2024 09:12:09 +0100
Subject: [PATCH] remove qa_checks.py

---
 .../connector_ops/connector_ops/qa_checks.py  | 320 ------------------
 .../connectors/connector_ops/pyproject.toml   |   2 +-
 .../connector_ops/tests/test_qa_checks.py     | 274 ---------------
 .../testing-connectors/README.md              |   2 +-
 4 files changed, 2 insertions(+), 596 deletions(-)
 delete mode 100644 airbyte-ci/connectors/connector_ops/connector_ops/qa_checks.py
 delete mode 100644 airbyte-ci/connectors/connector_ops/tests/test_qa_checks.py

diff --git a/airbyte-ci/connectors/connector_ops/connector_ops/qa_checks.py b/airbyte-ci/connectors/connector_ops/connector_ops/qa_checks.py
deleted file mode 100644
index 99bd8e1786b0ee..00000000000000
--- a/airbyte-ci/connectors/connector_ops/connector_ops/qa_checks.py
+++ /dev/null
@@ -1,320 +0,0 @@
-#
-# Copyright (c) 2023 Airbyte, Inc., all rights reserved.
-#
-
-
-import sys
-from pathlib import Path
-from typing import Callable, Iterable, Optional, Set, Tuple
-
-from connector_ops.utils import Connector, ConnectorLanguage
-from pydash.objects import get
-
-
-def check_migration_guide(connector: Connector) -> bool:
-    """Check if a migration guide is available for the connector if a breaking change was introduced."""
-
-    breaking_changes = get(connector.metadata, "releases.breakingChanges")
-    if not breaking_changes:
-        return True
-
-    migration_guide_file_path = connector.migration_guide_file_path
-    migration_guide_exists = migration_guide_file_path is not None and migration_guide_file_path.exists()
-    if not migration_guide_exists:
-        print(
-            f"Migration guide file is missing for {connector.name}. Please create a migration guide at {connector.migration_guide_file_path}"
-        )
-        return False
-
-    # Check that the migration guide begins with # {connector name} Migration Guide
-    expected_title = f"# {connector.name_from_metadata} Migration Guide"
-    expected_version_header_start = "## Upgrading to "
-    with open(migration_guide_file_path) as f:
-        first_line = f.readline().strip()
-        if not first_line == expected_title:
-            print(
-                f"Migration guide file for {connector.technical_name} does not start with the correct header. Expected '{expected_title}', got '{first_line}'"
-            )
-            return False
-
-        # Check that the migration guide contains a section for each breaking change key ## Upgrading to {version}
-        # Note that breaking change is a dict where the version is the key
-        # Note that the migration guide must have the sections in order of the version descending
-        # 3.0.0, 2.0.0, 1.0.0, etc
-        # This means we have to record the headings in the migration guide and then check that they are in order
-        # We also have to check that the headings are in the breaking changes dict
-
-        ordered_breaking_changes = sorted(breaking_changes.keys(), reverse=True)
-        ordered_expected_headings = [f"{expected_version_header_start}{version}" for version in ordered_breaking_changes]
-
-        ordered_heading_versions = []
-        for line in f:
-            stripped_line = line.strip()
-            if stripped_line.startswith(expected_version_header_start):
-                version = stripped_line.replace(expected_version_header_start, "")
-                ordered_heading_versions.append(version)
-
-        if ordered_breaking_changes != ordered_heading_versions:
-            print(f"Migration guide file for {connector.name} has incorrect version headings.")
-            print("Check for missing, extra, or misordered headings, or headers with typos.")
-            print(f"Expected headings: {ordered_expected_headings}")
-            return False
-
-    return True
-
-
-def check_documentation_file_exists(connector: Connector) -> bool:
-    """Check if a markdown file with connector documentation is available
-    in docs/integrations/<connector-type>s/<connector-name>.md
-
-    Args:
-        connector (Connector): a Connector dataclass instance.
-
-    Returns:
-        bool: Wether a documentation file was found.
-    """
-    file_path = connector.documentation_file_path
-
-    return file_path is not None and file_path.exists()
-
-
-def check_documentation_follows_guidelines(connector: Connector) -> bool:
-    """Documentation guidelines are defined here https://hackmd.io/Bz75cgATSbm7DjrAqgl4rw"""
-    follows_guidelines = True
-    with open(connector.documentation_file_path) as f:
-        doc_lines = [line.lower() for line in f.read().splitlines()]
-    if not doc_lines[0].startswith("# "):
-        print("The connector name is not used as the main header in the documentation.")
-        follows_guidelines = False
-    # We usually don't have a metadata if the connector is not published.
-    if connector.metadata:
-        if doc_lines[0].strip() != f"# {connector.metadata['name'].lower()}":
-            print("The connector name is not used as the main header in the documentation.")
-            follows_guidelines = False
-    elif not doc_lines[0].startswith("# "):
-        print("The connector name is not used as the main header in the documentation.")
-        follows_guidelines = False
-
-    expected_sections = ["## Prerequisites", "## Setup guide", "## Supported sync modes", "## Supported streams", "## Changelog"]
-
-    for expected_section in expected_sections:
-        if expected_section.lower() not in doc_lines:
-            print(f"Connector documentation is missing a '{expected_section.replace('#', '').strip()}' section.")
-            follows_guidelines = False
-    return follows_guidelines
-
-
-def check_changelog_entry_is_updated(connector: Connector) -> bool:
-    """Check that the changelog entry is updated for the latest connector version
-    in docs/integrations/<connector-type>/<connector-name>.md
-
-    Args:
-        connector (Connector): a Connector dataclass instance.
-
-    Returns:
-        bool: Wether a the changelog is up to date.
-    """
-    if not check_documentation_file_exists(connector):
-        return False
-    with open(connector.documentation_file_path) as f:
-        after_changelog = False
-        for line in f:
-            if "# changelog" in line.lower():
-                after_changelog = True
-            if after_changelog and connector.version in line:
-                return True
-    return False
-
-
-def check_connector_icon_is_available(connector: Connector) -> bool:
-    """Check an SVG icon exists for a connector in
-    in airbyte-config-oss/init-oss/src/main/resources/icons/<connector-name>.svg
-
-    Args:
-        connector (Connector): a Connector dataclass instance.
-
-    Returns:
-        bool: Wether an icon exists for this connector.
-    """
-    return connector.icon_path.exists()
-
-
-def read_all_files_in_directory(
-    directory: Path, ignored_directories: Optional[Set[str]] = None, ignored_filename_patterns: Optional[Set[str]] = None
-) -> Iterable[Tuple[str, str]]:
-    ignored_directories = ignored_directories if ignored_directories is not None else {}
-    ignored_filename_patterns = ignored_filename_patterns if ignored_filename_patterns is not None else {}
-
-    for path in directory.rglob("*"):
-        ignore_directory = any([ignored_directory in path.parts for ignored_directory in ignored_directories])
-        ignore_filename = any([path.match(ignored_filename_pattern) for ignored_filename_pattern in ignored_filename_patterns])
-        ignore = ignore_directory or ignore_filename
-        if path.is_file() and not ignore:
-            try:
-                for line in open(path, "r"):
-                    yield path, line
-            except UnicodeDecodeError:
-                print(f"{path} could not be decoded as it is not UTF8.")
-                continue
-
-
-IGNORED_DIRECTORIES_FOR_HTTPS_CHECKS = {
-    ".venv",
-    "tests",
-    "unit_tests",
-    "integration_tests",
-    "build",
-    "source-file",
-    ".pytest_cache",
-    "acceptance_tests_logs",
-    ".hypothesis",
-}
-
-IGNORED_FILENAME_PATTERN_FOR_HTTPS_CHECKS = {
-    "*Test.java",
-    "*.jar",
-    "*.pyc",
-    "*.gz",
-    "*.svg",
-    "expected_records.jsonl",
-    "expected_records.json",
-}
-IGNORED_URLS_PREFIX = {
-    "http://json-schema.org",
-    "http://localhost",
-}
-
-
-def is_comment(line: str, file_path: Path):
-    language_comments = {
-        ".py": "#",
-        ".yml": "#",
-        ".yaml": "#",
-        ".java": "//",
-        ".md": "<!--",
-    }
-
-    denote_comment = language_comments.get(file_path.suffix)
-    if not denote_comment:
-        return False
-
-    trimmed_line = line.lstrip()
-    return trimmed_line.startswith(denote_comment)
-
-
-def check_connector_https_url_only(connector: Connector) -> bool:
-    """Check a connector code contains only https url.
-
-    Args:
-        connector (Connector): a Connector dataclass instance.
-
-    Returns:
-        bool: Wether the connector code contains only https url.
-    """
-    files_with_http_url = set()
-    ignore_comment = "# ignore-https-check"  # Define the ignore comment pattern
-
-    for filename, line in read_all_files_in_directory(
-        connector.code_directory, IGNORED_DIRECTORIES_FOR_HTTPS_CHECKS, IGNORED_FILENAME_PATTERN_FOR_HTTPS_CHECKS
-    ):
-        line = line.lower()
-        if is_comment(line, filename):
-            continue
-        if ignore_comment in line:
-            continue
-        for prefix in IGNORED_URLS_PREFIX:
-            line = line.replace(prefix, "")
-        if "http://" in line:
-            files_with_http_url.add(str(filename))
-
-    if files_with_http_url:
-        files_with_http_url = "\n\t- ".join(files_with_http_url)
-        print(f"The following files have http:// URLs:\n\t- {files_with_http_url}")
-        return False
-    return True
-
-
-def check_connector_has_no_critical_vulnerabilities(connector: Connector) -> bool:
-    """Check if the connector image is free of critical Snyk vulnerabilities.
-    Runs a docker scan command.
-
-    Args:
-        connector (Connector): a Connector dataclass instance.
-
-    Returns:
-        bool: Wether the connector is free of critical vulnerabilities.
-    """
-    # TODO implement
-    return True
-
-
-def check_metadata_version_matches_dockerfile_label(connector: Connector) -> bool:
-    version_in_dockerfile = connector.version_in_dockerfile_label
-    if version_in_dockerfile is None:
-        # Java connectors don't have Dockerfiles.
-        return connector.language == ConnectorLanguage.JAVA
-    return version_in_dockerfile == connector.version
-
-
-DEFAULT_QA_CHECKS = (
-    check_documentation_file_exists,
-    check_migration_guide,
-    # Disabling the following check because it's likely to not pass on a lot of connectors.
-    # check_documentation_follows_guidelines,
-    check_changelog_entry_is_updated,
-    check_connector_icon_is_available,
-    # TODO - check_connector_https_url_only might be redundant once the following issues are closed
-    # https://github.com/airbytehq/airbyte/issues/20552
-    # https://github.com/airbytehq/airbyte/issues/21606
-    check_connector_https_url_only,
-    check_connector_has_no_critical_vulnerabilities,
-)
-
-
-def get_qa_checks_to_run(connector: Connector) -> Tuple[Callable]:
-    if connector.has_dockerfile:
-        return DEFAULT_QA_CHECKS + (check_metadata_version_matches_dockerfile_label,)
-    return DEFAULT_QA_CHECKS
-
-
-def remove_strict_encrypt_suffix(connector_technical_name: str) -> str:
-    """Remove the strict encrypt suffix from a connector name.
-
-    Args:
-        connector_technical_name (str): the connector name.
-
-    Returns:
-        str: the connector name without the strict encrypt suffix.
-    """
-    strict_encrypt_suffixes = [
-        "-strict-encrypt",
-        "-secure",
-    ]
-
-    for suffix in strict_encrypt_suffixes:
-        if connector_technical_name.endswith(suffix):
-            new_connector_technical_name = connector_technical_name.replace(suffix, "")
-            print("Checking connector " + new_connector_technical_name + " due to strict-encrypt")
-            return new_connector_technical_name
-    return connector_technical_name
-
-
-def run_qa_checks():
-    connector_technical_name = sys.argv[1].split("/")[-1]
-    if not connector_technical_name.startswith("source-") and not connector_technical_name.startswith("destination-"):
-        print("No QA check to run as this is not a connector.")
-        sys.exit(0)
-
-    connector_technical_name = remove_strict_encrypt_suffix(connector_technical_name)
-    connector = Connector(connector_technical_name)
-    print(f"Running QA checks for {connector_technical_name}:{connector.version}")
-    qa_check_results = {qa_check.__name__: qa_check(connector) for qa_check in get_qa_checks_to_run(connector)}
-    if not all(qa_check_results.values()):
-        print(f"QA checks failed for {connector_technical_name}:{connector.version}:")
-        for check_name, check_result in qa_check_results.items():
-            check_result_prefix = "✅" if check_result else "❌"
-            print(f"{check_result_prefix} - {check_name}")
-        sys.exit(1)
-    else:
-        print(f"All QA checks succeeded for {connector_technical_name}:{connector.version}")
-        sys.exit(0)
diff --git a/airbyte-ci/connectors/connector_ops/pyproject.toml b/airbyte-ci/connectors/connector_ops/pyproject.toml
index 3270d58e8be6b1..8aae2d66bbb3ee 100644
--- a/airbyte-ci/connectors/connector_ops/pyproject.toml
+++ b/airbyte-ci/connectors/connector_ops/pyproject.toml
@@ -4,7 +4,7 @@ build-backend = "poetry.core.masonry.api"
 
 [tool.poetry]
 name = "connector_ops"
-version = "0.3.3"
+version = "0.3.4"
 description = "Packaged maintained by the connector operations team to perform CI for connectors"
 authors = ["Airbyte <contact@airbyte.io>"]
 
diff --git a/airbyte-ci/connectors/connector_ops/tests/test_qa_checks.py b/airbyte-ci/connectors/connector_ops/tests/test_qa_checks.py
deleted file mode 100644
index f87bdbba384f2e..00000000000000
--- a/airbyte-ci/connectors/connector_ops/tests/test_qa_checks.py
+++ /dev/null
@@ -1,274 +0,0 @@
-#
-# Copyright (c) 2023 Airbyte, Inc., all rights reserved.
-#
-
-
-from pathlib import Path
-
-import pytest
-from connector_ops import qa_checks, utils
-
-
-@pytest.mark.parametrize(
-    "connector, expect_exists",
-    [
-        (utils.Connector("source-faker"), True),
-        (utils.Connector("source-foobar"), False),
-    ],
-)
-def test_check_documentation_file_exists(connector, expect_exists):
-    assert qa_checks.check_documentation_file_exists(connector) == expect_exists
-
-
-def test_check_changelog_entry_is_updated_missing_doc(mocker):
-    mocker.patch.object(qa_checks, "check_documentation_file_exists", mocker.Mock(return_value=False))
-    assert qa_checks.check_changelog_entry_is_updated(qa_checks.Connector("source-foobar")) is False
-
-
-def test_check_changelog_entry_is_updated_no_changelog_section(mocker, tmp_path):
-    mock_documentation_file_path = Path(tmp_path / "doc.md")
-    mock_documentation_file_path.touch()
-
-    mocker.patch.object(qa_checks.Connector, "documentation_file_path", mock_documentation_file_path)
-    assert qa_checks.check_changelog_entry_is_updated(qa_checks.Connector("source-foobar")) is False
-
-
-def test_check_changelog_entry_is_updated_version_not_in_changelog(mocker, tmp_path):
-    mock_documentation_file_path = Path(tmp_path / "doc.md")
-    with open(mock_documentation_file_path, "w") as f:
-        f.write("# Changelog")
-
-    mocker.patch.object(qa_checks.Connector, "documentation_file_path", mock_documentation_file_path)
-
-    mocker.patch.object(qa_checks.Connector, "version", "0.0.0")
-
-    assert qa_checks.check_changelog_entry_is_updated(qa_checks.Connector("source-foobar")) is False
-
-
-def test_check_changelog_entry_is_updated_version_in_changelog(mocker, tmp_path):
-    mock_documentation_file_path = Path(tmp_path / "doc.md")
-    with open(mock_documentation_file_path, "w") as f:
-        f.write("# Changelog\n0.0.0")
-
-    mocker.patch.object(qa_checks.Connector, "documentation_file_path", mock_documentation_file_path)
-
-    mocker.patch.object(qa_checks.Connector, "version", "0.0.0")
-    assert qa_checks.check_changelog_entry_is_updated(qa_checks.Connector("source-foobar"))
-
-
-@pytest.mark.parametrize(
-    "connector, expect_exists",
-    [
-        (utils.Connector("source-faker"), True),
-        (utils.Connector("source-foobar"), False),
-    ],
-)
-def test_check_connector_icon_is_available(connector, expect_exists):
-    assert qa_checks.check_connector_icon_is_available(connector) == expect_exists
-
-
-@pytest.mark.parametrize(
-    "user_input, expect_qa_checks_to_run",
-    [
-        ("not-a-connector", False),
-        ("connectors/source-faker", True),
-        ("source-faker", True),
-    ],
-)
-def test_run_qa_checks_success(capsys, mocker, user_input, expect_qa_checks_to_run):
-    mocker.patch.object(qa_checks.sys, "argv", ["", user_input])
-    mocker.patch.object(qa_checks, "Connector")
-    mock_qa_check = mocker.Mock(return_value=True, __name__="mock_qa_check")
-    if expect_qa_checks_to_run:
-        mocker.patch.object(qa_checks, "get_qa_checks_to_run", return_value=[mock_qa_check])
-    with pytest.raises(SystemExit) as wrapped_error:
-        qa_checks.run_qa_checks()
-    assert wrapped_error.value.code == 0
-    if not expect_qa_checks_to_run:
-        qa_checks.Connector.assert_not_called()
-        stdout, _ = capsys.readouterr()
-        assert "No QA check to run" in stdout
-    else:
-        expected_connector_technical_name = user_input.split("/")[-1]
-        qa_checks.Connector.assert_called_with(expected_connector_technical_name)
-        mock_qa_check.assert_called_with(qa_checks.Connector.return_value)
-        stdout, _ = capsys.readouterr()
-        assert f"Running QA checks for {expected_connector_technical_name}" in stdout
-        assert f"All QA checks succeeded for {expected_connector_technical_name}" in stdout
-
-
-def test_run_qa_checks_error(capsys, mocker):
-    mocker.patch.object(qa_checks.sys, "argv", ["", "source-faker"])
-    mocker.patch.object(qa_checks, "Connector")
-    mock_qa_check = mocker.Mock(return_value=False, __name__="mock_qa_check")
-    mocker.patch.object(qa_checks, "DEFAULT_QA_CHECKS", (mock_qa_check,))
-    with pytest.raises(SystemExit) as wrapped_error:
-        qa_checks.run_qa_checks()
-    assert wrapped_error.value.code == 1
-    stdout, _ = capsys.readouterr()
-    assert "QA checks failed for source-faker" in stdout
-    assert "❌ - mock_qa_check" in stdout
-
-
-@pytest.mark.parametrize(
-    "file_name, file_line, expected_in_stdout",
-    [
-        ("file_with_http_url.foo", "http://foo.bar", True),
-        ("file_with_http_url_and_ignore_comment.foo", "http://foo.bar # ignore-https-check", False),
-        ("file_without_https_url.foo", "", False),
-        ("file_with_https_url.foo", "https://airbyte.com", False),
-        ("file_with_http_url_and_ignored.foo", "http://localhost http://airbyte.com", True),
-        ("file_with_ignored_url.foo", "http://localhost", False),
-        ("file_with_http_url_in_comment.py", "# http://dev.foo", False),
-        ("file_with_http_url_in_comment.yml", "# http://dev.foo", False),
-        ("file_with_http_url_in_comment.yaml", "# http://dev.foo", False),
-        ("file_with_http_url_in_comment.java", "// http://dev.foo", False),
-        ("file_with_http_url_in_comment.md", "<!-- http://dev.foo", False),
-    ],
-)
-def test_check_connector_https_url_only(capsys, tmp_path, mocker, file_name, file_line, expected_in_stdout):
-    file_with_url = Path(tmp_path / file_name)
-    mocker.patch.object(qa_checks, "IGNORED_DIRECTORIES_FOR_HTTPS_CHECKS", set())
-    with open(file_with_url, "w") as f:
-        f.write(file_line)
-    connector = mocker.Mock(code_directory=tmp_path)
-    assert expected_in_stdout != qa_checks.check_connector_https_url_only(connector)
-    stdout, _ = capsys.readouterr()
-    if expected_in_stdout:
-        assert file_name in stdout
-
-
-@pytest.mark.skip(reason="This should only be run when we want to test all connectors for their https url only compliance")
-def test_check_connector_https_url_only_all_connectors():
-    failing_connectors = []
-    for raw_connector in utils.OSS_CATALOG["sources"] + utils.OSS_CATALOG["destinations"]:
-        technical_name = raw_connector["dockerRepository"].replace("airbyte/", "")
-        connector = utils.Connector(technical_name)
-        if not qa_checks.check_connector_https_url_only(connector):
-            failing_connectors.append(connector)
-    if failing_connectors:
-        by_ab_internal_sl = {}
-        for failing_connector in failing_connectors:
-            by_ab_internal_sl.setdefault(failing_connector.ab_internal_sl, [])
-            by_ab_internal_sl[failing_connector.ab_internal_sl].append(failing_connector)
-        failure_message = ""
-        for ab_internal_sl in by_ab_internal_sl.keys():
-            failure_message += f"\nFailing SL {ab_internal_sl} connectors:\n"
-            for connector in by_ab_internal_sl[ab_internal_sl]:
-                failure_message += f"\t- {connector.technical_name}\n"
-        pytest.fail(failure_message)
-
-
-@pytest.mark.parametrize(
-    "file_name, line, expect_is_comment",
-    [
-        ("foo.py", "# I'm a comment", True),
-        ("foo.py", "   # I'm a comment", True),
-        ("foo.py", "I'm not # a comment", False),
-        ("foo.yaml", "# I'm a comment", True),
-        ("foo.yaml", "   # I'm a comment", True),
-        ("foo.yaml", "I'm not # a comment", False),
-        ("foo.yml", "# I'm a comment", True),
-        ("foo.yml", "   # I'm a comment", True),
-        ("foo.yml", "I'm not # a comment", False),
-        ("foo.java", "// I'm a comment", True),
-        ("foo.java", "   // I'm a comment", True),
-        ("foo.java", "I'm not // a comment", False),
-        ("foo.md", "<!-- I'm a comment", True),
-        ("foo.md", "   <!-- I'm a comment", True),
-        ("foo.md", "I'm not <!-- a comment", False),
-    ],
-)
-def test_is_comment(tmp_path, file_name, line, expect_is_comment):
-    file_path = tmp_path / file_name
-    assert qa_checks.is_comment(line, file_path) is expect_is_comment
-
-
-def test_check_missing_migration_guide(mocker, tmp_path, capsys):
-    connector = qa_checks.Connector("source-foobar")
-    local_connector_documentation_directory = Path(tmp_path)
-    mocker.patch.object(qa_checks.Connector, "local_connector_documentation_directory", local_connector_documentation_directory)
-
-    mock_metadata_dict = {
-        "documentationUrl": tmp_path,
-        "releases": {
-            "breakingChanges": {
-                "2.0.0": {
-                    "upgradeDeadline": "2021-01-01",
-                    "message": "This is a breaking change",
-                }
-            }
-        },
-    }
-    mocker.patch.object(qa_checks.Connector, "metadata", mock_metadata_dict)
-
-    assert qa_checks.check_migration_guide(connector) is False
-    stdout, _ = capsys.readouterr()
-    # f"Migration guide file is missing for {connector.name}. Please create a migration guide at {connector.migration_guide_file_path}"
-
-    assert (
-        f"Migration guide file is missing for foobar. Please create a migration guide at {local_connector_documentation_directory}/foobar-migrations.md"
-        in stdout
-    )
-
-
-@pytest.mark.parametrize(
-    "test_file, expected_stdout",
-    [
-        ("bad-header.md", "has incorrect version headings"),
-        ("out-of-order.md", "has incorrect version headings"),
-        ("missing-entry.md", "has incorrect version headings"),
-        ("bad-title.md", "does not start with the correct header"),
-        ("extra-header.md", "has incorrect version headings"),
-    ],
-)
-def test_check_invalid_migration_guides(mocker, tmp_path, capsys, test_file, expected_stdout):
-    connector = qa_checks.Connector("source-foobar")
-    local_connector_documentation_directory = Path(tmp_path)
-    mocker.patch.object(qa_checks.Connector, "local_connector_documentation_directory", local_connector_documentation_directory)
-    mock_migration_file = local_connector_documentation_directory / f"{connector.name}-migrations.md"
-
-    mock_breaking_change_value = {
-        "upgradeDeadline": "2021-01-01",
-        "message": "This is a breaking change",
-    }
-
-    # transform metadata_breaking_changes into a dictionary
-    mock_breaking_change_dict = {version: mock_breaking_change_value for version in ["2.0.0", "1.0.0"]}
-
-    mock_metadata_dict = {"name": "Foobar", "documentationUrl": tmp_path, "releases": {"breakingChanges": mock_breaking_change_dict}}
-
-    test_file = Path("airbyte-ci/connectors/connector_ops/tests/test_migration_files") / test_file
-    with open(test_file, "r") as f:
-        contents = f.read()
-
-    with open(mock_migration_file, "w") as f:
-        f.write(contents)
-
-    mocker.patch.object(qa_checks.Connector, "metadata", mock_metadata_dict)
-
-    assert qa_checks.check_migration_guide(connector) is False
-    stdout, _ = capsys.readouterr()
-    assert expected_stdout in stdout
-
-
-def test_get_qa_checks_to_run(mocker):
-    mocker.patch.object(utils.Connector, "has_dockerfile", False)
-    connector = utils.Connector("source-faker")
-
-    assert (
-        qa_checks.get_qa_checks_to_run(connector) == qa_checks.DEFAULT_QA_CHECKS
-    ), "A connector without a Dockerfile should run the default set of QA checks"
-    mocker.patch.object(utils.Connector, "has_dockerfile", True)
-    connector = utils.Connector("source-faker")
-    assert qa_checks.get_qa_checks_to_run(connector) == qa_checks.DEFAULT_QA_CHECKS + (
-        qa_checks.check_metadata_version_matches_dockerfile_label,
-    ), "A connector with a Dockerfile should run the default set of QA checks plus check_metadata_version_matches_dockerfile_label"
-
-
-def test_check_metadata_version_matches_dockerfile_label_without_dockerfile(mocker):
-    mocker.patch.object(utils.Connector, "has_dockerfile", False)
-    connector_without_dockerfile = utils.Connector("source-faker")
-    assert (
-        qa_checks.check_metadata_version_matches_dockerfile_label(connector_without_dockerfile) is False
-    ), "A connector without a Dockerfile should fail check_metadata_version_matches_dockerfile_label"
diff --git a/docs/connector-development/testing-connectors/README.md b/docs/connector-development/testing-connectors/README.md
index e57d22904fde36..06377781fd210c 100644
--- a/docs/connector-development/testing-connectors/README.md
+++ b/docs/connector-development/testing-connectors/README.md
@@ -3,7 +3,7 @@
 Multiple tests suites compose the Airbyte connector testing pyramid
 
 ## Common to all connectors
-* [Connectors QA checks](https://github.com/airbytehq/airbyte/blob/master/airbyte-ci/connectors/connector_ops/connector_ops/qa_checks.py)
+* [Connectors QA checks](https://docs.airbyte.com/contributing-to-airbyte/resources/qa-checks)
 * [Connector Acceptance tests](https://docs.airbyte.com/connector-development/testing-connectors/connector-acceptance-tests-reference/)
 
 ## Connector specific tests