fix(config-version):Updated examples config-version and added terraform+cloud for ConfigurationSource enum

Author: isivaselvan

examples/configuration_version.py

@@ -277,29 +277,8 @@ def main():
                     print(f"       - {filename} ({size} bytes)")
 
                 try:
-                    # Create tar.gz archive manually since go-slug isn't available
-                    print("Creating tar.gz archive manually...")
-
-                    import tarfile
-
-                    # Create tar.gz archive in memory
-                    archive_buffer = io.BytesIO()
-                    with tarfile.open(fileobj=archive_buffer, mode="w:gz") as tar:
-                        # Add all files from the temp directory
-                        for filename in files:
-                            filepath = os.path.join(temp_dir, filename)
-                            tar.add(filepath, arcname=filename)
-
-                    archive_buffer.seek(0)
-                    archive_bytes = archive_buffer.getvalue()
-                    print(f"Created archive: {len(archive_bytes)} bytes")
-
-                    # Use the SDK's upload_tar_gzip method instead of direct HTTP calls
-                    print("Uploading archive using SDK method...")
-                    archive_buffer.seek(0)  # Reset buffer position
-                    client.configuration_versions.upload_tar_gzip(
-                        new_cv.upload_url, archive_buffer
-                    )
+                    print("Uploading Terraform configuration...")
+                    client.configuration_versions.upload(new_cv.upload_url, temp_dir)
                     print("Terraform configuration uploaded successfully!")
 
                     # Wait and check status
@@ -408,8 +387,6 @@ def main():
     # =====================================================
     # TEST 4: UPLOAD CONFIGURATION VERSION
     # =====================================================
-    # Test 4: Upload function (requires go-slug)
-    # =====================================================
     print("\n4. Testing upload() function:")
     try:
         # Create a fresh configuration version specifically for upload testing
@@ -441,33 +418,19 @@ def main():
                 print(f"\n Uploading configuration to CV: {fresh_cv.id}")
                 print(f"Upload URL: {upload_url[:60]}...")
 
-                try:
-                    client.configuration_versions.upload(upload_url, temp_dir)
-                    print("Configuration uploaded successfully!")
-
-                    # Check status after upload
-                    print("\n Checking status after upload:")
-                    time.sleep(3)  # Give TFE time to process
-                    updated_cv = client.configuration_versions.read(fresh_cv.id)
-                    print(f"Status after upload: {updated_cv.status}")
+                client.configuration_versions.upload(upload_url, temp_dir)
+                print("Configuration uploaded successfully!")
 
-                    if updated_cv.status.value != "pending":
-                        print("Status changed (upload processed)")
-                    else:
-                        print("Status still pending (may need more time)")
+                # Check status after upload
+                print("\n Checking status after upload:")
+                time.sleep(3)  # Give TFE time to process
+                updated_cv = client.configuration_versions.read(fresh_cv.id)
+                print(f"Status after upload: {updated_cv.status}")
 
-                except ImportError as e:
-                    if "go-slug" in str(e):
-                        print("go-slug package not available")
-                        print("Install with: pip install go-slug")
-                        print(
-                            "Upload function exists but requires go-slug for packaging"
-                        )
-                        print(
-                            "Function correctly raises ImportError when go-slug unavailable"
-                        )
-                    else:
-                        raise
+                if updated_cv.status.value != "pending":
+                    print("Status changed (upload processed)")
+                else:
+                    print("Status still pending (may need more time)")
 
     except Exception as e:
         print(f"Error: {e}")
@@ -868,7 +831,7 @@ def main():
         "TEST 2:  create() - Create new configuration versions with different options"
     )
     print("TEST 3:  read() - Read configuration version details and validate fields")
-    print("TEST 4:  upload() - Upload Terraform configurations (requires go-slug)")
+    print("TEST 4:  upload() - Upload Terraform configurations (stdlib tarfile)")
     print("TEST 5:  download() - Download configuration version archives")
     print("TEST 6:  archive() - Archive configuration versions")
     print("TEST 7:  read_with_options() - Read with include options")

src/pytfe/models/configuration_version.py

@@ -25,6 +25,7 @@ class ConfigurationSource(str, Enum):
     GITLAB = "gitlab"
     ADO = "ado"
     TERRAFORM = "terraform"
+    TERRAFORM_CLOUD = "terraform+cloud"
 
 
 class ConfigVerIncludeOpt(str, Enum):

src/pytfe/utils.py

@@ -366,7 +366,9 @@ def pack_contents(path: str) -> io.BytesIO:
         ValueError: If path is invalid
     """
     if not path or not os.path.isdir(path):
-        raise ValueError(f"Failed to pack directory {path}: path must be an existing directory")
+        raise ValueError(
+            f"Failed to pack directory {path}: path must be an existing directory"
+        )
 
     body = io.BytesIO()
 
@@ -375,7 +377,9 @@ def pack_contents(path: str) -> io.BytesIO:
             rel_root = os.path.relpath(root, path)
             for filename in files:
                 full_path = os.path.join(root, filename)
-                arcname = filename if rel_root == "." else os.path.join(rel_root, filename)
+                arcname = (
+                    filename if rel_root == "." else os.path.join(rel_root, filename)
+                )
                 tar.add(full_path, arcname=arcname, recursive=False)
 
     body.seek(0)

tests/units/test_configuration_version.py

@@ -17,7 +17,7 @@
 """
 
 import io
-from unittest.mock import Mock, patch
+from unittest.mock import Mock
 
 import pytest
 
@@ -343,9 +343,7 @@ def test_read_with_options_no_ingress(
 class TestConfigurationVersionsUpload:
     """Test configuration versions upload functionality."""
 
-    def test_upload_packs_with_tar(
-        self, configuration_versions_service, tmp_path
-    ):
+    def test_upload_packs_with_tar(self, configuration_versions_service, tmp_path):
         """Test upload works by packing a directory to tar.gz with stdlib."""
         upload_url = "https://example.com/upload"
         directory_path = tmp_path