Address PR commments.

Signed-off-by: alyssacgoins <agoins@redhat.com>

Author: alyssacgoins

.github/resources/scripts/get_components_for_validation.sh

@@ -0,0 +1,30 @@
+name: scripts-unit-tests
+env:
+  PYTHON_VERSION: 3.11
+on:
+  pull_request:
+    paths:
+      - 'scripts/validate_metadata/**'
+jobs:
+  execute-scripts-unit-tests:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v5
+        with:
+          fetch-depth: 0
+
+      - name: Set up Python
+        uses: actions/setup-python@v6
+        with:
+          python-version: ${{ env.PYTHON_VERSION }}
+
+      - name: Install test dependencies
+        run: |
+          pip install -r scripts/validate_metadata/requirements.txt
+
+      - name: Install Test dependencies
+        run: |
+          pip install -r scripts/validate_metadata/requirements.txt
+
+      - name: Run tests
+        run: pytest scripts/validate_metadata

.github/workflows/scripts-unit-tests.yml

@@ -0,0 +1,49 @@
+name: validate-metadata-schema
+env:
+  PYTHON_VERSION: 3.11
+  RETRIEVAL_SCRIPT_PATH: $GITHUB_WORKSPACE/.github/resources/scripts/get_items_for_validation.sh
+  VALIDATION_SCRIPT_PATH: $GITHUB_WORKSPACE/scripts/validate_metadata/validate_metadata.py
+on:
+  pull_request:
+    paths:
+      - 'components/**'
+      - 'pipelines/**'
+      - 'scripts/**'
+      - 'third_party/components/**'
+      - 'third_party/pipelines/**'
+
+jobs:
+  validate-metadata-schema:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v5
+        with:
+          fetch-depth: 0
+
+      - name: Set up Python
+        uses: actions/setup-python@v6
+        with:
+          python-version: ${{ env.PYTHON_VERSION }}
+
+      - name: Install Test dependencies
+        run: |
+          pip install -r scripts/validate_metadata/requirements.txt
+
+      - name: Retrieve new components and/or pipelines
+        id: get-new-items
+        run: ${{  env.RETRIEVAL_SCRIPT_PATH  }} "${{ github.event.pull_request.base.sha }}" "${{ github.event.pull_request.head.sha }}" validate_metadata
+
+      #ToDo: Utilize .github/scripts/find-changed-components-and-pipelines.sh script once merged in from https://github.com/kubeflow/pipelines-components/pull/7
+      - name: Validate new core/third-party components and/or pipelines 
+        if: ${{ steps.get-new-items.outputs.new_items_list != '' }}
+        run: |
+          NEW_ITEMS_ARRAY="${{ steps.get-new-items.outputs.new_items_list }}"
+
+          # Set IFS to a comma, so that the shell will split the string by commas.
+          IFS=','
+          
+          for item in $NEW_ITEMS_ARRAY; do
+            FILE_PATH="$GITHUB_WORKSPACE/$item"
+            echo "Processing item: $item"
+            python "${{  env.VALIDATION_SCRIPT_PATH  }}" --item $FILE_PATH
+          done

.github/workflows/validate-component-metadata-schema.yml

@@ -1,2 +1,3 @@
 PyYAML==6.0.3
 semver==3.0.4
+pytest==7.1.2

.github/workflows/validate-metadata-schema.yml

@@ -1,5 +1,5 @@
 name: component_valid
-tier: third-party
+tier: third_party
 stability: alpha
 dependencies:
   kubeflow:

scripts/__init__.py

@@ -57,6 +57,7 @@ def parse_args() -> argparse.Namespace:
     parser.add_argument(
         '--component',
         type=validate_component_directory,
+        required=True,
         help='Path to the component directory (must contain component.py and metadata.yaml)'
     )
 
@@ -290,25 +291,22 @@ def get_invalid_versions(dependencies: list[dict]) -> list[dict]:
 def main():
     """Main entry point for the CLI."""
     args = parse_args()
-    if not args.component:
-        raise ValidationError("Error: --component must be specified.")
-
     component_dir = args.component
 
     # Validate OWNERS
     try:
         owners_file_path = component_dir / OWNERS
         validate_owners_file(owners_file_path)
     except ValidationError as e:
-        logging.error("Validation Error: {}", e)
+        logging.error("Validation Error: %s", e)
         sys.exit(1)
 
     # Validate metadata.yaml
     try:
         metadata_file_path = component_dir / METADATA
         validate_metadata_yaml(metadata_file_path)
     except ValidationError as e:
-        logging.error("Validation Error: {}", e)
+        logging.error("Validation Error: %s", e)
         sys.exit(1)
 
     # Validation successful.