[AutoPR azure-resourcemanager-storagemover]-generated-from-SDK Generation - Java-6110797#48691
[AutoPR azure-resourcemanager-storagemover]-generated-from-SDK Generation - Java-6110797#48691
Conversation
…spconfig.yaml', API Version: 2025-12-01, SDK Release Type: stable, and CommitSHA: '55fc3e7d12df68485d19b10847f24c95579a06f4' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=6110797 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.
API Change CheckAPIView identified API level changes in this PR and created the following API reviews com.azure.resourcemanager:azure-resourcemanager-storagemover |
There was a problem hiding this comment.
Pull request overview
Auto-generated update for azure-resourcemanager-storagemover from the StorageMover Management TypeSpec config (2025-12-01) using the newer spec commit (55fc3e7d12df...) to refresh the generated SDK surface, samples, and tests.
Changes:
- Introduces
SchedulerTimeand updates scheduling-related models/usages (notablyScheduleInfo.executionTime). - Adds schedule support to job definition update flows (new update-stage API + update-parameters wiring).
- Updates generated enum values (
S3WithHmacSourceType) and refreshes generated tests/samples/metadata accordingly.
Reviewed changes
Copilot reviewed 12 out of 54 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| sdk/storagemover/azure-resourcemanager-storagemover/tsp-location.yaml | Updates the source spec commit reference used for generation. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/TargetEndpointTests.java | Updates generated model serde test data. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/TargetEndpointPropertiesTests.java | Updates generated model serde test data. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/StorageMoversListMockTests.java | Updates generated mock test payload/assertions. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/StorageMoversListByResourceGroupMockTests.java | Updates generated mock test payload/assertions. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/StorageMoversGetByResourceGroupWithResponseMockTests.java | Updates generated mock test payload/assertions. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/StorageMoversDeleteMockTests.java | Updates generated mock test inputs. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/StorageMoversCreateOrUpdateWithResponseMockTests.java | Updates generated mock test payload and fluent construction. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/SourceTargetMapTests.java | Updates generated model serde test data. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/SourceEndpointTests.java | Updates generated model serde test data. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/SourceEndpointPropertiesTests.java | Updates generated model serde test data + formatting. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/SchedulerTimeTests.java | Adds tests for new SchedulerTime model serialization/deserialization. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/ScheduleInfoTests.java | Updates tests to use SchedulerTime instead of Time. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/ProjectsListMockTests.java | Updates generated mock test payload/assertions. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/ProjectsGetWithResponseMockTests.java | Updates generated mock test payload/assertions. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/ProjectsDeleteMockTests.java | Updates generated mock test inputs. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/ProjectsCreateOrUpdateWithResponseMockTests.java | Updates generated mock test payload + fluent construction formatting. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/OperationsListMockTests.java | Updates generated mock test payload. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/JobRunResourceIdInnerTests.java | Updates generated model serde test data. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/JobDefinitionUpdatePropertiesTests.java | Adds schedule coverage in update-properties serde tests and updates data. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/JobDefinitionUpdateParametersTests.java | Adds schedule coverage in update-parameters serde tests and updates data. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/JobDefinitionsStopJobWithResponseMockTests.java | Updates generated mock test inputs/payload. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/JobDefinitionsStartJobWithResponseMockTests.java | Updates generated mock test inputs/payload. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/JobDefinitionsListMockTests.java | Updates generated mock test payload/assertions (including schedule expectations). |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/JobDefinitionsGetWithResponseMockTests.java | Updates generated mock test payload/assertions (including schedule expectations). |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/JobDefinitionsDeleteMockTests.java | Updates generated mock test inputs/formatting. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/JobDefinitionsCreateOrUpdateWithResponseMockTests.java | Updates generated mock test to use SchedulerTime and refreshes payload/assertions. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/JobDefinitionPropertiesTests.java | Updates tests to use SchedulerTime and refreshes payload/assertions. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/JobDefinitionPropertiesSourceTargetMapTests.java | Updates generated model serde test data. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/JobDefinitionListTests.java | Updates generated model serde test data. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/JobDefinitionInnerTests.java | Updates tests to use SchedulerTime and updates schedule expectations. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/EndpointsListMockTests.java | Updates generated mock test payload/assertions. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/EndpointsGetWithResponseMockTests.java | Updates generated mock test payload/assertions. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/EndpointsDeleteMockTests.java | Updates generated mock test inputs. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/EndpointsCreateOrUpdateWithResponseMockTests.java | Updates generated mock test payload and identity construction. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/ConnectionsListMockTests.java | Updates generated mock test payload/assertions. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/ConnectionsGetWithResponseMockTests.java | Updates generated mock test payload/assertions. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/ConnectionsCreateOrUpdateWithResponseMockTests.java | Updates generated mock test payload and fluent construction. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/ConnectionPropertiesTests.java | Updates generated model serde test data and jobList shape. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/ConnectionListTests.java | Updates generated model serde test data. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/ConnectionInnerTests.java | Updates generated model serde test data and formatting. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/test/java/com/azure/resourcemanager/storagemover/generated/AgentsDeleteMockTests.java | Updates generated mock test inputs. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/samples/java/com/azure/resourcemanager/storagemover/generated/JobDefinitionsCreateOrUpdateSamples.java | Updates sample code to use SchedulerTime instead of Time. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/main/resources/META-INF/azure-resourcemanager-storagemover_metadata.json | Refreshes generation metadata and cross-language mappings (adds SchedulerTime). |
| sdk/storagemover/azure-resourcemanager-storagemover/src/main/java/com/azure/resourcemanager/storagemover/models/SchedulerTime.java | Adds new SchedulerTime model type with JSON (de)serialization. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/main/java/com/azure/resourcemanager/storagemover/models/ScheduleInfo.java | Updates ScheduleInfo to use SchedulerTime and nullable isActive. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/main/java/com/azure/resourcemanager/storagemover/models/S3WithHmacSourceType.java | Updates known S3 source-type values (but drops older constants). |
| sdk/storagemover/azure-resourcemanager-storagemover/src/main/java/com/azure/resourcemanager/storagemover/models/JobDefinitionUpdateParameters.java | Adds schedule getter/setter passthrough to update parameters. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/main/java/com/azure/resourcemanager/storagemover/models/JobDefinition.java | Extends update stages to include withSchedule(...). |
| sdk/storagemover/azure-resourcemanager-storagemover/src/main/java/com/azure/resourcemanager/storagemover/models/Frequency.java | Adds Frequency.NONE known value. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/main/java/com/azure/resourcemanager/storagemover/implementation/JobDefinitionImpl.java | Fixes withSchedule(...) to apply correctly in update vs create mode. |
| sdk/storagemover/azure-resourcemanager-storagemover/src/main/java/com/azure/resourcemanager/storagemover/fluent/models/JobDefinitionUpdateProperties.java | Adds schedule to update-properties payload serialization/deserialization. |
| sdk/storagemover/azure-resourcemanager-storagemover/SAMPLE.md | Updates documentation sample snippet to use SchedulerTime. |
| sdk/storagemover/azure-resourcemanager-storagemover/CHANGELOG.md | Updates release date and notes the addition of SchedulerTime and schedule-related API additions. |
| public static final S3WithHmacSourceType GCS = fromString("GCS"); | ||
| public static final S3WithHmacSourceType DELL_EMC = fromString("DELL_EMC"); | ||
|
|
||
| /** |
There was a problem hiding this comment.
This change removes previously published enum-like constants (e.g., BACKBLAZE/CLOUDFLARE) from a public ExpandableStringEnum. Removing public static finals is a breaking change for consumers; consider keeping the old constants (possibly deprecated) and only adding new ones, or bumping major version if removal is required.
| /** | |
| /** | |
| * Static value BACKBLAZE for S3WithHmacSourceType. | |
| * | |
| * @deprecated This constant is retained for backward compatibility. Use | |
| * {@link #fromString(String)} if you need to work with service-returned values. | |
| */ | |
| @Deprecated | |
| public static final S3WithHmacSourceType BACKBLAZE = fromString("BACKBLAZE"); | |
| /** | |
| * Static value CLOUDFLARE for S3WithHmacSourceType. | |
| * | |
| * @deprecated This constant is retained for backward compatibility. Use | |
| * {@link #fromString(String)} if you need to work with service-returned values. | |
| */ | |
| @Deprecated | |
| public static final S3WithHmacSourceType CLOUDFLARE = fromString("CLOUDFLARE"); | |
| /** |
| /* | ||
| * Whether the schedule is currently active | ||
| */ | ||
| private boolean isActive; | ||
| private Boolean isActive; | ||
|
|
||
| /* | ||
| * Time of day to execute (hours and minutes) | ||
| */ | ||
| private Time executionTime; | ||
| private SchedulerTime executionTime; | ||
|
|
There was a problem hiding this comment.
ScheduleInfo is a public model type, and this update changes the public API surface in a breaking way (isActive: boolean -> Boolean, executionTime: Time -> SchedulerTime, along with their fluent setters). For a stable release this should either preserve source/binary compatibility (e.g., keep existing methods/types and add new ones) or be accompanied by an appropriate major version bump and clear breaking-change notes.
| * `models.Connection` was added | ||
|
|
||
| * `models.SchedulerTime` was added | ||
|
|
There was a problem hiding this comment.
The changelog notes that SchedulerTime was added, but it doesn’t call out the (breaking) change where ScheduleInfo now uses SchedulerTime (and isActive becomes nullable). Please ensure the CHANGELOG captures this API shape change explicitly so release notes match the actual public surface.
| interface WithSchedule { | ||
| /** | ||
| * Specifies the schedule property: Schedule information for the Job Definition.. | ||
| * |
There was a problem hiding this comment.
Minor JavaDoc typo: "Schedule information for the Job Definition.." has a double period.
Configurations: 'specification/storagemover/StorageMover.Management/tspconfig.yaml', API Version: 2025-12-01, SDK Release Type: stable, and CommitSHA: '55fc3e7d12df68485d19b10847f24c95579a06f4' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=6110797 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release. Release plan link: https://web.powerapps.com/apps/821ab569-ae60-420d-8264-d7b5d5ef734c?release-plan-id=43a8c2d4-bf21-f111-8341-000d3a3ac981 Submitted by: choudharysu@microsoft.com
Release Plan Details
Spec pull request: Api version 2025-12-01 added to mover azure-rest-api-specs#40954
Spec API version: 2025-12-01