Source Zoom: Fix authenticator bug and add missing fields

airbyte-integrations/connectors/source-zoom/README.md

@@ -17,19 +17,70 @@ and place them into `secrets/config.json`.
 ### Locally running the connector docker image
 
 
-#### Build
-**Via [`airbyte-ci`](https://github.com/airbytehq/airbyte/blob/master/airbyte-ci/connectors/pipelines/README.md) (recommended):**
+
+
+#### Use `airbyte-ci` to build your connector
+The Airbyte way of building this connector is to use our `airbyte-ci` tool.
+You can follow install instructions [here](https://github.com/airbytehq/airbyte/blob/master/airbyte-ci/connectors/pipelines/README.md#L1).
+Then running the following command will build your connector:
+
 ```bash
-airbyte-ci connectors --name=source-zoom build
+airbyte-ci connectors --name source-zoom build
+```
+Once the command is done, you will find your connector image in your local docker registry: `airbyte/source-zoom:dev`.
+
+##### Customizing our build process
+When contributing on our connector you might need to customize the build process to add a system dependency or set an env var.
+You can customize our build process by adding a `build_customization.py` module to your connector.
+This module should contain a `pre_connector_install` and `post_connector_install` async function that will mutate the base image and the connector container respectively.
+It will be imported at runtime by our build process and the functions will be called if they exist.
+
+Here is an example of a `build_customization.py` module:
+```python
+from __future__ import annotations
+
+from typing import TYPE_CHECKING
+
+if TYPE_CHECKING:
+    # Feel free to check the dagger documentation for more information on the Container object and its methods.
+    # https://dagger-io.readthedocs.io/en/sdk-python-v0.6.4/
+    from dagger import Container
+
+
+async def pre_connector_install(base_image_container: Container) -> Container:
+    return await base_image_container.with_env_variable("MY_PRE_BUILD_ENV_VAR", "my_pre_build_env_var_value")
+
+async def post_connector_install(connector_container: Container) -> Container:
+    return await connector_container.with_env_variable("MY_POST_BUILD_ENV_VAR", "my_post_build_env_var_value")
 ```
 
-An image will be built with the tag `airbyte/source-zoom:dev`.
+#### Build your own connector image
+This connector is built using our dynamic built process in `airbyte-ci`.
+The base image used to build it is defined within the metadata.yaml file under the `connectorBuildOptions`.
+The build logic is defined using [Dagger](https://dagger.io/) [here](https://github.com/airbytehq/airbyte/blob/master/airbyte-ci/connectors/pipelines/pipelines/builds/python_connectors.py).
+It does not rely on a Dockerfile.
+
+If you would like to patch our connector and build your own a simple approach would be to:
+
+1. Create your own Dockerfile based on the latest version of the connector image.
+```Dockerfile
+FROM airbyte/source-zoom:latest
+
+COPY . ./airbyte/integration_code
+RUN pip install ./airbyte/integration_code
 
-**Via `docker build`:**
+# The entrypoint and default env vars are already set in the base image
+# ENV AIRBYTE_ENTRYPOINT "python /airbyte/integration_code/main.py"
+# ENTRYPOINT ["python", "/airbyte/integration_code/main.py"]
+```
+Please use this as an example. This is not optimized.
+
+2. Build your image:
 ```bash
 docker build -t airbyte/source-zoom:dev .
+# Running the spec command against your patched connector
+docker run airbyte/source-zoom:dev spec
 ```
-
 #### Run
 Then run any of the connector commands as follows:
 ```
@@ -64,4 +115,3 @@ You've checked out the repo, implemented a million dollar feature, and you're re
 5. Create a Pull Request: use [our PR naming conventions](https://docs.airbyte.com/contributing-to-airbyte/resources/pull-requests-handbook/#pull-request-title-convention).
 6. Pat yourself on the back for being an awesome contributor.
 7. Someone from Airbyte will take a look at your PR and iterate with you to merge it into master.
-

airbyte-integrations/connectors/source-zoom/acceptance-test-config.yml

@@ -1,44 +1,55 @@
 # See [Connector Acceptance Tests](https://docs.airbyte.io/connector-development/testing-connectors/connector-acceptance-tests-reference)
 # for more information about how to configure these tests
 connector_image: airbyte/source-zoom:dev
-tests:
+acceptance_tests:
   spec:
-    - spec_path: "source_zoom/spec.yaml"
+    tests:
+      - spec_path: "source_zoom/spec.yaml"
   connection:
-    - config_path: "secrets/config.json"
-      status: "succeed"
-    - config_path: "integration_tests/invalid_config.json"
-      status: "failed"
+    tests:
+      - config_path: "secrets/config.json"
+        status: "succeed"
+      - config_path: "integration_tests/invalid_config.json"
+        status: "failed"
   discovery:
-    - config_path: "secrets/config.json"
+    tests:
+      - config_path: "secrets/config.json"
   basic_read:
-    - config_path: "secrets/config.json"
-      configured_catalog_path: "integration_tests/configured_catalog.json"
-      timeout_seconds: 3600
-      empty_streams:
-        - "meeting_registrants"
-        - "meeting_polls"
-        - "meeting_poll_results"
-        - "meeting_registration_questions"
-        - "webinars"
-        - "webinar_panelists"
-        - "webinar_registrants"
-        - "webinar_absentees"
-        - "webinar_polls"
-        - "webinar_poll_results"
-        - "webinar_registration_questions"
-        - "webinar_tracking_sources"
-        - "webinar_qna_results"
-        - "report_meetings"
-        - "report_meeting_participants"
-        - "report_webinars"
-        - "report_webinar_participants"
+    tests:
+      - config_path: "secrets/config.json"
+        expect_records:
+          path: "integration_tests/expected_records.jsonl"
+        configured_catalog_path: "integration_tests/configured_catalog.json"
+        timeout_seconds: 3600
+        empty_streams:
+          - name: "webinars"
+            bypass_reason: "Need paid Webinar license to populate data"
+          - name: "webinar_panelists"
+            bypass_reason: "Need paid Webinar license to populate data"
+          - name: "webinar_registrants"
+            bypass_reason: "Need paid Webinar license to populate data"
+          - name: "webinar_absentees"
+            bypass_reason: "Need paid Webinar license to populate data"
+          - name: "webinar_polls"
+            bypass_reason: "Need paid Webinar license to populate data"
+          - name: "webinar_poll_results"
+            bypass_reason: "Need paid Webinar license to populate data"
+          - name: "webinar_registration_questions"
+            bypass_reason: "Need paid Webinar license to populate data"
+          - name: "webinar_tracking_sources"
+            bypass_reason: "Need paid Webinar license to populate data"
+          - name: "webinar_qna_results"
+            bypass_reason: "Need paid Webinar license to populate data"
+          - name: "report_webinars"
+            bypass_reason: "Need paid Webinar license to populate data"
+          - name: "report_webinar_participants"
+            bypass_reason: "Need paid Webinar license to populate data"
   full_refresh:
-    - config_path: "secrets/config.json"
-      configured_catalog_path: "integration_tests/configured_catalog.json"
-      ignored_fields:
-        "meetings":
-          - "start_url"
-        "webinars":
-          - "start_url"
-      timeout_seconds: 3600
+    tests:
+      - config_path: "secrets/config.json"
+        configured_catalog_path: "integration_tests/configured_catalog.json"
+        ignored_fields:
+          meetings:
+            - name: "start_url"
+              bypass_reason: "Causes sequential_read test to fail as the value is unique upon each read"
+        timeout_seconds: 3600