Dynatrace / dynatrace-configuration-as-code

This tool automates deployment of Dynatrace Configuration to one or multiple Dynatrace environments.
https://docs.dynatrace.com/docs/manage/configuration-as-code/monaco
Apache License 2.0
164 stars 94 forks source link

fix: Add e2e test that is deleting classic api resource with OAuth #1626

Closed tomazpu closed 1 week ago

tomazpu commented 2 weeks ago

…ly manifest

What this PR does / Why we need it:

This PR adds a regression test for the behavior of deleting a Classic API resource with a manifest where only OAuth is defined. The expected behavior would be that the deletion of Classic API resources is being skipped. We are currently running into a null pointer exception.

After the merge of https://github.com/Dynatrace/dynatrace-configuration-as-code/pull/1623 this problem should be resolved, e2e suit green.

Special notes for your reviewer:

Does this PR introduce a user-facing change?

github-actions[bot] commented 2 weeks ago

Unit Test Results

1 891 tests  ±0   1 890 ✅ ±0   53s ⏱️ ±0s   133 suites ±0       1 💤 ±0      1 files   ±0       0 ❌ ±0 

Results for commit 799b690a. ± Comparison against base commit b5dc2de3.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 2 weeks ago

E2E Test Results

    4 files  ±0    268 suites  ±0   22m 45s ⏱️ - 15m 7s 2 009 tests +3  2 007 ✅ +3  2 💤 ±0  0 ❌ ±0  2 124 runs  +3  2 122 ✅ +3  2 💤 ±0  0 ❌ ±0 

Results for commit 3578ce68. ± Comparison against base commit 27d7eeda.

:recycle: This comment has been updated with latest results.

sonarcloud[bot] commented 1 week ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud