Commit f3b6d02
authored
mgmt monitor , fix live test failures (#47427)
* test(monitor): update test configurations and add VM size specifications
- Updated region from US_WEST to US_WEST2 in multiple test files
- Added disableSharedKeyAccess() to various resource creations
- Specified VirtualMachineSizeTypes.STANDARD_A1_V2 for VM instances
- Imported VirtualMachineSizeTypes in relevant test classes
* test(monitor): update autoscale tests to reflect email notification changes
- Removed admin and co-admin email notifications from autoscale settings
- Updated assertions to check for disabled email notifications
- Adjusted test cases to align with new autoscale profile configurations
- Verified autoscale enabled/disabled states in multiple scenarios
- Ensured custom email notifications remain functional after updates
- Cleaned up redundant autoscale profile removal logic
* test(monitor): update tests for Azure Monitor management changes
- Add ZoneOffset import for UTC time handling in AlertsTests
- Remove deprecated admin/co-admin email notification assertions in AutoscaleTests
- Comment out unsupported EventHub namespace creation in DiagnosticSettingsTests
- Update subscription ID format with leading slash in DiagnosticSettingsTests
- Set retention days to 0 for diagnostic settings due to policy changes
- Disable test requiring Microsoft Entra-only authentication for Azure SQL Server
* fix(monitor): update diagnostic settings test with correct retention days
- Change retention days from 7 to 0 in logs and metrics configuration
- Comment out invalid update operation that removes all data sinks
- Remove assertion for storage account ID null check
- Update test logic to align with diagnostic settings requirements
* test(monitor): disable flaky test due to Azure SQL Server authentication requirement
- Added @disabled annotation to canListEventsAndMetricsWithWhiteSpaceInResourceId test
- Included explanatory message about Microsoft Entra-only authentication requirement
- Preserved existing @DoNotRecord annotation behavior
- Test will be re-enabled once Azure SQL Server supports proper authentication during creation
* fix(monitor): prevent deletion error when resource group not found
- Add check to verify resource group existence before deletion
- Prevent IllegalArgumentException when resource group does not exist
- Improve cleanup method reliability in ActionGroupsTests
* chore(assets): update tag reference for azure-resourcemanager-monitor
- Updated tag from fb1a47b010 to 28c5828c2d
- Maintains consistency with latest asset changes
* fix(tests): improve resource cleanup and add documentation link
- Update resource group deletion logic to check existence before attempting removal
- Add Microsoft documentation link for diagnostic settings storage lifecycle policy
- Prevent potential errors during test cleanup phase
- Clarify comments for better maintainability
* test(monitor): remove unused event hubs imports in diagnostic settings tests
- Removed unused EventHubNamespace import
- Removed unused EventHubNamespaceAuthorizationRule import
- Clean up import statements for better code readability
* test(monitor): update autoscale tests to remove co-admin email notification assertions
- Removed assertions for co-admin email notification being disabled
- Commented out failing assertion lines in autoscale test cases
- Maintained existing test structure and other assertion logic
* fix(tests): update diagnostic settings test to assert storage account ID
- Uncomment and update assertion to verify storage account ID matches expected value
- Ensure storage account ID is correctly set in diagnostic settings configuration
- Maintain existing assertions for other diagnostic settings properties
* test(monitor): update region to US_WEST3 in diagnostic settings tests
- Changed region from US_WEST2 to US_WEST3 in DiagnosticSettingsTests
- Updated region configuration in MonitorActivityAndMetricsTests
- Ensured consistency across monitor-related test cases1 parent 10051b5 commit f3b6d02
File tree
7 files changed
+66
-38
lines changed- sdk/monitor/azure-resourcemanager-monitor
- src/test/java/com/azure/resourcemanager/monitor
7 files changed
+66
-38
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
6 | 6 | | |
Lines changed: 4 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
28 | | - | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
29 | 32 | | |
30 | 33 | | |
31 | 34 | | |
| |||
Lines changed: 8 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
| 12 | + | |
12 | 13 | | |
13 | 14 | | |
14 | 15 | | |
| |||
27 | 28 | | |
28 | 29 | | |
29 | 30 | | |
| 31 | + | |
30 | 32 | | |
31 | 33 | | |
32 | 34 | | |
| |||
56 | 58 | | |
57 | 59 | | |
58 | 60 | | |
| 61 | + | |
59 | 62 | | |
60 | 63 | | |
61 | 64 | | |
| |||
338 | 341 | | |
339 | 342 | | |
340 | 343 | | |
| 344 | + | |
341 | 345 | | |
342 | 346 | | |
343 | 347 | | |
| |||
350 | 354 | | |
351 | 355 | | |
352 | 356 | | |
| 357 | + | |
353 | 358 | | |
354 | 359 | | |
355 | 360 | | |
| |||
378 | 383 | | |
379 | 384 | | |
380 | 385 | | |
381 | | - | |
| 386 | + | |
| 387 | + | |
| 388 | + | |
382 | 389 | | |
383 | 390 | | |
384 | 391 | | |
| |||
Lines changed: 13 additions & 11 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
95 | 95 | | |
96 | 96 | | |
97 | 97 | | |
98 | | - | |
99 | | - | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
100 | 101 | | |
101 | 102 | | |
102 | 103 | | |
103 | 104 | | |
104 | 105 | | |
105 | 106 | | |
106 | 107 | | |
107 | | - | |
108 | | - | |
| 108 | + | |
| 109 | + | |
109 | 110 | | |
110 | 111 | | |
111 | 112 | | |
| |||
206 | 207 | | |
207 | 208 | | |
208 | 209 | | |
209 | | - | |
210 | | - | |
| 210 | + | |
| 211 | + | |
211 | 212 | | |
212 | 213 | | |
213 | 214 | | |
| |||
337 | 338 | | |
338 | 339 | | |
339 | 340 | | |
340 | | - | |
| 341 | + | |
| 342 | + | |
341 | 343 | | |
342 | 344 | | |
343 | 345 | | |
344 | 346 | | |
345 | 347 | | |
346 | | - | |
347 | | - | |
| 348 | + | |
| 349 | + | |
348 | 350 | | |
349 | 351 | | |
350 | 352 | | |
| |||
454 | 456 | | |
455 | 457 | | |
456 | 458 | | |
457 | | - | |
458 | | - | |
| 459 | + | |
| 460 | + | |
459 | 461 | | |
460 | 462 | | |
461 | 463 | | |
| |||
Lines changed: 35 additions & 23 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
11 | | - | |
12 | | - | |
13 | 11 | | |
14 | 12 | | |
15 | 13 | | |
| |||
23 | 21 | | |
24 | 22 | | |
25 | 23 | | |
| 24 | + | |
26 | 25 | | |
27 | 26 | | |
28 | 27 | | |
| |||
55 | 54 | | |
56 | 55 | | |
57 | 56 | | |
58 | | - | |
| 57 | + | |
59 | 58 | | |
60 | 59 | | |
61 | 60 | | |
| |||
73 | 72 | | |
74 | 73 | | |
75 | 74 | | |
| 75 | + | |
76 | 76 | | |
77 | 77 | | |
78 | | - | |
79 | | - | |
80 | | - | |
81 | | - | |
82 | | - | |
83 | | - | |
84 | | - | |
85 | | - | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
86 | 88 | | |
87 | | - | |
| 89 | + | |
88 | 90 | | |
89 | 91 | | |
90 | 92 | | |
| |||
96 | 98 | | |
97 | 99 | | |
98 | 100 | | |
99 | | - | |
100 | | - | |
| 101 | + | |
| 102 | + | |
101 | 103 | | |
102 | 104 | | |
103 | 105 | | |
104 | 106 | | |
105 | | - | |
| 107 | + | |
106 | 108 | | |
107 | 109 | | |
108 | 110 | | |
109 | 111 | | |
110 | 112 | | |
111 | | - | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
112 | 116 | | |
113 | 117 | | |
114 | | - | |
115 | | - | |
| 118 | + | |
| 119 | + | |
116 | 120 | | |
117 | 121 | | |
118 | 122 | | |
| |||
145 | 149 | | |
146 | 150 | | |
147 | 151 | | |
| 152 | + | |
148 | 153 | | |
149 | 154 | | |
150 | | - | |
| 155 | + | |
151 | 156 | | |
152 | 157 | | |
153 | 158 | | |
| |||
205 | 210 | | |
206 | 211 | | |
207 | 212 | | |
| 213 | + | |
208 | 214 | | |
209 | 215 | | |
210 | 216 | | |
| |||
226 | 232 | | |
227 | 233 | | |
228 | 234 | | |
229 | | - | |
| 235 | + | |
| 236 | + | |
| 237 | + | |
230 | 238 | | |
231 | 239 | | |
232 | 240 | | |
| |||
273 | 281 | | |
274 | 282 | | |
275 | 283 | | |
276 | | - | |
| 284 | + | |
277 | 285 | | |
278 | 286 | | |
279 | 287 | | |
| |||
316 | 324 | | |
317 | 325 | | |
318 | 326 | | |
| 327 | + | |
319 | 328 | | |
320 | 329 | | |
321 | 330 | | |
322 | 331 | | |
323 | 332 | | |
324 | 333 | | |
325 | 334 | | |
326 | | - | |
| 335 | + | |
| 336 | + | |
327 | 337 | | |
328 | 338 | | |
329 | 339 | | |
| |||
342 | 352 | | |
343 | 353 | | |
344 | 354 | | |
345 | | - | |
| 355 | + | |
346 | 356 | | |
347 | 357 | | |
348 | 358 | | |
| |||
352 | 362 | | |
353 | 363 | | |
354 | 364 | | |
| 365 | + | |
355 | 366 | | |
356 | 367 | | |
357 | 368 | | |
| |||
360 | 371 | | |
361 | 372 | | |
362 | 373 | | |
| 374 | + | |
363 | 375 | | |
364 | 376 | | |
365 | 377 | | |
| |||
Lines changed: 3 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
| 24 | + | |
24 | 25 | | |
25 | 26 | | |
26 | 27 | | |
| |||
46 | 47 | | |
47 | 48 | | |
48 | 49 | | |
49 | | - | |
| 50 | + | |
50 | 51 | | |
51 | 52 | | |
52 | 53 | | |
| |||
130 | 131 | | |
131 | 132 | | |
132 | 133 | | |
| 134 | + | |
133 | 135 | | |
134 | 136 | | |
135 | 137 | | |
| |||
Lines changed: 2 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
| 18 | + | |
18 | 19 | | |
19 | 20 | | |
20 | 21 | | |
| |||
82 | 83 | | |
83 | 84 | | |
84 | 85 | | |
| 86 | + | |
85 | 87 | | |
86 | 88 | | |
87 | 89 | | |
| |||
0 commit comments