Skip to content

[AMBARI-22995] Remove unit tests reintroduced by merge commit#446

Merged
adoroszlai merged 1 commit into
apache:branch-feature-AMBARI-14714from
adoroszlai:branch-feature-AMBARI-14714
Feb 23, 2018
Merged

[AMBARI-22995] Remove unit tests reintroduced by merge commit#446
adoroszlai merged 1 commit into
apache:branch-feature-AMBARI-14714from
adoroszlai:branch-feature-AMBARI-14714

Conversation

@adoroszlai
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

90aca63 removed some unit tests related to deprecated upgrade packs.

133febc renamed NON_ROLLING to EXPRESS.

6912c65 reintroduced some of the removed unit tests, causing compile errors due to still referencing NON_ROLLING.

The goal of this change is to remove these unit tests again.

How was this patch tested?

$ mvn -am -pl ambari-server -DskipTests -Drat.skip clean test
...
55352 [INFO] Reactor Summary:
55352 [INFO]
55353 [INFO] Ambari Main ........................................ SUCCESS [  1.448 s]
55353 [INFO] Apache Ambari Project POM .......................... SUCCESS [  0.005 s]
55353 [INFO] Ambari Views ....................................... SUCCESS [  1.034 s]
55353 [INFO] ambari-utility ..................................... SUCCESS [  1.257 s]
55353 [INFO] ambari-metrics ..................................... SUCCESS [  0.301 s]
55353 [INFO] Ambari Metrics Common .............................. SUCCESS [  0.809 s]
55353 [INFO] Ambari Service Advisor ............................. SUCCESS [  0.621 s]
55353 [INFO] Ambari Server ...................................... SUCCESS [ 48.619 s]

@adoroszlai adoroszlai merged commit 396e488 into apache:branch-feature-AMBARI-14714 Feb 23, 2018
@asfgit
Copy link
Copy Markdown

asfgit commented Feb 23, 2018

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/765/
Test FAILed.
Test FAILured.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants