Skip to content

Linodes and OBJ Integration Tests Improvements #434

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
lgarber-akamai merged 3 commits into from
Jul 10, 2024
Merged

Linodes and OBJ Integration Tests Improvements #434

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
1a306c0
Improve and deduplicate object storage tests
zliang-akamai Jul 10, 2024
bdb0862
Improve linode save test
zliang-akamai Jul 10, 2024
64e7b53
type annotations for clients
zliang-akamai Jul 10, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions test/integration/models/linode/test_linode.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -542,7 +542,7 @@ def test_get_linode_types_overrides(test_linode_client):
def test_save_linode_noforce(test_linode_client, create_linode):
linode = create_linode
old_label = linode.label
linode.label = "updated_no_force_label"
linode.label = old_label + "updated_no_force"
linode.save(force=False)

linode = test_linode_client.load(Instance, linode.id)
Expand All @@ -553,8 +553,8 @@ def test_save_linode_noforce(test_linode_client, create_linode):
def test_save_linode_force(test_linode_client, create_linode):
linode = create_linode
old_label = linode.label
linode.label = "updated_force_label"
linode.save(force=False)
linode.label = old_label + "updated_force"
linode.save(force=True)

linode = test_linode_client.load(Instance, linode.id)

Expand Down
39 changes: 39 additions & 0 deletions test/integration/models/object_storage/test_obj.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
from linode_api4.objects.object_storage import (
ObjectStorageACL,
ObjectStorageBucket,
ObjectStorageCluster,
ObjectStorageKeyPermission,
ObjectStorageKeys,
)
Expand Down Expand Up @@ -90,3 +91,41 @@ def test_bucket(
buckets = test_linode_client.object_storage.buckets_in_region(region=region)
assert len(buckets) >= 1
assert any(b.label == bucket.label for b in buckets)


def test_list_obj_storage_bucket(
test_linode_client: LinodeClient,
bucket: ObjectStorageBucket,
):
buckets = test_linode_client.object_storage.buckets()
target_bucket_id = bucket.id
assert any(target_bucket_id == b.id for b in buckets)


def test_bucket_access_modify(bucket: ObjectStorageBucket):
bucket.access_modify(ObjectStorageACL.PRIVATE, cors_enabled=True)


def test_bucket_access_update(bucket: ObjectStorageBucket):
bucket.access_update(ObjectStorageACL.PRIVATE, cors_enabled=True)


def test_get_ssl_cert(bucket: ObjectStorageBucket):
assert not bucket.ssl_cert().ssl


def test_get_cluster(
test_linode_client: LinodeClient, bucket: ObjectStorageBucket
):
cluster = test_linode_client.load(ObjectStorageCluster, bucket.cluster)

assert "linodeobjects.com" in cluster.domain
assert cluster.id == bucket.cluster
assert "available" == cluster.status


def test_get_buckets_in_cluster(
test_linode_client: LinodeClient, bucket: ObjectStorageBucket
):
cluster = test_linode_client.load(ObjectStorageCluster, bucket.cluster)
assert any(bucket.id == b.id for b in cluster.buckets_in_cluster())
105 changes: 0 additions & 105 deletions test/integration/models/objectstorage/test_obj_storage.py
View file
Open in desktop

This file was deleted.