Skip to content
This repository was archived by the owner on May 12, 2021. It is now read-only.

[MINOR] Fix unit test cases caused by forcefully using fasterxml-jackson#776

Closed
haoch wants to merge 8 commits intoapache:masterfrom
haoch:FixUT
Closed

[MINOR] Fix unit test cases caused by forcefully using fasterxml-jackson#776
haoch wants to merge 8 commits intoapache:masterfrom
haoch:FixUT

Conversation

@haoch
Copy link
Member

@haoch haoch commented Jan 13, 2017

  • Refactor TopologyCheckAppConfig and TopologyRackResolver
  • Fix unit test cases after using fasterxml-jackson:
    • TestGenericEntityServiceResource.java
    • TestResourceUnmarshal.java
  • Try to get eagle service port from config by key: service.port no matter STRING or INT.

@haoch haoch changed the title [MINOR] Fix unit test cases [MINOR] Fix unit test cases caused by forcefully using fasterxml-jackson Jan 13, 2017
@qingwen220
Copy link
Contributor

The changes look good to me.

@asfgit asfgit closed this in 2160673 Jan 14, 2017
koone pushed a commit to 1haodian/eagle that referenced this pull request Jan 17, 2017
Fix unit test cases after using `fasterxml-jackson`:

* `TestGenericEntityServiceResource.java`
* `TestResourceUnmarshal.java`

Author: Hao Chen <hao@apache.org>

Closes apache#776 from haoch/FixUT.
jhsenjaliya pushed a commit to jhsenjaliya/eagle that referenced this pull request Jul 12, 2017
Fix unit test cases after using `fasterxml-jackson`:

* `TestGenericEntityServiceResource.java`
* `TestResourceUnmarshal.java`

Author: Hao Chen <hao@apache.org>

Closes apache#776 from haoch/FixUT.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants